[29/35] libcamera: software_isp: Make isStandardBayerOrder static
diff mbox series

Message ID 20250611013245.133785-30-bryan.odonoghue@linaro.org
State New
Headers show
Series
  • Add GLES 2.0 GPUISP to libcamera
Related show

Commit Message

Bryan O'Donoghue June 11, 2025, 1:32 a.m. UTC
Make this member function static so that it may be called from static
members of derived classes.

Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
 src/libcamera/software_isp/debayer.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Bryan O'Donoghue June 16, 2025, 7:22 p.m. UTC | #1
On 11/06/2025 02:32, Bryan O'Donoghue wrote:
> Make this member function static so that it may be called from static
> members of derived classes.
> 
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
>   src/libcamera/software_isp/debayer.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/libcamera/software_isp/debayer.h b/src/libcamera/software_isp/debayer.h
> index 214bcdd3..3893318b 100644
> --- a/src/libcamera/software_isp/debayer.h
> +++ b/src/libcamera/software_isp/debayer.h
> @@ -88,7 +88,7 @@ private:
>   protected:
>   	void setParams(DebayerParams &params);
>   	void dmaSyncBegin(std::vector<DmaSyncer> &dmaSyncers, FrameBuffer *input, FrameBuffer *output);
> -	bool isStandardBayerOrder(BayerFormat::Order order);
> +	static bool isStandardBayerOrder(BayerFormat::Order order);
>   };
> 
>   } /* namespace libcamera */
> --
> 2.49.0
> 

Both this and the next patch can/should be dropped, provided we can tell 
GBM to do 24 BPP at a reasonable stage in the object instantiation logic.

We need not have static methods in the Bayer classes to ask what pixel 
formats they support.

---
bod

Patch
diff mbox series

diff --git a/src/libcamera/software_isp/debayer.h b/src/libcamera/software_isp/debayer.h
index 214bcdd3..3893318b 100644
--- a/src/libcamera/software_isp/debayer.h
+++ b/src/libcamera/software_isp/debayer.h
@@ -88,7 +88,7 @@  private:
 protected:
 	void setParams(DebayerParams &params);
 	void dmaSyncBegin(std::vector<DmaSyncer> &dmaSyncers, FrameBuffer *input, FrameBuffer *output);
-	bool isStandardBayerOrder(BayerFormat::Order order);
+	static bool isStandardBayerOrder(BayerFormat::Order order);
 };
 
 } /* namespace libcamera */