Hi, Fabio

Anson Huang
Best Regards!


> -----Original Message-----
> From: Fabio Estevam [mailto:[email protected]]
> Sent: Wednesday, June 27, 2018 10:31 PM
> To: Anson Huang <[email protected]>
> Cc: Shawn Guo <[email protected]>; Sascha Hauer
> <[email protected]>; Sascha Hauer <[email protected]>; Fabio
> Estevam <[email protected]>; Rob Herring <[email protected]>;
> Mark Rutland <[email protected]>; open list:OPEN FIRMWARE AND
> FLATTENED DEVICE TREE BINDINGS <[email protected]>;
> dl-linux-imx <[email protected]>; moderated list:ARM/FREESCALE IMX / MXC
> ARM ARCHITECTURE <[email protected]>; linux-kernel
> <[email protected]>
> Subject: Re: [PATCH] ARM: dts: imx6sll-evk: enable usdhc3 slot
> 
> Hi Anson,
> 
> On Wed, Jun 27, 2018 at 5:36 AM, Anson Huang <[email protected]>
> wrote:
> 
> > +&usdhc3 {
> > +       pinctrl-names = "default", "state_100mhz", "state_200mhz";
> > +       pinctrl-0 = <&pinctrl_usdhc3>;
> > +       pinctrl-1 = <&pinctrl_usdhc3_100mhz>;
> > +       pinctrl-2 = <&pinctrl_usdhc3_200mhz>;
> > +       cd-gpios = <&gpio3 22 GPIO_ACTIVE_LOW>;
> > +       keep-power-in-suspend;
> > +       enable-sdio-wakeup;
> > +       vmmc-supply = <&reg_sd3_vmmc>;
> > +       status = "okay";
> > +};
> > +
> >  &usbotg1 {
> 
> Please keep the nodes sorted in alphabetical order.

Thanks, it is because original code does NOT do it in alphabetical order, so I 
just added
usdhc3 at the end of usdhc1, but I sent V2 patch to re-sort it anyway, please 
review it.  

Anson.


Reply via email to