Re: [Patch] HID: Fix PicoLCD to allow it to run fbcon and handleunplug while FB in use

From: Jiri Kosina
Date: Thu Jun 24 2010 - 04:54:59 EST


On Wed, 23 Jun 2010, Bruno PrÃmont wrote:

> Do you think This patch can be applied as-is or should I break it up
> into 2 or 3 patches (one for the 8bpp NULL-pointer dereference,
> one for switch between 1bpp and 8bpp and one for the refcounting of
> framebuffer to get things polite on unplug while framebuffer is still
> in use?

Hi Bruno,

splitting would definitely improve readability and reviewability (even
more so for someone like me, who is not really familiar with the
framebuffer stuff).

Still it'd be nice if you could collect Ack from someone more familiar
with the framebuffer code.

Thanks,

--
Jiri Kosina
SUSE Labs, Novell Inc.
--
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/