RE: [PATCH] arm64: dts: nxp: ls208xa: add more thermal zone support

2018-12-19 Thread Andy Tang


> -Original Message-
> From: Shawn Guo 
> Sent: 2018年12月19日 12:08
> To: Andy Tang 
> Cc: mark.rutl...@arm.com; devicet...@vger.kernel.org;
> daniel.lezc...@linaro.org; linux-kernel@vger.kernel.org; Leo Li
> ; robh...@kernel.org; rui.zh...@intel.com;
> linux-arm-ker...@lists.infradead.org
> Subject: Re: [PATCH] arm64: dts: nxp: ls208xa: add more thermal zone support
> 
> On Tue, Dec 18, 2018 at 07:01:32AM +, Andy Tang wrote:
> > Hi,
> >
> > PING.
> >
> > BR,
> > Andy
> >
> > > -Original Message-
> > > From: Yuantian Tang 
> > > Sent: 2018年10月31日 12:48
> > > To: shawn...@kernel.org
> > > Cc: Leo Li ; robh...@kernel.org;
> > > mark.rutl...@arm.com; linux-arm-ker...@lists.infradead.org;
> > > devicet...@vger.kernel.org; linux-kernel@vger.kernel.org;
> > > rui.zh...@intel.com; daniel.lezc...@linaro.org; Andy Tang
> > > 
> > > Subject: [PATCH] arm64: dts: nxp: ls208xa: add more thermal zone
> > > support
> > >
> > > Ls208xa has several thermal sensors. Add all the sensor id to dts to
> > > enable them.
> > >
> > > To make the dts cleaner, re-organize the nodes to split out the
> > > common part so that it can be shared with other SoCs.
> > >
> > > Signed-off-by: Yuantian Tang 
> 
> Please take a look at patch below.
> 
> https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatch
> work.kernel.org%2Fpatch%2F10685815%2Fdata=02%7C01%7Candy.tang
> %40nxp.com%7C440d607feede45068dc608d66567c978%7C686ea1d3bc2b4c6fa
> 92cd99c5c301635%7C0%7C0%7C636807893765936983sdata=QC%2By9f
> mvI42t9eS3LS4YGD23Kqq6EUD0EpIGnDEDhrA%3Dreserved=0
> 
Thanks, will rebase my patch.

BR,
Andy
> Shawn


Re: [PATCH] arm64: dts: nxp: ls208xa: add more thermal zone support

2018-12-18 Thread Shawn Guo
On Tue, Dec 18, 2018 at 07:01:32AM +, Andy Tang wrote:
> Hi,
> 
> PING.
> 
> BR,
> Andy
> 
> > -Original Message-
> > From: Yuantian Tang 
> > Sent: 2018年10月31日 12:48
> > To: shawn...@kernel.org
> > Cc: Leo Li ; robh...@kernel.org; mark.rutl...@arm.com;
> > linux-arm-ker...@lists.infradead.org; devicet...@vger.kernel.org;
> > linux-kernel@vger.kernel.org; rui.zh...@intel.com; 
> > daniel.lezc...@linaro.org;
> > Andy Tang 
> > Subject: [PATCH] arm64: dts: nxp: ls208xa: add more thermal zone support
> > 
> > Ls208xa has several thermal sensors. Add all the sensor id to dts to enable
> > them.
> > 
> > To make the dts cleaner, re-organize the nodes to split out the common part 
> > so
> > that it can be shared with other SoCs.
> > 
> > Signed-off-by: Yuantian Tang 

Please take a look at patch below.

https://patchwork.kernel.org/patch/10685815/

Shawn


RE: [PATCH] arm64: dts: nxp: ls208xa: add more thermal zone support

2018-12-17 Thread Andy Tang
Hi,

PING.

BR,
Andy

