add class/video to fb drivers
RFC 1/1 add class/video to fb drivers
Hi,
As promised last week, here's a patch against the latest 2.6.0-test3-bk tree that adds /sys/class/video support to the fb drivers.
With this patch, my /sys/class/video looks like:
$ tree /sys/class/video/
/sys/class/video/
`-- fb0
|-- dev
|-- device -> ../../../devices/pci0000:00/0000:00:14.0
`-- driver -> ../../../bus/pci/drivers/radeonfb$ cat /sys/class/video/fb0/dev 29:0
If you use a fb driver that is not attached to a pci device, then the "device" and "driver" symlinks will not be present.
Yeah, it's a hack for now, and it requires that all callers of register_framebuffer() be modified (I know I missed some, I only got the ones that I was able to actually build, and then a few more.)
To do this patch "correctly", I would like to add a struct class_device to the struct fb_info structure, but that would require that all fb drivers dynamically create this structure, when almost all of them statically create it right now :(
But if that was done, then lots of good fb info could be exported to sysfs much easier, with some of the core information done by the fb core, and other fb driver specific info exported by the individual drivers as they want to. I'll work on trying to get that to work at a later time...
So for now, the patch below is the simplest way to get video support in sysfs, which is needed for udev and other persistent device naming schemes.
Comments?
thanks,
greg k-h
drivers/video/68328fb.c +1 -1
@@ -429,7 +429,7 @@ fix->visual =FB_VISUAL_DIRECTCOLOR ; info->screen_base = (u_char *) mc68328_fix.smem_start; -if (register_framebuffer (&fb_info) <0 ) +if (register_framebuffer (&fb_info,NULL ) <0 )panic ("Cannot register frame buffer\n" );return 0 ; }
drivers/video/aty/aty128fb.c +1 -1
@@ -1615,7 +1615,7 @@aty128_init_engine (par); -if (register_framebuffer (info) <0 ) +if (register_framebuffer (info, &pdev->dev) <0 )return 0 ;#ifdef CONFIG_PMAC_BACKLIGHT
drivers/video/aty/atyfb_base.c +1 -1
@@ -1895,7 +1895,7 @@fb_alloc_cmap (&info->cmap,256 ,0 ); -if (register_framebuffer (info) <0 ) +if (register_framebuffer (info,NULL ) <0 )return 0 ; fb_list = info;
drivers/video/cirrusfb.c +1 -1
@@ -2817,7 +2817,7 @@fbgen_set_disp (-1 , &fb_info->gen);do_install_cmap (0 , &fb_info->gen.info); - err =register_framebuffer (&fb_info->gen.info); + err =register_framebuffer (&fb_info->gen.info, fb_info->pdev);if (err) {printk (KERN_ERR "clgen: ERROR - could not register fb device; err = %d!\n" , err);DPRINTK ("EXIT, returning -EINVAL\n" );
drivers/video/cyber2000fb.c +1 -1
@@ -1399,7 +1399,7 @@ cfb->fb.var.xres, cfb->fb.var.yres, h_sync / 1000, h_sync % 1000, v_sync); - err =register_framebuffer (&cfb->fb); + err =register_framebuffer (&cfb->fb, &cfb->dev->dev); failed:return err;
drivers/video/fbmem.c +84 -1
121 lines not on this page · raw message
drivers/video/hgafb.c +1 -1
@@ -560,7 +560,7 @@ fb_info.fbops = &hgafb_ops; fb_info.screen_base = (char *)hga_fix.smem_start; -if (register_framebuffer (&fb_info) <0 ) +if (register_framebuffer (&fb_info,NULL ) <0 )return -EINVAL ; printk(KERN_INFO "fb%d: %s frame buffer device\n",
drivers/video/hitfb.c +1 -1
@@ -164,7 +164,7 @@ size = (fb_info.var.bits_per_pixel ==8 ) ?256 :16 ;fb_alloc_cmap (&fb_info.cmap, size,0 ); -if (register_framebuffer (&fb_info) <0 ) +if (register_framebuffer (&fb_info,NULL ) <0 )return -EINVAL ; printk(KERN_INFO "fb%d: %s frame buffer device\n",
drivers/video/hpfb.c +1 -1
@@ -158,7 +158,7 @@fb_alloc_cmap (&fb_info.cmap,256 ,0 ); -if (register_framebuffer (&fb_info) <0 ) +if (register_framebuffer (&fb_info,NULL ) <0 )return 1 ;return 0 ; }
drivers/video/i810/i810_main.c +1 -1
@@ -1890,7 +1890,7 @@encode_fix (&info->fix, info);i810fb_init_ringbuffer (info); - err =register_framebuffer (info); + err =register_framebuffer (info, &dev->dev);if (err <0 ) {i810fb_release_resource (info, par);printk ("i810fb_init: cannot register framebuffer device\n" );
drivers/video/igafb.c +3 -3
@@ -332,7 +332,7 @@#endif }; -static int __init iga_init (struct fb_info *info,struct iga_par *par) +static int __init iga_init (struct fb_info *info,struct iga_par *par,struct pci_dev *dev) {char vramsz =iga_inb (par,IGA_EXT_CNTRL ,IGA_IDX_EXT_BUS_CNTL ) &MEM_SIZE_ALIAS ;
@@ -361,7 +361,7 @@fb_alloc_cmap (info->cmap, video_cmap_len,0 ); -if (register_framebuffer (info) <0 ) +if (register_framebuffer (info, &dev->dev) <0 )return 0 ; printk("fb%d: %s frame buffer device at 0x%08lx [%dMB VRAM]\n",
@@ -529,7 +529,7 @@ info->fix = igafb_fix; info->pseudo_palette = (void *)(par +1 ); -if (!iga_init (info, par)) { +if (!iga_init (info, par, pdev)) {iounmap ((void *)par->io_base);iounmap (info->screen_base);if (par->mmap_map)
drivers/video/imsttfb.c +3 -3
@@ -1348,7 +1348,7 @@ };static void __init -init_imstt (struct fb_info *info) +init_imstt (struct fb_info *info,struct pci_dev *pdev) {struct imstt_par *par = (struct imstt_par *) info->par;__u32 i, tmp, *ip, *end;
@@ -1445,7 +1445,7 @@fb_alloc_cmap (&info->cmap,0 ,0 ); -if (register_framebuffer (info) <0 ) { +if (register_framebuffer (info, &pdev->dev) <0 ) {kfree (info);return ; }
@@ -1506,7 +1506,7 @@ par->cmap_regs = (__u8 *)ioremap (addr +0x840000 ,0x1000 ); info->par = par; info->pseudo_palette = (void *) (par +1 ); -init_imstt (info); +init_imstt (info, pdev);pci_set_drvdata (pdev, info);return 0 ;
drivers/video/matrox/matroxfb_base.c +1 -1
@@ -1824,7 +1824,7 @@/* We do not have to set currcon to 0... register_framebuffer do it for us on first console * and we do not want currcon == 0 for subsequent framebuffers */ -if (register_framebuffer (&ACCESS_FBINFO (fbcon)) <0 ) { +if (register_framebuffer (&ACCESS_FBINFO (fbcon),NULL ) <0 ) { goto failVideoIO; } printk("fb%d: %s frame buffer device\n",
drivers/video/matrox/matroxfb_crtc2.c +1 -1
@@ -638,7 +638,7 @@ }matroxfb_dh_init_fix (m2info); -if (register_framebuffer (&m2info->fbcon)) { +if (register_framebuffer (&m2info->fbcon, &m2info->primary_dev->pcidev->dev)) {return -ENXIO ; }if (m2info->fbcon.currcon <0 ) {
drivers/video/neofb.c +1 -1
@@ -2016,7 +2016,7 @@ info->var.yres, h_sync / 1000, h_sync % 1000, v_sync); - err =register_framebuffer (info); + err =register_framebuffer (info, &dev->dev);if (err <0 ) goto failed;
drivers/video/pmag-ba-fb.c +1 -1
@@ -146,7 +146,7 @@fb_alloc_cmap (&fb_info.cmap,256 ,0 ); -if (register_framebuffer (info) <0 ) +if (register_framebuffer (info,NULL ) <0 )return 1 ;return 0 ; }
drivers/video/radeonfb.c +1 -1
@@ -3034,7 +3034,7 @@ rinfo->next = board_list; board_list = rinfo; -if (register_framebuffer ((struct fb_info *) rinfo) <0 ) { +if (register_framebuffer ((struct fb_info *) rinfo, &pdev->dev) <0 ) {printk ("radeonfb: could not register framebuffer\n" );iounmap ((void *)rinfo->fb_base);iounmap ((void *)rinfo->mmio_base);
drivers/video/retz3fb.c +1 -1
@@ -1424,7 +1424,7 @@do_install_cmap (0 , fb_info); -if (register_framebuffer (fb_info) <0 ) +if (register_framebuffer (fb_info,NULL ) <0 )return -EINVAL ; printk(KERN_INFO "fb%d: %s frame buffer device, using %ldK of "
drivers/video/riva/fbdev.c +1 -1
@@ -1866,7 +1866,7 @@ goto err_out_iounmap_fb; } -if (register_framebuffer (info) <0 ) { +if (register_framebuffer (info, &pd->dev) <0 ) { printk(KERN_ERR PFX "error registering riva framebuffer\n"); goto err_out_iounmap_fb;
drivers/video/sis/sis_main.c +1 -1
@@ -4528,7 +4528,7 @@TWDEBUG ("Before calling register_framebuffer" ); -if (register_framebuffer (&sis_fb_info) <0 ) +if (register_framebuffer (&sis_fb_info, &pdev->dev) <0 )return -EINVAL ; sisfb_registered =1 ;
drivers/video/skeletonfb.c +5 -1
@@ -563,7 +563,11 @@*/ info.var = xxxfb_var; -if (register_framebuffer (&info) <0 ) +/* +* Pass in a pointer to the struct device that this framebuffer is +* associated with. If there is no struct device available, use NULL. +*/ +if (register_framebuffer (&info,NULL ) <0 )return -EINVAL ;printk (KERN_INFO "fb%d: %s frame buffer device\n" , info.node, info.fix.id);
drivers/video/sstfb.c +1 -1
@@ -1508,7 +1508,7 @@fb_alloc_cmap (&info->cmap,256 ,0 );/* register fb */ -if (register_framebuffer (info) <0 ) { +if (register_framebuffer (info, &pdev->dev) <0 ) {eprintk ("can't register framebuffer.\n" ); goto fail; }
drivers/video/stifb.c +1 -1
@@ -1300,7 +1300,7 @@ goto out_err2; } -if (register_framebuffer (&fb->info) <0 ) +if (register_framebuffer (&fb->info,NULL ) <0 ) goto out_err3; sti->info = info;/* save for unregister_framebuffer() */
drivers/video/tdfxfb.c +2 -1
@@ -1261,7 +1261,7 @@ size = (info->var.bits_per_pixel ==8 ) ?256 :16 ;fb_alloc_cmap (&info->cmap, size,0 ); -if (register_framebuffer (info) <0 ) { +if (register_framebuffer (info, &pdev->dev) <0 ) {printk ("tdfxfb: can't register framebuffer\n" ); goto out_err; }
@@ -1356,4 +1356,5 @@ } }#endif +
drivers/video/tgafb.c +1 -1
@@ -1451,7 +1451,7 @@tgafb_set_par (&all->info);tgafb_init_fix (&all->info); -if (register_framebuffer (&all->info) <0 ) { +if (register_framebuffer (&all->info, &pdev->dev) <0 ) {printk (KERN_ERR "tgafb: Could not register framebuffer\n" ); ret = -EINVAL ; goto err1;
drivers/video/tridentfb.c +2 -2
@@ -1129,7 +1129,7 @@return -1 ; } -output ("%s board found\n" , dev->dev.name); +output ("%s board found\n" ,pci_name (dev));#if 0 output ("Trident board found : mem = %X,io = %X, mem_v = %X, io_v = %X\n" , tridentfb_fix.smem_start, tridentfb_fix.mmio_start, fb_info.screen_base, default_par.io_virt);
@@ -1155,7 +1155,7 @@ default_var.accel_flags &= ~FB_ACCELF_TEXT ; default_var.activate |=FB_ACTIVATE_NOW ; fb_info.var = default_var; -if (register_framebuffer (&fb_info) <0 ) { +if (register_framebuffer (&fb_info, &dev->dev) <0 ) {output ("Could not register Trident framebuffer\n" );return -EINVAL ; }
drivers/video/tx3912fb.c +1 -1
@@ -304,7 +304,7 @@fb_alloc_cmap (&info->cmap, size,0 ); -if (register_framebuffer (&fb_info) <0 ) +if (register_framebuffer (&fb_info,NULL ) <0 )return -1 ; printk(KERN_INFO "fb%d: TX3912 frame buffer using %uKB.\n",
drivers/video/vesafb.c +1 -1
@@ -366,7 +366,7 @@fb_alloc_cmap (&fb_info.cmap, video_cmap_len,0 ); -if (register_framebuffer (&fb_info)<0 ) +if (register_framebuffer (&fb_info,NULL )<0 )return -EINVAL ; printk(KERN_INFO "fb%d: %s frame buffer device\n",
drivers/video/vfb.c +1 -1
@@ -439,7 +439,7 @@fb_alloc_cmap (&fb_info.cmap,256 ,0 ); -if (register_framebuffer (&fb_info) <0 ) { +if (register_framebuffer (&fb_info,NULL ) <0 ) {vfree (videomemory);return -EINVAL ; }
drivers/video/vga16fb.c +1 -1
@@ -1378,7 +1378,7 @@vga16fb_update_fix (&vga16fb); -if (register_framebuffer (&vga16fb) <0 ) { +if (register_framebuffer (&vga16fb,NULL ) <0 ) {iounmap (vga16fb.screen_base);return -EINVAL ; }
drivers/video/virgefb.c +1 -1
@@ -1797,7 +1797,7 @@virgefb_set_disp (-1 , &fb_info);do_install_cmap (0 , &fb_info); -if (register_framebuffer (&fb_info) <0 ) { +if (register_framebuffer (&fb_info,NULL ) <0 ) {#warning release resourcesprintk (KERN_ERR "virgefb.c: register_framebuffer failed\n" );DPRINTK ("EXIT\n" );
include/linux/fb.h +2 -1
@@ -352,6 +352,7 @@struct fb_info ;struct vm_area_struct ;struct file ; +struct device ;/* * Frame buffer operations
@@ -477,7 +478,7 @@extern void cfb_imageblit (struct fb_info *info,const struct fb_image *image);/* drivers/video/fbmem.c */ -extern int register_framebuffer (struct fb_info *fb_info); +extern int register_framebuffer (struct fb_info *fb_info,struct device *dev);extern int unregister_framebuffer (struct fb_info *fb_info);extern int fb_prepare_logo (struct fb_info *fb_info);extern int fb_show_logo (struct fb_info *fb_info);
-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including Data Reports, E-commerce, Portals, and Forums are available now. Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100003ave/direct;at.aspnet_072303_01/01
RFC 1/1 add class/video to fb drivers - Take 2
Ok, here's a smaller version of the patch, that doesn't break any fb drivers. Only those that have a struct device associated with them should be changed (and that's just a 1 line addition). I think it's much better because of this.
Any comments?
thanks,
greg k-h
drivers/video/aty/aty128fb.c +1 -0
@@ -1536,6 +1536,7 @@/* fill in info */ info->fbops = &aty128fb_ops; info->flags =FBINFO_FLAG_DEFAULT ; + info->dev = &pdev->dev;#ifdef CONFIG_PMAC_PBOOK par->lcd_on = default_lcd_on;
drivers/video/cirrusfb.c +1 -0
@@ -2787,6 +2787,7 @@ fb_info->gen.info.switch_con = &fbgen_switch; fb_info->gen.info.updatevar = &fbgen_update_var; fb_info->gen.info.flags =FBINFO_FLAG_DEFAULT ; + fb_info->gen.info.dev = fb_info->pdev;for (j =0 ; j <256 ; j++) {if (j <16 ) {
drivers/video/cyber2000fb.c +1 -0
@@ -1366,6 +1366,7 @@ cfb->fb.fix.smem_len = smem_size; cfb->fb.fix.mmio_len =MMIO_SIZE ; cfb->fb.screen_base = cfb->region; + cfb->fb.dev = &cfb->dev->dev; err = -EINVAL ; if (!fb_find_mode(&cfb->fb.var, &cfb->fb, NULL, NULL, 0,
drivers/video/fbmem.c +83 -0
113 lines not on this page · raw message
drivers/video/i810/i810_main.c +1 -0
@@ -1880,6 +1880,7 @@ info->fbops = &par->i810fb_ops; info->pseudo_palette = par->pseudo_palette; info->flags =FBINFO_FLAG_DEFAULT ; + info->dev = &dev->dev;fb_alloc_cmap (&info->cmap,256 ,0 );
drivers/video/igafb.c +3 -2
@@ -332,7 +332,7 @@#endif }; -static int __init iga_init (struct fb_info *info,struct iga_par *par) +static int __init iga_init (struct fb_info *info,struct iga_par *par,struct pci_dev *dev) {char vramsz =iga_inb (par,IGA_EXT_CNTRL ,IGA_IDX_EXT_BUS_CNTL ) &MEM_SIZE_ALIAS ;
@@ -358,6 +358,7 @@ info->fbops = &igafb_ops; info->flags =FBINFO_FLAG_DEFAULT ; + info->dev = &dev->dev;fb_alloc_cmap (info->cmap, video_cmap_len,0 );
@@ -529,7 +530,7 @@ info->fix = igafb_fix; info->pseudo_palette = (void *)(par +1 ); -if (!iga_init (info, par)) { +if (!iga_init (info, par, pdev)) {iounmap ((void *)par->io_base);iounmap (info->screen_base);if (par->mmap_map)
drivers/video/imsttfb.c +3 -2
@@ -1348,7 +1348,7 @@ };static void __init -init_imstt (struct fb_info *info) +init_imstt (struct fb_info *info,struct pci_dev *pdev) {struct imstt_par *par = (struct imstt_par *) info->par;__u32 i, tmp, *ip, *end;
@@ -1442,6 +1442,7 @@ info->fbops = &imsttfb_ops; info->flags =FBINFO_FLAG_DEFAULT ; + info->dev = &pdev->dev;fb_alloc_cmap (&info->cmap,0 ,0 );
@@ -1506,7 +1507,7 @@ par->cmap_regs = (__u8 *)ioremap (addr +0x840000 ,0x1000 ); info->par = par; info->pseudo_palette = (void *) (par +1 ); -init_imstt (info); +init_imstt (info, pdev);pci_set_drvdata (pdev, info);return 0 ;
drivers/video/matrox/matroxfb_crtc2.c +1 -0
@@ -605,6 +605,7 @@ m2info->fbcon.flags =FBINFO_FLAG_DEFAULT ; m2info->fbcon.currcon =-1 ; m2info->fbcon.pseudo_palette = m2info->cmap; + m2info->fbcon.dev = &m2info->primary_dev->pcidev->dev;fb_alloc_cmap (&m2info->fbcon.cmap,256 ,1 );if (mem <64 )
drivers/video/neofb.c +1 -0
@@ -1943,6 +1943,7 @@ info->flags =FBINFO_FLAG_DEFAULT ; info->par = par; info->pseudo_palette = (void *) (par +1 ); + info->dev = &dev->dev;fb_alloc_cmap (&info->cmap,NR_PALETTE ,0 );
drivers/video/radeonfb.c +1 -0
@@ -3033,6 +3033,7 @@pci_set_drvdata (pdev, rinfo); rinfo->next = board_list; board_list = rinfo; + rinfo->info.dev = &pdev->dev;if (register_framebuffer ((struct fb_info *) rinfo) <0 ) {printk ("radeonfb: could not register framebuffer\n" );
drivers/video/riva/fbdev.c +1 -0
@@ -1751,6 +1751,7 @@if (info->pixmap.addr ==NULL ) goto err_out_kfree1;memset (info->pixmap.addr,0 ,64 *1024 ); + info->dev = &pd->dev;strcat (rivafb_fix.id, rci->name); default_par->riva.Architecture = rci->arch_rev;
drivers/video/sis/sis_main.c +1 -0
@@ -4507,6 +4507,7 @@ sis_fb_info.par = &ivideo; sis_fb_info.screen_base = ivideo.video_vbase; sis_fb_info.fbops = &sisfb_ops; + sis_fb_info.dev = &pdev->dev;sisfb_get_fix (&sis_fb_info.fix,-1 , &sis_fb_info); sis_fb_info.pseudo_palette = pseudo_palette;
drivers/video/sstfb.c +1 -0
@@ -1478,6 +1478,7 @@ info->fbops = &sstfb_ops; info->currcon =-1 ; info->pseudo_palette = &all->pseudo_palette; + info->dev = &pdev->dev; fix->type =FB_TYPE_PACKED_PIXELS ; fix->visual =FB_VISUAL_TRUECOLOR ;
drivers/video/tdfxfb.c +1 -0
@@ -1248,6 +1248,7 @@ info->par = default_par; info->pseudo_palette = (void *)(default_par +1 ); info->flags =FBINFO_FLAG_DEFAULT ; + info->dev = &pdev->dev;#ifndef MODULE if (!mode_option)
drivers/video/tgafb.c +1 -0
@@ -1430,6 +1430,7 @@ all->info.currcon =-1 ; all->info.par = &all->par; all->info.pseudo_palette = all->pseudo_palette; + all->info.dev = &pdev->dev;/* This should give a reasonable default video mode. */
drivers/video/tridentfb.c +2 -1
@@ -1129,7 +1129,7 @@return -1 ; } -output ("%s board found\n" , dev->dev.name); +output ("%s board found\n" ,pci_name (dev));#if 0 output ("Trident board found : mem = %X,io = %X, mem_v = %X, io_v = %X\n" , tridentfb_fix.smem_start, tridentfb_fix.mmio_start, fb_info.screen_base, default_par.io_virt);
@@ -1155,6 +1155,7 @@ default_var.accel_flags &= ~FB_ACCELF_TEXT ; default_var.activate |=FB_ACTIVATE_NOW ; fb_info.var = default_var; + fb_info.dev = &dev->dev;if (register_framebuffer (&fb_info) <0 ) {output ("Could not register Trident framebuffer\n" );return -EINVAL ;
include/linux/fb.h +2 -0
@@ -352,6 +352,7 @@struct fb_info ;struct vm_area_struct ;struct file ; +struct device ;/* * Frame buffer operations
@@ -412,6 +413,7 @@struct vc_data *display_fg;/* Console visible on this display */ int currcon;/* Current VC. */ void *pseudo_palette;/* Fake palette of 16 colors */ +struct device *dev;/* pointer to the device for this fb */ /* From here on everything is device dependent */ void *par; };
Re: [RFC] add class/video to fb drivers - Take 2
Quoted 5 lines
Ok, here's a smaller version of the patch, that doesn't break any fb drivers. Only those that have a struct device associated with them should be changed (and that's just a 1 line addition). I think it's much better because of this.
Nice :-) Will apply tonight. Do you have any issues still with this approach. You talked about the issue of dynamically creating the data. Is this probnlem still present in this patch.
Re: [RFC] add class/video to fb drivers - Take 2
Quoted 9 lines · James Simmons
Quoted 5 lines
Ok, here's a smaller version of the patch, that doesn't break any fb drivers. Only those that have a struct device associated with them should be changed (and that's just a 1 line addition). I think it's much better because of this.
Nice :-) Will apply tonight. Do you have any issues still with this approach. You talked about the issue of dynamically creating the data. Is this probnlem still present in this patch.
Well, it's not "ideal" but it will do for 2.6 to use this patch. If you look at the tty core, it's the same way that currently works.
Ideally in 2.7 I'd like to convert to have all fb drivers create the fb_info structure dynamically, but that's much to big of a change to do this late in the 2.6 development cycle.
thanks,
greg k-h
Re: [RFC] add class/video to fb drivers - Take 2
Quoted 7 lines
Well, it's not "ideal" but it will do for 2.6 to use this patch. If you look at the tty core, it's the same way that currently works.
Ideally in 2.7 I'd like to convert to have all fb drivers create the
fb_infostructure dynamically, but that's much to big of a change to do this late in the 2.6 development cycle.
What is the advantage of creating every fb_info dynamically?
Re: [RFC] add class/video to fb drivers - Take 2
Quoted 9 lines · James Simmons
Quoted 7 lines
Well, it's not "ideal" but it will do for 2.6 to use this patch. If you look at the tty core, it's the same way that currently works.
Ideally in 2.7 I'd like to convert to have all fb drivers create the
fb_infostructure dynamically, but that's much to big of a change to do this late in the 2.6 development cycle.What is the advantage of creating every
fb_infodynamically?
Then we can add a struct class_device into fb_info, which will allow us to export sysfs attributes for each fb_info device to userspace. It will provide the proper lifetime rules for such structures, and allow the individual drivers a place within sysfs to export different things they might want to export. Any /proc files you currently have could then be removed.
For an example of what needs to be done, see the recent v4l core changes that were just introduced into the kernel tree.
Again, this is a 2.7 thing (which could easily be backported to 2.6.late afterwards), nothing I want to attempt to do at this time.
thanks,
greg k-h