All of lore.kernel.org
 help / color / mirror / Atom feed
From: Li Ye-B37916 <b37916@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] imximage: Fix the bootdata.size calculation
Date: Wed, 22 Oct 2014 16:32:40 +0800	[thread overview]
Message-ID: <54476BA8.6080804@freescale.com> (raw)
In-Reply-To: <5447675C.3070708@denx.de>

Hi Stefano,

On 10/22/2014 4:14 PM, Stefano Babic wrote:
> Hi Ye,
>
> On 22/10/2014 09:38, Li Ye-B37916 wrote:
>
>> You can look into the "Figure 8-21. Image Vector Table" in the system boot chapter of i.MX6Q manual. The bootdata.start points
>> to the beginning of the destination memory, which means the bootdata.size should have IVT offset included.
> Well, ok, this is an interpretation....
>
>>  We also have checked
>> the boot rom codes for this.
> ok - this is the most important thing. Is it the same for all MX6 and
> MX53 ?

Yes. This is same for all MX6 and MX53.

>
>> We found this issue when booting from QSPI NOR on i.MX6SX. The u-boot runs into abnormal (crash or stop) after booting.  We checked the
>> destination memory where the image is loaded to, and found hundreds of bytes at the image end are not loaded into memory. Since there
>> is a 4096 bytes round in the calculation, so the image size decides if the issue can be reproduced. It is not easy to see the issue by SD boot.
> ok - the only thing that it looks weird is that the issue happens now
> and not with a MX6Q(D) booting from SPI-NOR. There are several boards
> booting from SPI, but I have not heard about such as problems.
>
> Best regards,
> Stefano Babic
>
>
There are some reasons:
1. The SPI-NOR IVT offset is 0x400, but QSPI-NOR IVT offset is 0x1000.  Because of the 0x1000 round, the possibility of showing the issue on SPI-NOR is less than QSPI-NOR.  Even for QSPI-NOR, we only
reproduce it on some image with particular size .

2. The IVT_offset was once included in the calculation before this patch "2013-08-31     Stefano Babic    tools: imx_header should not include flash_offset".

3. At the end of the image, most of data are zero.

Best regards,
Ye Li

  reply	other threads:[~2014-10-22  8:32 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-22  6:39 [U-Boot] [PATCH] imximage: Fix the bootdata.size calculation Ye.Li
2014-10-22  7:22 ` Stefano Babic
2014-10-22  7:38   ` Li Ye-B37916
2014-10-22  8:14     ` Stefano Babic
2014-10-22  8:32       ` Li Ye-B37916 [this message]
2014-10-22 16:52     ` Fabio Estevam

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=54476BA8.6080804@freescale.com \
    --to=b37916@freescale.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.