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

Groups > linux.kernel > #1531204

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

From Uwe Kleine-König <uwe@kleine-koenig.org>
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 <sIrPc-12d-19@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> <sIrPb-12d-11@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

Hello,

On 11/28/2016 11:52 AM, Andreas Färber wrote:
>> 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?

I'd even go as far as:

	1: add labels to .dtsi
	2: use labels on .dts#1
	3: use labels on .dts#2
	...

Best regards
Uwe

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