Re: fbdev sysfs support.

From: viro
Date: Fri Feb 06 2004 - 20:29:06 EST


On Fri, Feb 06, 2004 at 05:15:44PM -0800, Greg KH wrote:
> On Sat, Feb 07, 2004 at 01:01:35AM +0000, James Simmons wrote:
> > diff -urN -X /home/jsimmons/dontdiff linus-2.6/drivers/video/fbmem.c fbdev-2.6/drivers/video/fbmem.c
> > --- linus-2.6/drivers/video/fbmem.c 2004-01-27 19:48:11.000000000 -0800
> > +++ fbdev-2.6/drivers/video/fbmem.c 2004-02-06 03:45:53.000000000 -0800
> > @@ -1228,6 +1228,9 @@
> > break;
> > fb_info->node = i;
> >
> > + if (fb_add_class_device(fb_info))
> > + return -EINVAL;
> > +
>
> Woah, what about all of the fb drivers that statically allocate their
> fb_info structure? This will die a horrible death for them when they
> try to unload themselves.
>
> Are you going to fix up every fb driver to call framebuffer_alloc() to
> fix this problem? That's quite a task...

Netdev mess, mk.II...
-
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/