spi: imx: rename 'bpw' variables
authorSascha Hauer <s.hauer@pengutronix.de>
Fri, 2 Jun 2017 05:38:04 +0000 (07:38 +0200)
committerMark Brown <broonie@kernel.org>
Tue, 6 Jun 2017 18:48:23 +0000 (19:48 +0100)
'bpw' is ambiguous and only the context makes sure if bytes_per_word
or bits_per_word is meant. Use the full names instead to make reading
the code easier.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-imx.c

index 12e5ef7beb0b9d14dae17aa6647595fcfcd8e608..e544f45348cc3d0c9f60338f3db277191fcecf77 100644 (file)
@@ -200,27 +200,27 @@ out:
        return i;
 }
 
-static int spi_imx_bytes_per_word(const int bpw)
+static int spi_imx_bytes_per_word(const int bits_per_word)
 {
-       return DIV_ROUND_UP(bpw, BITS_PER_BYTE);
+       return DIV_ROUND_UP(bits_per_word, BITS_PER_BYTE);
 }
 
 static bool spi_imx_can_dma(struct spi_master *master, struct spi_device *spi,
                         struct spi_transfer *transfer)
 {
        struct spi_imx_data *spi_imx = spi_master_get_devdata(master);
-       unsigned int bpw, i;
+       unsigned int bytes_per_word, i;
 
        if (!master->dma_rx)
                return false;
 
-       bpw = spi_imx_bytes_per_word(transfer->bits_per_word);
+       bytes_per_word = spi_imx_bytes_per_word(transfer->bits_per_word);
 
-       if (bpw != 1 && bpw != 2 && bpw != 4)
+       if (bytes_per_word != 1 && bytes_per_word != 2 && bytes_per_word != 4)
                return false;
 
        for (i = spi_imx_get_fifosize(spi_imx) / 2; i > 0; i--) {
-               if (!(transfer->len % (i * bpw)))
+               if (!(transfer->len % (i * bytes_per_word)))
                        break;
        }