Re: [PATCH v5 6/8] Documentation: bindings: add compatible specific to legacy SCPI protocol

From: Sudeep Holla
Date: Tue Nov 15 2016 - 11:36:41 EST




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@xxxxxxx>
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