On 16 December 2015 at 13:23, Leif Lindholm <leif.lindh...@linaro.org> wrote:
> On Wed, Dec 16, 2015 at 12:16:01PM +0000, Mark Rutland wrote:
>> On Wed, Dec 16, 2015 at 12:24:30PM +0100, Ard Biesheuvel wrote:
>> > On 16 December 2015 at 12:18, Mark Rutland <mark.rutl...@arm.com> wrote:
>> > > On Wed, Dec 16, 2015 at 10:37:39AM +0100, Ard Biesheuvel wrote:
>> > >> diff --git a/ArmPkg/Include/Chipset/AArch64.h 
>> > >> b/ArmPkg/Include/Chipset/AArch64.h
>> > >> index 5e1653bcb69d..7be18546799c 100644
>> > >> --- a/ArmPkg/Include/Chipset/AArch64.h
>> > >> +++ b/ArmPkg/Include/Chipset/AArch64.h
>> > >> @@ -119,15 +119,18 @@
>> > >>  #define ARM_VECTOR_LOW_A32_SERR 0x780
>> > >>
>> > >>  #define VECTOR_BASE(tbl)          \
>> > >> +  .section .text.##tbl##,"aw";    \
>> > >
>> > > Shouldn't this have "x", too?
>> > >
>> > > Do we overwrite this? If not, it could just be "ax" rather than "awx".
>> > >
>> >
>> > Ah yes, I meant "ax" not "aw"
>>
>> Ok. With that:
>>
>> Acked-by: Mark Rutland <mark.rutl...@arm.com>
>
> If Mark's happy:
> Reviewed-by: Leif Lindholm <leif.lindh...@linaro.org>
>

Thanks guys.

Committed as SVN r19303
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to