RE: [PATCH 3/3] arm64: dts: imx95-15x15-frdm: fix duplicated can-phy node names

Bough Chen <[email protected]> Thu, 30 Jul 2026 05:17:27 +0000
Newsgroups org.kernel.vger.linux-can,dev.linux.lists.imx,org.infradead.lists.linux-arm-kernel,org.infradead.lists.linux-phy,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel
Message-ID <AM9PR04MB8681A8151B95F61A4AB57D1F90C92@AM9PR04MB8681.eurprd04.prod.outlook.com>
> -----Original Message-----
> From: Bough Chen (OSS) <[email protected]>
> Sent: 2026年7月30日 12:33
> To: Marc Kleine-Budde <[email protected]>; Vincent Mailhol
> <[email protected]>; Vinod Koul <[email protected]>; Neil Armstrong
> <[email protected]>; Catalin Marinas <[email protected]>;
> Will Deacon <[email protected]>; Rob Herring <[email protected]>; Krzysztof
> Kozlowski <[email protected]>; Conor Dooley <[email protected]>;
> Frank Li <[email protected]>; Sascha Hauer <[email protected]>;
> Pengutronix Kernel Team <[email protected]>; Fabio Estevam
> <[email protected]>
> Cc: [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]; [email protected]; Bough Chen
> <[email protected]>; Joseph Guo <[email protected]>
> Subject: [PATCH 3/3] arm64: dts: imx95-15x15-frdm: fix duplicated can-phy
> node names
> 
> From: Haibo Chen <[email protected]>
> 
> Both the flexcan2 and flexcan5 CAN transceiver PHY nodes use the same
> node name "can-phy", which results in a node name collision. Distinguish
> them by using unit-less unique names "can-phy0" and "can-phy1".
> 
> These two CAN PHYs share their silent pin, this shared GPIO can be handled
> by GPIO shared proxy.

Sorry, forget to add the fix tag:

Fixes: 9bdfeed989b1 ("arm64: dts: freescale: imx95: Add support for i.MX95 15x15 FRDM board")


Regards
Haibo Chen
> 
> Signed-off-by: Haibo Chen <[email protected]>
> ---
>  arch/arm64/boot/dts/freescale/imx95-15x15-frdm.dts | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx95-15x15-frdm.dts
> b/arch/arm64/boot/dts/freescale/imx95-15x15-frdm.dts
> index
> 0f43e3be70589532763f65f1699f31dd361cc841..7ed9ff978432edeff4198f1
> 89247a154fc1016b5 100644
> --- a/arch/arm64/boot/dts/freescale/imx95-15x15-frdm.dts
> +++ b/arch/arm64/boot/dts/freescale/imx95-15x15-frdm.dts
> @@ -56,7 +56,7 @@ dmic: dmic {
>  		num-channels = <2>;
>  	};
> 
> -	flexcan2_phy: can-phy {
> +	flexcan2_phy: can-phy0 {
>  		compatible = "nxp,tja1051";
>  		#phy-cells = <0>;
>  		max-bitrate = <5000000>;
> @@ -67,7 +67,7 @@ flexcan2_phy: can-phy {
>  		silent-gpios = <&pcal6524 7 GPIO_ACTIVE_HIGH>;
>  	};
> 
> -	flexcan5_phy: can-phy {
> +	flexcan5_phy: can-phy1 {
>  		compatible = "nxp,tja1051";
>  		#phy-cells = <0>;
>  		max-bitrate = <5000000>;
> 
> --
> 2.34.1