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