From: Lee Jones Date: Mon, 2 Nov 2020 14:23:48 +0000 (+0000) Subject: scsi: esas2r: esas2r_disc: Place brackets around a potentially empty if() X-Git-Tag: accepted/tizen/unified/20230118.172025~8143^2~224 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=9a5cf98d7ee69e7705705b40b5d947a252c134ac;p=platform%2Fkernel%2Flinux-rpi.git scsi: esas2r: esas2r_disc: Place brackets around a potentially empty if() Fixes the following W=1 kernel build warning(s): drivers/scsi/esas2r/esas2r_disc.c: In function ‘esas2r_disc_get_phys_addr’: drivers/scsi/esas2r/esas2r_disc.c:1035:17: warning: suggest braces around empty body in an ‘if’ statement [-Wempty-body] Link: https://lore.kernel.org/r/20201102142359.561122-9-lee.jones@linaro.org Cc: Bradley Grove Signed-off-by: Lee Jones Signed-off-by: Martin K. Petersen --- diff --git a/drivers/scsi/esas2r/esas2r_disc.c b/drivers/scsi/esas2r/esas2r_disc.c index 1c079f4..ba42536d 100644 --- a/drivers/scsi/esas2r/esas2r_disc.c +++ b/drivers/scsi/esas2r/esas2r_disc.c @@ -1031,8 +1031,9 @@ static u32 esas2r_disc_get_phys_addr(struct esas2r_sg_context *sgc, u64 *addr) { struct esas2r_adapter *a = sgc->adapter; - if (sgc->length > ESAS2R_DISC_BUF_LEN) + if (sgc->length > ESAS2R_DISC_BUF_LEN) { esas2r_bugon(); + } *addr = a->uncached_phys + (u64)((u8 *)a->disc_buffer - a->uncached);