Em Sun, 1 Dec 2013 22:06:54 +0100 Frank Schäfer <fschaefer.oss@xxxxxxxxxxxxxx> escreveu: > For GPI-connected buttons without (hardware) debouncing, the polling interval > needs to be reduced to detect button presses properly. > > Signed-off-by: Frank Schäfer <fschaefer.oss@xxxxxxxxxxxxxx> > --- > drivers/media/usb/em28xx/em28xx-input.c | 2 +- > 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-) > > diff --git a/drivers/media/usb/em28xx/em28xx-input.c b/drivers/media/usb/em28xx/em28xx-input.c > index ebc5387..c8f7ecb 100644 > --- a/drivers/media/usb/em28xx/em28xx-input.c > +++ b/drivers/media/usb/em28xx/em28xx-input.c > @@ -31,7 +31,7 @@ > #include "em28xx.h" > > #define EM28XX_SNAPSHOT_KEY KEY_CAMERA > -#define EM28XX_BUTTONS_QUERY_INTERVAL 500 > +#define EM28XX_BUTTONS_QUERY_INTERVAL 100 > > static unsigned int ir_debug; > module_param(ir_debug, int, 0644); I don't like this patch. If the reduced timeout is needed for the GPI connected buttons, you should not change it for the other buttons, as polling has a high cost, in terms of CPU sleep state (so, a more frequent polling time drains more power - meaning a lower time when battery is in usage). -- Cheers, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
