summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJoão Paulo Gonçalves <joao.goncalves@toradex.com>2025-04-02 13:37:32 -0300
committerJoão Paulo Gonçalves <joao.goncalves@toradex.com>2025-04-04 10:21:12 -0300
commit29ddb5b33673b5b1ae8492446ef953e27e2e9c3d (patch)
tree24ac8c316bea5a67ef2296117c356fbf0d6d7524
parent82c9ac97060f675e434dbfa656ae8cfdbd98963b (diff)
verdin-imx8mm/mp: ov5640: Refactor camera overlays
Reorganize the properties on the camera node to follow Toradex DTS coding standards and remove dtc warnings on remote endpoint ports nodes. Related-to: ELB-6321 Signed-off-by: João Paulo Gonçalves <joao.goncalves@toradex.com>
-rw-r--r--overlays/verdin-imx8mm_ov5640_overlay.dts11
-rw-r--r--overlays/verdin-imx8mp_ov5640_overlay.dts11
2 files changed, 14 insertions, 8 deletions
diff --git a/overlays/verdin-imx8mm_ov5640_overlay.dts b/overlays/verdin-imx8mm_ov5640_overlay.dts
index 755897d..581acc2 100644
--- a/overlays/verdin-imx8mm_ov5640_overlay.dts
+++ b/overlays/verdin-imx8mm_ov5640_overlay.dts
@@ -37,16 +37,16 @@
camera@3c {
compatible = "ovti,ov5640";
reg = <0x3c>;
- clocks = <&clk IMX8MM_CLK_CLKO1>;
- clock-names = "xclk";
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_gpio5>, <&pinctrl_gpio6>;
assigned-clocks = <&clk IMX8MM_CLK_CLKO1>;
assigned-clock-parents = <&clk IMX8MM_CLK_24M>;
assigned-clock-rates = <24000000>;
+ clocks = <&clk IMX8MM_CLK_CLKO1>;
+ clock-names = "xclk";
AVDD-supply = <&regulator_camera>;
DOVDD-supply = <&regulator_camera>;
DVDD-supply = <&regulator_camera>;
- pinctrl-names = "default";
- pinctrl-0 = <&pinctrl_gpio5>, <&pinctrl_gpio6>;
/* Verdin GPIO6 - Camera Connector 22 */
powerdown-gpios = <&gpio1 11 GPIO_ACTIVE_HIGH>;
/* Verdin GPIO5 - Camera Connector 11 */
@@ -66,6 +66,9 @@
status = "okay";
ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
port@0 {
reg = <0>;
diff --git a/overlays/verdin-imx8mp_ov5640_overlay.dts b/overlays/verdin-imx8mp_ov5640_overlay.dts
index 3fa0447..7ba9530 100644
--- a/overlays/verdin-imx8mp_ov5640_overlay.dts
+++ b/overlays/verdin-imx8mp_ov5640_overlay.dts
@@ -33,16 +33,16 @@
camera@3c {
compatible = "ovti,ov5640";
reg = <0x3c>;
- clocks = <&clk IMX8MP_CLK_IPP_DO_CLKO2>;
- clock-names = "xclk";
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_gpio5>, <&pinctrl_gpio6>;
assigned-clocks = <&clk IMX8MP_CLK_IPP_DO_CLKO2>;
assigned-clock-parents = <&clk IMX8MP_CLK_24M>;
assigned-clock-rates = <24000000>;
+ clocks = <&clk IMX8MP_CLK_IPP_DO_CLKO2>;
+ clock-names = "xclk";
AVDD-supply = <&regulator_camera>;
DOVDD-supply = <&regulator_camera>;
DVDD-supply = <&regulator_camera>;
- pinctrl-names = "default";
- pinctrl-0 = <&pinctrl_gpio5>, <&pinctrl_gpio6>;
/* Verdin GPIO6 - Camera Connector 22 */
powerdown-gpios = <&gpio1 8 GPIO_ACTIVE_HIGH>;
/* Verdin GPIO5 - Camera Connector 11 */
@@ -66,6 +66,9 @@
status = "okay";
ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
port@0 {
reg = <0>;