Re: [PATCH v1 2/3] arm64: dts: freescale: imx8mp-verdin: add labels to som nodes

[Frank Li <Frank.li@xxxxxxx>]

On Tue, Sep 24, 2024 at 01:40:52PM +0200, Francesco Dolcini wrote:
> From: João Paulo Gonçalves <joao.goncalves@xxxxxxxxxxx>
>
> Add labels to ti-ads1015 and fec ethernet mdio node to make it easier to
> reference them from other nodes.
>
> Signed-off-by: João Paulo Gonçalves <joao.goncalves@xxxxxxxxxxx>
> Signed-off-by: Francesco Dolcini <francesco.dolcini@xxxxxxxxxxx>
> ---
>  arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
> index a19ad5ee7f79..e9518b7c7aa8 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-verdin.dtsi
> @@ -320,7 +320,7 @@ &fec {
>  	pinctrl-0 = <&pinctrl_fec>;
>  	pinctrl-1 = <&pinctrl_fec_sleep>;
>
> -	mdio {
> +	verdin_eth2_mdio: mdio {
>  		#address-cells = <1>;
>  		#size-cells = <0>;
>
> @@ -591,11 +591,12 @@ hwmon_temp_module: sensor@48 {
>  		vs-supply = <&reg_vdd_1v8>;
>  	};
>
> -	adc@49 {
> +	verdin_som_adc: adc@49 {
>  		compatible = "ti,ads1015";
>  		reg = <0x49>;
>  		#address-cells = <1>;
>  		#size-cells = <0>;
> +		#io-channel-cells = <1>;

Why add this? it is not related with what descript in commit message

Frank

>
>  		/* Verdin I2C_1 (ADC_4 - ADC_3) */
>  		channel@0 {
> --
> 2.39.5
>