[libcamera-devel] [PATCH 1/2] ipa: raspberrypi: AWB: Remove unused code

[Kieran Bingham]

Hi David,

On 25/02/2021 17:01, David Plowman wrote:
> Some more code from pre-libcamera versions is removed. There is no
> functional change.

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

> 
> Signed-off-by: David Plowman <david.plowman at raspberrypi.com>
> ---
>  src/ipa/raspberrypi/controller/rpi/awb.cpp | 4 +---
>  src/ipa/raspberrypi/controller/rpi/awb.hpp | 7 +------
>  2 files changed, 2 insertions(+), 9 deletions(-)
> 
> diff --git a/src/ipa/raspberrypi/controller/rpi/awb.cpp b/src/ipa/raspberrypi/controller/rpi/awb.cpp
> index bb637f10..791a7039 100644
> --- a/src/ipa/raspberrypi/controller/rpi/awb.cpp
> +++ b/src/ipa/raspberrypi/controller/rpi/awb.cpp
> @@ -21,8 +21,6 @@ LOG_DEFINE_CATEGORY(RPiAwb)
>  #define AWB_STATS_SIZE_X DEFAULT_AWB_REGIONS_X
>  #define AWB_STATS_SIZE_Y DEFAULT_AWB_REGIONS_Y
>  
> -const double Awb::RGB::INVALID = -1.0;
> -
>  // todo - the locking in this algorithm needs some tidying up as has been done
>  // elsewhere (ALSC and AGC).
>  
> @@ -340,7 +338,7 @@ static void generate_stats(std::vector<Awb::RGB> &zones,
>  			   double min_G)
>  {
>  	for (int i = 0; i < AWB_STATS_SIZE_X * AWB_STATS_SIZE_Y; i++) {
> -		Awb::RGB zone; // this is "invalid", unless R gets overwritten later
> +		Awb::RGB zone;
>  		double counted = stats[i].counted;
>  		if (counted >= min_pixels) {
>  			zone.G = stats[i].g_sum / counted;
> diff --git a/src/ipa/raspberrypi/controller/rpi/awb.hpp b/src/ipa/raspberrypi/controller/rpi/awb.hpp
> index 45ba9e25..2fe59c37 100644
> --- a/src/ipa/raspberrypi/controller/rpi/awb.hpp
> +++ b/src/ipa/raspberrypi/controller/rpi/awb.hpp
> @@ -90,21 +90,16 @@ public:
>  	void Prepare(Metadata *image_metadata) override;
>  	void Process(StatisticsPtr &stats, Metadata *image_metadata) override;
>  	struct RGB {
> -		RGB(double _R = INVALID, double _G = INVALID,
> -		    double _B = INVALID)
> +		RGB(double _R = -1.0, double _G = -1.0, double _B = -1.0)
>  			: R(_R), G(_G), B(_B)
>  		{
>  		}
>  		double R, G, B;
> -		static const double INVALID;
> -		bool Valid() const { return G != INVALID; }
> -		bool Invalid() const { return G == INVALID; }
>  		RGB &operator+=(RGB const &other)
>  		{
>  			R += other.R, G += other.G, B += other.B;
>  			return *this;
>  		}
> -		RGB Square() const { return RGB(R * R, G * G, B * B); }
>  	};
>  
>  private:
> 

-- 
Regards
--
Kieran