Hi Simon, Thanks for your patch.
On 2018-01-17 17:17:05 +0100, Simon Horman wrote: > Sort subnodes of root node to aid maintenance. > > This patch should not introduce any functional change. > > Signed-off-by: Simon Horman <[email protected]> Reviewed-by: Niklas Söderlund <[email protected]> > --- > v2 > * New patch > --- > arch/arm/boot/dts/r8a7790.dtsi | 104 > ++++++++++++++++++++--------------------- > 1 file changed, 52 insertions(+), 52 deletions(-) > > diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi > index 54c5a2d7ea89..3bbcc0b93f1c 100644 > --- a/arch/arm/boot/dts/r8a7790.dtsi > +++ b/arch/arm/boot/dts/r8a7790.dtsi > @@ -40,6 +40,35 @@ > vin3 = &vin3; > }; > > + /* > + * The external audio clocks are configured as 0 Hz fixed frequency > + * clocks by default. > + * Boards that provide audio clocks should override them. > + */ > + audio_clk_a: audio_clk_a { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + clock-frequency = <0>; > + }; > + audio_clk_b: audio_clk_b { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + clock-frequency = <0>; > + }; > + audio_clk_c: audio_clk_c { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + clock-frequency = <0>; > + }; > + > + /* External CAN clock */ > + can_clk: can { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + /* This value must be overridden by the board. */ > + clock-frequency = <0>; > + }; > + > cpus { > #address-cells = <1>; > #size-cells = <0>; > @@ -158,6 +187,29 @@ > }; > }; > > + /* External root clock */ > + extal_clk: extal { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + /* This value must be overridden by the board. */ > + clock-frequency = <0>; > + }; > + > + /* External PCIe clock - can be overridden by the board */ > + pcie_bus_clk: pcie_bus { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + clock-frequency = <0>; > + }; > + > + /* External SCIF clock */ > + scif_clk: scif { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + /* This value must be overridden by the board. */ > + clock-frequency = <0>; > + }; > + > soc { > compatible = "simple-bus"; > interrupt-parent = <&gic>; > @@ -1678,62 +1730,10 @@ > <&gic GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) > | IRQ_TYPE_LEVEL_LOW)>; > }; > > - /* External root clock */ > - extal_clk: extal { > - compatible = "fixed-clock"; > - #clock-cells = <0>; > - /* This value must be overridden by the board. */ > - clock-frequency = <0>; > - }; > - > - /* External PCIe clock - can be overridden by the board */ > - pcie_bus_clk: pcie_bus { > - compatible = "fixed-clock"; > - #clock-cells = <0>; > - clock-frequency = <0>; > - }; > - > - /* > - * The external audio clocks are configured as 0 Hz fixed frequency > - * clocks by default. > - * Boards that provide audio clocks should override them. > - */ > - audio_clk_a: audio_clk_a { > - compatible = "fixed-clock"; > - #clock-cells = <0>; > - clock-frequency = <0>; > - }; > - audio_clk_b: audio_clk_b { > - compatible = "fixed-clock"; > - #clock-cells = <0>; > - clock-frequency = <0>; > - }; > - audio_clk_c: audio_clk_c { > - compatible = "fixed-clock"; > - #clock-cells = <0>; > - clock-frequency = <0>; > - }; > - > - /* External SCIF clock */ > - scif_clk: scif { > - compatible = "fixed-clock"; > - #clock-cells = <0>; > - /* This value must be overridden by the board. */ > - clock-frequency = <0>; > - }; > - > /* External USB clock - can be overridden by the board */ > usb_extal_clk: usb_extal { > compatible = "fixed-clock"; > #clock-cells = <0>; > clock-frequency = <48000000>; > }; > - > - /* External CAN clock */ > - can_clk: can { > - compatible = "fixed-clock"; > - #clock-cells = <0>; > - /* This value must be overridden by the board. */ > - clock-frequency = <0>; > - }; > }; > -- > 2.11.0 > -- Regards, Niklas Söderlund
