On Sun, Oct 09, 2011 at 07:42:51PM +0300, Aaro Koskinen wrote: > - switch (xgi_video_info.disp_state & DISPTYPE_DISP2) { > + switch (xgifb_info->disp_state & DISPTYPE_DISP2) { This isn't related to this patch, but this data type is weird. It would be probably be more natural to break up ->disp_state into two variables where one stored the display type and one stored the if the displays were mirrored or single. Can you connect a CRT and an LCD at the same time? Sorry if I'm asking stupid questions... regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel