From: Timur Tabi <ti...@codeaurora.org>
Date: Tue, 14 Jun 2016 17:22:35 -0500

> +/* Free all descriptors of given receive queue */
> +static void emac_rx_q_free_descs(struct emac_adapter *adpt)
> +{
> +     struct emac_rx_queue *rx_q = &adpt->rx_q;
> +     struct device *dev = adpt->netdev->dev.parent;
> +     size_t size;
> +     int i;

Please always order local variable declarations from longest to shortest
line.

Audit your entire driver for this issue, thanks.

Reply via email to