[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20140319.140609.2180508884816449731.davem@davemloft.net>
Date: Wed, 19 Mar 2014 14:06:09 -0400 (EDT)
From: David Miller <davem@...emloft.net>
To: jeffrey.t.kirsher@...el.com
Cc: mitch.a.williams@...el.com, netdev@...r.kernel.org,
gospo@...hat.com, sassmann@...hat.com, catherine.sullivan@...el.com
Subject: Re: [net-next v2 01/15] i40e: support VF link state ndo
From: Jeff Kirsher <jeffrey.t.kirsher@...el.com>
Date: Tue, 18 Mar 2014 20:41:59 -0700
> + vf = &(pf->vf[vf_id]);
These parenthesis are unnecessary, and in fact confusing. Someone reading
this is going to look again and again trying to figure out why they are
needed.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists