MLK-20233 imx8: Fix QSPI read in container parser
authorAbel Vesa <abel.vesa@nxp.com>
Wed, 7 Nov 2018 16:12:22 +0000 (18:12 +0200)
committerYe Li <ye.li@nxp.com>
Fri, 24 May 2019 09:39:10 +0000 (02:39 -0700)
The check for CONFIG_SPL_SPI_LOAD is fixed, get rid of ret local variable
(that's actually a bug) and fix the length for the spi_flash_read call.

Signed-off-by: Abel Vesa <abel.vesa@nxp.com>
Reviewed-by: Ye Li <ye.li@nxp.com>
(cherry picked from commit 32adc4aed8864fe274de13f14f4e29759062ee1d)

arch/arm/mach-imx/imx8/parser.c

index 8b94cf7..631f4a3 100644 (file)
@@ -55,12 +55,13 @@ static int read(int start, int len, void *load_addr)
                        return -EIO;
                }
        }
-#elif CONFIG_SPL_SPI_LOAD
+#endif
+#ifdef CONFIG_SPL_SPI_LOAD
        if (current_dev_type == QSPI_DEV) {
                struct spi_flash *flash = (struct spi_flash *)device;
 
-               int ret = spi_flash_read(flash, start,
-                                       CONTAINER_HDR_ALIGNMENT, load_addr);
+               ret = spi_flash_read(flash, start,
+                                    len, load_addr);
                if (ret != 0) {
                        debug("Read container image from QSPI failed\n");
                        return -EIO;