Re: [PATCH v2 1/5] ARM: dts: aspeed: ventura2: Add LTC2305 ADCs

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] ARM: dts: aspeed: ventura2: missing vref-supply violates
  lltc,ltc2497.yaml
--

commit caeadf546c0d6980ef4365a1de76028a3b8cfe39
Author: Kyle Hsieh <kylehsieh1995@gmail.com>

ARM: dts: aspeed: ventura2: Add LTC2305 ADCs

This commit adds three lltc,ltc2305 adc nodes to the ventura2 devicetree and
wires the i2c3 channels into iio-hwmon.

> diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura2.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura2.dts
> index f3dbdece30767..50e652cc37a17 100644
> --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura2.dts
> +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-ventura2.dts

[ ... ]

> @@ -392,6 +393,12 @@ i2c0mux0ch2: i2c@2 {
>  			reg = <2>;
>  			#address-cells = <1>;
>  			#size-cells = <0>;
> +
> +			adc@8 {
> +				compatible = "lltc,ltc2305";
> +				reg = <0x08>;
> +				#io-channel-cells = <1>;
> +			};
>  		};
>  
>  		i2c0mux0ch3: i2c@3 {
> @@ -583,6 +590,18 @@ &i2c3 {
>  
>  	status = "okay";
>  
> +	adc_op_fb1: adc@8 {
> +		compatible = "lltc,ltc2305";
> +		reg = <0x08>;
> +		#io-channel-cells = <1>;
> +	};
> +
> +	adc_op_fb2: adc@9 {
> +		compatible = "lltc,ltc2305";
> +		reg = <0x09>;
> +		#io-channel-cells = <1>;
> +	};

[Severity: Low]
Will omitting vref-supply on these lltc,ltc2305 nodes cause dtbs_check to
fail?  Documentation/devicetree/bindings/iio/adc/lltc,ltc2497.yaml lists
vref-supply under required unconditionally, so the binding schema needs to
make vref-supply optional for lltc,ltc2305 and lltc,ltc2309 before omitting it
here.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261111-ventura2-dts-update-v2-0-bce5b5dafc1f@gmail.com?part=1