[coreboot-gerrit] New patch to review for coreboot: fb07d5a vendorcode/amd/agesa/*/gcc-intrin.h: Invaild inline asm

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Thu May 15 13:49:45 CEST 2014


Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5758

-gerrit

commit fb07d5a6b9f9824d92e8b71d256b054a0b86a204
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Thu May 15 21:13:33 2014 +1000

    vendorcode/amd/agesa/*/gcc-intrin.h: Invaild inline asm
    
    immediate or register makes no sense here, only m is sensible (a memory
    reference).
    
    Change-Id: Iaf3ad65791640e1060a2029e7ebb043f57b338a9
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 src/vendorcode/amd/agesa/f14/Include/gcc-intrin.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/vendorcode/amd/agesa/f14/Include/gcc-intrin.h b/src/vendorcode/amd/agesa/f14/Include/gcc-intrin.h
index 5ce3ee3..519c257 100644
--- a/src/vendorcode/amd/agesa/f14/Include/gcc-intrin.h
+++ b/src/vendorcode/amd/agesa/f14/Include/gcc-intrin.h
@@ -544,7 +544,7 @@ static __inline__ __attribute__((always_inline)) void __writefsdword(const unsig
 static __inline__ __attribute__((always_inline)) unsigned char __readfsbyte(const unsigned long Offset)
 {
   unsigned char value;
-  __asm__("movb %%fs:%a[Offset], %b[value]" : [value] "=q" (value) : [Offset] "irm" (Offset));
+  __asm__("movb %%fs:%[Offset], %b[value]" : [value] "=q" (value) : [Offset] "m" (Offset));
   return value;
 }
 



More information about the coreboot-gerrit mailing list