Hi, Fabio Best Regards! Anson Huang > -----Original Message----- > From: Fabio Estevam [mailto:festevam@xxxxxxxxx] > Sent: 2018年11月8日 23:38 > To: Anson Huang <anson.huang@xxxxxxx> > Cc: Rob Herring <robh+dt@xxxxxxxxxx>; Mark Rutland > <mark.rutland@xxxxxxx>; Shawn Guo <shawnguo@xxxxxxxxxx>; Sascha > Hauer <s.hauer@xxxxxxxxxxxxxx>; Sascha Hauer <kernel@xxxxxxxxxxxxxx>; > Fabio Estevam <fabio.estevam@xxxxxxx>; open list:OPEN FIRMWARE AND > FLATTENED DEVICE TREE BINDINGS <devicetree@xxxxxxxxxxxxxxx>; linux-kernel > <linux-kernel@xxxxxxxxxxxxxxx>; moderated list:ARM/FREESCALE IMX / MXC > ARM ARCHITECTURE <linux-arm-kernel@xxxxxxxxxxxxxxxxxxx>; dl-linux-imx > <linux-imx@xxxxxxx> > Subject: Re: [PATCH] ARM: dts: imx7d-sdb: add rev-a board support > > Hi Anson, > > On Thu, Nov 8, 2018 at 12:30 AM Anson Huang <anson.huang@xxxxxxx> > wrote: > > > 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? > > You could pass "phy-supply" property and model it as gpio controlled > regulator. Thanks for the suggestion, I tried it and it works, please help review V2 patch. Anson.