ARM: dts: renesas: Add missing ADV751[13] power supply properties
authorGeert Uytterhoeven <geert+renesas@glider.be>
Mon, 23 Oct 2023 13:52:16 +0000 (15:52 +0200)
committerGeert Uytterhoeven <geert+renesas@glider.be>
Mon, 13 Nov 2023 13:35:36 +0000 (14:35 +0100)
make dtbs_check:

    arch/arm/boot/dts/renesas/r8a7791-koelsch.dtb: hdmi@39: 'avdd-supply' is a required property
    from schema $id: http://devicetree.org/schemas/display/bridge/adi,adv7511.yaml#
    arch/arm/boot/dts/renesas/r8a7791-koelsch.dtb: hdmi@39: 'dvdd-supply' is a required property
    from schema $id: http://devicetree.org/schemas/display/bridge/adi,adv7511.yaml#
    arch/arm/boot/dts/renesas/r8a7791-koelsch.dtb: hdmi@39: 'pvdd-supply' is a required property
    from schema $id: http://devicetree.org/schemas/display/bridge/adi,adv7511.yaml#
    arch/arm/boot/dts/renesas/r8a7791-koelsch.dtb: hdmi@39: 'dvdd-3v-supply' is a required property
    from schema $id: http://devicetree.org/schemas/display/bridge/adi,adv7511.yaml#
    arch/arm/boot/dts/renesas/r8a7791-koelsch.dtb: hdmi@39: 'bgvdd-supply' is a required property
    from schema $id: http://devicetree.org/schemas/display/bridge/adi,adv7511.yaml#
    ...

Fix this by adding the missing power supply properties, and by adding
fixed regulators where needed.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Link: https://lore.kernel.org/r/6c49fd83d327a68040f052bd9fd96fe25b0fc301.1698068647.git.geert+renesas@glider.be
arch/arm/boot/dts/renesas/iwg20d-q7-dbcm-ca.dtsi
arch/arm/boot/dts/renesas/r8a7745-iwg22d-sodimm-dbhd-ca.dts
arch/arm/boot/dts/renesas/r8a7790-lager.dts
arch/arm/boot/dts/renesas/r8a7790-stout.dts
arch/arm/boot/dts/renesas/r8a7791-koelsch.dts
arch/arm/boot/dts/renesas/r8a7791-porter.dts
arch/arm/boot/dts/renesas/r8a7792-blanche.dts
arch/arm/boot/dts/renesas/r8a7792-wheat.dts
arch/arm/boot/dts/renesas/r8a7793-gose.dts
arch/arm/boot/dts/renesas/r8a7794-silk.dts

index e10f992..de52218 100644 (file)
                        };
                };
        };
+
+       reg_1p8v: regulator-1p8v {
+               compatible = "regulator-fixed";
+               regulator-name = "1P8V";
+               regulator-min-microvolt = <1800000>;
+               regulator-max-microvolt = <1800000>;
+               regulator-boot-on;
+               regulator-always-on;
+       };
 };
 
 &can1 {
                clocks = <&cec_clock>;
                clock-names = "cec";
 
+               avdd-supply = <&reg_1p8v>;
+               dvdd-supply = <&reg_1p8v>;
+               pvdd-supply = <&reg_1p8v>;
+               dvdd-3v-supply = <&reg_3p3v>;
+               bgvdd-supply = <&reg_1p8v>;
+
                adi,input-depth = <8>;
                adi,input-colorspace = "rgb";
                adi,input-clock = "1x";
index b1f679d..a0b5743 100644 (file)
                        };
                };
        };
+
+       reg_1p8v: regulator-1p8v {
+               compatible = "regulator-fixed";
+               regulator-name = "1P8V";
+               regulator-min-microvolt = <1800000>;
+               regulator-max-microvolt = <1800000>;
+               regulator-boot-on;
+               regulator-always-on;
+       };
 };
 
 &du {
                clock-names = "cec";
                pd-gpios = <&gpio2 24 GPIO_ACTIVE_HIGH>;
 
+               avdd-supply = <&reg_1p8v>;
+               dvdd-supply = <&reg_1p8v>;
+               pvdd-supply = <&reg_1p8v>;
+               dvdd-3v-supply = <&reg_3p3v>;
+               bgvdd-supply = <&reg_1p8v>;
+
                adi,input-depth = <8>;
                adi,input-colorspace = "rgb";
                adi,input-clock = "1x";
index bc04567..2fba4d0 100644 (file)
                };
        };
 
+       fixedregulator1v8: regulator-1v8 {
+               compatible = "regulator-fixed";
+               regulator-name = "fixed-1.8V";
+               regulator-min-microvolt = <1800000>;
+               regulator-max-microvolt = <1800000>;
+               regulator-boot-on;
+               regulator-always-on;
+       };
+
        fixedregulator3v3: regulator-3v3 {
                compatible = "regulator-fixed";
                regulator-name = "fixed-3.3V";
                        clocks = <&cec_clock>;
                        clock-names = "cec";
 
+                       avdd-supply = <&fixedregulator1v8>;
+                       dvdd-supply = <&fixedregulator1v8>;
+                       pvdd-supply = <&fixedregulator1v8>;
+                       dvdd-3v-supply = <&fixedregulator3v3>;
+                       bgvdd-supply = <&fixedregulator1v8>;
+
                        adi,input-depth = <8>;
                        adi,input-colorspace = "rgb";
                        adi,input-clock = "1x";
index fe14727..f9bc5b4 100644 (file)
                };
        };
 
+       fixedregulator1v8: regulator-1v8 {
+               compatible = "regulator-fixed";
+               regulator-name = "fixed-1.8V";
+               regulator-min-microvolt = <1800000>;
+               regulator-max-microvolt = <1800000>;
+               regulator-boot-on;
+               regulator-always-on;
+       };
+
        fixedregulator3v3: regulator-3v3 {
                compatible = "regulator-fixed";
                regulator-name = "fixed-3.3V";
                clocks = <&osc4_clk>;
                clock-names = "cec";
 
+               avdd-supply = <&fixedregulator1v8>;
+               dvdd-supply = <&fixedregulator1v8>;
+               pvdd-supply = <&fixedregulator1v8>;
+               dvdd-3v-supply = <&fixedregulator3v3>;
+               bgvdd-supply = <&fixedregulator1v8>;
+
                adi,input-depth = <8>;
                adi,input-colorspace = "rgb";
                adi,input-clock = "1x";
index 63ca025..e9c13bb 100644 (file)
                };
        };
 
+       reg_1p8v: regulator-1v8 {
+               compatible = "regulator-fixed";
+               regulator-name = "fixed-1.8V";
+               regulator-min-microvolt = <1800000>;
+               regulator-max-microvolt = <1800000>;
+               regulator-boot-on;
+               regulator-always-on;
+       };
+
+       reg_3p3v: regulator-3v3 {
+               compatible = "regulator-fixed";
+               regulator-name = "fixed-3.3V";
+               regulator-min-microvolt = <3300000>;
+               regulator-max-microvolt = <3300000>;
+               regulator-boot-on;
+               regulator-always-on;
+       };
+
        vcc_sdhi0: regulator-vcc-sdhi0 {
                compatible = "regulator-fixed";
 
                        clocks = <&cec_clock>;
                        clock-names = "cec";
 
+                       avdd-supply = <&reg_1p8v>;
+                       dvdd-supply = <&reg_1p8v>;
+                       pvdd-supply = <&reg_1p8v>;
+                       dvdd-3v-supply = <&reg_3p3v>;
+                       bgvdd-supply = <&reg_1p8v>;
+
                        adi,input-depth = <8>;
                        adi,input-colorspace = "rgb";
                        adi,input-clock = "1x";
index 65fd462..7e8bc06 100644 (file)
                reg = <2 0x00000000 0 0x40000000>;
        };
 
+       reg_1p8v: regulator-1v8 {
+               compatible = "regulator-fixed";
+               regulator-name = "fixed-1.8V";
+               regulator-min-microvolt = <1800000>;
+               regulator-max-microvolt = <1800000>;
+               regulator-boot-on;
+               regulator-always-on;
+       };
+
+       reg_3p3v: regulator-3v3 {
+               compatible = "regulator-fixed";
+               regulator-name = "fixed-3.3V";
+               regulator-min-microvolt = <3300000>;
+               regulator-max-microvolt = <3300000>;
+               regulator-boot-on;
+               regulator-always-on;
+       };
+
        vcc_sdhi0: regulator-vcc-sdhi0 {
                compatible = "regulator-fixed";
 
                        interrupt-parent = <&gpio3>;
                        interrupts = <29 IRQ_TYPE_LEVEL_LOW>;
 
+                       avdd-supply = <&reg_1p8v>;
+                       dvdd-supply = <&reg_1p8v>;
+                       pvdd-supply = <&reg_1p8v>;
+                       dvdd-3v-supply = <&reg_3p3v>;
+                       bgvdd-supply = <&reg_1p8v>;
+
                        adi,input-depth = <8>;
                        adi,input-colorspace = "rgb";
                        adi,input-clock = "1x";
index 4ee83b8..4f9838c 100644 (file)
                reg = <0 0x40000000 0 0x40000000>;
        };
 
+       d1_8v: regulator-1v8 {
+               compatible = "regulator-fixed";
+               regulator-name = "D1.8V";
+               regulator-min-microvolt = <1800000>;
+               regulator-max-microvolt = <1800000>;
+               regulator-boot-on;
+               regulator-always-on;
+       };
+
        d3_3v: regulator-3v3 {
                compatible = "regulator-fixed";
                regulator-name = "D3.3V";
                interrupt-parent = <&irqc>;
                interrupts = <3 IRQ_TYPE_EDGE_FALLING>;
 
+               avdd-supply = <&d1_8v>;
+               dvdd-supply = <&d1_8v>;
+               pvdd-supply = <&d1_8v>;
+               dvdd-3v-supply = <&d3_3v>;
+               bgvdd-supply = <&d1_8v>;
+
                adi,input-depth = <8>;
                adi,input-colorspace = "rgb";
                adi,input-clock = "1x";
index f87e78f..000f21a 100644 (file)
                reg = <0 0x40000000 0 0x40000000>;
        };
 
+       d1_8v: regulator-1v8 {
+               compatible = "regulator-fixed";
+               regulator-name = "D1.8V";
+               regulator-min-microvolt = <1800000>;
+               regulator-max-microvolt = <1800000>;
+               regulator-boot-on;
+               regulator-always-on;
+       };
+
        d3_3v: regulator-3v3 {
                compatible = "regulator-fixed";
                regulator-name = "D3.3V";
                reg = <0x3d>, <0x4d>, <0x2d>, <0x5d>;
                reg-names = "main", "edid", "cec", "packet";
 
+               avdd-supply = <&d1_8v>;
+               dvdd-supply = <&d1_8v>;
+               pvdd-supply = <&d1_8v>;
+               dvdd-3v-supply = <&d3_3v>;
+               bgvdd-supply = <&d1_8v>;
+
                adi,input-depth = <8>;
                adi,input-colorspace = "rgb";
                adi,input-clock = "1x";
                reg = <0x39>, <0x49>, <0x29>, <0x59>;
                reg-names = "main", "edid", "cec", "packet";
 
+               avdd-supply = <&d1_8v>;
+               dvdd-supply = <&d1_8v>;
+               pvdd-supply = <&d1_8v>;
+               dvdd-3v-supply = <&d3_3v>;
+               bgvdd-supply = <&d1_8v>;
+
                adi,input-depth = <8>;
                adi,input-colorspace = "rgb";
                adi,input-clock = "1x";
index 58e2670..1744fdb 100644 (file)
                };
        };
 
+       reg_1p8v: regulator-1v8 {
+               compatible = "regulator-fixed";
+               regulator-name = "fixed-1.8V";
+               regulator-min-microvolt = <1800000>;
+               regulator-max-microvolt = <1800000>;
+               regulator-boot-on;
+               regulator-always-on;
+       };
+
+       reg_3p3v: regulator-3v3 {
+               compatible = "regulator-fixed";
+               regulator-name = "fixed-3.3V";
+               regulator-min-microvolt = <3300000>;
+               regulator-max-microvolt = <3300000>;
+               regulator-boot-on;
+               regulator-always-on;
+       };
+
        vcc_sdhi0: regulator-vcc-sdhi0 {
                compatible = "regulator-fixed";
 
                        interrupt-parent = <&gpio3>;
                        interrupts = <29 IRQ_TYPE_LEVEL_LOW>;
 
+                       avdd-supply = <&reg_1p8v>;
+                       dvdd-supply = <&reg_1p8v>;
+                       pvdd-supply = <&reg_1p8v>;
+                       dvdd-3v-supply = <&reg_3p3v>;
+                       bgvdd-supply = <&reg_1p8v>;
+
                        adi,input-depth = <8>;
                        adi,input-colorspace = "rgb";
                        adi,input-clock = "1x";
index 01fb06d..43d480a 100644 (file)
                };
        };
 
+       d1_8v: regulator-d1-8v {
+               compatible = "regulator-fixed";
+               regulator-name = "D1.8V";
+               regulator-min-microvolt = <1800000>;
+               regulator-max-microvolt = <1800000>;
+               regulator-boot-on;
+               regulator-always-on;
+       };
+
        d3_3v: regulator-d3-3v {
                compatible = "regulator-fixed";
                regulator-name = "D3.3V";
                        interrupt-parent = <&gpio5>;
                        interrupts = <23 IRQ_TYPE_LEVEL_LOW>;
 
+                       avdd-supply = <&d1_8v>;
+                       dvdd-supply = <&d1_8v>;
+                       pvdd-supply = <&d1_8v>;
+                       dvdd-3v-supply = <&d3_3v>;
+                       bgvdd-supply = <&d1_8v>;
+
                        adi,input-depth = <8>;
                        adi,input-colorspace = "rgb";
                        adi,input-clock = "1x";