diff mbox

[v4,1/2] Documentation: bindings: add amlogic, meson-gxbb-scpi-sensors compatible

Message ID 20170530090507.16611-2-carlo@caione.org (mailing list archive)
State New, archived
Headers show

Commit Message

Carlo Caione May 30, 2017, 9:05 a.m. UTC
From: Carlo Caione <carlo@endlessm.com>

Introduce a new Amlogic specific compatible to handle how Amlogic reports
the sensor readings from SCPI using a different scale.

Signed-off-by: Carlo Caione <carlo@endlessm.com>
---
 Documentation/devicetree/bindings/arm/arm,scpi.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Guenter Roeck June 2, 2017, 1:14 p.m. UTC | #1
On 05/30/2017 02:05 AM, Carlo Caione wrote:
> From: Carlo Caione <carlo@endlessm.com>
> 
> Introduce a new Amlogic specific compatible to handle how Amlogic reports
> the sensor readings from SCPI using a different scale.
> 
> Signed-off-by: Carlo Caione <carlo@endlessm.com>

Formally waiting for DT approval. However, I don't see any problems with it,
so

Acked-by: Guenter Roeck <linux@roeck-us.net>

Not sure what branch this is supposed to go through. For now I assume it
will be arm/arm64.

Thanks,
Guenter

> ---
>   Documentation/devicetree/bindings/arm/arm,scpi.txt | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/arm,scpi.txt b/Documentation/devicetree/bindings/arm/arm,scpi.txt
> index 4018319..40f50c1 100644
> --- a/Documentation/devicetree/bindings/arm/arm,scpi.txt
> +++ b/Documentation/devicetree/bindings/arm/arm,scpi.txt
> @@ -76,7 +76,7 @@ Sensor bindings for the sensors based on SCPI Message Protocol
>   SCPI provides an API to access the various sensors on the SoC.
>   
>   Required properties:
> -- compatible : should be "arm,scpi-sensors".
> +- compatible : should be "arm,scpi-sensors" or "amlogic,meson-gxbb-scpi-sensors"
>   - #thermal-sensor-cells: should be set to 1. This property follows the
>   			 thermal device tree bindings[2].
>   
>
Sudeep Holla June 2, 2017, 1:26 p.m. UTC | #2
On 02/06/17 14:14, Guenter Roeck wrote:
> On 05/30/2017 02:05 AM, Carlo Caione wrote:
>> From: Carlo Caione <carlo@endlessm.com>
>>
>> Introduce a new Amlogic specific compatible to handle how Amlogic reports
>> the sensor readings from SCPI using a different scale.
>>
>> Signed-off-by: Carlo Caione <carlo@endlessm.com>
> 
> Formally waiting for DT approval. However, I don't see any problems with
> it,
> so
> 
> Acked-by: Guenter Roeck <linux@roeck-us.net>
> 
> Not sure what branch this is supposed to go through. For now I assume it
> will be arm/arm64.
> 

Initially I submitted it via arm-soc. But I think you can take it along
with the driver change in your tree as I don't see any other patch
touching arm,scpi.txt in time around.
Rob Herring June 7, 2017, 8:51 p.m. UTC | #3
On Tue, May 30, 2017 at 11:05:06AM +0200, Carlo Caione wrote:
> From: Carlo Caione <carlo@endlessm.com>
> 
> Introduce a new Amlogic specific compatible to handle how Amlogic reports
> the sensor readings from SCPI using a different scale.
> 
> Signed-off-by: Carlo Caione <carlo@endlessm.com>
> ---
>  Documentation/devicetree/bindings/arm/arm,scpi.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Acked-by: Rob Herring <robh@kernel.org>
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/arm/arm,scpi.txt b/Documentation/devicetree/bindings/arm/arm,scpi.txt
index 4018319..40f50c1 100644
--- a/Documentation/devicetree/bindings/arm/arm,scpi.txt
+++ b/Documentation/devicetree/bindings/arm/arm,scpi.txt
@@ -76,7 +76,7 @@  Sensor bindings for the sensors based on SCPI Message Protocol
 SCPI provides an API to access the various sensors on the SoC.
 
 Required properties:
-- compatible : should be "arm,scpi-sensors".
+- compatible : should be "arm,scpi-sensors" or "amlogic,meson-gxbb-scpi-sensors"
 - #thermal-sensor-cells: should be set to 1. This property follows the
 			 thermal device tree bindings[2].