Messages in this thread | | | Subject | Re: [PATCH] ARM: dts: mvebu: Add Armada 38x labels and clean up Turris Omnia | From | Andreas Färber <> | Date | Mon, 28 Nov 2016 11:58:31 +0100 |
| |
Hi,
Am 28.11.2016 um 11:54 schrieb Uwe Kleine-König: > On 11/28/2016 11:52 AM, Andreas Färber wrote: >> 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 > ...
That was what I had in mind. :) I even considered reusing the existing labels first, then adding more and converting more nodes.
Making the patches smaller will hopefully make them more easily reviewable at the same time.
Cheers, Andreas
-- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg)
| |