Am Montag, den 22.01.2018, 10:28 -0600 schrieb Rob Herring: > On Fri, Jan 19, 2018 at 6:06 AM, Lucas Stach <l.stach@xxxxxxxxxxxxxx> > wrote: > > While the clocks were documented as required, the driver always > > treated them > > as optional and there are existing Marvell Dove DTs, which would > > break if > > changed to required. Accept reality and document the clocks as > > optional. > > The fact that clocks are optional for a driver doesn't mean they are > optional for the binding. Now they could be optional because you have > power-domains and all clock management is done within the power > domain > (though that too is driver specifics leaking into the binding because > power domain has come to mean power management domain). So you prefer the binding to stay as-is and accept the Dove DTs as non- compliant to the bindings, but still okay? That's fine with me and I'll drop that patch in this case. Regards, Lucas -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html