[libcamera-devel] [PATCH v5 8/8] ipa: raspberrypi: Make variable names consistent

[Kieran Bingham]

Hi Naush,

On 09/07/2021 10:56, Naushir Patuck wrote:
> s/DefaultAnalogueGain/defaultAnalogueGain/
> s/DefaultExposureTime/defaultExposureTime/
> 
> Change these for consistency with the other static const variables.
> 
> Signed-off-by: Naushir Patuck <naush at raspberrypi.com>
> Reviewed-by: David Plowman <david.plowman at raspberrypi.com>
> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>


> ---
>  src/ipa/raspberrypi/raspberrypi.cpp | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/src/ipa/raspberrypi/raspberrypi.cpp b/src/ipa/raspberrypi/raspberrypi.cpp
> index 8fb98a49da63..1dd7fd68c3e9 100644
> --- a/src/ipa/raspberrypi/raspberrypi.cpp
> +++ b/src/ipa/raspberrypi/raspberrypi.cpp
> @@ -59,8 +59,8 @@ using namespace std::literals::chrono_literals;
>  using utils::Duration;
>  
>  /* Configure the sensor with these values initially. */
> -constexpr double DefaultAnalogueGain = 1.0;
> -constexpr Duration DefaultExposureTime = 20.0ms;
> +constexpr double defaultAnalogueGain = 1.0;
> +constexpr Duration defaultExposureTime = 20.0ms;
>  constexpr Duration defaultMinFrameDuration = 1.0s / 30.0;
>  constexpr Duration defaultMaxFrameDuration = 250.0s;
>  
> @@ -390,8 +390,8 @@ int IPARPi::configure(const IPACameraSensorInfo &sensorInfo,
>  		/* Supply initial values for gain and exposure. */
>  		ControlList ctrls(sensorCtrls_);
>  		AgcStatus agcStatus;
> -		agcStatus.shutter_time = DefaultExposureTime;
> -		agcStatus.analogue_gain = DefaultAnalogueGain;
> +		agcStatus.shutter_time = defaultExposureTime;
> +		agcStatus.analogue_gain = defaultAnalogueGain;
>  		applyAGC(&agcStatus, ctrls);
>  
>  		ASSERT(controls);
>