[PATCH v6 24/25] gpio/omap: restore OE only after setting the output level
DebBarma, Tarun Kanti
tarun.kanti at ti.com
Wed Sep 7 02:18:49 EDT 2011
On Wed, Sep 7, 2011 at 5:36 AM, Kevin Hilman <khilman at ti.com> wrote:
> Tarun Kanti DebBarma <tarun.kanti at ti.com> writes:
>
>> From: Nishanth Menon <nm at ti.com>
>>
>> Setup the dataout register before setting the GPIO to output mode
>> in restore path.
>
> Please summarize why. (again, it may seem obvious now, but may not be for
> those not familiar with the driver or when coming back to it after a few
> months looking at other code.
Ok.
>
> Kevin
>
>> Signed-off-by: Nishanth Menon <nm at ti.com>
>> Signed-off-by: Tarun Kanti DebBarma <tarun.kanti at ti.com>
>> Reviewed-by: Santosh Shilimkar <santosh.shilimkar at ti.com>
>> ---
>> drivers/gpio/gpio-omap.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/gpio/gpio-omap.c b/drivers/gpio/gpio-omap.c
>> index db22df8..a629498 100644
>> --- a/drivers/gpio/gpio-omap.c
>> +++ b/drivers/gpio/gpio-omap.c
>> @@ -1326,7 +1326,6 @@ static void omap_gpio_restore_context(struct gpio_bank *bank)
>> __raw_writel(bank->context.wake_en,
>> bank->base + bank->regs->wkup_en);
>> __raw_writel(bank->context.ctrl, bank->base + bank->regs->ctrl);
>> - __raw_writel(bank->context.oe, bank->base + bank->regs->direction);
>> __raw_writel(bank->context.leveldetect0,
>> bank->base + bank->regs->leveldetect0);
>> __raw_writel(bank->context.leveldetect1,
>> @@ -1336,6 +1335,7 @@ static void omap_gpio_restore_context(struct gpio_bank *bank)
>> __raw_writel(bank->context.fallingdetect,
>> bank->base + bank->regs->fallingdetect);
>> __raw_writel(bank->context.dataout, bank->base + bank->regs->dataout);
>> + __raw_writel(bank->context.oe, bank->base + bank->regs->direction);
>>
>> if (bank->dbck_enable_mask) {
>> __raw_writel(bank->context.debounce, bank->base +
>
More information about the linux-arm-kernel
mailing list