ARM: dts: exynos: move DP and MIPI phys to PMU node in Exynos5250
authorKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Fri, 27 Jan 2023 19:40:56 +0000 (20:40 +0100)
committerKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Mon, 6 Mar 2023 11:33:32 +0000 (12:33 +0100)
The DisplayPort and MIPI phys are actually part of the Power Management
Unit system controller.  They do not have their own address space, thus
keeping the nodes under soc causes warnings:

  exynos5250-arndale.dtb: soc: video-phy-0: {'compatible': ['samsung,exynos5250-dp-video-phy'],
    'samsung,pmu-syscon': [[20]], '#phy-cells': [[0]], 'phandle': [[24]]} should not be valid under {'type': 'object'}

Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>
Link: https://lore.kernel.org/r/20230127194057.186458-8-krzysztof.kozlowski@linaro.org
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
arch/arm/boot/dts/exynos5250.dtsi

index 60a623e3a200352e954e8fb5a28d466d9dfaa23b..89c8665ac9aacd28eb0329ebe4ab4e8ba3b4f44f 100644 (file)
                };
 
                pmu_system_controller: system-controller@10040000 {
-                       compatible = "samsung,exynos5250-pmu", "syscon";
+                       compatible = "samsung,exynos5250-pmu", "simple-mfd", "syscon";
                        reg = <0x10040000 0x5000>;
                        clock-names = "clkout16";
                        clocks = <&clock CLK_FIN_PLL>;
                        interrupt-controller;
                        #interrupt-cells = <3>;
                        interrupt-parent = <&gic>;
+
+                       dp_phy: dp-phy {
+                               compatible = "samsung,exynos5250-dp-video-phy";
+                               #phy-cells = <0>;
+                       };
+
+                       mipi_phy: mipi-phy {
+                               compatible = "samsung,s5pv210-mipi-video-phy";
+                               #phy-cells = <1>;
+                       };
                };
 
                watchdog@101d0000 {
                        status = "disabled";
                };
 
-               dp_phy: video-phy-0 {
-                       compatible = "samsung,exynos5250-dp-video-phy";
-                       samsung,pmu-syscon = <&pmu_system_controller>;
-                       #phy-cells = <0>;
-               };
-
-               mipi_phy: video-phy-1 {
-                       compatible = "samsung,s5pv210-mipi-video-phy";
-                       #phy-cells = <1>;
-                       syscon = <&pmu_system_controller>;
-               };
-
                dsi_0: dsi@14500000 {
                        compatible = "samsung,exynos4210-mipi-dsi";
                        reg = <0x14500000 0x10000>;