On Wed, Feb 26, 2014 at 10:21:15AM -0700, Simon Glass wrote: > Hi Tom, > > On 26 February 2014 09:34, Tom Rini <tr...@ti.com> wrote: > > > On Mon, Feb 17, 2014 at 03:06:47PM -0700, Simon Glass wrote: > > > > > Now that named GPIO banks are supported, along with a way of obtaining > > > the status of a GPIO (input or output), we can provide an enhanced > > > GPIO command for driver model. Where the driver provides its own > > operation > > > for obtaining the GPIO state, this is used, otherwise a generic version > > > is sufficient. > > [snip] > > > +#if defined(CONFIG_DM_GPIO) > > > + /* > > > + * TODO(s...@chromium.org): Convert this code over to use the GPIO > > > + * uclass interface instead of the numbered GPIO compatibility > > > + * layer. > > > + */ > > > + ret = gpio_lookup_name(str_gpio, NULL, NULL, &gpio); > > > + if (ret) > > > + return cmd_process_error(cmdtp, ret); > > > > Since this is part of the initial "lets show DM code" shouldn't we do > > this TODO before merging? > > > > The problem is that this is generic code, which must work with and without > driver model. So we can't move the command code over to exclusively use > driver model until all GPIO drivers use driver model.
OK, since we're inside a CONFIG_DM_GPIO can you just expand the comment a little so it's clear it's blocked by further conversion? -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot