2002-01-05 07:16:50

by Adam J. Richter

[permalink] [raw]
Subject: Patch?: linux-2.5.2-pre8/drivers/video kdev_t compilation fixes

Most (all?) of the frame buffer drivers in
linux-2.5.2-pre8/drivers/video have a kdev_t compilation error,
because they try to set their fb_info->node field to -1 (it is
a kdev_t). Shortly thereafter, regsiter_framebuffer sets it to
a more useful value. So far, I have been unable to spot any
code that relies on fb_info->node being initialized to a
particular value prior to the call to register_framebuffer,
although I have not looked very hard. What I did see was
some "???" comments beside some of the initializations to -1.
I suspsect that this is just useless initialization.

Anyhow, deleting all of those initializations to -1
allows all of the framebuffer drivers to compile, which is
that this patch does. I have also included a tiny patch to
a commented out line in include/linux/fb.h, updating it to
use minor() instead of MINOR(), in case it is every uncommented.

I am still a long way from getting pre8 to run, so I
have not tested change. Can anyone point out a place in
drivers/framebuffer that relies on fb_info->node being set to -1,
or minor(fb_info->node) being -1?

By the way, don't worry if a few of the line offsets in this
patch are off. That's because of some other changes I have in my
drivers/framebuffer (which I have previous posted, specifically
PCI ID tables, fb font modularization with demand loading, and the
fb core as a loadable module).


--
Adam J. Richter __ ______________ 4880 Stevens Creek Blvd, Suite 104
[email protected] \ / San Jose, California 95129-1034
+1 408 261-6630 | g g d r a s i l United States of America
fax +1 408 261-6631 "Free Software For The Rest Of Us."


Attachments:
(No filename) (1.64 kB)
video.diff (21.38 kB)
Download all attachments