ARM: dts: Group omap3 CM_FCLKEN_WKUP clocks
authorTony Lindgren <tony@atomide.com>
Fri, 29 Apr 2022 06:57:36 +0000 (09:57 +0300)
committerTony Lindgren <tony@atomide.com>
Tue, 3 May 2022 06:15:42 +0000 (09:15 +0300)
The clksel related registers on omap3 cause unique_unit_address and
node_name_chars_strict warnings with the W=1 or W=2 make flags enabled.

With the clock drivers updated, we can now avoid most of these warnings
by grouping the TI component clocks using the TI clksel binding, and
with the use of clock-output-names property to avoid non-standard node
names for the clocks.

Signed-off-by: Tony Lindgren <tony@atomide.com>
arch/arm/boot/dts/omap34xx-omap36xx-clocks.dtsi
arch/arm/boot/dts/omap36xx-omap3430es2plus-clocks.dtsi
arch/arm/boot/dts/omap3xxx-clocks.dtsi

index b6316be894ce0266434e0b1b9b22a8c5ad31d491..edc28654d158aac8874cc90710a44d939a3f9edc 100644 (file)
                clock-div = <1>;
        };
 
-       sr1_fck: sr1_fck@c00 {
-               #clock-cells = <0>;
-               compatible = "ti,wait-gate-clock";
-               clocks = <&sys_ck>;
-               reg = <0x0c00>;
-               ti,bit-shift = <6>;
-       };
+       clock@c00 {
+               compatible = "ti,clksel";
+               reg = <0xc00>;
+               #clock-cells = <2>;
+               #address-cells = <0>;
 
-       sr2_fck: sr2_fck@c00 {
-               #clock-cells = <0>;
-               compatible = "ti,wait-gate-clock";
-               clocks = <&sys_ck>;
-               reg = <0x0c00>;
-               ti,bit-shift = <7>;
+               sr1_fck: clock-sr1-fck {
+                       #clock-cells = <0>;
+                       compatible = "ti,wait-gate-clock";
+                       clock-output-names = "sr1_fck";
+                       clocks = <&sys_ck>;
+                       ti,bit-shift = <6>;
+               };
+
+               sr2_fck: clock-sr2-fck {
+                       #clock-cells = <0>;
+                       compatible = "ti,wait-gate-clock";
+                       clock-output-names = "sr2_fck";
+                       clocks = <&sys_ck>;
+                       ti,bit-shift = <7>;
+               };
        };
 
        sr_l4_ick: sr_l4_ick {
index c6ab51a36f393ea16bed999d6e6f489a25d73b98..7c25739d561272450e45cbb3e092688016326219 100644 (file)
                clock-div = <1>;
        };
 
-       usim_gate_fck: usim_gate_fck@c00 {
-               #clock-cells = <0>;
-               compatible = "ti,composite-gate-clock";
-               clocks = <&omap_96m_fck>;
-               ti,bit-shift = <9>;
-               reg = <0x0c00>;
+       clock@c00 {
+               compatible = "ti,clksel";
+               reg = <0xc00>;
+               #clock-cells = <2>;
+               #address-cells = <0>;
+
+               usim_gate_fck: clock-usim-gate-fck {
+                       #clock-cells = <0>;
+                       compatible = "ti,composite-gate-clock";
+                       clock-output-names = "usim_gate_fck";
+                       clocks = <&omap_96m_fck>;
+                       ti,bit-shift = <9>;
+               };
        };
 
        sys_d2_ck: sys_d2_ck {
index 430dca208dc840f357ada9504ff62c7ad1d0d260..2cb77eb15eae0244b1beac5854bb9d20785e389f 100644 (file)
                clock-frequency = <0>;
        };
 
-       gpt1_gate_fck: gpt1_gate_fck@c00 {
-               #clock-cells = <0>;
-               compatible = "ti,composite-gate-clock";
-               clocks = <&sys_ck>;
-               ti,bit-shift = <0>;
-               reg = <0x0c00>;
+       /* CM_FCLKEN_WKUP */
+       clock@c00 {
+               compatible = "ti,clksel";
+               reg = <0xc00>;
+               #clock-cells = <2>;
+               #address-cells = <0>;
+
+               gpt1_gate_fck: clock-gpt1-gate-fck {
+                       #clock-cells = <0>;
+                       compatible = "ti,composite-gate-clock";
+                       clock-output-names = "gpt1_gate_fck";
+                       clocks = <&sys_ck>;
+                       ti,bit-shift = <0>;
+               };
+
+               gpio1_dbck: clock-gpio1-dbck {
+                       #clock-cells = <0>;
+                       compatible = "ti,gate-clock";
+                       clock-output-names = "gpio1_dbck";
+                       clocks = <&wkup_32k_fck>;
+                       ti,bit-shift = <3>;
+               };
+
+               wdt2_fck: clock-wdt2-fck {
+                       #clock-cells = <0>;
+                       compatible = "ti,wait-gate-clock";
+                       clock-output-names = "wdt2_fck";
+                       clocks = <&wkup_32k_fck>;
+                       ti,bit-shift = <5>;
+               };
        };
 
        gpt1_mux_fck: gpt1_mux_fck@c40 {
                clock-div = <1>;
        };
 
-       gpio1_dbck: gpio1_dbck@c00 {
-               #clock-cells = <0>;
-               compatible = "ti,gate-clock";
-               clocks = <&wkup_32k_fck>;
-               reg = <0x0c00>;
-               ti,bit-shift = <3>;
-       };
-
-       wdt2_fck: wdt2_fck@c00 {
-               #clock-cells = <0>;
-               compatible = "ti,wait-gate-clock";
-               clocks = <&wkup_32k_fck>;
-               reg = <0x0c00>;
-               ti,bit-shift = <5>;
-       };
-
        wdt2_ick: wdt2_ick@c10 {
                #clock-cells = <0>;
                compatible = "ti,omap3-interface-clock";