Re: [dpdk-dev] [PATCH 2/5] cxgbe: add cxgbe poll mode driver.

[Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>]

On Fri, May 22, 2015 at 09:46:38 -0700, Stephen Hemminger wrote:
> On Fri, 22 May 2015 18:54:20 +0530
> Rahul Lakkireddy <rahul.lakkireddy@chelsio.com> wrote:
> 
> > +static int cxgbe_dev_link_update(struct rte_eth_dev *eth_dev,
> > +				 __rte_unused int wait_to_complete)
> > +{
> > +	struct port_info *pi = (struct port_info *)(eth_dev->data->dev_private);
> > +	struct adapter *adapter = pi->adapter;
> > +	struct sge *s = &adapter->sge;
> > +	unsigned int work_done, budget = 4;
> > +	int ret;
> > +
> > +	cxgbe_poll(&s->fw_evtq, NULL, budget, &work_done);
> > +	ret = pi->link_cfg.link_ok;
> > +	eth_dev->data->dev_link.link_status = pi->link_cfg.link_ok;
> > +	eth_dev->data->dev_link.link_duplex = ETH_LINK_FULL_DUPLEX;
> > +	eth_dev->data->dev_link.link_speed = pi->link_cfg.speed;
> > +	return ret;
> 
> It is well not documented but link update is supposed to return 0
> on link status changed and -1 if not changed.

Ok. We will correct the return value in v2 posting.
We need to rebase anyway since PMDs now seem to have moved to
drivers/net directory.


Thanks,
Rahul.