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

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.
>=20
> This defines the hardware current limit operating mode (low/high)
> for each sensor to match the physical board design.
>=20
> Specific configurations:
> - HSC1 (0x11): low
> - HSC2 (0x13): high
> - HSC3 (0x15): high
> - HSC4 (0x17): low
>=20
> Signed-off-by: Mike Hsieh <[email protected]>
> ---
> =C2=A0arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-sanmiguel.dts | 4 ++++
> =C2=A01 file changed, 4 insertions(+)
>=20
> 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 {
> =C2=A0		compatible =3D "ti,lm5066i";
> =C2=A0		reg =3D <0x11>;
> =C2=A0		shunt-resistor-micro-ohms =3D <1000>;
> +		ti,current-range =3D "low";
> =C2=A0	};
> =C2=A0
> =C2=A0	pdb_mps_hsc2: power-monitor@12 {
> @@ -462,6 +463,7 @@ pdb_ti_hsc2: power-monitor@13 {
> =C2=A0		compatible =3D "ti,lm5066i";
> =C2=A0		reg =3D <0x13>;
> =C2=A0		shunt-resistor-micro-ohms =3D <321>;
> +		ti,current-range =3D "high";
> =C2=A0	};
> =C2=A0
> =C2=A0	pdb_mps_hsc3: power-monitor@14 {
> @@ -473,6 +475,7 @@ pdb_ti_hsc3: power-monitor@15 {
> =C2=A0		compatible =3D "ti,lm5066i";
> =C2=A0		reg =3D <0x15>;
> =C2=A0		shunt-resistor-micro-ohms =3D <321>;
> +		ti,current-range =3D "high";
> =C2=A0	};
> =C2=A0
> =C2=A0	pdb_mps_hsc4: power-monitor@16 {
> @@ -484,6 +487,7 @@ pdb_ti_hsc4: power-monitor@17 {
> =C2=A0		compatible =3D "ti,lm5066i";
> =C2=A0		reg =3D <0x17>;
> =C2=A0		shunt-resistor-micro-ohms =3D <500>;
> +		ti,current-range =3D "low";
> =C2=A0	};
> =C2=A0
> =C2=A0	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-monito=
r@11 (ti,lm5066i): Unevaluated properties are not allowed ('ti,current-rang=
e' was unexpected)
           from schema $id: http://devicetree.org/schemas/hwmon/pmbus/ti,lm=
25066.yaml
   arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-sanmiguel.dtb: power-monito=
r@13 (ti,lm5066i): Unevaluated properties are not allowed ('ti,current-rang=
e' was unexpected)
           from schema $id: http://devicetree.org/schemas/hwmon/pmbus/ti,lm=
25066.yaml
   arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-sanmiguel.dtb: power-monito=
r@15 (ti,lm5066i): Unevaluated properties are not allowed ('ti,current-rang=
e' was unexpected)
           from schema $id: http://devicetree.org/schemas/hwmon/pmbus/ti,lm=
25066.yaml
   arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-sanmiguel.dtb: power-monito=
r@17 (ti,lm5066i): Unevaluated properties are not allowed ('ti,current-rang=
e' was unexpected)
           from schema $id: http://devicetree.org/schemas/hwmon/pmbus/ti,lm=
25066.yaml

Andrew