From mboxrd@z Thu Jan  1 00:00:00 1970
From: Anatolij Gustschin <agust@denx.de>
Date: Tue, 22 Oct 2019 12:16:52 +0200
Subject: [U-Boot] [PATCH] imx: imx8: fix loading container image from
 eMMC boot partitions
In-Reply-To: <AM0PR04MB448179F5527773C12197C8BE88680@AM0PR04MB4481.eurprd04.prod.outlook.com>
References: <20191021152154.15385-1-agust@denx.de>
 <DB7PR04MB4490F29A0222A4786D2A508188680@DB7PR04MB4490.eurprd04.prod.outlook.com>
 <20191022093422.54bc10f8@crub>
 <AM0PR04MB4481D2F2CFB825C902ADED1088680@AM0PR04MB4481.eurprd04.prod.outlook.com>
 <20191022114906.1386b50c@crub>
 <AM0PR04MB448179F5527773C12197C8BE88680@AM0PR04MB4481.eurprd04.prod.outlook.com>
Message-ID: <20191022121652.6ed5674b@crub>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Tue, 22 Oct 2019 10:08:42 +0000
Peng Fan peng.fan at nxp.com wrote:
... 
> > > From common/spl/spl_mmc.c, there is boot partition switch there, Why
> > > need to add such logic in imx8/image.c?  
> > 
> > spl_mmc_load() reads the container header before partition switching.  
> 
> Oh. Indeeded. Not took care of emmc when adding spl_mmc_get_uboot_raw_sector (:

I think we should avoid adding partition switching in get_container_size(),
it looks like container header is read several times which is suboptimal.
I'll have to rethink this patch, don't have time for this now.

--
Anatolij