[libcamera-devel] [PATCH] ipa: raspberrrypi: Remove duplicate comment

[Sebastian Fricke]

Hey Jean-Michel,

On 28.03.2021 23:04, Jean-Michel Hautbois wrote:
>Hi Sebastian,
>
>On 28/03/2021 18:51, Laurent Pinchart wrote:
>> Hi Sebastian,
>>
>> Thank you for the patch.
>>
>> On Sun, Mar 28, 2021 at 09:59:54AM +0200, Sebastian Fricke wrote:
>>> The exact same comment is used two times within 5 lines.
>>> Remove one of them as it is redundant.
>>>
>>> Signed-off-by: Sebastian Fricke <sebastian.fricke at posteo.net>
>>
>> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
>
>There is a typo in the commit title (too much r's :-)).
>Otherwise:
>Reviewed-by: Jean-Michel Hautbois <jeanmichel.hautbois at ideasonboard.com>

Woops fixing a typo with a typo ;)
Thank you for notifying.

Greetings,
Sebastian

>
>>> ---
>>>  src/ipa/raspberrypi/raspberrypi.cpp | 1 -
>>>  1 file changed, 1 deletion(-)
>>>
>>> diff --git a/src/ipa/raspberrypi/raspberrypi.cpp b/src/ipa/raspberrypi/raspberrypi.cpp
>>> index 1c928b72..ed2d9a54 100644
>>> --- a/src/ipa/raspberrypi/raspberrypi.cpp
>>> +++ b/src/ipa/raspberrypi/raspberrypi.cpp
>>> @@ -208,7 +208,6 @@ void IPARPi::start(const ControlList &controls, ipa::RPi::StartConfig *startConf
>>>  	agcStatus.shutter_time = 0.0;
>>>  	agcStatus.analogue_gain = 0.0;
>>>
>>> -	/* SwitchMode may supply updated exposure/gain values to use. */
>>>  	metadata.Get("agc.status", agcStatus);
>>>  	if (agcStatus.shutter_time != 0.0 && agcStatus.analogue_gain != 0.0) {
>>>  		ControlList ctrls(sensorCtrls_);
>>