Re: [PATCH 00/13] Add HD44780 Character LCD support

From: Geert Uytterhoeven
Date: Thu Mar 23 2017 - 16:29:26 EST


Hi Willy,

On Thu, Mar 23, 2017 at 8:03 PM, Willy TARREAU <wtarreau@xxxxxxxxxxx> wrote:
> On Thu, Mar 23, 2017 at 07:49:53PM +0100, Geert Uytterhoeven wrote:
>> I take it you're using v1 of the patchset?
>>
>> v2 had this change:
>>
>> - Move backlight mutex initialization before call to
>> charlcd_init_display(), to prevent locking an uninitialized mutex
>> (reported by 0day).
>>
>> https://lkml.org/lkml/2017/3/10/1151
>
> Ah yes good point sorry about that. I was surprized that the patches I
> applied were that old as I remembered reading about them not that long
> ago :-) I have too much stuff in this mailbox...
>
> Given the few differences, I don't think I need to re-test it. But
> just let me know if you need another test.

I don't think there's a need to re-test.
I did manage to reproduce the original problem when 0day reported it,
and verified that it was fixed.

Thanks!

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds