On 02/18/2017 02:50 PM, Lino Sanfilippo wrote:
Hi,

On 17.02.2017 22:07, Pavel Belous wrote:
From: Pavel Belous <pavel.bel...@aquantia.com>

We should allocate the number of pages based on the config parameter
AQ_CFG_RX_FRAME_MAX.

Signed-off-by: Pavel Belous <pavel.bel...@aquantia.com>

        do {
                if (spin_trylock(&ring->header.lock)) {
-                       frags = aq_nic_map_skb(self, skb, &buffers[0]);
+                       frags = aq_nic_map_skb(self, skb, buffers);

-                       aq_ring_tx_append_buffs(ring, &buffers[0], frags);
+                       aq_ring_tx_append_buffs(ring, buffers, frags);


This change has nothing to do with what the commit message claims that the
patch is about. Please dont mix fixes and totally unrelated cleanups in one
patch.

Regards,
Lino


Sorry, its just small fix for readability.
I will remove it or put in separate patch in v3.

Regards,
Pavel

Reply via email to