Techwizz/kernel-fxtec-pro1x
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Staging: xgifb: XGI_main_26.c: Refactored the function

Browse source 
Refactored the function `XGIfb_search_refresh_rate` by removing a level
of `if...else` block nesting. Removed unnecessary parantheses. Removed
potential bug of array underflow.

Signed-off-by: Pratik Jain <pratik.jain0509@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Pratik Jain 2018-03-21 15:20:43 +05:30 committed by Greg Kroah-Hartman
parent f81746470b
commit 65197a7d36
1 changed files with 30 additions and 29 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

59
drivers/staging/xgifb/XGI_main_26.c
View file

@ -544,41 +544,42 @@ static u8 XGIfb_search_refresh_rate(struct xgifb_video_info *xgifb_info,
yres = XGIbios_mode[xgifb_info->mode_idx].yres; yres = XGIbios_mode[xgifb_info->mode_idx].yres;
xgifb_info->rate_idx = 0; xgifb_info->rate_idx = 0;
while ((XGIfb_vrate[i].idx != 0) && (XGIfb_vrate[i].xres <= xres)) {
if ((XGIfb_vrate[i].xres == xres) && while (XGIfb_vrate[i].idx != 0 && XGIfb_vrate[i].xres <= xres) {
(XGIfb_vrate[i].yres == yres)) { /* Skip values with xres or yres less than specified */
if (XGIfb_vrate[i].refresh == rate) { if ((XGIfb_vrate[i].yres != yres) ||
(XGIfb_vrate[i].xres != xres)) {
i++;
continue;
}
if (XGIfb_vrate[i].refresh == rate) {
xgifb_info->rate_idx = XGIfb_vrate[i].idx;
break;
} else if (XGIfb_vrate[i].refresh > rate) {
if (XGIfb_vrate[i].refresh - rate <= 3) {
pr_debug("Adjusting rate from %d up to %d\n",
rate, XGIfb_vrate[i].refresh);
xgifb_info->rate_idx = XGIfb_vrate[i].idx; xgifb_info->rate_idx = XGIfb_vrate[i].idx;
break; xgifb_info->refresh_rate =
} else if (XGIfb_vrate[i].refresh > rate) { XGIfb_vrate[i].refresh;
if ((XGIfb_vrate[i].refresh - rate) <= 3) { } else if ((XGIfb_vrate[i].idx != 1) &&
pr_debug("Adjusting rate from %d up to %d\n", (rate - XGIfb_vrate[i - 1].refresh <= 2)) {
rate, XGIfb_vrate[i].refresh);
xgifb_info->rate_idx =
XGIfb_vrate[i].idx;
xgifb_info->refresh_rate =
XGIfb_vrate[i].refresh;
} else if (((rate - XGIfb_vrate[i - 1].refresh)
<= 2) && (XGIfb_vrate[i].idx
!= 1)) {
pr_debug("Adjusting rate from %d down to %d\n",
rate,
XGIfb_vrate[i - 1].refresh);
xgifb_info->rate_idx =
XGIfb_vrate[i - 1].idx;
xgifb_info->refresh_rate =
XGIfb_vrate[i - 1].refresh;
}
break;
} else if ((rate - XGIfb_vrate[i].refresh) <= 2) {
pr_debug("Adjusting rate from %d down to %d\n", pr_debug("Adjusting rate from %d down to %d\n",
rate, XGIfb_vrate[i].refresh); rate, XGIfb_vrate[i - 1].refresh);
xgifb_info->rate_idx = XGIfb_vrate[i].idx; xgifb_info->rate_idx = XGIfb_vrate[i - 1].idx;
break; xgifb_info->refresh_rate =
XGIfb_vrate[i - 1].refresh;
} }
break;
} else if (rate - XGIfb_vrate[i].refresh <= 2) {
pr_debug("Adjusting rate from %d down to %d\n",
rate, XGIfb_vrate[i].refresh);
xgifb_info->rate_idx = XGIfb_vrate[i].idx;
break;
} }
i++; i++;
} }
if (xgifb_info->rate_idx > 0) if (xgifb_info->rate_idx > 0)
return xgifb_info->rate_idx; return xgifb_info->rate_idx;
pr_info("Unsupported rate %d for %dx%d\n", pr_info("Unsupported rate %d for %dx%d\n",