Messages in this thread | | | Subject | Re: [PATCH v5 6/8] Documentation: bindings: add compatible specific to legacy SCPI protocol | From | Sudeep Holla <> | Date | Tue, 15 Nov 2016 16:36:34 +0000 |
| |
On 11/11/16 14:19, Sudeep Holla wrote: > > > On 11/11/16 13:34, Rob Herring wrote: >> On Fri, Nov 11, 2016 at 1:48 AM, Sudeep Holla <sudeep.holla@arm.com> >> wrote:
[...]
>>> >>> True and I agree, how about "arm,scpi-pre-1.0" instead ? >> >> That's still meaningless. Convince me that multiple implementations >> are identical, then we can have a common property. For example, how >> many releases did ARM make before 1.0. >> > > None officially, so I tend to agree with you on this. > > But so far we have seen some commonality between Rockchip and Amlogic > implementations, which in fact shares some commonality with early > release of SCPI from ARM (there are based on the same SCP code base, > which is closed source and released to partners only). ARM improved the > specification and the code base before the official release but by then > it was adopted(as usual we were late ;)) > > IMO, it's might be useful to have more generic say "arm,scpi-pre-1.0" > and platform specific "amlogic,meson-gxbb-scpi" >
Rob and Olof, is it convincing enough reason to have generic compatible? Or you prefer to drop it ?
I prefer to have "arm,scpi-pre-1.0". IMO it's useful, let me know. I need to send PR as it's getting late now.
-- Regards, Sudeep
| |