Re: [PATCH 2/2] ARM: dts: aspeed: sanmiguel: add current-range property for PDB HSC

[Andrew Jeffery]

On Thu, 2026-06-18 at 15:41 +0800, Mike Hsieh wrote:
> Configure the ti,current-range property for the four TI LM5066i
> hot-swap controllers on PDB board.
> 
> This defines the hardware current limit operating mode (low/high)
> for each sensor to match the physical board design.
> 
> Specific configurations:
> - HSC1 (0x11): low
> - HSC2 (0x13): high
> - HSC3 (0x15): high
> - HSC4 (0x17): low
> 
> Signed-off-by: Mike Hsieh <mike.quanta.115@xxxxxxxxx>
> ---
>  arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-sanmiguel.dts | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-sanmiguel.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-sanmiguel.dts
> index 3faac0925a79..e518e3fc1c97 100644
> --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-sanmiguel.dts
> +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-sanmiguel.dts
> @@ -451,6 +451,7 @@ pdb_ti_hsc1: power-monitor@11 {
>  		compatible = "ti,lm5066i";
>  		reg = <0x11>;
>  		shunt-resistor-micro-ohms = <1000>;
> +		ti,current-range = "low";
>  	};
>  
>  	pdb_mps_hsc2: power-monitor@12 {
> @@ -462,6 +463,7 @@ pdb_ti_hsc2: power-monitor@13 {
>  		compatible = "ti,lm5066i";
>  		reg = <0x13>;
>  		shunt-resistor-micro-ohms = <321>;
> +		ti,current-range = "high";
>  	};
>  
>  	pdb_mps_hsc3: power-monitor@14 {
> @@ -473,6 +475,7 @@ pdb_ti_hsc3: power-monitor@15 {
>  		compatible = "ti,lm5066i";
>  		reg = <0x15>;
>  		shunt-resistor-micro-ohms = <321>;
> +		ti,current-range = "high";
>  	};
>  
>  	pdb_mps_hsc4: power-monitor@16 {
> @@ -484,6 +487,7 @@ pdb_ti_hsc4: power-monitor@17 {
>  		compatible = "ti,lm5066i";
>  		reg = <0x17>;
>  		shunt-resistor-micro-ohms = <500>;
> +		ti,current-range = "low";
>  	};
>  
>  	pdb_ioexp_20: gpio@20 {

The property isn't inspected by the associated driver, nor is it
allowed by the binding:

   arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-sanmiguel.dtb: power-monitor@11 (ti,lm5066i): Unevaluated properties are not allowed ('ti,current-range' was unexpected)
           from schema $id: http://devicetree.org/schemas/hwmon/pmbus/ti,lm25066.yaml
   arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-sanmiguel.dtb: power-monitor@13 (ti,lm5066i): Unevaluated properties are not allowed ('ti,current-range' was unexpected)
           from schema $id: http://devicetree.org/schemas/hwmon/pmbus/ti,lm25066.yaml
   arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-sanmiguel.dtb: power-monitor@15 (ti,lm5066i): Unevaluated properties are not allowed ('ti,current-range' was unexpected)
           from schema $id: http://devicetree.org/schemas/hwmon/pmbus/ti,lm25066.yaml
   arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-sanmiguel.dtb: power-monitor@17 (ti,lm5066i): Unevaluated properties are not allowed ('ti,current-range' was unexpected)
           from schema $id: http://devicetree.org/schemas/hwmon/pmbus/ti,lm25066.yaml

Andrew