Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: ti: k3-am62x-sk: Add orientation to IMX219 sensor node
@ 2026-11-02 21:15 Thorsten Lannynd
  2026-11-04 16:50 ` Kieran Bingham
  0 siblings, 1 reply; 3+ messages in thread
From: Thorsten Lannynd @ 2026-11-02 21:15 UTC (permalink / raw)
  To: linux-arm-kernel
  Cc: b-brnich, praneeth, r-donadkar, devarsht, Nishanth Menon,
	Vignesh Raghavendra, Tero Kristo, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, devicetree, linux-kernel

The IMX219 CSI2 expansion module is externally attached. Without the
orientation property, V4L2_CID_CAMERA_ORIENTATION (0x009a0922) is not
exposed and libcamera fails with:

  ERROR CameraSensor camera_sensor_raw.cpp:465 'imx219 4-0010':
  Mandatory V4L2 control 0x009a0922 not available

Set orientation = <2> (V4L2_CAMERA_ORIENTATION_EXTERNAL) to fix this.

Signed-off-by: Thorsten Lannynd <t-lannynd@ti.com>
---
 arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-imx219.dtso | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-imx219.dtso b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-imx219.dtso
index 149c59c07182..829ed98345dd 100644
--- a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-imx219.dtso
+++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-imx219.dtso
@@ -72,6 +72,7 @@ ov5640: camera@10 {
 				VDDL-supply = <&reg_1p2v>;
 
 				reset-gpios = <&exp1 13 GPIO_ACTIVE_HIGH>;
+				orientation = <2>;
 
 				port {
 					csi2_cam0: endpoint {
-- 
2.34.1



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: dts: ti: k3-am62x-sk: Add orientation to IMX219 sensor node
  2026-11-02 21:15 [PATCH] arm64: dts: ti: k3-am62x-sk: Add orientation to IMX219 sensor node Thorsten Lannynd
@ 2026-11-04 16:50 ` Kieran Bingham
  2026-11-04 17:14   ` Lannynd, Thorsten
  0 siblings, 1 reply; 3+ messages in thread
From: Kieran Bingham @ 2026-11-04 16:50 UTC (permalink / raw)
  To: Thorsten Lannynd, linux-arm-kernel
  Cc: b-brnich, praneeth, r-donadkar, devarsht, Nishanth Menon,
	Vignesh Raghavendra, Tero Kristo, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, devicetree, linux-kernel

Hi Thorsten,


Quoting Thorsten Lannynd (2026-11-02 21:15:49)
> The IMX219 CSI2 expansion module is externally attached. Without the
> orientation property, V4L2_CID_CAMERA_ORIENTATION (0x009a0922) is not
> exposed and libcamera fails with:
> 
>   ERROR CameraSensor camera_sensor_raw.cpp:465 'imx219 4-0010':
>   Mandatory V4L2 control 0x009a0922 not available
> 
> Set orientation = <2> (V4L2_CAMERA_ORIENTATION_EXTERNAL) to fix this.
> 
> Signed-off-by: Thorsten Lannynd <t-lannynd@ti.com>
> ---
>  arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-imx219.dtso | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-imx219.dtso b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-imx219.dtso
> index 149c59c07182..829ed98345dd 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-imx219.dtso
> +++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-imx219.dtso

#include <dt-bindings/media/video-interface-devices.h>

> @@ -72,6 +72,7 @@ ov5640: camera@10 {
>                                 VDDL-supply = <&reg_1p2v>;
>  
>                                 reset-gpios = <&exp1 13 GPIO_ACTIVE_HIGH>;
> +                               orientation = <2>;

and then you can do:

				orientation = <MEDIA_ORIENTATION_EXTERNAL>;

--
Kieran


>  
>                                 port {
>                                         csi2_cam0: endpoint {
> -- 
> 2.34.1
>


^ permalink raw reply	[flat|nested] 3+ messages in thread

* RE: [PATCH] arm64: dts: ti: k3-am62x-sk: Add orientation to IMX219 sensor node
  2026-11-04 16:50 ` Kieran Bingham
@ 2026-11-04 17:14   ` Lannynd, Thorsten
  0 siblings, 0 replies; 3+ messages in thread
From: Lannynd, Thorsten @ 2026-11-04 17:14 UTC (permalink / raw)
  To: Kieran Bingham, linux-arm-kernel@lists.infradead.org
  Cc: Brnich, Brandon, Bajjuri, Praneeth, Donadkar, Rishikesh,
	Thakkar, Devarsh, Menon, Nishanth, Raghavendra, Vignesh,
	Tero Kristo, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org

On Wed, 4 Nov 2026, Kieran Bingham wrote:
> > +                               orientation = <2>;
> 
> and then you can do:
> 
> 				orientation =
> <MEDIA_ORIENTATION_EXTERNAL>;

Good catch, will fix in v2

--
Thorsten

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-11-04 17:15 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-11-02 21:15 [PATCH] arm64: dts: ti: k3-am62x-sk: Add orientation to IMX219 sensor node Thorsten Lannynd
2026-11-04 16:50 ` Kieran Bingham
2026-11-04 17:14   ` Lannynd, Thorsten

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox