On 31/03/2024 13:58, Peng Fan wrote: >>> + const: 1 >>> + description: >>> + The clock consumer should specify the desired clock by having the >> clock >>> + ID in its "clocks" phandle cell. See >>> + include/dt-bindings/clock/nxp,imx95-clock.h >> >> In such case, put header as your first patch in the patchset. I don't understand >> why it was split in the first place... > > Rob gave a comment in v4, so I split the headers. > " > If this number can change, then it is not ABI and doesn't go in this > header. With that dropped, > " Nothing here speaks about splitting headers. Absolutely NOTHING. Rob commented that you added number which can change, thus this is not a binding. Best regards, Krzysztof