Hi, Fabio Best Regards! Anson Huang
> -----Original Message----- > From: Fabio Estevam [mailto:[email protected]] > Sent: 2018年11月8日 10:21 > To: Anson Huang <[email protected]> > Cc: Rob Herring <[email protected]>; Mark Rutland > <[email protected]>; Shawn Guo <[email protected]>; Sascha > Hauer <[email protected]>; Sascha Hauer <[email protected]>; > Fabio Estevam <[email protected]>; open list:OPEN FIRMWARE AND > FLATTENED DEVICE TREE BINDINGS <[email protected]>; linux-kernel > <[email protected]>; moderated list:ARM/FREESCALE IMX / MXC > ARM ARCHITECTURE <[email protected]>; dl-linux-imx > <[email protected]> > Subject: Re: [PATCH] ARM: dts: imx7d-sdb: add rev-a board support > > Hi Anson, > > On Wed, Nov 7, 2018 at 11:24 PM Anson Huang <[email protected]> > wrote: > > > Regarding the fec2 settings in imx7d-sdb.dts, originally it uses Rev-A > > board (but some nodes are using Rev-B board), in this patch, I clean > > up all the settings in imx7d-sdb.dts using Rev-B board, so below is > > added to imx7d-sdb.dts for Rev-B board, and in imx7d-sdb-reva.dts I have to > delete it, since Rev-A board has no this pin. > > > > - pinctrl-0 = <&pinctrl_enet2>; > > + pinctrl-0 = <&pinctrl_enet2 &pinctrl_enet2_epdc0_en>; > > + pinctrl-assert-gpios = <&gpio1 4 GPIO_ACTIVE_LOW>; > > "pinctrl-assert-gpios" only exists in the NXP vendor kernel, so you cannot use > such property in mainline. Thanks for pointing out this issue, I made a mistake when doing the search in git repo.. So, if we have to assert a GPIO, there is no common way in pinctrl driver? Have to do it in driver itself? Like this case, we have to assert the GPIO in fec driver? Or, by default the GPIO is LOW on the board which is for FEC2, we can just skip it now? The EPDC driver is NOT enabled yet in upstream mainline. Anson.
