From: James S. <jsi...@tr...> - 2001-11-14 17:35:25
|
I have started to make "global" options more specific to the driver. For example to invert the color map. It would be really nice if we asked to invert the colormap for certain framebuffer drivers. Alot of drivers do this in their own setup functions. What I was thinking was to put these "general" options in video_setup and call them from their. This would work if we can pass strut fb_info to these general functions i.e fb_invert_cmap(struct fb_info). In order to do this tho I need the struct fb_info for that frmaebuffer. So I like to suggest for a solution in 2.5.X is to change static struct { const char *name; int (*init)(void); int (*setup)(char*); } fb_drivers[] __initdata = { to: static struct { struct fb_info* (*init)(void); int (*setup)(char*); } fb_drivers[] __initdata = { We don't need the name field since this is already in struct fb_info. What do you think? |