Re: [PATCH] Fix braille keyboard keysym generation

From: Samuel Thibault
Date: Fri Nov 20 2009 - 18:01:30 EST


Andrew Morton, le Fri 20 Nov 2009 13:19:54 -0800, a écrit :
> > if (keycode >= NR_KEYS)
> > if (keycode >= KEY_BRL_DOT1 && keycode <= KEY_BRL_DOT8)
> > - keysym = K(KT_BRL, keycode - KEY_BRL_DOT1 + 1);
> > + keysym = U(K(KT_BRL, keycode - KEY_BRL_DOT1 + 1));
> > else
> > return;
> > else
>
> Sorry, but I cannot work out (with a reasonable amount of effort) what
> the user-visible effects of this bug are likely to be. So I am not in
> a position to decide which kernel(s) this patch should be merged into.

It will make braille keyboards actually work. Apparently I had never
really tried it and so it has never worked up to now.

> Please be careful to include this aspect in the changelog when
> appropriate, thanks.

Right, sorry. Braille keyboards announced as KEY_BRL_DOT* are still
pretty rare (that's why the bug completely went unnoticed from the
start), so it's maybe not worth including the patch in the stable
kernels, but making it for 2.6.32 would be a good thing.

Samuel
--
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/