On Fri, Dec 23, 2011 at 1:43 AM, Dmitry Torokhov
<dmitry.torokhov@xxxxxxxxx> wrote:
> Hi Haojian,
>
> On Thu, Dec 22, 2011 at 04:55:51PM +0800, Haojian Zhuang wrote:
>> From: Kevin Liu <kliu5@xxxxxxxxxxx>
>>
>> Signed-off-by: Kevin Liu <kliu5@xxxxxxxxxxx>
>
> Since I am getting the patch from you you need to add your own
> "Signed-off-by: ..." as well.
>
>> ---
>> drivers/input/misc/max8925_onkey.c | 35 +++++++++++++++++++++++++++++++++++
>> 1 files changed, 35 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/input/misc/max8925_onkey.c b/drivers/input/misc/max8925_onkey.c
>> index 7de0ded..ce07ae2 100644
>> --- a/drivers/input/misc/max8925_onkey.c
>> +++ b/drivers/input/misc/max8925_onkey.c
>> @@ -122,6 +122,7 @@ static int __devinit max8925_onkey_probe(struct platform_device *pdev)
>> info->idev->evbit[0] = BIT_MASK(EV_KEY);
>> info->idev->keybit[BIT_WORD(KEY_POWER)] = BIT_MASK(KEY_POWER);
>>
>> + device_init_wakeup(&pdev->dev, 1);
>>
>> error = input_register_device(info->idev);
>> if (error) {
>> @@ -158,10 +159,44 @@ static int __devexit max8925_onkey_remove(struct platform_device *pdev)
>> return 0;
>> }
>>
>> +#ifdef CONFIG_PM
>
> #ifdef CONFIG_PM_SLEEP is better choice here.
>
>> +static int max8925_onkey_suspend(struct device *dev)
>> +{
>> + struct platform_device *pdev = to_platform_device(dev);
>> + struct max8925_chip *chip = dev_get_drvdata(pdev->dev.parent);
>> +
>> + if (device_may_wakeup(dev)) {
>> + chip->wakeup_flag |= 1 << (onkey_irq[0] - chip->irq_base);
>> + chip->wakeup_flag |= 1 << (onkey_irq[1] - chip->irq_base);
>
> You don't need to actually communicate this to the device?
>
Excuse me that I missed another patch in this series. So I'll send it again.
Best Regards
Haojian
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
