[U-Boot] [PATCH v5] board/fsl/lx2160a: Fix MC firmware loading during SD boot

[Prabhakar Kushwaha]

> -----Original Message-----
> From: Pankaj Bansal
> Sent: Monday, April 15, 2019 9:39 AM
> To: Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>; Meenakshi
> Aggarwal <meenakshi.aggarwal at nxp.com>; Priyanka Jain
> <priyanka.jain at nxp.com>
> Cc: u-boot at 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 at nxp.com>; Meenakshi Aggarwal
> > <meenakshi.aggarwal at nxp.com>; Priyanka Jain <priyanka.jain at nxp.com>
> > Cc: u-boot at 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 at nxp.com>; Priyanka Jain
> > > <priyanka.jain at nxp.com>; Prabhakar Kushwaha
> > > <prabhakar.kushwaha at nxp.com>
> > > Cc: u-boot at lists.denx.de; Pankaj Bansal <pankaj.bansal at 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