spi: mpc8xxx: Reduce scope of loop variables
authorMario Six <mario.six@gdsys.cc>
Sun, 28 Apr 2019 20:28:46 +0000 (01:58 +0530)
committerJagan Teki <jagan@amarulasolutions.com>
Mon, 10 Jun 2019 12:29:48 +0000 (17:59 +0530)
The transmission loop starts with setting some variables, which are only
used inside the loop. Reduce the scope to the loop to make the
declaration and initialization of these variables coincide.

In the case of char_size this also always initializes the variable
immediately with the final value actually used in the loop (instead of
the placeholder value 32).

Signed-off-by: Mario Six <mario.six@gdsys.cc>
Acked-by: Jagan Teki <jagan@amarulasolutions.com>
drivers/spi/mpc8xxx_spi.c

index a2e698ea170292a1ed552d70d4bedc1ca6ee5def..2a0f3cc06a9b5007f3335c39ccef16941c3e6ad1 100644 (file)
@@ -88,10 +88,8 @@ int spi_xfer(struct spi_slave *slave, uint bitlen, const void *dout, void *din,
             ulong flags)
 {
        spi8xxx_t *spi = &((immap_t *)(CONFIG_SYS_IMMR))->spi;
-       uint tmpdout, tmpdin, event;
+       u32 tmpdin;
        int num_blks = DIV_ROUND_UP(bitlen, 32);
-       int tm;
-       uchar char_size = 32;
 
        debug("%s: slave %u:%u dout %08X din %08X bitlen %u\n", __func__,
              slave->bus, slave->cs, *(uint *)dout, *(uint *)din, bitlen);
@@ -104,8 +102,9 @@ int spi_xfer(struct spi_slave *slave, uint bitlen, const void *dout, void *din,
 
        /* Handle data in 32-bit chunks */
        while (num_blks--) {
-               tmpdout = 0;
-               char_size = (bitlen >= 32 ? 32 : bitlen);
+               int tm;
+               u32 tmpdout = 0;
+               uchar char_size = (bitlen >= 32 ? 32 : bitlen);
 
                /* Shift data so it's msb-justified */
                tmpdout = *(u32 *)dout >> (32 - char_size);
@@ -145,7 +144,7 @@ int spi_xfer(struct spi_slave *slave, uint bitlen, const void *dout, void *din,
                 * The NE event must be read and cleared first
                 */
                for (tm = 0; tm < SPI_TIMEOUT; ++tm) {
-                       event = in_be32(&spi->event);
+                       u32 event = in_be32(&spi->event);
                        bool have_ne = event & SPI_EV_NE;
                        bool have_nf = event & SPI_EV_NF;