<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 29 Jun 2022 at 12:06, Kieran Bingham <<a href="mailto:kieran.bingham@ideasonboard.com" target="_blank">kieran.bingham@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">Quoting Naushir Patuck via libcamera-devel (2022-06-29 09:57:18)<br>
> Hi,<br>
> <br>
> The only change in this version is using the suggested std::optional::value_or()<br>
> function instead of the ternary operator in lux.cpp.<br>
<br>
And you've collected tags too, so all that makes it easier for me to<br>
merge. I'll get to this next. Thanks.<br></blockquote><div><br></div><div>Thanks Kieran!</div><div><br></div><div>PS the "Correct ControlInfoMap from the IPA" series needs only one more R-B tag for 3/3 if you are feeling bored ;-)</div><div><br></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>
Kieran<br>
<br>
<br>
> <br>
> Thanks,<br>
> Naush<br>
> <br>
> Naushir Patuck (4):<br>
>   libcamera: controls: Add SensorTemperature control<br>
>   ipa: raspberrypi: Add sensor temperature to DeviceStatus<br>
>   ipa: raspberrypi: imx477: Get sensor temperature from embedded data<br>
>   ipa: raspberrypi: Use std::optional in DeviceStatus<br>
> <br>
>  src/ipa/raspberrypi/cam_helper.cpp              |  8 +++++---<br>
>  src/ipa/raspberrypi/cam_helper_imx477.cpp       |  5 ++++-<br>
>  .../raspberrypi/controller/device_status.cpp    | 17 +++++++++++++----<br>
>  src/ipa/raspberrypi/controller/device_status.h  | 12 +++++++-----<br>
>  src/ipa/raspberrypi/controller/rpi/lux.cpp      |  4 +---<br>
>  src/ipa/raspberrypi/raspberrypi.cpp             |  2 ++<br>
>  src/libcamera/control_ids.yaml                  | 10 ++++++++++<br>
>  7 files changed, 42 insertions(+), 16 deletions(-)<br>
> <br>
> -- <br>
> 2.25.1<br>
><br>
</blockquote></div></div>