You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

Jean Delvare 0728bacbba matroxfb: make CONFIG_FB_MATROX_MULTIHEAD=y mandatory ... I would like to get rid of option CONFIG_FB_MATROX_MULTIHEAD and just always enable it. There are many reasons for doing this: * CONFIG_FB_MATROX_MULTIHEAD=y is what all x86 distributions do, so it definitely works or we would know by now. * Building the matroxfb driver with CONFIG_FB_MATROX_MULTIHEAD not set results in the following build warning: drivers/video/matrox/matroxfb_crtc2.c: In function 'matroxfb_dh_open': drivers/video/matrox/matroxfb_crtc2.c:265: warning: the address of 'matroxfb_global_mxinfo' will always evaluate as 'true' drivers/video/matrox/matroxfb_crtc2.c: In function 'matroxfb_dh_release': drivers/video/matrox/matroxfb_crtc2.c:285: warning: the address of 'matroxfb_global_mxinfo' will always evaluate as 'true' This is nothing to be worried about, the driver will work fine, but build warnings are still annoying. * The trick to get multihead support without CONFIG_FB_MATROX_MULTIHEAD, which is described in the config help text, no longer works: you can't load the same kernel module more than once. * I fail to see how CONFIG_FB_MATROX_MULTIHEAD=y would make the code significantly slower, contrary to what the help text says. A few extra parameters on the stack here and there can't really slow things down in comaprison to the rest of the code, and register access. * The driver built without CONFIG_FB_MATROX_MULTIHEAD is larger than the driver build with CONFIG_FB_MATROX_MULTIHEAD=y by 8%. * One less configuration option makes things simpler. We add options all the time, being able to remove one for once is nice. It improves testing coverage. And I don't think the Matrox adapters are still popular enough to warrant overdetailed configuration settings. * We should be able to unobfuscate the driver code quite a bit after this change (patches follow.) Signed-off-by: Jean Delvare <khali@linux-fr.org> Acked-by: Petr Vandrovec <vandrove@vc.cvut.cz> Cc: Krzysztof Helt <krzysztof.h1@poczta.fm> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>		15 years ago
..
00-INDEX	fbdev: remove cyblafb driver	16 years ago
arkfb.txt	…
aty128fb.txt	…
cirrusfb.txt	…
cmap_xfbdev.txt	…
deferred_io.txt	…
ep93xx-fb.txt	ep93xx video driver	15 years ago
fbcon.txt	…
framebuffer.txt	…
gxfb.txt	…
imacfb.txt	…
intel810.txt	…
intelfb.txt	intelfb: support 945GME (as used in ASUS Eee 901)	16 years ago
internals.txt	…
lxfb.txt	…
matroxfb.txt	matroxfb: make CONFIG_FB_MATROX_MULTIHEAD=y mandatory	15 years ago
metronomefb.txt	…
modedb.txt	…
pvr2fb.txt	…
pxafb.txt	[ARM] pxafb: add support for overlay1 and overlay2 as framebuffer devices	16 years ago
s3fb.txt	…
sa1100fb.txt	…
sh7760fb.txt	trivial: Miscellaneous documentation typo fixes	16 years ago
sisfb.txt	…
sstfb.txt	…
tgafb.txt	…
tridentfb.txt	…
uvesafb.txt	uvesafb: documentation update	16 years ago
vesafb.txt	Documentation/fb/vesafb.txt: fix typo	16 years ago
viafb.modes	…
viafb.txt	…
vt8623fb.txt	…