From: Andrew Liu <[email protected]> For exmaple, keycode: KEY_OK(0x160) is changed by "and" operation with KEY_MAX(0x2ff) to KEY_KPENTER(96).
Signed-off-by: Andrew Liu <[email protected]> --- drivers/input/keyboard/adp5588-keys.c | 8 ++++++-- drivers/input/keyboard/adp5589-keys.c | 8 ++++++-- drivers/input/keyboard/bf54x-keys.c | 8 ++++++-- drivers/input/misc/pcf8574_keypad.c | 6 +++++- drivers/media/pci/ttpci/av7110_ir.c | 1 + 5 files changed, 24 insertions(+), 7 deletions(-) diff --git a/drivers/input/keyboard/adp5588-keys.c b/drivers/input/keyboard/adp5588-keys.c index dbd2047..1ce559e 100644 --- a/drivers/input/keyboard/adp5588-keys.c +++ b/drivers/input/keyboard/adp5588-keys.c @@ -535,8 +535,12 @@ static int adp5588_probe(struct i2c_client *client, if (pdata->repeat) __set_bit(EV_REP, input->evbit); - for (i = 0; i < input->keycodemax; i++) - __set_bit(kpad->keycode[i] & KEY_MAX, input->keybit); + for (i = 0; i < input->keycodemax; i++) { + if (kpad->keycode[i] > KEY_MAX) + kpad->keycode[i] = 0; + else if (kpad->keycode[i] > KEY_RESERVED) + __set_bit(kpad->keycode[i], input->keybit); + } __clear_bit(KEY_RESERVED, input->keybit); if (kpad->gpimapsize) diff --git a/drivers/input/keyboard/adp5589-keys.c b/drivers/input/keyboard/adp5589-keys.c index 67d12b3..22ca2a5 100644 --- a/drivers/input/keyboard/adp5589-keys.c +++ b/drivers/input/keyboard/adp5589-keys.c @@ -991,8 +991,12 @@ static int adp5589_probe(struct i2c_client *client, if (pdata->repeat) __set_bit(EV_REP, input->evbit); - for (i = 0; i < input->keycodemax; i++) - __set_bit(kpad->keycode[i] & KEY_MAX, input->keybit); + for (i = 0; i < input->keycodemax; i++) { + if (kpad->keycode[i] > KEY_MAX) + kpad->keycode[i] = 0; + else if (kpad->keycode[i] > KEY_RESERVED) + __set_bit(kpad->keycode[i], input->keybit); + } __clear_bit(KEY_RESERVED, input->keybit); if (kpad->gpimapsize) diff --git a/drivers/input/keyboard/bf54x-keys.c b/drivers/input/keyboard/bf54x-keys.c index fc88fb4..96f54e7 100644 --- a/drivers/input/keyboard/bf54x-keys.c +++ b/drivers/input/keyboard/bf54x-keys.c @@ -288,8 +288,12 @@ static int bfin_kpad_probe(struct platform_device *pdev) if (pdata->repeat) __set_bit(EV_REP, input->evbit); - for (i = 0; i < input->keycodemax; i++) - __set_bit(bf54x_kpad->keycode[i] & KEY_MAX, input->keybit); + for (i = 0; i < input->keycodemax; i++) { + if (bf54x_kpad->keycode[i] > KEY_MAX) + bf54x_kpad->keycode[i] = 0; + else if (bf54x_kpad->keycode[i] > KEY_RESERVED) + __set_bit(bf54x_kpad->keycode[i], input->keybit); + } __clear_bit(KEY_RESERVED, input->keybit); error = input_register_device(input); diff --git a/drivers/input/misc/pcf8574_keypad.c b/drivers/input/misc/pcf8574_keypad.c index e373929..48829f3 100644 --- a/drivers/input/misc/pcf8574_keypad.c +++ b/drivers/input/misc/pcf8574_keypad.c @@ -114,8 +114,12 @@ static int pcf8574_kp_probe(struct i2c_client *client, const struct i2c_device_i for (i = 0; i < ARRAY_SIZE(pcf8574_kp_btncode); i++) { lp->btncode[i] = pcf8574_kp_btncode[i]; - __set_bit(lp->btncode[i] & KEY_MAX, idev->keybit); + if (lp->btncode[i] > KEY_MAX) + lp->btncode[i] = 0; + else if (lp->btncode[i] > KEY_RESERVED) + __set_bit(lp->btncode[i], idev->keybit); } + __clear_bit(KEY_RESERVED, idev->keybit); sprintf(lp->name, DRV_NAME); sprintf(lp->phys, "kp_data/input0"); diff --git a/drivers/media/pci/ttpci/av7110_ir.c b/drivers/media/pci/ttpci/av7110_ir.c index 0e763a7..7fdac45 100644 --- a/drivers/media/pci/ttpci/av7110_ir.c +++ b/drivers/media/pci/ttpci/av7110_ir.c @@ -201,6 +201,7 @@ static void input_register_keys(struct infrared *ir) else if (ir->key_map[i] > KEY_RESERVED) set_bit(ir->key_map[i], ir->input_dev->keybit); } + __clear_bit(KEY_RESERVED, ir->input_dev->keybit); ir->input_dev->keycode = ir->key_map; ir->input_dev->keycodesize = sizeof(ir->key_map[0]); -- 1.7.1 -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
