linux-kernel - Re: [PATCH v1 3/4] ARM: dts: aspeed: harma: add sq52206 power monitor device

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <f76e867ca4dff82744958a8b555cf226139bcd78.camel@codeconstruct.com.au>
Date: Mon, 22 Sep 2025 13:06:12 +0930
From: Andrew Jeffery <andrew@...econstruct.com.au>
To: Peter Yin <peteryin.openbmc@...il.com>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
  Conor Dooley <conor+dt@...nel.org>, Joel Stanley <joel@....id.au>,
 devicetree@...r.kernel.org, 	linux-arm-kernel@...ts.infradead.org,
 linux-aspeed@...ts.ozlabs.org, 	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1 3/4] ARM: dts: aspeed: harma: add sq52206 power
 monitor device

On Wed, 2025-09-17 at 18:18 +0800, Peter Yin wrote:
> Add the SQ52206 power monitor device and reorder the sequence.
> 
> Signed-off-by: Peter Yin <peteryin.openbmc@...il.com>
> ---
>  .../dts/aspeed/aspeed-bmc-facebook-harma.dts  | 28 +++++++++++------
> --
>  1 file changed, 17 insertions(+), 11 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-harma.dts
> b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-harma.dts
> index bcef91e6eb54..fe72d47a7632 100644
> --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-harma.dts
> +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-harma.dts
> @@ -353,14 +353,15 @@ eeprom@52 {
>  		reg = <0x52>;
>  	};
>  
> -	power-monitor@69 {
> -		compatible = "pmbus";
> -		reg = <0x69>;
> +	power-monitor@40 {
> +		compatible = "infineon,xdp710";
> +		reg = <0x40>;
>  	};
>  
> -	temperature-sensor@49 {
> -		compatible = "ti,tmp75";
> -		reg = <0x49>;
> +	power-monitor@41 {
> +		compatible = "silergy,sq52206";
> +		reg = <0x41>;
> +		shunt-resistor = <500>;
>  	};
>  
>  	power-monitor@44 {
> @@ -369,16 +370,21 @@ power-monitor@44 {
>  		shunt-resistor-micro-ohms = <250>;
>  	};
>  
> -	power-monitor@40 {
> -		compatible = "infineon,xdp710";
> -		reg = <0x40>;
> -	};
> -
>  	power-monitor@45 {
>  		compatible = "ti,ina238";
>  		reg = <0x45>;
>  		shunt-resistor = <500>;
>  	};
> +
> +	power-monitor@69 {
> +		compatible = "pmbus";

I realise you're just moving this node, but I'm surprised it hasn't
caused trouble otherwise. This happens to work due to a quirk of I2C
device IDs in the kernel but it's not a documented compatible.

Compatible strings need to represent the physical device. Can you
please split out a patch either dropping this node, or replacing the
compatible string with something appropriate?

Andrew