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 = <®_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.

