Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756793AbcKKOTR (ORCPT ); Fri, 11 Nov 2016 09:19:17 -0500 Received: from foss.arm.com ([217.140.101.70]:44292 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756626AbcKKOTP (ORCPT ); Fri, 11 Nov 2016 09:19:15 -0500 Subject: Re: [PATCH v5 6/8] Documentation: bindings: add compatible specific to legacy SCPI protocol To: Rob Herring References: <1478148731-11712-1-git-send-email-sudeep.holla@arm.com> <1478148731-11712-7-git-send-email-sudeep.holla@arm.com> <20161110012249.ed56ik6kdffoikym@rob-hp-laptop> <14e563ae-36c5-4bf9-0d51-3b07830de3db@arm.com> <7ccc12bc-9a05-47e3-8ab8-d1b0ad31159e@arm.com> Cc: Sudeep Holla , Olof Johansson , Neil Armstrong , "linux-arm-kernel@lists.infradead.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , linux-amlogic@lists.infradead.org From: Sudeep Holla Organization: ARM Message-ID: <4e31f1d9-61b9-53a9-bd0c-dd5e452faece@arm.com> Date: Fri, 11 Nov 2016 14:19:10 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2217 Lines: 60 On 11/11/16 13:34, Rob Herring wrote: > On Fri, Nov 11, 2016 at 1:48 AM, Sudeep Holla wrote: >> On 10/11/16 19:03, Olof Johansson wrote: >>> On Thu, Nov 10, 2016 at 6:34 AM, Sudeep Holla >>> wrote: > > [...] > >>>> E.g. Amlogic follows most of the legacy protocol though it deviates in >>>> couple of things which we can handle with platform specific compatible >>>> (in the following patch in the series). When another user(Rockchip ?) >>>> make use of this legacy protocol, we can start using those platform >>>> specific compatible for deviations only. >>>> >>>> Is that not acceptable ? >>> >>> >>> If there's no shared legacy feature set, then it's probably less >>> useful to have a shared less precise compatible value. >>> >> >> There is and will be some shared feature set for sure. At the least the >> standard command set will be shared. >> >>> What the main point I was trying to get across was that we shouldn't >>> expand the generic binding with per-vendor compatible fields, instead >>> we should have those as extensions on the side. >>> >> >> Yes I get the point. We will have per-vendor compatibles for handle the >> deviations but generic one to handle the shared set. >> >>> I'm also a little apprehensive of using "legacy", it goes in the same >>> bucket as "misc". At some point 1.0 will be legacy too, etc. >>> >> >> 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" -- Regards, Sudeep