[PATCH] android: camera_device: Save capture settings unconditionally

[Jacopo Mondi]

Hi,

On Tue, Feb 27, 2024 at 10:13:20AM +0800, Fang Hui wrote:
> If not, it will be deferred to the next frame,

I proposed an extended commit message in the previous review. If
ignored on purpose, could you suggest me how it could be improved.
Just this one line is too short and not explicative imo

> as there's a bug assigned to this issue
> https://bugs.libcamera.org/show_bug.cgi?id=210

What I meant was to add:

Bug: https://bugs.libcamera.org/show_bug.cgi?id=210
>
> Signed-off-by: Fang Hui <hui.fang at nxp.com>

Before the Signed-off-by tag

> ---
>  src/android/camera_device.cpp | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/android/camera_device.cpp b/src/android/camera_device.cpp
> index 25cedd44..d45ed1a5 100644
> --- a/src/android/camera_device.cpp
> +++ b/src/android/camera_device.cpp
> @@ -952,8 +952,8 @@ int CameraDevice::processCaptureRequest(camera3_capture_request_t *camera3Reques
>  	 */
>  	if (camera3Request->settings)
>  		lastSettings_ = camera3Request->settings;
> -	else
> -		descriptor->settings_ = lastSettings_;
> +
> +	descriptor->settings_ = lastSettings_;

I still think you should re-initialize lastSettings_ as I proposed in
the review of the previous version. Was it left out intentionally
because you think it's not needed ? Could you explain why ?

As I've said, with your ack, I could have made the changes I proposed
before applying the patch, but as you sent a new version I deduce you
disagree with the proposed changes ?

>
>  	LOG(HAL, Debug) << "Queueing request " << descriptor->request_->cookie()
>  			<< " with " << descriptor->buffers_.size() << " streams";
> --
> 2.25.1
>