Messages in this thread | | | Subject | Re: [PATCH] Staging: comedi: move trailing statement to next line in ni_mio_common.c | From | Joe Perches <> | Date | Mon, 13 Jan 2014 19:16:14 -0800 |
| |
On Mon, 2014-01-13 at 21:13 -0600, Chase Southwood wrote: > This patch for ni_mio_common.c silences a checkpatch error due to a > trailing statement. [] > diff --git a/drivers/staging/comedi/drivers/ni_mio_common.c b/drivers/staging/comedi/drivers/ni_mio_common.c [] > @@ -692,7 +692,8 @@ static void ni_clear_ai_fifo(struct comedi_device *dev) > /* Flush the 6143 data FIFO */ > ni_writel(0x10, AIFIFO_Control_6143); /* Flush fifo */ > ni_writel(0x00, AIFIFO_Control_6143); /* Flush fifo */ > - while (ni_readl(AIFIFO_Status_6143) & 0x10) ; /* Wait for complete */ > + while (ni_readl(AIFIFO_Status_6143) & 0x10) > + ; /* Wait for complete */
It's generally better to use timeouts too.
| |