[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAHrpEqSyuToC6QFEZygO0YyA=gV2cZEtkpRJMeZFOjhgTLdexQ@mail.gmail.com>
Date: Tue, 1 Mar 2016 15:06:40 -0600
From: Zhi Li <lznuaa@...il.com>
To: Troy Kisky <troy.kisky@...ndarydevices.com>
Cc: "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
David Miller <davem@...emloft.net>,
Duan Fugang-B38611 <B38611@...escale.com>,
Fabio Estevam <fabio.estevam@...escale.com>,
Lucas Stach <l.stach@...gutronix.de>,
Andrew Lunn <andrew@...n.ch>, tremyfr@...il.com,
Russell King - ARM Linux <linux@....linux.org.uk>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>, laci@...ndarydevices.com,
Shawn Guo <shawnguo@...nel.org>, johannes@...solutions.net,
Joshua Clayton <stillcompiling@...il.com>,
Sergei Shtylyov <sergei.shtylyov@...entembedded.com>,
Arnd Bergmann <arnd@...db.de>
Subject: Re: [PATCH net-next V2 03/16] net: fec: pass txq to fec_enet_tx_queue
instead of queue_id
On Wed, Feb 24, 2016 at 6:36 PM, Troy Kisky
<troy.kisky@...ndarydevices.com> wrote:
> queue_id is the qid member of struct bufdesc_prop.
>
> Signed-off-by: Troy Kisky <troy.kisky@...ndarydevices.com>
> ---
> drivers/net/ethernet/freescale/fec_main.c | 17 ++++++-----------
> 1 file changed, 6 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/net/ethernet/freescale/fec_main.c b/drivers/net/ethernet/freescale/fec_main.c
> index 9619b9e..c517194 100644
> --- a/drivers/net/ethernet/freescale/fec_main.c
> +++ b/drivers/net/ethernet/freescale/fec_main.c
> @@ -1156,25 +1156,18 @@ fec_enet_hwtstamp(struct fec_enet_private *fep, unsigned ts,
> hwtstamps->hwtstamp = ns_to_ktime(ns);
> }
>
> -static void
> -fec_enet_tx_queue(struct net_device *ndev, u16 queue_id)
> +static void fec_txq(struct net_device *ndev, struct fec_enet_private *fep,
> + struct fec_enet_priv_tx_q *txq)
you can get fep from ndev.
best regards
Frank Li
> {
> - struct fec_enet_private *fep;
> struct bufdesc *bdp;
> unsigned short status;
> struct sk_buff *skb;
> - struct fec_enet_priv_tx_q *txq;
> struct netdev_queue *nq;
> int index = 0;
> int entries_free;
>
> - fep = netdev_priv(ndev);
> -
> - queue_id = FEC_ENET_GET_QUQUE(queue_id);
> -
> - txq = fep->tx_queue[queue_id];
> /* get next bdp of dirty_tx */
> - nq = netdev_get_tx_queue(ndev, queue_id);
> + nq = netdev_get_tx_queue(ndev, txq->bd.qid);
> bdp = txq->dirty_tx;
>
> /* get next bdp of dirty_tx */
> @@ -1268,11 +1261,13 @@ static void
> fec_enet_tx(struct net_device *ndev)
> {
> struct fec_enet_private *fep = netdev_priv(ndev);
> + struct fec_enet_priv_tx_q *txq;
> u16 queue_id;
> /* First process class A queue, then Class B and Best Effort queue */
> for_each_set_bit(queue_id, &fep->work_tx, FEC_ENET_MAX_TX_QS) {
> clear_bit(queue_id, &fep->work_tx);
> - fec_enet_tx_queue(ndev, queue_id);
> + txq = fep->tx_queue[FEC_ENET_GET_QUQUE(queue_id)];
> + fec_txq(ndev, fep, txq);
> }
> return;
> }
> --
> 2.5.0
>
Powered by blists - more mailing lists