Re: [PATCH v2 5/5] arm64: dts: imx93-11x11-evk: Use phys to replace xceiver-supply
From: Frank Li <Frank.li@nxp.com>
Date: 2025-08-25 15:59:01
Also in:
imx, linux-can, linux-devicetree, linux-phy, lkml
On Mon, Aug 25, 2025 at 04:36:46PM +0800, Peng Fan wrote:
The TJA1057 used on i.MX93 EVK is actually high-speed CAN transceiver, not a regulator supply. So use phys to reflect the truth. And TJA1057 is compatible with TJA1051, so use nxp,tja1051 compatible string.
Need add TJA1057 into binding, "nxp,tja1057", "ti,tcan1043". You already touch binding doc, just add nxp,tja1057. Frank
quoted hunk ↗ jump to hunk
Signed-off-by: Peng Fan <peng.fan@nxp.com> --- arch/arm64/boot/dts/freescale/imx93-11x11-evk.dts | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-)diff --git a/arch/arm64/boot/dts/freescale/imx93-11x11-evk.dts b/arch/arm64/boot/dts/freescale/imx93-11x11-evk.dts index e24e12f04526c3a08c0bdc6134297fb010e6e926..94be59771cd05c93a6569ee08fff71bc11da21c1 100644 --- a/arch/arm64/boot/dts/freescale/imx93-11x11-evk.dts +++ b/arch/arm64/boot/dts/freescale/imx93-11x11-evk.dts@@ -62,6 +62,13 @@ vdevbuffer: vdevbuffer@a4020000 { }; + flexcan_phy: can-phy { + compatible = "nxp,tja1051", "ti,tcan1043"; + #phy-cells = <0>; + max-bitrate = <5000000>; + standby-gpios = <&adp5585 6 GPIO_ACTIVE_HIGH>; + }; + reg_vdd_12v: regulator-vdd-12v { compatible = "regulator-fixed"; regulator-name = "VDD_12V";@@ -87,14 +94,6 @@ reg_audio_pwr: regulator-audio-pwr { enable-active-high; }; - reg_can2_standby: regulator-can2-standby { - compatible = "regulator-fixed"; - regulator-name = "can2-stby"; - regulator-min-microvolt = <3300000>; - regulator-max-microvolt = <3300000>; - gpio = <&adp5585 6 GPIO_ACTIVE_LOW>; - }; - reg_m2_pwr: regulator-m2-pwr { compatible = "regulator-fixed"; regulator-name = "M.2-power";@@ -284,7 +283,7 @@ ethphy2: ethernet-phy@2 { &flexcan2 { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_flexcan2>; - xceiver-supply = <®_can2_standby>; + phys = <&flexcan_phy>; status = "okay"; }; --2.37.1