From: "Robert P. J. Day" Following the programming advice laid down in the gcc manual, make sure the case "..." operator has spaces on either side. Signed-off-by: Robert P. J. Day Cc: "Antonino A. Daplas" Signed-off-by: Andrew Morton --- drivers/video/modedb.c | 4 ++-- drivers/video/pxafb.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff -puN drivers/video/modedb.c~video-add-spaces-on-either-side-of-the-case-operator drivers/video/modedb.c --- a/drivers/video/modedb.c~video-add-spaces-on-either-side-of-the-case-operator +++ a/drivers/video/modedb.c @@ -395,7 +395,7 @@ static int my_atoi(const char *name) for (;; name++) { switch (*name) { - case '0'...'9': + case '0' ... '9': val = 10*val+(*name-'0'); break; default: @@ -548,7 +548,7 @@ int fb_find_mode(struct fb_var_screeninf } else goto done; break; - case '0'...'9': + case '0' ... '9': break; case 'M': if (!yres_specified) diff -puN drivers/video/pxafb.c~video-add-spaces-on-either-side-of-the-case-operator drivers/video/pxafb.c --- a/drivers/video/pxafb.c~video-add-spaces-on-either-side-of-the-case-operator +++ a/drivers/video/pxafb.c @@ -1203,7 +1203,7 @@ static int __init pxafb_parse_options(st } else goto done; break; - case '0'...'9': + case '0' ... '9': break; default: goto done; _