[libcamera-devel,0/2] VIMC IPA nitpicks
mbox series

Message ID 20220408105439.144182-1-umang.jain@ideasonboard.com
Headers show
Series
  • VIMC IPA nitpicks
Related show

Message

Umang Jain April 8, 2022, 10:54 a.m. UTC 
Summary says it all :)
Umang Jain (2):
  ipa: vimc: Establish logical order of operations
  ipa: vimc: Synchronise parameter buffer ops naming

 include/libcamera/ipa/vimc.mojom     |  6 +++---
 src/ipa/vimc/vimc.cpp                | 16 ++++++++--------
 src/libcamera/pipeline/vimc/vimc.cpp |  8 ++++----
 3 files changed, 15 insertions(+), 15 deletions(-)

Comments

Nicolas Dufresne via libcamera-devel April 11, 2022, 7:07 a.m. UTC | #1 
Hi Umang,

On Fri, Apr 08, 2022 at 04:24:37PM +0530, Umang Jain via libcamera-devel wrote:
> Summary says it all :)
> Umang Jain (2):
>   ipa: vimc: Establish logical order of operations
>   ipa: vimc: Synchronise parameter buffer ops naming

For both,

Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>

> 
>  include/libcamera/ipa/vimc.mojom     |  6 +++---
>  src/ipa/vimc/vimc.cpp                | 16 ++++++++--------
>  src/libcamera/pipeline/vimc/vimc.cpp |  8 ++++----
>  3 files changed, 15 insertions(+), 15 deletions(-)
> 
> -- 
> 2.31.0
>