Re: [PATCH 3/4] arm64: dts: marvell: Clean up armada-7040-db

On Thu, Mar 24, 2016 at 02:37:23PM +0100, Andrew Lunn wrote: > > > I agree that converting the Marvell Armada 32-bits SoCs would produce a > > > lot of churn. But if some binding are common there is no file at all are > > > in common, so we could use this solution for the 64 bits SoCs only. > > >

Re: [PATCH 3/4] arm64: dts: marvell: Clean up armada-7040-db

On Thu, Mar 24, 2016 at 02:37:23PM +0100, Andrew Lunn wrote: > > > I agree that converting the Marvell Armada 32-bits SoCs would produce a > > > lot of churn. But if some binding are common there is no file at all are > > > in common, so we could use this solution for the 64 bits SoCs only. > > >

Re: [PATCH 3/4] arm64: dts: marvell: Clean up armada-7040-db

> > I agree that converting the Marvell Armada 32-bits SoCs would produce a > > lot of churn. But if some binding are common there is no file at all are > > in common, so we could use this solution for the 64 bits SoCs only. > > Yes, we could. I'm fine with it. I was merely pointing out that it

Re: [PATCH 3/4] arm64: dts: marvell: Clean up armada-7040-db

> > I agree that converting the Marvell Armada 32-bits SoCs would produce a > > lot of churn. But if some binding are common there is no file at all are > > in common, so we could use this solution for the 64 bits SoCs only. > > Yes, we could. I'm fine with it. I was merely pointing out that it

Re: [PATCH 3/4] arm64: dts: marvell: Clean up armada-7040-db

Hello, On Thu, 24 Mar 2016 11:27:27 +0100, Gregory CLEMENT wrote: > > We haven't used this solution on Marvell Armada 32-bits SoCs, so there > > needs to be a discussion on whether we want to go in this direction for > > the 64 bits SoCs. > > At first view using the label helps to have simpler

Re: [PATCH 3/4] arm64: dts: marvell: Clean up armada-7040-db

Hello, On Thu, 24 Mar 2016 11:27:27 +0100, Gregory CLEMENT wrote: > > We haven't used this solution on Marvell Armada 32-bits SoCs, so there > > needs to be a discussion on whether we want to go in this direction for > > the 64 bits SoCs. > > At first view using the label helps to have simpler

Re: [PATCH 3/4] arm64: dts: marvell: Clean up armada-7040-db

Hi Thomas, On jeu., mars 24 2016, Thomas Petazzoni wrote: > Hello, > > On Wed, 23 Mar 2016 23:24:20 +0100, Andreas Färber wrote: >> Instead of duplicating the node hierarchy, reference the nodes by label, >> adding labels where necessary. >> >> Drop some

Re: [PATCH 3/4] arm64: dts: marvell: Clean up armada-7040-db

Hi Thomas, On jeu., mars 24 2016, Thomas Petazzoni wrote: > Hello, > > On Wed, 23 Mar 2016 23:24:20 +0100, Andreas Färber wrote: >> Instead of duplicating the node hierarchy, reference the nodes by label, >> adding labels where necessary. >> >> Drop some trailing or inconsistent white lines

Re: [PATCH 3/4] arm64: dts: marvell: Clean up armada-7040-db

Hello, On Wed, 23 Mar 2016 23:24:20 +0100, Andreas Färber wrote: > Instead of duplicating the node hierarchy, reference the nodes by label, > adding labels where necessary. > > Drop some trailing or inconsistent white lines while at it. > > Fixes: ec7e5a569bce ("arm64: dts: marvell: add Device

Re: [PATCH 3/4] arm64: dts: marvell: Clean up armada-7040-db

Hello, On Wed, 23 Mar 2016 23:24:20 +0100, Andreas Färber wrote: > Instead of duplicating the node hierarchy, reference the nodes by label, > adding labels where necessary. > > Drop some trailing or inconsistent white lines while at it. > > Fixes: ec7e5a569bce ("arm64: dts: marvell: add Device

[PATCH 3/4] arm64: dts: marvell: Clean up armada-7040-db

Instead of duplicating the node hierarchy, reference the nodes by label, adding labels where necessary. Drop some trailing or inconsistent white lines while at it. Fixes: ec7e5a569bce ("arm64: dts: marvell: add Device Tree files for Armada 7K/8K") Cc: Thomas Petazzoni

[PATCH 3/4] arm64: dts: marvell: Clean up armada-7040-db

Instead of duplicating the node hierarchy, reference the nodes by label, adding labels where necessary. Drop some trailing or inconsistent white lines while at it. Fixes: ec7e5a569bce ("arm64: dts: marvell: add Device Tree files for Armada 7K/8K") Cc: Thomas Petazzoni Signed-off-by: Andreas