<div dir="ltr">HI Laurent<div><br></div><div>Yes, I think that would be fine.</div><div><br></div><div>Thanks</div><div>David</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 29 Jun 2021 at 14:55, Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com">laurent.pinchart@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi David,<br>
<br>
On Tue, Jun 29, 2021 at 02:22:25PM +0100, David Plowman wrote:<br>
> Hi Jean-Michel<br>
> <br>
> Thanks for your work - very happy to see our code being useful elsewhere!<br>
> <br>
> One minor thing - would it be ok to include a Raspberry Pi copyright<br>
> alongside the IdeasonBoard one?<br>
<br>
I haven't looked at the code yet, but if a substantial part of it comes<br>
from the RPi implementation, it should certainly mention appropriate<br>
authorship and copyright information. In the kernel, we often mention it<br>
as follows:<br>
<br>
 * Based on the implementation from the Raspberry Pi IPA,<br>
 * Copyright (C) 2019-2021, Raspberry Pi (Trading) Ltd.<br>
<br>
Would that be fine with you ?<br>
<br>
> On Mon, 28 Jun 2021 at 21:23, Jean-Michel Hautboiswrote:<br>
> <br>
> > This one comes from RPi for most if it, except that we are not<br>
> > exchanging any metadata between algorithms for now.<br>
> > When process() is called, the current analogue gain and shutter time are<br>
> > calculated. The AWB stats from IPU3 are then parsed to generate new<br>
> > statistics dedicated to AGC. This new grid is used to estimate the<br>
> > luminance and each region is weighted. A default centered metering is<br>
> > used as is should be the most used one.<br>
> ><br>
> > After calculating weighted regions, analogue gain and shutter time are<br>
> > divided up and the values are then sent back to the IPAIPU3.<br>
> ><br>
> > Signed-off-by: Jean-Michel Hautbois <<a href="mailto:jeanmichel.hautbois@ideasonboard.com" target="_blank">jeanmichel.hautbois@ideasonboard.com</a>><br>
> > ---<br>
> >  src/ipa/ipu3/ipu3_agc.cpp | 276 ++++++++++++++++++++++++--------------<br>
> >  src/ipa/ipu3/ipu3_agc.h   |  23 +++-<br>
> >  2 files changed, 196 insertions(+), 103 deletions(-)<br>
<br>
-- <br>
Regards,<br>
<br>
Laurent Pinchart<br>
</blockquote></div>