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

From	Sudeep Holla <sudeep.holla@arm.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v5 6/8] Documentation: bindings: add compatible specific to legacy SCPI protocol
Date	2016-11-15 17:40 +0100
Message-ID	<sDOW5-7mP-13@gated-at.bofh.it> (permalink)
References	(5 earlier) <sBYGd-7rk-5@gated-at.bofh.it> <sC2Tw-1Yn-29@gated-at.bofh.it> <sCeKZ-1mm-5@gated-at.bofh.it> <sCkdI-4SU-37@gated-at.bofh.it> <sCkQp-5qC-5@gated-at.bofh.it>
Organization	ARM

Show all headers | View raw


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

Back to linux.kernel | Previous | Next | Find similar | Unroll thread

Thread

Re: [PATCH v5 6/8] Documentation: bindings: add compatible specific  to legacy SCPI protocol Sudeep Holla <sudeep.holla@arm.com> - 2016-11-15 17:40 +0100

csiph-web