Messages in this thread | | | From | Rob Herring <> | Date | Tue, 2 Jun 2020 15:44:32 -0600 | Subject | Re: [PATCH v25 01/16] dt: bindings: Add multicolor class dt bindings documention |
| |
On Tue, Jun 2, 2020 at 2:04 PM Pavel Machek <pavel@ucw.cz> wrote: > > On Wed 2020-05-27 08:35:06, Rob Herring wrote: > > On Wed, May 27, 2020 at 7:39 AM Pavel Machek <pavel@ucw.cz> wrote: > > > > > > Hi! > > > > > > Thanks for reviews! > > > > > > > > +additionalProperties: false > > > > > +... > > > > > diff --git a/drivers/leds/led-core.c b/drivers/leds/led-core.c > > > > > > > > This isn't a binding file. Belongs in another patch. > > > > > > These constants are directly related to the binding. It makes sense to > > > go in one patch... > > > > Yes, the header does go in this patch, but kernel subsystem files do not. > > > > Part of the reason for separating is we generate a DT only repository > > which filters out all the kernel code. Ideally this is just filtering > > out commits and the commit messages still make sens > > Well, but the patch can't be split like that. Otherwise we risk null > pointer dereferences when one part is applied but not the second one.
There's no risk because you are supposed to apply both patches. I don't apply binding patches that are a part of a series like this.
Rob
| |