Commit 7dd22a86 authored by Shannon Nelson's avatar Shannon Nelson Committed by David S. Miller

ionic: add lif param to ionic_qcq_disable

Add the lif parameter for use in an error message, and
to better match the style of most of the functions calls.
Signed-off-by: default avatarShannon Nelson <snelson@pensando.io>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 3a5e0faf
...@@ -287,10 +287,9 @@ static int ionic_qcq_enable(struct ionic_qcq *qcq) ...@@ -287,10 +287,9 @@ static int ionic_qcq_enable(struct ionic_qcq *qcq)
return ionic_adminq_post_wait(lif, &ctx); return ionic_adminq_post_wait(lif, &ctx);
} }
static int ionic_qcq_disable(struct ionic_qcq *qcq, int fw_err) static int ionic_qcq_disable(struct ionic_lif *lif, struct ionic_qcq *qcq, int fw_err)
{ {
struct ionic_queue *q; struct ionic_queue *q;
struct ionic_lif *lif;
struct ionic_admin_ctx ctx = { struct ionic_admin_ctx ctx = {
.work = COMPLETION_INITIALIZER_ONSTACK(ctx.work), .work = COMPLETION_INITIALIZER_ONSTACK(ctx.work),
...@@ -300,11 +299,12 @@ static int ionic_qcq_disable(struct ionic_qcq *qcq, int fw_err) ...@@ -300,11 +299,12 @@ static int ionic_qcq_disable(struct ionic_qcq *qcq, int fw_err)
}, },
}; };
if (!qcq) if (!qcq) {
netdev_err(lif->netdev, "%s: bad qcq\n", __func__);
return -ENXIO; return -ENXIO;
}
q = &qcq->q; q = &qcq->q;
lif = q->lif;
if (qcq->flags & IONIC_QCQ_F_INTR) { if (qcq->flags & IONIC_QCQ_F_INTR) {
struct ionic_dev *idev = &lif->ionic->idev; struct ionic_dev *idev = &lif->ionic->idev;
...@@ -1948,19 +1948,19 @@ static void ionic_txrx_disable(struct ionic_lif *lif) ...@@ -1948,19 +1948,19 @@ static void ionic_txrx_disable(struct ionic_lif *lif)
if (lif->txqcqs) { if (lif->txqcqs) {
for (i = 0; i < lif->nxqs; i++) for (i = 0; i < lif->nxqs; i++)
err = ionic_qcq_disable(lif->txqcqs[i], err); err = ionic_qcq_disable(lif, lif->txqcqs[i], err);
} }
if (lif->hwstamp_txq) if (lif->hwstamp_txq)
err = ionic_qcq_disable(lif->hwstamp_txq, err); err = ionic_qcq_disable(lif, lif->hwstamp_txq, err);
if (lif->rxqcqs) { if (lif->rxqcqs) {
for (i = 0; i < lif->nxqs; i++) for (i = 0; i < lif->nxqs; i++)
err = ionic_qcq_disable(lif->rxqcqs[i], err); err = ionic_qcq_disable(lif, lif->rxqcqs[i], err);
} }
if (lif->hwstamp_rxq) if (lif->hwstamp_rxq)
err = ionic_qcq_disable(lif->hwstamp_rxq, err); err = ionic_qcq_disable(lif, lif->hwstamp_rxq, err);
ionic_lif_quiesce(lif); ionic_lif_quiesce(lif);
} }
...@@ -2160,7 +2160,7 @@ static int ionic_txrx_enable(struct ionic_lif *lif) ...@@ -2160,7 +2160,7 @@ static int ionic_txrx_enable(struct ionic_lif *lif)
err = ionic_qcq_enable(lif->txqcqs[i]); err = ionic_qcq_enable(lif->txqcqs[i]);
if (err) { if (err) {
derr = ionic_qcq_disable(lif->rxqcqs[i], err); derr = ionic_qcq_disable(lif, lif->rxqcqs[i], err);
goto err_out; goto err_out;
} }
} }
...@@ -2182,13 +2182,13 @@ static int ionic_txrx_enable(struct ionic_lif *lif) ...@@ -2182,13 +2182,13 @@ static int ionic_txrx_enable(struct ionic_lif *lif)
err_out_hwstamp_tx: err_out_hwstamp_tx:
if (lif->hwstamp_rxq) if (lif->hwstamp_rxq)
derr = ionic_qcq_disable(lif->hwstamp_rxq, derr); derr = ionic_qcq_disable(lif, lif->hwstamp_rxq, derr);
err_out_hwstamp_rx: err_out_hwstamp_rx:
i = lif->nxqs; i = lif->nxqs;
err_out: err_out:
while (i--) { while (i--) {
derr = ionic_qcq_disable(lif->txqcqs[i], derr); derr = ionic_qcq_disable(lif, lif->txqcqs[i], derr);
derr = ionic_qcq_disable(lif->rxqcqs[i], derr); derr = ionic_qcq_disable(lif, lif->rxqcqs[i], derr);
} }
return err; return err;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment