"Varadarajan, Charulatha" <[email protected]> writes:

> On Sat, Apr 23, 2011 at 04:32, Kevin Hilman <[email protected]> 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 <[email protected]>

[...]

>> -#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
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to