Re: [PATCH v2 16/16] ARM: dts: r8a7794: sort subnodes of root node

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jan 18, 2018 at 03:20:00AM +0100, Niklas Söderlund wrote:
> Hi Simon,
> 
> Thanks for your patch.
> 
> On 2018-01-17 17:17:17 +0100, Simon Horman wrote:
> > Sort the subnodes of the soc node to improve maintainability.
> > The sort key is the address on the bus with instances of the same
> > IP block grouped together.
> > 
> > This patch should not introduce any functional change.
> > 
> > Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx>
> 
> This patch also aligns the comment for audio_clka. This is good but not 
> mentioned in the commit message which confused the review bot, i.e me :-)

Sorry, its probably worth adding that to the changelog.

> Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>
> 
> > ---
> > v2
> > * New patch
> > ---
> >  arch/arm/boot/dts/r8a7794.dtsi | 90 +++++++++++++++++++++---------------------
> >  1 file changed, 45 insertions(+), 45 deletions(-)
> > 
> > diff --git a/arch/arm/boot/dts/r8a7794.dtsi b/arch/arm/boot/dts/r8a7794.dtsi
> > index 9e5f886f53c5..d588efa6aeaa 100644
> > --- a/arch/arm/boot/dts/r8a7794.dtsi
> > +++ b/arch/arm/boot/dts/r8a7794.dtsi
> > @@ -33,6 +33,35 @@
> >  		vin1 = &vin1;
> >  	};
> >  
> > +	/*
> > +	 * The external audio clocks are configured as 0 Hz fixed frequency
> > +	 * clocks by default.
> > +	 * Boards that provide audio clocks should override them.
> > +	 */
> > +	audio_clka: audio_clka {
> > +		compatible = "fixed-clock";
> > +		#clock-cells = <0>;
> > +		clock-frequency = <0>;
> > +	};
> > +	audio_clkb: audio_clkb {
> > +		compatible = "fixed-clock";
> > +		#clock-cells = <0>;
> > +		clock-frequency = <0>;
> > +	};
> > +	audio_clkc: audio_clkc {
> > +		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>;
> > @@ -66,6 +95,22 @@
> >  		};
> >  	};
> >  
> > +	/* External root clock */
> > +	extal_clk: extal {
> > +		compatible = "fixed-clock";
> > +		#clock-cells = <0>;
> > +		/* This value must be overridden by the board. */
> > +		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>;
> > @@ -1353,55 +1398,10 @@
> >  				      <&gic GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(2) | 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 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>;
> > -	};
> > -
> > -	/* External SCIF clock */
> > -	scif_clk: scif {
> > -		compatible = "fixed-clock";
> > -		#clock-cells = <0>;
> > -		/* This value must be overridden by the board. */
> > -		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_clka: audio_clka {
> > -		compatible = "fixed-clock";
> > -		#clock-cells = <0>;
> > -		clock-frequency = <0>;
> > -	};
> > -	audio_clkb: audio_clkb {
> > -		compatible = "fixed-clock";
> > -		#clock-cells = <0>;
> > -		clock-frequency = <0>;
> > -	};
> > -	audio_clkc: audio_clkc {
> > -		compatible = "fixed-clock";
> > -		#clock-cells = <0>;
> > -		clock-frequency = <0>;
> > -	};
> >  };
> > -- 
> > 2.11.0
> > 
> 
> -- 
> Regards,
> Niklas Söderlund
> 



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux