On 7/1/21 6:07 PM, Rob Herring wrote: > On Mon, Jun 28, 2021 at 10:03 AM Daniel Mack <daniel@xxxxxxxxxx> wrote: >> >> On 6/24/21 11:33 PM, Rob Herring wrote: >>> On Thu, Jun 17, 2021 at 09:09:06PM +0200, Daniel Mack wrote: >>>> CLK_IN is only used in dynamic mode and is hence optional. >>>> >>>> Re-order the clocks so REF_CLK is specified first. >>> >>> It's not a compatible change. You can't do that unless you can explain >>> why it doesn't matter. >> >> Hmm okay. And that rule also applies even if I patch all mainline users? > > Explaining why you are doing something and the implications of the > change, yes that's required for every commit. Whether you can make > incompatible changes, that's up to platform maintainers. I only ask > that doing that is documented clearly. > > Do you know that every user of affected h/w is okay with you requiring > them update their dtb? Alright, I'll rework that to use a new property then for mode selection. and repost. Thanks, Daniel
