Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1531199

Re: [PATCH] ARM: dts: mvebu: Add Armada 38x labels and clean up Turris Omnia

From Andreas Färber <afaerber@suse.de>
Newsgroups linux.kernel
Subject Re: [PATCH] ARM: dts: mvebu: Add Armada 38x labels and clean up Turris Omnia
Date 2016-11-28 12:00 +0100
Message-ID <sIrPb-12d-11@gated-at.bofh.it> (permalink)
References <sIcQ9-8az-5@gated-at.bofh.it> <sIcQ9-8az-3@gated-at.bofh.it> <sIrFv-YK-15@gated-at.bofh.it>
Organization SUSE Linux GmbH

Show all headers | View raw

Hi Russell,

Am 28.11.2016 um 11:37 schrieb Russell King - ARM Linux:
> On Sun, Nov 27, 2016 at 07:51:39PM +0100, Andreas Färber wrote:
>> To more consistently reference nodes by label, add labels for sata,
>> usb2, sdhci and usb3 nodes.
>>
>> Convert all other 38x boards for consistency. Add labels for nfc and rtc.
> 
> Please don't do this for clearfog - there's changes in the pipeline which
> completely replace armada-388-clearfog.dts because there's a "base" and
> "pro" versions of this hardware now, and making such a huge change will
> effectively mean we have to start over with the DT files.

Would it help to split it back up into a series of add-labels,
use-labels like I had originally? Then you could start using them in
your refactoring as soon as the add-labels patch gets applied. Or are
you completely against this style?

Thanks for pointing this out,

Andreas

-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH] ARM: dts: mvebu: Add Armada 38x labels and clean up  Turris Omnia Russell King - ARM Linux <linux@armlinux.org.uk> - 2016-11-28 11:50 +0100
  Re: [PATCH] ARM: dts: mvebu: Add Armada 38x labels and clean up  Turris Omnia Andreas Färber <afaerber@suse.de> - 2016-11-28 12:00 +0100
    Re: [PATCH] ARM: dts: mvebu: Add Armada 38x labels and clean up  Turris Omnia Russell King - ARM Linux <linux@armlinux.org.uk> - 2016-11-28 12:00 +0100
    Re: [PATCH] ARM: dts: mvebu: Add Armada 38x labels and clean up  Turris Omnia Uwe Kleine-König <uwe@kleine-koenig.org> - 2016-11-28 12:00 +0100
      Re: [PATCH] ARM: dts: mvebu: Add Armada 38x labels and clean up  Turris Omnia Andreas Färber <afaerber@suse.de> - 2016-11-28 12:00 +0100

csiph-web