struct gfar_private *priv = netdev_priv(dev);
        unsigned long rx_packets = 0, rx_bytes = 0, rx_dropped = 0;
        unsigned long tx_packets = 0, tx_bytes = 0;
-       int i = 0;
+       int i;
 
        for (i = 0; i < priv->num_rx_queues; i++) {
                rx_packets += priv->rx_queue[i]->stats.rx_packets;
 
 void lock_rx_qs(struct gfar_private *priv)
 {
-       int i = 0x0;
+       int i;
 
        for (i = 0; i < priv->num_rx_queues; i++)
                spin_lock(&priv->rx_queue[i]->rxlock);
 
 void lock_tx_qs(struct gfar_private *priv)
 {
-       int i = 0x0;
+       int i;
 
        for (i = 0; i < priv->num_tx_queues; i++)
                spin_lock(&priv->tx_queue[i]->txlock);
 
 void unlock_rx_qs(struct gfar_private *priv)
 {
-       int i = 0x0;
+       int i;
 
        for (i = 0; i < priv->num_rx_queues; i++)
                spin_unlock(&priv->rx_queue[i]->rxlock);
 
 void unlock_tx_qs(struct gfar_private *priv)
 {
-       int i = 0x0;
+       int i;
 
        for (i = 0; i < priv->num_tx_queues; i++)
                spin_unlock(&priv->tx_queue[i]->txlock);
 
 static void free_tx_pointers(struct gfar_private *priv)
 {
-       int i = 0;
+       int i;
 
        for (i = 0; i < priv->num_tx_queues; i++)
                kfree(priv->tx_queue[i]);
 
 static void free_rx_pointers(struct gfar_private *priv)
 {
-       int i = 0;
+       int i;
 
        for (i = 0; i < priv->num_rx_queues; i++)
                kfree(priv->rx_queue[i]);
 
 static void unmap_group_regs(struct gfar_private *priv)
 {
-       int i = 0;
+       int i;
 
        for (i = 0; i < MAXGROUPS; i++)
                if (priv->gfargrp[i].regs)
 
 static void disable_napi(struct gfar_private *priv)
 {
-       int i = 0;
+       int i;
 
        for (i = 0; i < priv->num_grps; i++)
                napi_disable(&priv->gfargrp[i].napi);
 
 static void enable_napi(struct gfar_private *priv)
 {
-       int i = 0;
+       int i;
 
        for (i = 0; i < priv->num_grps; i++)
                napi_enable(&priv->gfargrp[i].napi);
 {
        struct gfar_private *priv = netdev_priv(dev);
        struct gfar __iomem *regs = NULL;
-       int i = 0;
+       int i;
 
        for (i = 0; i < priv->num_grps; i++) {
                regs = priv->gfargrp[i].regs;
        struct gfar_private *priv = netdev_priv(dev);
        struct gfar __iomem *regs = NULL;
        u32 tempval;
-       int i = 0;
+       int i;
 
        for (i = 0; i < priv->num_grps; i++) {
                regs = priv->gfargrp[i].regs;
 static inline void gfar_tx_checksum(struct sk_buff *skb, struct txfcb *fcb,
                                    int fcb_length)
 {
-       u8 flags = 0;
-
        /* If we're here, it's a IP packet with a TCP or UDP
         * payload.  We set it to checksum, using a pseudo-header
         * we provide
         */
-       flags = TXFCB_DEFAULT;
+       u8 flags = TXFCB_DEFAULT;
 
        /* Tell the controller what the protocol is
         * And provide the already calculated phcs
 static void gfar_netpoll(struct net_device *dev)
 {
        struct gfar_private *priv = netdev_priv(dev);
-       int i = 0;
+       int i;
 
        /* If the device has multiple interrupts, run tx/rx */
        if (priv->device_flags & FSL_GIANFAR_DEV_HAS_MULTI_INTR) {