All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alex G. <mr.nuke.me@gmail.com>
To: u-boot@lists.denx.de
Subject: [PATCH] Revert "Fix data abort caused by mis-aligning FIT data"
Date: Mon, 19 Oct 2020 19:31:12 -0500	[thread overview]
Message-ID: <bdc00453-bd0a-67d6-bafa-53d0e184fdfa@gmail.com> (raw)
In-Reply-To: <15879d1e49eb40de8859944e7ab7b7f4@4rf.com>

On 10/19/20 6:13 PM, Reuben Dowle wrote:
> 
>> The reverted change linked to some kernel documentation that requires 64-
>> bit alignment. I agree with the alignment requirement.
>>
>> Im my opinion, there are two things that need to be done:
>>
>> First is to look at an ALIGNED address for the fdt. A summary inspection of
>> board_fdt_blob_setup() tells us this is done via the "_end" linker symbol.
> 
> The linker script can only control padding of the executable, but won't 
> affect the alignment of the fdt that can be appended to this later by 
> mkimage.

Which I've addressed in the second paragraph :)

>> Second is to put things in the right place. For FIT, the code, as is, is correct,
>> but this alignment is not guaranteed for legacy images. I think somebody
>> mentioned changing the arguments to mkimage to achieve this.
>>
>> I've tried to fix the first point by aligning the _end symbol (appendix A).
>> Unfortunately, this is causing other build issues that I don't know how to deal
>> with.
>>
>> Alex

> ------------------------------------------------------------------------
> The information in this email communication (inclusive of attachments) 
> is confidential to 4RF Limited and the intended recipient(s). If you are 
> not the intended recipient(s), please note that any use, disclosure, 
> distribution or copying of this information or any part thereof is 
> strictly prohibited and that the author accepts no liability for the 
> consequences of any action taken on the basis of the information 
> provided. If you have received this email in error, please notify the 
> sender immediately by return email and then delete all instances of this 
> email from your system. 4RF Limited will not accept responsibility for 
> any consequences associated with the use of this email (including, but 
> not limited to, damages sustained as a result of any viruses and/or any 
> action or lack of action taken in reliance on it).

Per your instructions, I have deleted this message and all copies of it 
from my computer.

  reply	other threads:[~2020-10-20  0:31 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-19 21:40 [PATCH] Revert "Fix data abort caused by mis-aligning FIT data" Marek Vasut
2020-10-19 21:50 ` Reuben Dowle
2020-10-19 21:59   ` Marek Vasut
2020-10-19 22:17     ` Reuben Dowle
2020-10-19 22:46       ` Marek Vasut
2020-10-19 22:45     ` Tom Rini
2020-10-19 22:54       ` Marek Vasut
2020-10-19 22:58         ` Tom Rini
2020-10-19 23:02           ` Marek Vasut
2020-10-19 23:09             ` Alex G.
2020-10-19 23:13               ` Reuben Dowle
2020-10-20  0:31                 ` Alex G. [this message]
2020-10-19 23:02         ` Reuben Dowle
2020-10-19 23:11           ` Marek Vasut
2020-10-20  0:27             ` Reuben Dowle
2020-10-20  9:05               ` Marek Vasut
2020-10-20 14:07                 ` Tom Rini
2020-10-20 14:29                   ` Marek Vasut
2020-10-20 14:32                     ` Tom Rini
2020-10-20 14:38                       ` Alex G.
2020-10-20 15:54                         ` Tom Rini
2020-10-20 17:01                           ` Alex G.
2020-10-20 18:10                             ` Tom Rini
2020-10-21 17:11                               ` Alex G.
2020-10-20 14:42                       ` Marek Vasut
2020-10-21 23:12 ` Tom Rini

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=bdc00453-bd0a-67d6-bafa-53d0e184fdfa@gmail.com \
    --to=mr.nuke.me@gmail.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.