<div dir="ltr">Hi everyone<div><br></div><div>Can I give this patch a little prod, see how folks would like to move forward on this one?</div><div><br></div><div>Thanks!</div><div>David</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 27 Oct 2020 at 14:12, David Plowman <<a href="mailto:david.plowman@raspberrypi.com">david.plowman@raspberrypi.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 everyone<br>
<br>
I wanted to raise a topic on which there was already some discussion a<br>
few weeks back, prompted by my noticing a gap in the metadata that we<br>
provide with images.<br>
<br>
My observation was that I couldn't correctly set the ISO value in my<br>
JPEG files because we don't report how much digital gain has been<br>
supplied by the image processing pipeline. To address this problem, it<br>
seemed as good a way as any actually to include a proposal for a new<br>
DigitalGain control (see the associated patch). Nevertheless, I think<br>
there's a discussion to be had first. Notably:<br>
<br>
* Should we let the pipeline report a single global "digital gain"<br>
  value or, given that different gains may be applied to the colour<br>
  channels, should we report three gain values instead?<br>
<br>
* In the Pi world I'd like this to be a read-only control, i.e. you<br>
  can't force the pipeline digital gain to a particular value, as our<br>
  AGC doesn't work like that. But there may be platforms that do allow<br>
  you to set the digital gain.<br>
<br>
* You could imagine this being "per-stream". I think it's another case<br>
  of "practically everyone will just want a single value", though<br>
  technically you might be able to imagine some platform and use-cases<br>
  where different outputs might reflect different digital gains.<br>
<br>
* There's a question about digital gain being applied by the sensor<br>
  itself, but I'm inclined to view that as a separate topic. The issue<br>
  at hand, at least for me, is being able to distinguish the gain<br>
  applied by the sensor, and which we see in the raw frames, from that<br>
  applied by the ISP, which appears in the JPEGs.<br>
<br>
You may have other comments too - so everyone's thoughts would be much<br>
appreciated.<br>
<br>
Thanks!<br>
David<br>
<br>
David Plowman (1):<br>
  libcamera: controls: Add DigitalGain control<br>
<br>
 src/libcamera/control_ids.yaml | 11 +++++++++++<br>
 1 file changed, 11 insertions(+)<br>
<br>
-- <br>
2.20.1<br>
<br>
</blockquote></div>