RE: [PATCH -mm] [RFC] IOAT: Add support for version 2 of ioatdma device

From: Nelson, Shannon
Date: Fri Sep 07 2007 - 11:11:39 EST


>From: Andy Whitcroft [mailto:apw@xxxxxxxxxxxx]
>
>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.
Shannon.Nelson@xxxxxxxxx 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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/