From: Vadim Lomovtsev <vadim.lomovt...@caviumnetworks.com>
Date: Fri,  6 Apr 2018 12:53:54 -0700

> @@ -1929,7 +1929,7 @@ static void nicvf_set_rx_mode_task(struct work_struct 
> *work_arg)
>                                                 work.work);
>       struct nicvf *nic = container_of(vf_work, struct nicvf, rx_mode_work);
>       union nic_mbx mbx = {};
> -     struct xcast_addr *xaddr, *next;
> +     int idx = 0;

No need to initialize idx.

> +             for (idx = 0; idx < vf_work->mc->count; idx++) {

As it is always explicitly initialized at, and only used inside of,
this loop.

Reply via email to