Re: [PATCH 6/8] Extend KD?BENT to handle > 256 keycodes.

[Vojtech Pavlik]

On Fri, Sep 26, 2003 at 01:38:37AM +0200, Andries Brouwer wrote:
> 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,

Well, as already said, there are two ways to go:

1) Keep NR_KEYS at 512 (or more in the future, iff needed), and have new
   ioctls.

2) Decrease NR_KEYS to 256 or better 255 or 254, so that for() cycles
   don't break because of limited range of the iterator, and optimize the
   KEY_* macros in input.h to fit in that range.

   This however means moving some already #defined macros to other values,
   which hopefully is not too big a problem, as few are using it.

   And it's likely we overflow the 254 entries soon, nevertheless, so we
   will need the new ioctls anyway.

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

Ok, I'm willing to give approach #2 a chance. But I'll need help from
you - as you have the most extensive list of extended keyboards - would
you please help me by creating a list of most used (and most standard)
keys on those keyboards and scancodes (unxlated set2) associated with
those?  

I'd then add what USB and Sun and others use and this way we should be
able to trim down the amount of defined keys down a bit.

I'm postponing the KD?KBENT patch for the moment ...

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