[PATCH 3/3 v2] clk: qoriq: update clock driver
Scott Wood
oss at buserror.net
Thu Oct 25 05:37:28 AEDT 2018
On Wed, 2018-10-24 at 10:11 +0800, andy.tang at nxp.com wrote:
> From: Yuantian Tang <andy.tang at nxp.com>
>
> Legacy bindings are deleted. So the legacy support in driver
> can be deleted safely.
NACK (both this and 2/3). The legacy support is intended to preserve
compatibility, regardless of what the dts files in the current kernel tree do.
If years later we find it's been broken for a while and nobody complained,
then maybe it'll be time to remove it, but why deliberately throw away
compatibility the instant the users have been removed from reference DTs that
might be copied by board vendors, etc?
Note that even if we didn't care about long-term compatibility at all,
removing the support in the same patchset as the change to the dts files means
that the patches can't go in via separate trees (though if that's still the
intent, you should make it clear who you're asking to take what by putting
them in separate patchsets).
-Scott
More information about the Linuxppc-dev
mailing list