video: fbdev: tgafb: Avoid comma separated statements

Use semicolons and braces.

Signed-off-by: Joe Perches <joe@perches.com>
Cc: Jiri Kosina <trivial@kernel.org>
[b.zolnierkie: minor patch summary fixup]
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Link: https://patchwork.freedesktop.org/patch/msgid/

diff --git a/drivers/video/fbdev/tgafb.c b/drivers/video/fbdev/tgafb.c
index e9869135d833a3a67cdb4e19c23932a70fd7ed0a..666fbe2f671c9358bb469737f2820779a88a1f1b 100644 (file)
--- a/drivers/video/fbdev/tgafb.c
+++ b/drivers/video/fbdev/tgafb.c
@@ -989,8 +989,10 @@ tgafb_fillrect(struct fb_info *info, const struct fb_fillrect *rect)
        /* We can fill 2k pixels per operation.  Notice blocks that fit
           the width of the screen so that we can take advantage of this
           and fill more than one line per write.  */
-       if (width == line_length)
-               width *= height, height = 1;
+       if (width == line_length) {
+               width *= height;
+               height = 1;
+       }
 
        /* The write into the frame buffer must be aligned to 4 bytes,
           but we are allowed to encode the offset within the word in
@@ -1171,8 +1173,10 @@ copyarea_8bpp(struct fb_info *info, u32 dx, u32 dy, u32 sx, u32 sy,
           More than anything else, these control how we do copies.  */
        depos = dy * line_length + dx;
        sepos = sy * line_length + sx;
-       if (backward)
-               depos += width, sepos += width;
+       if (backward) {
+               depos += width;
+               sepos += width;
+       }
 
        /* Next copy full words at a time.  */
        n32 = width / 32;