<div dir="ltr"><div dir="ltr">Hi Jacopo,<div><br></div><div>Thank you for the feedback.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 28 Sept 2022 at 08:33, Jacopo Mondi <<a href="mailto:jacopo@jmondi.org">jacopo@jmondi.org</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 Naush<br>
<br>
On Mon, Sep 26, 2022 at 01:29:11PM +0100, Naushir Patuck via libcamera-devel wrote:<br>
> The sensor black level is 60 (in 10-bits), fix this in the mono variant tuning<br>
> file.<br>
><br>
> Signed-off-by: Naushir Patuck <<a href="mailto:naush@raspberrypi.com" target="_blank">naush@raspberrypi.com</a>><br>
> ---<br>
>  src/ipa/raspberrypi/data/imx296_mono.json | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/src/ipa/raspberrypi/data/imx296_mono.json b/src/ipa/raspberrypi/data/imx296_mono.json<br>
> index 490c0fda0cca..b68b11dbd609 100644<br>
> --- a/src/ipa/raspberrypi/data/imx296_mono.json<br>
> +++ b/src/ipa/raspberrypi/data/imx296_mono.json<br>
> @@ -5,7 +5,7 @@<br>
>          {<br>
>              "rpi.black_level":<br>
>              {<br>
> -                "black_level": 4096<br>
> +                "black_level": 3840<br>
<br>
Out of curiosity, what encoding is this one ? You mentioned 10-bits<br>
but this doesn't look like Q10, or is it ?<br></blockquote><div><br></div><div>All our tuning parameters are specified in 16-bits, i.e. a white-point of 65535.</div><div>So the black level above is computed as 60 << (16bits - 10bits) == 3840.</div><div><br></div><div>Regards,</div><div>Naush</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
>              }<br>
>          },<br>
>          {<br>
> --<br>
> 2.25.1<br>
><br>
</blockquote></div></div>