Commit bae31702 authored by Peter Huewe's avatar Peter Huewe Committed by Greg Kroah-Hartman

staging/xgifb: rewrite XGIfb_get_cmap_len

We don't need to use this switch-case here for a simple two case
if-else.
Signed-off-by: default avatarPeter Huewe <peterhuewe@gmx.de>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 76a5899b
...@@ -1131,22 +1131,10 @@ static int XGIfb_release(struct fb_info *info, int user) ...@@ -1131,22 +1131,10 @@ static int XGIfb_release(struct fb_info *info, int user)
return 0; return 0;
} }
/* similar to sisfb_get_cmap_len */
static int XGIfb_get_cmap_len(const struct fb_var_screeninfo *var) static int XGIfb_get_cmap_len(const struct fb_var_screeninfo *var)
{ {
int rc = 16; return (var->bits_per_pixel == 8) ? 256 : 16;
switch (var->bits_per_pixel) {
case 8:
rc = 256;
break;
case 16:
rc = 16;
break;
case 32:
rc = 16;
break;
}
return rc;
} }
static int XGIfb_setcolreg(unsigned regno, unsigned red, unsigned green, static int XGIfb_setcolreg(unsigned regno, unsigned red, unsigned green,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment