On Mon, Sep 22, 2014 at 08:58:14PM +0100, Mark Einon wrote:
>  static void et131x_rx_dma_memory_free(struct et131x_adapter *adapter)
>  {
>       u8 id;
> -     u32 index;
> +     u32 ii;

Just use "int i;".  Making everything a u32 is a bad habbit and the
second 'i' character doesn't seem to stand for anything.

regards,
dan carpenter

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to