[PATCH v4 RESEND 6/7] gpio/omap: fix incorrect update to context.irqenable1
Kevin Hilman
khilman at ti.com
Tue Mar 20 14:01:00 EDT 2012
Tarun Kanti DebBarma <tarun.kanti at ti.com> writes:
> In _enable_gpio_irqbank() when bank->regs->set_irqenable is valid,
> gpio_mask can be directly set by writing to set_irqenable register
> without overwriting current value. In order to ensure the same is
> stored in context.irqenable1, we must read from regs->irqenable
> instead of overwriting it with gpio_mask.
> The overwriting makes sense only in the second case where irqenable
> is explicitly read and updated with new gpio_mask before writing it
> back. However, for consistency reading regs->irqenable into the
> bank->context.irqenable1 takes care of both the scenarios.
...except that the code doesn't do this anymore.
I like the newer version (I hope so, since I suggested it :), but please
update the changlog to describe what the code is actually doing.
Thanks,
Kevin
More information about the linux-arm-kernel
mailing list