| Message ID | 20250409134128.2098195-8-y-abhilashchandra@ti.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | Fix dtbs_check warnings in CSI overlays | expand |
On 4/9/2025 7:11 PM, Yemike Abhilash Chandra wrote: > The OV5640 device tree overlay incorrectly defined an I2C switch instead > of an I2C mux. According to the DT bindings, the correct terminology and > node definition should use "i2c-mux" instead of "i2c-switch". Hence, > update the same to avoid dtbs_check warnings. > > Fixes: 635ed9715194 ("arm64: dts: ti: k3-am62x: Add overlays for OV5640") > Cc: stable@vger.kernel.org > Signed-off-by: Yemike Abhilash Chandra <y-abhilashchandra@ti.com> > --- > arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-ov5640.dtso | 2 +- > arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-tevi-ov5640.dtso | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-ov5640.dtso b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-ov5640.dtso > index ccc7f5e43184..7fc7c95f5cd5 100644 > --- a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-ov5640.dtso > +++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-ov5640.dtso > @@ -22,7 +22,7 @@ &main_i2c2 { > #size-cells = <0>; > status = "okay"; > > - i2c-switch@71 { > + i2c-mux@71 { > compatible = "nxp,pca9543"; > #address-cells = <1>; > #size-cells = <0>; > diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-tevi-ov5640.dtso b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-tevi-ov5640.dtso > index 4eaf9d757dd0..b6bfdfbbdd98 100644 > --- a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-tevi-ov5640.dtso > +++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-tevi-ov5640.dtso > @@ -22,7 +22,7 @@ &main_i2c2 { > #size-cells = <0>; > status = "okay"; > > - i2c-switch@71 { > + i2c-mux@71 { > compatible = "nxp,pca9543"; > #address-cells = <1>; > #size-cells = <0>; Reviewed-by: Neha Malcom Francis <n-francis@ti.com>
On Wed, Apr 09, 2025 at 07:11:28PM +0530, Yemike Abhilash Chandra wrote: > The OV5640 device tree overlay incorrectly defined an I2C switch instead > of an I2C mux. According to the DT bindings, the correct terminology and > node definition should use "i2c-mux" instead of "i2c-switch". Hence, > update the same to avoid dtbs_check warnings. > > Fixes: 635ed9715194 ("arm64: dts: ti: k3-am62x: Add overlays for OV5640") > Cc: stable@vger.kernel.org > Signed-off-by: Yemike Abhilash Chandra <y-abhilashchandra@ti.com> Reviewed-by: Jai Luthra <jai.luthra@linux.dev> > --- > arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-ov5640.dtso | 2 +- > arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-tevi-ov5640.dtso | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-ov5640.dtso b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-ov5640.dtso > index ccc7f5e43184..7fc7c95f5cd5 100644 > --- a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-ov5640.dtso > +++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-ov5640.dtso > @@ -22,7 +22,7 @@ &main_i2c2 { > #size-cells = <0>; > status = "okay"; > > - i2c-switch@71 { > + i2c-mux@71 { > compatible = "nxp,pca9543"; > #address-cells = <1>; > #size-cells = <0>; > diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-tevi-ov5640.dtso b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-tevi-ov5640.dtso > index 4eaf9d757dd0..b6bfdfbbdd98 100644 > --- a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-tevi-ov5640.dtso > +++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-tevi-ov5640.dtso > @@ -22,7 +22,7 @@ &main_i2c2 { > #size-cells = <0>; > status = "okay"; > > - i2c-switch@71 { > + i2c-mux@71 { > compatible = "nxp,pca9543"; > #address-cells = <1>; > #size-cells = <0>; > -- > 2.34.1 >
diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-ov5640.dtso b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-ov5640.dtso index ccc7f5e43184..7fc7c95f5cd5 100644 --- a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-ov5640.dtso +++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-ov5640.dtso @@ -22,7 +22,7 @@ &main_i2c2 { #size-cells = <0>; status = "okay"; - i2c-switch@71 { + i2c-mux@71 { compatible = "nxp,pca9543"; #address-cells = <1>; #size-cells = <0>; diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-tevi-ov5640.dtso b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-tevi-ov5640.dtso index 4eaf9d757dd0..b6bfdfbbdd98 100644 --- a/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-tevi-ov5640.dtso +++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-tevi-ov5640.dtso @@ -22,7 +22,7 @@ &main_i2c2 { #size-cells = <0>; status = "okay"; - i2c-switch@71 { + i2c-mux@71 { compatible = "nxp,pca9543"; #address-cells = <1>; #size-cells = <0>;
The OV5640 device tree overlay incorrectly defined an I2C switch instead of an I2C mux. According to the DT bindings, the correct terminology and node definition should use "i2c-mux" instead of "i2c-switch". Hence, update the same to avoid dtbs_check warnings. Fixes: 635ed9715194 ("arm64: dts: ti: k3-am62x: Add overlays for OV5640") Cc: stable@vger.kernel.org Signed-off-by: Yemike Abhilash Chandra <y-abhilashchandra@ti.com> --- arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-ov5640.dtso | 2 +- arch/arm64/boot/dts/ti/k3-am62x-sk-csi2-tevi-ov5640.dtso | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)