On 4/4/21 3:50 PM, Richard Cochran wrote:
On Thu, Apr 01, 2021 at 10:56:00AM -0700, Shannon Nelson wrote:
@@ -1722,11 +1722,15 @@ static void ionic_txrx_free(struct ionic_lif *lif)
static int ionic_txrx_alloc(struct ionic_lif *lif)
  {
-       unsigned int sg_desc_sz;
+       unsigned int num_desc, desc_sz, comp_sz, sg_desc_sz;
        unsigned int flags;
        unsigned int i;
Coding Style nit: List of ints wants alphabetically order,
List can also fir 'flags' and 'i' with the others.

@@ -2246,9 +2258,9 @@ static void ionic_swap_queues(struct ionic_qcq *a, struct 
ionic_qcq *b)
  int ionic_reconfigure_queues(struct ionic_lif *lif,
                             struct ionic_queue_params *qparam)
  {
+       unsigned int num_desc, desc_sz, comp_sz, sg_desc_sz;
        struct ionic_qcq **tx_qcqs = NULL;
        struct ionic_qcq **rx_qcqs = NULL;
-       unsigned int sg_desc_sz;
        unsigned int flags;
Ditto.

Thanks,
Richard

I can tweak this is a followup patch.  Thanks for your review time.
sln

Reply via email to