Re: [PATCH v2] input: sirfsoc_pwrc: add onkey input driver for CSR SiRFprimaII PWRC

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, May 28, 2013 at 04:07:46PM +0800, Barry Song wrote:
> 2013/5/28 Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx>:
> > Hi Barry,
> >
> > On Tue, May 28, 2013 at 10:47:38AM +0800, Barry Song wrote:
> >> +     /*
> >> +      * For a typical Linux system, we report KEY_SUSPEND to trigger apm-power.c
> >> +      * to queue a SUSPEND APM event
> >> +      */
> >> +     input_event(pwrcdrv->input, EV_PWR, KEY_SUSPEND, 1);
> >> +     input_sync(pwrcdrv->input);
> >> +
> >> +     /*
> >> +      * Todo: report KEY_POWER event for Android platforms, Android PowerManager
> >> +      * will handle the suspend and powerdown/hibernation
> >> +      */
> >> +
> > ...
> >
> >> +
> >> +     pwrcdrv->input->evbit[0] = BIT_MASK(EV_PWR) | BIT_MASK(EV_KEY);
> >> +     set_bit(KEY_POWER, pwrcdrv->input->keybit);
> >
> > Why are we setting up KEY_POWER if we do not send this event?
> 
> For Android products, we send KEY_POWER to let Android PM trigger
> SUSPEND. that is left to be done.

OK, but then we should be adding capabilities when we are ready to send
KEY_POWER events.

I'll apply without KEY_POWER.

Thanks.

-- 
Dmitry
--
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




[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux