Thanks for refreshing Luming's patch series, Rui.
<added linux-input list>
I have only minor feedback: on the actual key-codes.
It seems to me that we should define keys for events that are in the spec
rather than leaving two of them as UNKNOWN just because the text machine
on hand doesn't have them.
On Friday 10 August 2007 17:07, Zhang Rui wrote:
> From: Yu Luming <luming.yu@xxxxxxxxx>
>
> Add keycode for ACPI video hotkey event.
>
> Signed-off-by: Zhang Rui <rui.zhang@xxxxxxxxx>
> ---
> include/linux/input.h | 5 +++++
> 1 file changed, 5 insertions(+)
>
> Index: linux-2.6.23-rc2/include/linux/input.h
> ===================================================================
> --- linux-2.6.23-rc2.orig/include/linux/input.h
> +++ linux-2.6.23-rc2/include/linux/input.h
> @@ -553,6 +553,11 @@ struct input_absinfo {
> #define KEY_BRL_DOT7 0x1f7
> #define KEY_BRL_DOT8 0x1f8
>
> +#define KEY_VIDEO_NEXT 0x1f9
> +#define KEY_VIDEO_PREV 0x1fa
> +#define KEY_BRIGHTNESS_ZERO 0x1fb
> +#define KEY_BRIGHTNESS_OFF 0x1fc
> +
should we also define KEY_VIDEO_CYCLE_MODE?
(though i'm not sure, exactly, how this is supposed to differ from KEY_VIDEO_NEXT...)
should we also define KEY_BRIGHTNESS_CYCLE?
(acts like KEY_BRIGHTNESSUP, but it goes to min after it hits max.)
> /* We avoid low common keys in module aliases so they don't get huge. */
> #define KEY_MIN_INTERESTING KEY_MUTE
> #define KEY_MAX 0x1ff
> -
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
