spi: spi-sun6i: sun6i_spi_fill_fifo(): remove not needed length argument

The function sun6i_spi_fill_fifo() is called with a length argument of
"sspi->fifo_depth" and "SUN6I_FIFO_DEPTH".

The driver reads the number of free bytes in the FIFO from the hardware and
uses the length argument to limit this value. This is not needed as the number
of free bytes in the FIFO is always less or equal the depth of the FIFO.

This patch removes the length argument and check.

Signed-off-by: Marc Kleine-Budde <[email protected]>
Acked-by: Maxime Ripard <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Mark Brown <[email protected]>

diff --git a/drivers/spi/spi-sun6i.c b/drivers/spi/spi-sun6i.c
index 5849b65c3b1cd9a3dc3aa883b25fa8337f1f3b4a..2442006c42293d178c0a977a370a2be8fb5af575 100644 (file)
--- a/drivers/spi/spi-sun6i.c
+++ b/drivers/spi/spi-sun6i.c
@@ -150,15 +150,16 @@ static inline void sun6i_spi_drain_fifo(struct sun6i_spi *sspi)
        }
 }
 
        }
 }
 

-static inline void sun6i_spi_fill_fifo(struct sun6i_spi *sspi, int len)
+static inline void sun6i_spi_fill_fifo(struct sun6i_spi *sspi)

 {
        u32 cnt;
 {
        u32 cnt;

+       int len;

        u8 byte;
 
        /* See how much data we can fit */
        cnt = sspi->fifo_depth - sun6i_spi_get_tx_fifo_count(sspi);
 
        u8 byte;
 
        /* See how much data we can fit */
        cnt = sspi->fifo_depth - sun6i_spi_get_tx_fifo_count(sspi);
 

-       len = min3(len, (int)cnt, sspi->len);
+       len = min((int)cnt, sspi->len);

 
        while (len--) {
                byte = sspi->tx_buf ? *sspi->tx_buf++ : 0;
 
        while (len--) {
                byte = sspi->tx_buf ? *sspi->tx_buf++ : 0;


@@ -306,7 +307,7 @@ static int sun6i_spi_transfer_one(struct spi_master *master,
        sun6i_spi_write(sspi, SUN6I_BURST_CTL_CNT_REG, tx_len);
 
        /* Fill the TX FIFO */
        sun6i_spi_write(sspi, SUN6I_BURST_CTL_CNT_REG, tx_len);
 
        /* Fill the TX FIFO */

-       sun6i_spi_fill_fifo(sspi, sspi->fifo_depth);
+       sun6i_spi_fill_fifo(sspi);

 
        /* Enable the interrupts */
        sun6i_spi_write(sspi, SUN6I_INT_CTL_REG, SUN6I_INT_CTL_TC);
 
        /* Enable the interrupts */
        sun6i_spi_write(sspi, SUN6I_INT_CTL_REG, SUN6I_INT_CTL_TC);


@@ -360,7 +361,7 @@ static irqreturn_t sun6i_spi_handler(int irq, void *dev_id)
 
        /* Transmit FIFO 3/4 empty */
        if (status & SUN6I_INT_CTL_TF_ERQ) {
 
        /* Transmit FIFO 3/4 empty */
        if (status & SUN6I_INT_CTL_TF_ERQ) {

-               sun6i_spi_fill_fifo(sspi, SUN6I_FIFO_DEPTH);
+               sun6i_spi_fill_fifo(sspi);

 
                if (!sspi->len)
                        /* nothing left to transmit */
 
                if (!sspi->len)
                        /* nothing left to transmit */