On Sun, Sep 19, 2010 at 18:47, Florian Tobias Schandinat <FlorianSchandinat@xxxxxx> wrote: > Bruno PrÃmont schrieb: >> diff --git a/drivers/video/fbsysfs.c b/drivers/video/fbsysfs.c >> index 0a08f13..be5f342 100644 >> --- a/drivers/video/fbsysfs.c >> +++ b/drivers/video/fbsysfs.c >> @@ -58,6 +58,7 @@ struct fb_info *framebuffer_alloc(size_t size, struct >> device *dev) >> Â Â Â Â Â Â Â Âinfo->par = p + fb_info_size; >> Â Â Â Âinfo->device = dev; >> + Â Â Â kref_init(&info->refcount); > > As far as I know there exist framebuffer drivers which do not call > framebuffer_alloc but contain their own fb_info. I guess these would be > broken as well. Those should be converted to framebuffer_alloc() anyway. Janitors? 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 -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html