Re: [U-Boot] [PATCH v8 13/14] dm: Enable gpio command to support driver model
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? -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v8 13/14] dm: Enable gpio command to support driver model
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. Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v8 13/14] dm: Enable gpio command to support driver model
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
Re: [U-Boot] [PATCH v8 13/14] dm: Enable gpio command to support driver model
Hi Tom, On 26 February 2014 10:28, Tom Rini tr...@ti.com wrote: 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? Sure, I'll do that and retest, resend. Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot