From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754504AbbINNet (ORCPT ); Mon, 14 Sep 2015 09:34:49 -0400 Received: from fw-tnat.cambridge.arm.com ([217.140.96.140]:56351 "EHLO cam-smtp0.cambridge.arm.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751304AbbINNer (ORCPT ); Mon, 14 Sep 2015 09:34:47 -0400 From: Punit Agrawal To: Mark Rutland Cc: "linux-arm-kernel\@lists.infradead.org" , "devicetree\@vger.kernel.org" , Sudeep Holla , "linux-pm\@vger.kernel.org" , "lm-sensors\@lm-sensors.org" , Liviu Dudau , "edubezval\@gmail.com" , "linux\@roeck-us.net" , "linux-kernel\@vger.kernel.org" , "robh+dt\@kernel.org" Subject: Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI sensors References: <1442235619-4029-1-git-send-email-punit.agrawal@arm.com> <1442235619-4029-2-git-send-email-punit.agrawal@arm.com> <20150914131448.GC7002@leverpostej> Date: Mon, 14 Sep 2015 14:34:03 +0100 In-Reply-To: <20150914131448.GC7002@leverpostej> (Mark Rutland's message of "Mon, 14 Sep 2015 14:14:48 +0100") Message-ID: <9hhh9mxf7lg.fsf@e105922-lin.cambridge.arm.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Mark, Thanks for taking a look. Mark Rutland writes: > On Mon, Sep 14, 2015 at 02:00:14PM +0100, Punit Agrawal wrote: >> The System Control Processor (SCP) provides access to SoC sensors via >> the System Control and Power Interface (SCPI) Message Protocol. Add >> bindings to allow probing of these sensors. Also support referencing >> of the sensors for setting up thermal zones via the thermal DT >> bindings. >> >> Signed-off-by: Punit Agrawal >> Cc: Rob Herring >> Cc: Mark Rutland >> Cc: Sudeep Holla >> --- >> Documentation/devicetree/bindings/arm/arm,scpi.txt | 15 +++++++++++++++ >> 1 file changed, 15 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/arm/arm,scpi.txt b/Documentation/devicetree/bindings/arm/arm,scpi.txt >> index f002460..af66f7f 100644 >> --- a/Documentation/devicetree/bindings/arm/arm,scpi.txt >> +++ b/Documentation/devicetree/bindings/arm/arm,scpi.txt >> @@ -72,8 +72,18 @@ Required sub-node properties: >> - compatible : should be "arm,juno-scp-shmem" for Non-secure SRAM based >> shared memory on Juno platforms >> >> +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". >> +- #thermal-sensor-cells: should be set to 1. This property follows the >> + thermal device tree bindings[2]. > > You need to specify what the valid values for this cell are. The enumeration depends on the number of sensors exported by SCP firmware - which is platform dependent. I could add add something like if you think that is helpful - "Valid cell value is a number between 0..n-1, where n is the number of sensors exported by SCP firmware." Thanks, Punit > > Mark. > >> + >> [0] http://infocenter.arm.com/help/topic/com.arm.doc.dui0922b/index.html >> [1] Documentation/devicetree/bindings/clock/clock-bindings.txt >> +[2] Documentation/devicetree/bindings/thermal/thermal.txt >> >> Example: >> >> @@ -122,6 +132,11 @@ scpi_protocol: scpi@2e000000 { >> clock-output-names = "pxlclk0", "pxlclk1"; >> }; >> }; >> + >> + scpi_sensors0: sensors { >> + compatible = "arm,scpi-sensors"; >> + #thermal-sensor-cells = <1>; >> + }; >> }; >> >> cpu@0 { >> -- >> 2.5.0 >> > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: Punit Agrawal Subject: Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI sensors Date: Mon, 14 Sep 2015 14:34:03 +0100 Message-ID: <9hhh9mxf7lg.fsf@e105922-lin.cambridge.arm.com> References: <1442235619-4029-1-git-send-email-punit.agrawal@arm.com> <1442235619-4029-2-git-send-email-punit.agrawal@arm.com> <20150914131448.GC7002@leverpostej> Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: <20150914131448.GC7002@leverpostej> (Mark Rutland's message of "Mon, 14 Sep 2015 14:14:48 +0100") Sender: linux-pm-owner@vger.kernel.org To: Mark Rutland Cc: "linux-arm-kernel@lists.infradead.org" , "devicetree@vger.kernel.org" , Sudeep Holla , "linux-pm@vger.kernel.org" , "lm-sensors@lm-sensors.org" , Liviu Dudau , "edubezval@gmail.com" , "linux@roeck-us.net" , "linux-kernel@vger.kernel.org" , "robh+dt@kernel.org" List-Id: devicetree@vger.kernel.org Hi Mark, Thanks for taking a look. Mark Rutland writes: > On Mon, Sep 14, 2015 at 02:00:14PM +0100, Punit Agrawal wrote: >> The System Control Processor (SCP) provides access to SoC sensors via >> the System Control and Power Interface (SCPI) Message Protocol. Add >> bindings to allow probing of these sensors. Also support referencing >> of the sensors for setting up thermal zones via the thermal DT >> bindings. >> >> Signed-off-by: Punit Agrawal >> Cc: Rob Herring >> Cc: Mark Rutland >> Cc: Sudeep Holla >> --- >> Documentation/devicetree/bindings/arm/arm,scpi.txt | 15 +++++++++++++++ >> 1 file changed, 15 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/arm/arm,scpi.txt b/Documentation/devicetree/bindings/arm/arm,scpi.txt >> index f002460..af66f7f 100644 >> --- a/Documentation/devicetree/bindings/arm/arm,scpi.txt >> +++ b/Documentation/devicetree/bindings/arm/arm,scpi.txt >> @@ -72,8 +72,18 @@ Required sub-node properties: >> - compatible : should be "arm,juno-scp-shmem" for Non-secure SRAM based >> shared memory on Juno platforms >> >> +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". >> +- #thermal-sensor-cells: should be set to 1. This property follows the >> + thermal device tree bindings[2]. > > You need to specify what the valid values for this cell are. The enumeration depends on the number of sensors exported by SCP firmware - which is platform dependent. I could add add something like if you think that is helpful - "Valid cell value is a number between 0..n-1, where n is the number of sensors exported by SCP firmware." Thanks, Punit > > Mark. > >> + >> [0] http://infocenter.arm.com/help/topic/com.arm.doc.dui0922b/index.html >> [1] Documentation/devicetree/bindings/clock/clock-bindings.txt >> +[2] Documentation/devicetree/bindings/thermal/thermal.txt >> >> Example: >> >> @@ -122,6 +132,11 @@ scpi_protocol: scpi@2e000000 { >> clock-output-names = "pxlclk0", "pxlclk1"; >> }; >> }; >> + >> + scpi_sensors0: sensors { >> + compatible = "arm,scpi-sensors"; >> + #thermal-sensor-cells = <1>; >> + }; >> }; >> >> cpu@0 { >> -- >> 2.5.0 >> > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: punit.agrawal@arm.com (Punit Agrawal) Date: Mon, 14 Sep 2015 14:34:03 +0100 Subject: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI sensors In-Reply-To: <20150914131448.GC7002@leverpostej> (Mark Rutland's message of "Mon, 14 Sep 2015 14:14:48 +0100") References: <1442235619-4029-1-git-send-email-punit.agrawal@arm.com> <1442235619-4029-2-git-send-email-punit.agrawal@arm.com> <20150914131448.GC7002@leverpostej> Message-ID: <9hhh9mxf7lg.fsf@e105922-lin.cambridge.arm.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Mark, Thanks for taking a look. Mark Rutland writes: > On Mon, Sep 14, 2015 at 02:00:14PM +0100, Punit Agrawal wrote: >> The System Control Processor (SCP) provides access to SoC sensors via >> the System Control and Power Interface (SCPI) Message Protocol. Add >> bindings to allow probing of these sensors. Also support referencing >> of the sensors for setting up thermal zones via the thermal DT >> bindings. >> >> Signed-off-by: Punit Agrawal >> Cc: Rob Herring >> Cc: Mark Rutland >> Cc: Sudeep Holla >> --- >> Documentation/devicetree/bindings/arm/arm,scpi.txt | 15 +++++++++++++++ >> 1 file changed, 15 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/arm/arm,scpi.txt b/Documentation/devicetree/bindings/arm/arm,scpi.txt >> index f002460..af66f7f 100644 >> --- a/Documentation/devicetree/bindings/arm/arm,scpi.txt >> +++ b/Documentation/devicetree/bindings/arm/arm,scpi.txt >> @@ -72,8 +72,18 @@ Required sub-node properties: >> - compatible : should be "arm,juno-scp-shmem" for Non-secure SRAM based >> shared memory on Juno platforms >> >> +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". >> +- #thermal-sensor-cells: should be set to 1. This property follows the >> + thermal device tree bindings[2]. > > You need to specify what the valid values for this cell are. The enumeration depends on the number of sensors exported by SCP firmware - which is platform dependent. I could add add something like if you think that is helpful - "Valid cell value is a number between 0..n-1, where n is the number of sensors exported by SCP firmware." Thanks, Punit > > Mark. > >> + >> [0] http://infocenter.arm.com/help/topic/com.arm.doc.dui0922b/index.html >> [1] Documentation/devicetree/bindings/clock/clock-bindings.txt >> +[2] Documentation/devicetree/bindings/thermal/thermal.txt >> >> Example: >> >> @@ -122,6 +132,11 @@ scpi_protocol: scpi at 2e000000 { >> clock-output-names = "pxlclk0", "pxlclk1"; >> }; >> }; >> + >> + scpi_sensors0: sensors { >> + compatible = "arm,scpi-sensors"; >> + #thermal-sensor-cells = <1>; >> + }; >> }; >> >> cpu at 0 { >> -- >> 2.5.0 >> > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html