linux-kernel - Re: [PATCH v6 08/10] ARM: dts: aspeed: system1: Remove VRs max8952

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: <804bffdf-a029-4bed-a6f1-42cf4c129f2a@linux.ibm.com>
Date: Wed, 22 Jan 2025 10:11:02 -0600
From: Eddie James <eajames@...ux.ibm.com>
To: Ninad Palsule <ninad@...ux.ibm.com>, minyard@....org, robh@...nel.org,
        krzk+dt@...nel.org, conor+dt@...nel.org, andrew+netdev@...n.ch,
        davem@...emloft.net, edumazet@...gle.com, kuba@...nel.org,
        pabeni@...hat.com, openipmi-developer@...ts.sourceforge.net,
        netdev@...r.kernel.org, joel@....id.au, andrew@...econstruct.com.au,
        devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-aspeed@...ts.ozlabs.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v6 08/10] ARM: dts: aspeed: system1: Remove VRs max8952


On 1/16/25 14:35, Ninad Palsule wrote:
> Removing voltage regulators max8952 from device tree. Those are fully
> controlled by hardware and firmware should not touch them.


Reviewed-by: Eddie James <eajames@...ux.ibm.com>


>
> Signed-off-by: Ninad Palsule <ninad@...ux.ibm.com>
> ---
>   .../dts/aspeed/aspeed-bmc-ibm-system1.dts     | 34 -------------------
>   1 file changed, 34 deletions(-)
>
> diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts
> index 1e0b1111ea9a..089a8315753a 100644
> --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts
> +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts
> @@ -486,23 +486,6 @@ eeprom@50 {
>   		compatible = "atmel,24c64";
>   		reg = <0x50>;
>   	};
> -
> -	regulator@60 {
> -		compatible = "maxim,max8952";
> -		reg = <0x60>;
> -
> -		max8952,default-mode = <0>;
> -		max8952,dvs-mode-microvolt = <1250000>, <1200000>,
> -						<1050000>, <950000>;
> -		max8952,sync-freq = <0>;
> -		max8952,ramp-speed = <0>;
> -
> -		regulator-name = "VR_v77_1v4";
> -		regulator-min-microvolt = <770000>;
> -		regulator-max-microvolt = <1400000>;
> -		regulator-always-on;
> -		regulator-boot-on;
> -	};
>   };
>   
>   &i2c1 {
> @@ -1198,23 +1181,6 @@ eeprom@50 {
>   		compatible = "atmel,24c64";
>   		reg = <0x50>;
>   	};
> -
> -	regulator@60 {
> -		compatible = "maxim,max8952";
> -		reg = <0x60>;
> -
> -		max8952,default-mode = <0>;
> -		max8952,dvs-mode-microvolt = <1250000>, <1200000>,
> -						<1050000>, <950000>;
> -		max8952,sync-freq = <0>;
> -		max8952,ramp-speed = <0>;
> -
> -		regulator-name = "VR_v77_1v4";
> -		regulator-min-microvolt = <770000>;
> -		regulator-max-microvolt = <1400000>;
> -		regulator-always-on;
> -		regulator-boot-on;
> -	};
>   };
>   
>   &i2c11 {