lkml.org 
[lkml]   [2003]   [Sep]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 6/8] Extend KD?BENT to handle > 256 keycodes.
On Thu, Sep 25, 2003 at 06:50:12PM +0200, Vojtech Pavlik wrote:

> -struct kbentry {
> +struct kbentry_old {
> unsigned char kb_table;
> unsigned char kb_index;
> unsigned short kb_value;
> };
> +#define KDGKBENT_OLD 0x4B46 /* gets one entry in translation table (old) */
> +#define KDSKBENT_OLD 0x4B47 /* sets one entry in translation table (old) */
> +
> +struct kbentry {
> + unsigned int kb_table;
> + unsigned int kb_index;
> + unsigned short kb_value;
> +};

> -#define KDGKBENT 0x4B46 /* gets one entry in translation table */
> -#define KDSKBENT 0x4B47 /* sets one entry in translation table */
> +#define KDGKBENT 0x4B73 /* gets one entry in translation table */
> +#define KDSKBENT 0x4B74 /* sets one entry in translation table */

Please don't.

As said already, no new ioctls are needed today, but if you add some
anyway, please leave old names unchanged and add something new, like
KDSKBENT32 or so.

Reusing old ioctl names for new uses is very bad practice.

Andries

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:48    [W:0.121 / U:0.504 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site