arm64: dts: imx8mp-evk: Use phys to replace xceiver-supply

The TJA1048 used on i.MX8MP-EVK is actually high-speed CAN transceiver,
not a regulator supply. So use phys to reflect the truth.

Reviewed-by: Frank Li <Frank.Li@nxp.com>
Acked-by: Marc Kleine-Budde <mkl@pengutronix.de>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
Signed-off-by: Shawn Guo <shawnguo@kernel.org>

diff --git a/arch/arm64/boot/dts/freescale/imx8mp-evk.dts b/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
index c6facb2ad9aaa210d355a0f40eecde5e4b2f4c5d..7ededc1a06179808070f1f82172d313c8d043de8 100644 (file)
--- a/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
@@ -56,6 +56,16 @@
                      <0x1 0x00000000 0 0xc0000000>;
        };
 
+       flexcan_phy: can-phy {
+               compatible = "nxp,tja1048";
+               #phy-cells = <1>;
+               max-bitrate = <5000000>;
+               pinctrl-names = "default";
+               pinctrl-0 = <&pinctrl_flexcan_phy>;
+               standby-gpios = <&gpio5 5 GPIO_ACTIVE_LOW>,
+                               <&gpio4 27 GPIO_ACTIVE_LOW>;
+       };
+
        native-hdmi-connector {
                compatible = "hdmi-connector";
                label = "HDMI OUT";
@@ -103,28 +113,6 @@
                enable-active-high;
        };
 
-       reg_can1_stby: regulator-can1-stby {
-               compatible = "regulator-fixed";
-               regulator-name = "can1-stby";
-               pinctrl-names = "default";
-               pinctrl-0 = <&pinctrl_flexcan1_reg>;
-               regulator-min-microvolt = <3300000>;
-               regulator-max-microvolt = <3300000>;
-               gpio = <&gpio5 5 GPIO_ACTIVE_HIGH>;
-               enable-active-high;
-       };
-
-       reg_can2_stby: regulator-can2-stby {
-               compatible = "regulator-fixed";
-               regulator-name = "can2-stby";
-               pinctrl-names = "default";
-               pinctrl-0 = <&pinctrl_flexcan2_reg>;
-               regulator-min-microvolt = <3300000>;
-               regulator-max-microvolt = <3300000>;
-               gpio = <&gpio4 27 GPIO_ACTIVE_HIGH>;
-               enable-active-high;
-       };
-
        reg_pcie0: regulator-pcie {
                compatible = "regulator-fixed";
                pinctrl-names = "default";
@@ -431,14 +419,14 @@
 &flexcan1 {
        pinctrl-names = "default";
        pinctrl-0 = <&pinctrl_flexcan1>;
-       xceiver-supply = <&reg_can1_stby>;
+       phys = <&flexcan_phy 0>;
        status = "okay";
 };
 
 &flexcan2 {
        pinctrl-names = "default";
        pinctrl-0 = <&pinctrl_flexcan2>;
-       xceiver-supply = <&reg_can2_stby>;
+       phys = <&flexcan_phy 1>;
        status = "disabled";/* can2 pin conflict with pdm */
 };
 
@@ -909,14 +897,9 @@
                >;
        };
 
-       pinctrl_flexcan1_reg: flexcan1reggrp {
+       pinctrl_flexcan_phy: flexcanphygrp {
                fsl,pins = <
                        MX8MP_IOMUXC_SPDIF_EXT_CLK__GPIO5_IO05  0x154   /* CAN1_STBY */
-               >;
-       };
-
-       pinctrl_flexcan2_reg: flexcan2reggrp {
-               fsl,pins = <
                        MX8MP_IOMUXC_SAI2_MCLK__GPIO4_IO27      0x154   /* CAN2_STBY */
                >;
        };