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

From: Uwe Kleine-KÃnig
Date: Mon Nov 28 2016 - 05:54:48 EST


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


Attachment: signature.asc
Description: OpenPGP digital signature