Hi, Fabio Anson Huang Best Regards!
> -----Original Message----- > From: Fabio Estevam [mailto:[email protected]] > Sent: Monday, April 23, 2018 7:09 PM > To: Anson Huang <[email protected]> > Cc: Shawn Guo <[email protected]>; Sascha Hauer > <[email protected]>; Fabio Estevam <[email protected]>; Rob > Herring <[email protected]>; Mark Rutland <[email protected]>; > dl-linux-imx <[email protected]>; moderated list:ARM/FREESCALE IMX / MXC > ARM ARCHITECTURE <[email protected]>; open list:OPEN > FIRMWARE AND FLATTENED DEVICE TREE BINDINGS > <[email protected]>; linux-kernel <[email protected]> > Subject: Re: [PATCH 4/5] ARM: dts: imx6sx-sabreauto: add fec support > > Hi Anson, > > On Mon, Apr 23, 2018 at 12:46 AM, Anson Huang <[email protected]> > wrote: > > Add FEC support on i.MX6SX Sabre Auto board. > > > > Signed-off-by: Anson Huang <[email protected]> > > --- > > arch/arm/boot/dts/imx6sx-sabreauto.dts | 70 > ++++++++++++++++++++++++++++++++++ > > 1 file changed, 70 insertions(+) > > > > diff --git a/arch/arm/boot/dts/imx6sx-sabreauto.dts > b/arch/arm/boot/dts/imx6sx-sabreauto.dts > > index 812f40b..eadd483 100644 > > --- a/arch/arm/boot/dts/imx6sx-sabreauto.dts > > +++ b/arch/arm/boot/dts/imx6sx-sabreauto.dts > > @@ -41,6 +41,40 @@ > > clock-frequency = <24576000>; > > }; > > > > +&fec1 { > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_enet1_1>; > > + phy-mode = "rgmii"; > > + phy-handle = <ðphy1>; > > + pinctrl-assert-gpios = <&max7322 0 GPIO_ACTIVE_HIGH>; > > This property does not exist in mainline, only in the NXP vendor kernel. Ah, yes, thanks for pointing out this issue, I just removed it and the function is still working, already sent out V2 patch set, thanks. Anson.
