Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

Staging: vt6655: Remove do { } while (0) from single-statement macros in upc.h.

This patch fixes checkpatch.pl warnings in upc.h regarding single-statement macros
embedded within do { } while (0) blocks.

Signed-off-by: Nicholas Parkanyi <n.parkanyi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

authored by

Nicholas Parkanyi and committed by
Greg Kroah-Hartman
11208b0b 353710ce

+12 -24
+12 -24
drivers/staging/vt6655/upc.h
··· 37 37 /* For memory mapped IO */ 38 38 39 39 40 - #define VNSvInPortB(dwIOAddress, pbyData) \ 41 - do { \ 42 - *(pbyData) = ioread8(dwIOAddress); \ 43 - } while (0) 40 + #define VNSvInPortB(dwIOAddress, pbyData) \ 41 + (*(pbyData) = ioread8(dwIOAddress)) 44 42 45 - #define VNSvInPortW(dwIOAddress, pwData) \ 46 - do { \ 47 - *(pwData) = ioread16(dwIOAddress); \ 48 - } while (0) 43 + #define VNSvInPortW(dwIOAddress, pwData) \ 44 + (*(pwData) = ioread16(dwIOAddress)) 49 45 50 - #define VNSvInPortD(dwIOAddress, pdwData) \ 51 - do { \ 52 - *(pdwData) = ioread32(dwIOAddress); \ 53 - } while (0) 46 + #define VNSvInPortD(dwIOAddress, pdwData) \ 47 + (*(pdwData) = ioread32(dwIOAddress)) 54 48 55 - #define VNSvOutPortB(dwIOAddress, byData) \ 56 - do { \ 57 - iowrite8((u8)byData, dwIOAddress); \ 58 - } while (0) 49 + #define VNSvOutPortB(dwIOAddress, byData) \ 50 + iowrite8((u8)(byData), dwIOAddress) 59 51 60 - #define VNSvOutPortW(dwIOAddress, wData) \ 61 - do { \ 62 - iowrite16((u16)wData, dwIOAddress); \ 63 - } while (0) 52 + #define VNSvOutPortW(dwIOAddress, wData) \ 53 + iowrite16((u16)(wData), dwIOAddress) 64 54 65 - #define VNSvOutPortD(dwIOAddress, dwData) \ 66 - do { \ 67 - iowrite32((u32)dwData, dwIOAddress); \ 68 - } while (0) 55 + #define VNSvOutPortD(dwIOAddress, dwData) \ 56 + iowrite32((u32)(dwData), dwIOAddress) 69 57 70 58 #define PCAvDelayByIO(uDelayUnit) \ 71 59 do { \