The variable "char_size" holds the number of bits to be transferred in
the current loop iteration. A better name would be "xfer_bitlen", which
we rename this variable to.

Signed-off-by: Mario Six <mario....@gdsys.cc>
---

Changes v1 -> v2:
None

---
 drivers/spi/mpc8xxx_spi.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/spi/mpc8xxx_spi.c b/drivers/spi/mpc8xxx_spi.c
index af3762737f..ce2a77c1aa 100644
--- a/drivers/spi/mpc8xxx_spi.c
+++ b/drivers/spi/mpc8xxx_spi.c
@@ -115,10 +115,10 @@ int spi_xfer(struct spi_slave *slave, uint bitlen, const 
void *dout, void *din,
        while (num_blks--) {
                int tm;
                u32 tmpdout = 0;
-               uchar char_size = (bitlen >= 32 ? 32 : bitlen);
+               uchar xfer_bitlen = (bitlen >= 32 ? 32 : bitlen);

                /* Shift data so it's msb-justified */
-               tmpdout = *(u32 *)dout >> (32 - char_size);
+               tmpdout = *(u32 *)dout >> (32 - xfer_bitlen);

                /* The LEN field of the SPMODE register is set as follows:
                 *
@@ -166,8 +166,8 @@ int spi_xfer(struct spi_slave *slave, uint bitlen, const 
void *dout, void *din,
                        tmpdin = in_be32(&spi->rx);
                        setbits_be32(&spi->event, SPI_EV_NE);

-                       *(u32 *)din = (tmpdin << (32 - char_size));
-                       if (char_size == 32) {
+                       *(u32 *)din = (tmpdin << (32 - xfer_bitlen));
+                       if (xfer_bitlen == 32) {
                                /* Advance output buffer by 32 bits */
                                din += 4;
                        }
--
2.16.1

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to