[PATCH 13/15] OMAP: GPIO: cleanup _set_gpio_wakeup(), remove ifdefs

Kevin Hilman khilman at ti.com
Tue May 3 12:35:46 EDT 2011


"Varadarajan, Charulatha" <charu at ti.com> writes:

> On Sat, Apr 23, 2011 at 04:32, Kevin Hilman <khilman at ti.com> wrote:
>> Make _set_gpio_wakeup() generic by removing ifdefs.  Code for the
>> various SoCs/bank-methods was already the same, except for the
>> non-wakeup GPIO checking.  But that flag is set on a per-SoC basis, so
>> can be used for all SoCs.
>>
>> While here, use pr_err() and remove GPIO bank calculation assumption
>> based on subtracting bank pointers.
>>
>> Signed-off-by: Kevin Hilman <khilman at ti.com>

[...]

>> -#endif
>> -       default:
>> -               printk(KERN_ERR "Can't enable GPIO wakeup for method %i\n",
>> -                      bank->method);
>> +       if (bank->non_wakeup_gpios & gpio_bit) {
>> +               pr_err("Unable to modify wakeup on non-wakeup GPIO%d\n", gpio);
>
> use dev_err instead.

Agreed, thanks.

Kevin



More information about the linux-arm-kernel mailing list