[U-Boot] [PATCH v3 09/10] mx23_olinuxino: Enable USB support
Marek Vasut
marex at denx.de
Tue Feb 19 13:21:26 CET 2013
Dear Otavio Salvador,
> On Mon, Feb 18, 2013 at 10:38 PM, Marek Vasut <marex at denx.de> wrote:
> > Dear Otavio Salvador,
> >
> >> On Mon, Feb 18, 2013 at 8:16 PM, Fabio Estevam <festevam at gmail.com> wrote:
> >> > On Sun, Feb 17, 2013 at 4:45 PM, Otavio Salvador
> >> >
> >> > <otavio at ossystems.com.br> wrote:
> >> >> +#ifdef CONFIG_CMD_USB
> >> >> + /* Enable LAN9512 */
> >> >> + gpio_direction_output(MX23_PAD_GPMI_ALE__GPIO_0_17, 1);
> >> >> +#endif
> >> >
> >> > What about doing like we do with other imx devices and write something
> >> > like:
> >> >
> >> > gpio_direction_output(MXS_GPIO_NR(0, 17), 1);
> >> >
> >> > ,where
> >> >
> >> > #define MXS_GPIO_NR(bank, nr) ((bank) * 32 + (nr))
> >> >
> >> > This also aligns with the kernel style.
> >>
> >> Awesome; I have added it to v5. Thx!
> >
> > This will bring in quite a lot of inconsistency, will it not?
>
> Really? why?
Because this will not be used globally with all GPIOs, but only locally for this
one case. So my suggestion is to postpone this change and do this in one large
swipe afterwards.
Best regards,
Marek Vasut
More information about the U-Boot
mailing list