>From: Andy Whitcroft [mailto:[EMAIL PROTECTED] 
>
>Those two look suspect to checkpatch, and I have to agree with it.
>
>WARNING: Trailing semicolon indicates no statements, indent implies
>otherwise
>#346: FILE: drivers/dma/ioat_dma.c:168:
>+               if (--cnt == 0);
>+                       break;
>WARNING: Trailing semicolon indicates no statements, indent implies
>otherwise
>#363: FILE: drivers/dma/ioat_dma.c:188:
>+               if (--cnt == 0);
>+                       break;
>
>-apw
>

Ouch - how did I miss those?  Thanks!

sln
--
======================================================================
Mr. Shannon Nelson                 LAN Access Division, Intel Corp.
[EMAIL PROTECTED]                I don't speak for Intel
(503) 712-7659                    Parents can't afford to be squeamish.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to