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

Staging: comedi: fix macro coding style issue in adl_pci9111.c

This is a patch that fixes up, macros with multiple statements should be enclosed in a do - while loop, coding style issue in the adl_pci9111.c file found by the checkpatch.pl tool

Signed-off-by: Maurice Dawson <mauricedawson2699@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

authored by

Maurice Dawson and committed by
Greg Kroah-Hartman
8be11ffc ff75f96b

+15 -6
+15 -6
drivers/staging/comedi/drivers/adl_pci9111.c
··· 266 266 outb(flags, PCI9111_IO_BASE+PCI9111_REGISTER_8254_CONTROL) 267 267 268 268 #define pci9111_8254_counter_0_set(data) \ 269 - outb(data & 0xFF, PCI9111_IO_BASE+PCI9111_REGISTER_8254_COUNTER_0); \ 269 + do { \ 270 + outb(data & 0xFF, \ 271 + PCI9111_IO_BASE+PCI9111_REGISTER_8254_COUNTER_0); \ 270 272 outb((data >> 8) & 0xFF, \ 271 - PCI9111_IO_BASE+PCI9111_REGISTER_8254_COUNTER_0) 273 + PCI9111_IO_BASE+PCI9111_REGISTER_8254_COUNTER_0); \ 274 + } while (0) 272 275 273 276 #define pci9111_8254_counter_1_set(data) \ 274 - outb(data & 0xFF, PCI9111_IO_BASE+PCI9111_REGISTER_8254_COUNTER_1); \ 277 + do { \ 278 + outb(data & 0xFF, \ 279 + PCI9111_IO_BASE+PCI9111_REGISTER_8254_COUNTER_1); \ 275 280 outb((data >> 8) & 0xFF, \ 276 - PCI9111_IO_BASE+PCI9111_REGISTER_8254_COUNTER_1) 281 + PCI9111_IO_BASE+PCI9111_REGISTER_8254_COUNTER_1); \ 282 + } while (0) 277 283 278 284 #define pci9111_8254_counter_2_set(data) \ 279 - outb(data & 0xFF, PCI9111_IO_BASE+PCI9111_REGISTER_8254_COUNTER_2); \ 285 + do { \ 286 + outb(data & 0xFF, \ 287 + PCI9111_IO_BASE+PCI9111_REGISTER_8254_COUNTER_2); \ 280 288 outb((data >> 8) & 0xFF, \ 281 - PCI9111_IO_BASE+PCI9111_REGISTER_8254_COUNTER_2) 289 + PCI9111_IO_BASE+PCI9111_REGISTER_8254_COUNTER_2); \ 290 + } while (0) 282 291 283 292 /* Function prototypes */ 284 293