On Wed, 2018-10-24 at 10:11 +0800, andy.tang@xxxxxxx wrote: > From: Yuantian Tang <andy.tang@xxxxxxx> > > 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
