Skip to content

Commit

Permalink
staging: xgifb: delete duplicated default video mode setup code
Browse files Browse the repository at this point in the history
The code for determining the default video mode is unnecessarily
duplicated in XGIfb_search_mode() and XGIfb_search_vesamode() functions.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
  • Loading branch information
Aaro Koskinen authored and Greg Kroah-Hartman committed Sep 6, 2011
1 parent 69a1869 commit 35c064d
Showing 1 changed file with 0 additions and 12 deletions.
12 changes: 0 additions & 12 deletions drivers/staging/xgifb/XGI_main_26.c
Original file line number Diff line number Diff line change
Expand Up @@ -460,12 +460,6 @@ static void XGIfb_search_mode(const char *name)

if (!strcmp(name, XGIbios_mode[MODE_INDEX_NONE].name)) {
printk(KERN_ERR "XGIfb: Mode 'none' not supported anymore. Using default.\n");
xgifb_mode_idx = DEFAULT_MODE;
if ((xgi_video_info.chip == XG21)
&& ((xgi_video_info.disp_state & DISPTYPE_DISP2)
== DISPTYPE_LCD)) {
xgifb_mode_idx = XGIfb_GetXG21DefaultLVDSModeIdx();
}
return;
}

Expand All @@ -489,12 +483,6 @@ static void XGIfb_search_vesamode(unsigned int vesamode)
if (vesamode == 0) {

printk(KERN_ERR "XGIfb: Mode 'none' not supported anymore. Using default.\n");
xgifb_mode_idx = DEFAULT_MODE;
if ((xgi_video_info.chip == XG21)
&& ((xgi_video_info.disp_state & DISPTYPE_DISP2)
== DISPTYPE_LCD)) {
xgifb_mode_idx = XGIfb_GetXG21DefaultLVDSModeIdx();
}
return;
}

Expand Down

0 comments on commit 35c064d

Please sign in to comment.