Re: [Patch RFC 02/37] input: keyboard/hil_kbd: semaphore cleanup

From: Dmitry Torokhov
Date: Tue Jul 28 2009 - 03:36:15 EST


On Sun, Jul 26, 2009 at 08:17:15AM -0000, Thomas Gleixner wrote:
> The usage of this "mutex" is non obvious and probably a completion in
> some places. Make it a semaphore.
>
> Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Cc: Dmitry Torokhov <dtor@xxxxxxx>
> ---
> drivers/input/keyboard/hil_kbd.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Index: linux-2.6-tip/drivers/input/keyboard/hil_kbd.c
> ===================================================================
> --- linux-2.6-tip.orig/drivers/input/keyboard/hil_kbd.c
> +++ linux-2.6-tip/drivers/input/keyboard/hil_kbd.c
> @@ -277,7 +277,7 @@ static int hil_kbd_connect(struct serio
> serio_set_drvdata(serio, kbd);
> kbd->serio = serio;
>
> - init_MUTEX_LOCKED(&kbd->sem);
> + semaphore_init_locked(&kbd->sem);
>

I started working on conversion of that into a completion, I'll
commit as soon as I get someone to test it.

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