On 09.06.20 10:31, Sébastien Szymanski wrote:
Hi,
On 6/9/20 6:03 AM, Ye Li wrote:
Hi Sébastien,
-----Original Message----- From: U-Boot u-boot-bounces@lists.denx.de On Behalf Of sbabic@denx.de Sent: 2020年6月9日 1:22 To: Sébastien Szymanski sebastien.szymanski@armadeus.com; u- boot@lists.denx.de Subject: [EXT] [PATCH v2 1/1] imx: rom api: fix image offset computation
Caution: EXT Email
When not booting from FlexSPI, the offset computation is: offset = image_offset +
CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR * 512
- 0x8000; When booting from SD card or eMMC user partition,
image_offset is 0x8000. It is useless to add and remove 0x8000. When booting from other device, image_offset is 0 so this computation is
wrong.
Simplfy this computation to work on all booting devices. Signed-off-by: Sébastien Szymanski sebastien.szymanski@armadeus.com
Applied to u-boot-imx, master, thanks !
I just notice this patch. Can you elaborate the problem you met? Because from my view, your change is wrong.
- Removing the image_offset will break secondary (redundant) boot support for sd and emmc.
- When booting from emmc boot partition, the image_offset is 0. But the flash.bin
generated by mkimage with imximage-8mp-lpddr4.cfg is for sd. It expects to be burn at 32KB offset. The fit offset 0x60000 has already included the 32KB offset. So when you burn this flash.bin to emmc boot partition at offset 0, the fit offset should subtract the 32KB (0x60000 - 0x8000).
You are right ! When I tried, I used a .cfg file without SECOND_LOADER and flashed proper U-Boot "by hand" at offset 0x60000 which is wrong.
Stefano, can you drop my patch please ?
I revert it immediately - thanks Ye to have checked this before I send my PR.
Stefano
Sorry :/
Regards,
Best regards, Ye Li
Best regards, Stefano Babic
--
=========== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de ========================================================== ===========