When referring to array of phandles, using <> to separate the array entries is better notation as it makes potential errors with phandle and cell arguments easier to catch. Fix the outliers to be consistent with the rest of the usage. Cc: Wadim Egorov <w.egorov@xxxxxxxxx> Fixes: 0ad58871f63c ("arm64: dts: ti: Add basic support for phyBOARD-Lyra-AM625") Signed-off-by: Nishanth Menon <nm@xxxxxx> --- Vignesh, I missed in the review, just noticed. just cosmetic though, might be worthwhile to pick or squash. arch/arm64/boot/dts/ti/k3-am625-phyboard-lyra-rdk.dts | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/arch/arm64/boot/dts/ti/k3-am625-phyboard-lyra-rdk.dts b/arch/arm64/boot/dts/ti/k3-am625-phyboard-lyra-rdk.dts index 5024f1e38fc4..a438baf542c2 100644 --- a/arch/arm64/boot/dts/ti/k3-am625-phyboard-lyra-rdk.dts +++ b/arch/arm64/boot/dts/ti/k3-am625-phyboard-lyra-rdk.dts @@ -57,7 +57,7 @@ key-menu { leds { compatible = "gpio-leds"; pinctrl-names = "default"; - pinctrl-0 = <&leds_pins_default &user_leds_pins_default>; + pinctrl-0 = <&leds_pins_default>, <&user_leds_pins_default>; led-1 { gpios = <&main_gpio0 32 GPIO_ACTIVE_HIGH>; @@ -167,8 +167,7 @@ AM62X_IOPAD(0x084, PIN_OUTPUT, 7) /* (L23) GPMC0_ADVn_ALE.GPIO0_32 */ &cpsw3g { pinctrl-names = "default"; - pinctrl-0 = <&main_rgmii1_pins_default - &main_rgmii2_pins_default>; + pinctrl-0 = <&main_rgmii1_pins_default>, <&main_rgmii2_pins_default>; }; &cpsw_port2 { -- 2.40.0