From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Subject: Re: [PATCH v3 1/2] Documentation: bindings: Introduce scpi,sensors-scale Date: Wed, 15 Mar 2017 12:30:23 -0700 Message-ID: <20170315193023.GA7096@roeck-us.net> References: <20170309101759.18081-1-carlo@caione.org> <20170309101759.18081-2-carlo@caione.org> <57ff035a-b462-6f5d-0100-051e99e9a2b6@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <57ff035a-b462-6f5d-0100-051e99e9a2b6@arm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Sudeep Holla Cc: linux-hwmon@vger.kernel.org, devicetree@vger.kernel.org, jdelvare@suse.com, punit.agrawal@arm.com, robh+dt@kernel.org, Carlo Caione , Carlo Caione , linux-amlogic@lists.infradead.org, linux@endlessm.com, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org On Wed, Mar 15, 2017 at 06:20:33PM +0000, Sudeep Holla wrote: > > > On 09/03/17 10:17, Carlo Caione wrote: > > From: Carlo Caione > > > > Document the new property `scpi,sensors-scale` used by the hwmon-scpi > > driver to convert the sensor readings to the correct / expected scale. > > > > I am fine with DT bindings if DT maintainers agree with having one but I > would like to add couple of points to get their feedback: > > 1. ARM recognized the drawbacks of SCPI which started informally with > ARM Ltd platforms and got adopted by few vendors. It's now being > worked on and a much better and scalable replacement protocol is > being discussed with vendors now. So future enhancement to this SCPI > protocol is unlikely. > > 2. AmLogic has diverted from base protocol in many ways and most of them > are handled with just compatible so far without any need for > additional bindings. Can we avoid it ? > > So based on these, I prefer not to add more bindings to handle such > deviations but make it just AmLogic specific. > I like the idea. Guenter