RE: [PATCH] ARM: dts: imx7d-sdb: add rev-a board support

From: Anson Huang
Date: Thu Nov 08 2018 - 20:28:43 EST


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.