-----Original Message----- From: Pankaj Bansal Sent: Monday, April 15, 2019 9:39 AM To: Prabhakar Kushwaha prabhakar.kushwaha@nxp.com; Meenakshi Aggarwal meenakshi.aggarwal@nxp.com; Priyanka Jain priyanka.jain@nxp.com Cc: u-boot@lists.denx.de Subject: RE: [PATCH v5] board/fsl/lx2160a: Fix MC firmware loading during SD boot
-----Original Message----- From: Prabhakar Kushwaha Sent: Wednesday, 27 March, 2019 11:50 AM To: Pankaj Bansal pankaj.bansal@nxp.com; Meenakshi Aggarwal meenakshi.aggarwal@nxp.com; Priyanka Jain priyanka.jain@nxp.com Cc: u-boot@lists.denx.de Subject: RE: [PATCH v5] board/fsl/lx2160a: Fix MC firmware loading during SD boot
-----Original Message----- From: Pankaj Bansal Sent: Monday, March 25, 2019 5:28 PM To: Meenakshi Aggarwal meenakshi.aggarwal@nxp.com; Priyanka Jain priyanka.jain@nxp.com; Prabhakar Kushwaha prabhakar.kushwaha@nxp.com Cc: u-boot@lists.denx.de; Pankaj Bansal pankaj.bansal@nxp.com Subject: [PATCH v5] board/fsl/lx2160a: Fix MC firmware loading during SD boot
during SD boot, following error comes: MMC read: dev # 0, block # 20480, count 2048 ... 2048 blocks read: OK
MMC read: dev # 0, block # 28672, count 2048 ... 2048 blocks read: OK fsl-mc: ERR: Bad firmware image (bad FIT header) Hit any key to stop autoboot: 0
it's occurring because mc 10.14.3 file size is 1064880, which means 0x820 SD blocks which is more than 0x800 blocks (1MB). This results in DPC loading address 0x80100000 overlapping with MC loading address
0x80000000.
so, update the MC/dpl/dpc addresses as per their addresses in SD card. Assuming that SD card block size is 512 bytes and 0x0 block in SD card would get loaded at 0x80000000 (DDR base address), this gives following addresses for various binaries:
Binary | SD block | DDR offset
MC | 0x5000 | 0x80a00000 DPL | 0x6800 | 0x80d00000 DPC | 0x7000 | 0x80e00000
This info should be part of README present in board folder. No need to add in commit message.
It is. Please check https://elixir.bootlin.com/u- boot/latest/source/board/freescale/lx2160a/README#L68
Yes, It is part of README.
This is the reason, I asking to remove from commit message.
--pk