[U-Boot] [PATCH] part_efi: fix protective_mbr struct allocation

Palacios, Hector Hector.Palacios at digi.com
Wed Feb 19 13:52:07 CET 2014


On 02/19/2014 11:16 AM, Albert ARIBAUD wrote:
> On Wed, 19 Feb 2014 11:08:03 +0100, Albert ARIBAUD
>
>>> Thanks for pointing out. Now it is perfectly visible :-)
>>>
>>>>
>>>>> Inclusion of v2 has been postponed since there was a discussion if
>>>>> we shall allow unaligned access (-mno-unaligned-access flag) at
>>>>> armv7 (after patches posted by Tom).
>>>>>
>>>>> As fair as I can tell, we will keep the current approach so, I think
>>>>> that Tom will be willing to pull this patch (v2) now.
>>>>
>>>> Agreed, but then we should make sure no one has comments on V2 that
>>>> they might have withheld due to the initial rejection of V2.
>>>
>>> Any comments?
>>>
>>> This patch do fix unaligned access problem on Trats2 (Exynos4412), when
>>> we restore/create GPT, so I would like to know if there are any new
>>> inquires regarding this patch.
>>
>> Does not seem to be, so I will apply V2.
>
> Correction: I would like it to be applied as per current ARM alignment
> policy, but this patch is not ARM per se and is in Tom's hands.
>
> Tom, can you apply http://patchwork.ozlabs.org/patch/314717/ ? This
> would by no means close the discussion I opened, and in the event
> of a policy change, the patch could always be reverted; meanwhile, it
> matches our current policy.

I tested Piotr's patch on i.MX6 (armv7) custom board and it is working fine without 
the -mno-unaligned-access flag.

Tested-by: Hector Palacios <hector.palacios at digi.com>

-- 
Héctor Palacios


More information about the U-Boot mailing list