> -Original Message-
> From: Yuantian Tang 
> Sent: 2018年10月31日 12:48
> To: shawn...@kernel.org
> Cc: Leo Li ; robh...@kernel.org; mark.rutl...@arm.com;
> linux-arm-ker...@lists.infradead.org; devicet...@vger.kernel.org;
> linux-kernel@vger.kernel.org; rui.zh...@intel.com; daniel.lezc...@linaro.org;
> Andy Tang 
> Subject: [PATCH] arm64: dts: nxp: ls208xa: add more thermal zone support
> 
> Ls208xa has several thermal sensors. Add all the sensor id to dts to enable
> them.
> 
> To make the dts cleaner, re-organize the nodes to split out the common part so
> that it can be shared with other SoCs.
> 
> Signed-off-by: Yuantian Tang 
> ---
>  arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi  |8 +-
>  arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi  |8 +-
>  arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi  |   83 +++-
>  arch/arm64/boot/dts/freescale/fsl-tmu-map1.dtsi |   99 +
>  arch/arm64/boot/dts/freescale/fsl-tmu-map2.dtsi |   99 +
>  arch/arm64/boot/dts/freescale/fsl-tmu-map3.dtsi |   99 +
>  arch/arm64/boot/dts/freescale/fsl-tmu.dtsi  |  251
> +++
>  7 files changed, 591 insertions(+), 56 deletions(-)  create mode 100644
> arch/arm64/boot/dts/freescale/fsl-tmu-map1.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/fsl-tmu-map2.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/fsl-tmu-map3.dtsi
>  create mode 100644 arch/arm64/boot/dts/freescale/fsl-tmu.dtsi
> 
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi
> b/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi
> index f9c1d30..8f9788c 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi
> @@ -12,7 +12,7 @@
>  #include "fsl-ls208xa.dtsi"
> 
>   {
> - cpu0: cpu@0 {
> + cooling_map0: cpu0: cpu@0 {
>   device_type = "cpu";
>   compatible = "arm,cortex-a57";
>   reg = <0x0>;
> @@ -32,7 +32,7 @@
>   #cooling-cells = <2>;
>   };
> 
> - cpu2: cpu@100 {
> + cooling_map1: cpu2: cpu@100 {
>   device_type = "cpu";
>   compatible = "arm,cortex-a57";
>   reg = <0x100>;
> @@ -52,7 +52,7 @@
>   #cooling-cells = <2>;
>   };
> 
> - cpu4: cpu@200 {
> + cooling_map2: cpu4: cpu@200 {
>   device_type = "cpu";
>   compatible = "arm,cortex-a57";
>   reg = <0x200>;
> @@ -72,7 +72,7 @@
>   #cooling-cells = <2>;
>   };
> 
> - cpu6: cpu@300 {
> + cooling_map3: cpu6: cpu@300 {
>   device_type = "cpu";
>   compatible = "arm,cortex-a57";
>   reg = <0x300>;
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi
> b/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi
> index 7c882da..013fe16 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls2088a.dtsi
> @@ -12,7 +12,7 @@
>  #include "fsl-ls208xa.dtsi"
> 
>   {
> - cpu0: cpu@0 {
> + cooling_map0: cpu0: cpu@0 {
>   device_type = "cpu";
>   compatible = "arm,cortex-a72";
>   reg = <0x0>;
> @@ -32,7 +32,7 @@
>   #cooling-cells = <2>;
>   };
> 
> - cpu2: cpu@100 {
> + cooling_map1: cpu2: cpu@100 {
>   device_type = "cpu";
>   compatible = "arm,cortex-a72";
>   reg = <0x100>;
> @@ -52,7 +52,7 @@
>   #cooling-cells = <2>;
>   };
> 
> - cpu4: cpu@200 {
> + cooling_map2: cpu4: cpu@200 {
>   device_type = "cpu";
>   compatible = "arm,cortex-a72";
>   reg = <0x200>;
> @@ -72,7 +72,7 @@
>   #cooling-cells = <2>;
>   };
> 
> - cpu6: cpu@300 {
> + cooling_map3: cpu6: cpu@300 {
>   device_type = "cpu";
>   compatible = "arm,cortex-a72";
>   reg = <0x300>;
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
> b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
> index 8cb78dd..4102317 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
> @@ -75,54 +75,7 @@
>   mask = <0x2>;
>   };
> 
> - thermal-zones {
> - cpu_thermal: cpu-thermal {
> - polling-delay-passive = <1000>;
> - polling-delay = <5000>;
> -
> - thermal-sensors = < 4>;
> -
> - trips {
> - cpu_alert: cpu-alert {
> - temperature = <75000>;
> - hysteresis = <2000>;
> - type = "passive";
> - };
> - cpu_crit: cpu-crit {
> - temperature = <85000>;
> - hysteresis = <2000>;
> -