linux-kernel - Re: [PATCH 2/3] ARM: dts: aspeed: yosemite5: Update sensor configuration

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <ad11cd30eddc1b7c496cc92eef129eb5c0f2d604.camel@codeconstruct.com.au>
Date: Mon, 24 Nov 2025 15:12:34 +1030
From: Andrew Jeffery <andrew@...econstruct.com.au>
To: Kevin Tung <kevin.tung.openbmc@...il.com>
Cc: devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, 
	linux-aspeed@...ts.ozlabs.org, linux-kernel@...r.kernel.org, Amithash
 Prasasd	 <amithash@...a.com>, Kevin Tung <Kevin.Tung@...ntatw.com>, Ken
 Chen	 <Ken.Chen@...ntatw.com>, Leo Yang <Leo-Yang@...ntatw.com>, Rob
 Herring	 <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor
 Dooley	 <conor+dt@...nel.org>, Joel Stanley <joel@....id.au>
Subject: Re: [PATCH 2/3] ARM: dts: aspeed: yosemite5: Update sensor
 configuration

On Tue, 2025-11-18 at 18:53 +0800, Kevin Tung wrote:
> Remove sensors 10-0040 and 10-0045 to align with the latest
> hardware design changes.

Can you please elaborate on this commit message? I'd like this
description to be much more specific about revisions of the hardware
design, and why you expect all instances of the previous design to
suddenly stop existing.

I can't imagine this is the only hardware change that occurred. What
other changes are lurking?

Andrew

> 
> Signed-off-by: Kevin Tung <kevin.tung.openbmc@...il.com>
> ---
>  arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite5.dts | 14 --------------
>  1 file changed, 14 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite5.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite5.dts
> index 7991e9360847532cff9aad4ad4ed57d4c30668a0..45b8ac2e8c65a4f672e64571631b7f6944f26213 100644
> --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite5.dts
> +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite5.dts
> @@ -674,20 +674,6 @@ gpio-expander@22 {
>  			"PWRGD_P3V3_AUX","ALERT_TEMP";
>  	};
>  
> -	power-sensor@40 {
> -		compatible = "ti,ina233";
> -		reg = <0x40>;
> -		shunt-resistor = <2000>;
> -		ti,maximum-expected-current-microamp = <32768000>;
> -	};
> -
> -	power-sensor@45 {
> -		compatible = "ti,ina233";
> -		reg = <0x45>;
> -		shunt-resistor = <2000>;
> -		ti,maximum-expected-current-microamp = <32768000>;
> -	};
> -
>  	adc@48 {
>  		compatible = "ti,ads7830";
>  		reg = <0x48>;