qca_spi: Introduce QCASPI_RX_MAX_FRAMES
authorStefan Wahren <wahrenst@gmx.net>
Sun, 28 Jan 2024 20:10:52 +0000 (21:10 +0100)
committerDavid S. Miller <davem@davemloft.net>
Wed, 31 Jan 2024 13:38:50 +0000 (13:38 +0000)
Currently qca_spi reserves enough space for 4 complete Ethernet over SPI
frames in the receive buffer. Unfortunately this is hidden under a magic
number. So replace it with a more self explaining define.

Signed-off-by: Stefan Wahren <wahrenst@gmx.net>
Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/qualcomm/qca_spi.c
drivers/net/ethernet/qualcomm/qca_spi.h

index f53438c4aea2337c24a9446a887cb7fd8153d63d..c0f6bd3d331ab76898a9ef14ef47ebe1c73a37dc 100644 (file)
@@ -829,7 +829,7 @@ qcaspi_netdev_init(struct net_device *dev)
        qca->burst_len = qcaspi_burst_len;
        qca->spi_thread = NULL;
        qca->buffer_size = (dev->mtu + VLAN_ETH_HLEN + QCAFRM_HEADER_LEN +
-               QCAFRM_FOOTER_LEN + 4) * 4;
+               QCAFRM_FOOTER_LEN + 4) * QCASPI_RX_MAX_FRAMES;
 
        memset(&qca->stats, 0, sizeof(struct qcaspi_stats));
 
index dcecb072b8eba643a96bbc8406045b49e2862da0..f735ad77402f497dcb178cb223eac68bad801152 100644 (file)
@@ -41,6 +41,7 @@
 
 #define QCASPI_TX_RING_MAX_LEN 10
 #define QCASPI_TX_RING_MIN_LEN 2
+#define QCASPI_RX_MAX_FRAMES 4
 
 /* sync related constants */
 #define QCASPI_SYNC_UNKNOWN 0