From: Joakim Tjernlund <joakim.tjernl...@infinera.com>
Date: Tue, 19 Jun 2018 18:30:36 +0200

> @@ -3242,6 +3243,8 @@ static int ucc_geth_tx(struct net_device *dev, u8 txQ)
>       struct ucc_geth_private *ugeth = netdev_priv(dev);
>       u8 __iomem *bd;         /* BD pointer */
>       u32 bd_status;
> +     int howmany = 0;
> +     unsigned int bytes_sent = 0;

Please keep the function local variable declarations ordered from
longest to shortest line.

Thank you.

Reply via email to