Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Fri, 30 Sep 2022 14:46:18 +0200
From:   Pali =?utf-8?B?Um9ow6Fy?= <pali@kernel.org>
To:     Michael Ellerman <mpe@ellerman.id.au>,
        Nicholas Piggin <npiggin@gmail.com>,
        Christophe Leroy <christophe.leroy@csgroup.eu>
Cc:     linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org,
        linux-hwmon@vger.kernel.org,
        Marek =?utf-8?B?QmVow7pu?= <kabel@kernel.org>,
        Josef Schlehofer <josef.schlehofer@nic.cz>
Subject: Re: [PATCH] powerpc: dts: turris1x.dts: Add channel labels for
 temperature sensor
Message-ID: <20220930124618.kyaansrl7ls5kn3i@pali>
References: <20220930123901.10251-1-pali@kernel.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20220930123901.10251-1-pali@kernel.org>
User-Agent: NeoMutt/20180716
Precedence: bulk

+ CC hwmon ML

On Friday 30 September 2022 14:39:01 Pali Rohár wrote:
> Channel 0 of SA56004ED chip refers to internal SA56004ED chip sensor (chip
> itself is located on the board) and channel 1 of SA56004ED chip refers to
> external sensor which is connected to temperature diode of the P2020 CPU.
> 
> Fixes: 54c15ec3b738 ("powerpc: dts: Add DTS file for CZ.NIC Turris 1.x routers")
> Signed-off-by: Pali Rohár <pali@kernel.org>
> ---
> With this change userspace 'sensors' applications prints labels:
> 
>     $ sensors
>     sa56004-i2c-0-4c
>     Adapter: MPC adapter (i2c@3000)
>     board:        +34.2°C  (low  =  +0.0°C, high = +70.0°C)
>                            (crit = +85.0°C, hyst = +75.0°C)
>     cpu:          +58.9°C  (low  =  +0.0°C, high = +70.0°C)
>                            (crit = +85.0°C, hyst = +75.0°C)
> 
> And without this change it prints just generic tempX names:
> 
>     $ sensors
>     sa56004-i2c-0-4c
>     Adapter: MPC adapter (i2c@3000)
>     temp1:        +43.0°C  (low  =  +0.0°C, high = +70.0°C)
>                            (crit = +85.0°C, hyst = +75.0°C)
>     temp2:        +63.4°C  (low  =  +0.0°C, high = +70.0°C)
>                            (crit = +85.0°C, hyst = +75.0°C)
> ---
>  arch/powerpc/boot/dts/turris1x.dts | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
> 
> diff --git a/arch/powerpc/boot/dts/turris1x.dts b/arch/powerpc/boot/dts/turris1x.dts
> index 4033c554b06a..5b5278c32e43 100644
> --- a/arch/powerpc/boot/dts/turris1x.dts
> +++ b/arch/powerpc/boot/dts/turris1x.dts
> @@ -69,6 +69,20 @@
>  				interrupt-parent = <&gpio>;
>  				interrupts = <12 IRQ_TYPE_LEVEL_LOW>, /* GPIO12 - ALERT pin */
>  					     <13 IRQ_TYPE_LEVEL_LOW>; /* GPIO13 - CRIT pin */
> +				#address-cells = <1>;
> +				#size-cells = <0>;
> +
> +				/* Local temperature sensor (SA56004ED internal) */
> +				channel@0 {
> +					reg = <0>;
> +					label = "board";
> +				};
> +
> +				/* Remote temperature sensor (D+/D- connected to P2020 CPU Temperature Diode) */
> +				channel@1 {
> +					reg = <1>;
> +					label = "cpu";
> +				};

I'm not sure if you want UPPERCASE, lowercase, PascalCase, kebab-case
or snake_case format of labels. Or if you want also "temp" or
"temperature" keyword in the label. So please adjust label to the
preferred one, if proposed format is not the correct.

>  			};
>  
>  			/* DDR3 SPD/EEPROM */
> -- 
> 2.20.1
>