Skip to content

Commit

Permalink
video: fbdev: tgafb: Avoid comma separated statements
Browse files Browse the repository at this point in the history
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/ <3e56046e67e583ac93030297ff6d58c08ca54086.1598331149.git.joe@perches.com
  • Loading branch information
JoePerches authored and bzolnier committed Sep 8, 2020
1 parent 3f61423 commit 9258964
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions drivers/video/fbdev/tgafb.c
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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;
Expand Down

0 comments on commit 9258964

Please sign in to comment.