<p>Damien Zammit has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21218">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">cpu/x86 asm: Just use the correct op suffix<br><br>LLVM AS doesn't support as much GNU junk extensions, data16/32<br>is almost never needed in truth if we just use the correct op<br>suffix. So do that here, fixes clang/llvm builds with the<br>integrated-as toggled on.<br><br>Change-Id: I6095d03d0289b418a49a10f135de5eb0e117cae0<br>Also-by: Damien Zammit <damien@zamaudio.com><br>Signed-off-by: Damien Zammit <damien@zamaudio.com><br>---<br>M src/cpu/x86/lapic/secondary.S<br>M src/cpu/x86/sipi_vector.S<br>M src/cpu/x86/smm/smm_stub.S<br>M src/cpu/x86/smm/smmhandler.S<br>M src/device/oprom/realmode/x86_asm.S<br>5 files changed, 10 insertions(+), 10 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/18/21218/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/cpu/x86/lapic/secondary.S b/src/cpu/x86/lapic/secondary.S<br>index 671114a..48360ad 100644<br>--- a/src/cpu/x86/lapic/secondary.S<br>+++ b/src/cpu/x86/lapic/secondary.S<br>@@ -31,7 +31,7 @@<br>   movw    %cs, %ax<br>      movw    %ax, %ds<br> <br>-  data32  lgdt    gdtaddr  - _secondary_start<br>+  lgdtl   gdtaddr  - _secondary_start<br> <br>        movl    %cr0, %eax<br>    andl    $0x7FFAFFD1, %eax /* PG,AM,WP,NE,TS,EM,MP = 0 */<br>diff --git a/src/cpu/x86/sipi_vector.S b/src/cpu/x86/sipi_vector.S<br>index e654915..8e6925f 100644<br>--- a/src/cpu/x86/sipi_vector.S<br>+++ b/src/cpu/x86/sipi_vector.S<br>@@ -79,7 +79,7 @@<br>      movl    $(gdtaddr), %ebx<br>      sub     $(_start), %ebx<br> <br>-   data32 lgdt (%ebx)<br>+   lgdtl (%ebx)<br> <br>       movl    %cr0, %eax<br>    andl    $~CR0_CLEAR_FLAGS, %eax<br>diff --git a/src/cpu/x86/smm/smm_stub.S b/src/cpu/x86/smm/smm_stub.S<br>index 824a26b..32435a0 100644<br>--- a/src/cpu/x86/smm/smm_stub.S<br>+++ b/src/cpu/x86/smm/smm_stub.S<br>@@ -68,7 +68,7 @@<br> .global _start<br> _start:<br>        movl    $(smm_relocate_gdt), %ebx<br>-    data32  lgdt (%ebx)<br>+  lgdtl   (%ebx)<br> <br>     movl    %cr0, %eax<br>    andl    $~CR0_CLEAR_FLAGS, %eax<br>@@ -76,7 +76,7 @@<br>    movl    %eax, %cr0<br> <br>         /* Enable protected mode */<br>-  data32  ljmp $0x8, $smm_trampoline32<br>+ ljmpl   $0x8, $smm_trampoline32<br> <br> .align 4<br> smm_relocate_gdt:<br>diff --git a/src/cpu/x86/smm/smmhandler.S b/src/cpu/x86/smm/smmhandler.S<br>index dd8a0c0..b57d015 100644<br>--- a/src/cpu/x86/smm/smmhandler.S<br>+++ b/src/cpu/x86/smm/smmhandler.S<br>@@ -110,7 +110,7 @@<br> untampered_lapic:<br> #endif<br>  movw    $(smm_gdtptr16 - smm_handler_start + SMM_HANDLER_OFFSET), %bx<br>-        data32  lgdt %cs:(%bx)<br>+       lgdtl   %cs:(%bx)<br> <br>  movl    %cr0, %eax<br>    andl    $0x7FFAFFD1, %eax /* PG,AM,WP,NE,TS,EM,MP = 0 */<br>@@ -118,7 +118,7 @@<br>         movl    %eax, %cr0<br> <br>         /* Enable protected mode */<br>-  data32  ljmp    $0x08, $1f<br>+   ljmpl   $0x08, $1f<br> <br> .code32<br> 1:<br>diff --git a/src/device/oprom/realmode/x86_asm.S b/src/device/oprom/realmode/x86_asm.S<br>index 2bb2a3f..87348cd 100644<br>--- a/src/device/oprom/realmode/x86_asm.S<br>+++ b/src/device/oprom/realmode/x86_asm.S<br>@@ -177,7 +177,7 @@<br>        /* Now that we are in protected mode<br>   * jump to a 32 bit code segment.<br>      */<br>-  data32  ljmp    $0x10, $RELOCATED(1f)<br>+        ljmpl   $0x10, $RELOCATED(1f)<br> 1:<br>    .code32<br>       mov     $0x18, %ax<br>@@ -253,7 +253,7 @@<br>       movl    %eax, %cr0<br> <br>         /* Now really going into real mode */<br>-        data32 ljmp     $0, $RELOCATED(1f)<br>+   ljmpl   $0, $RELOCATED(1f)<br> 1:<br> <br>    /* put the stack at the end of page zero.  That way we can easily<br>@@ -297,7 +297,7 @@<br>        movl    %eax, %cr0<br> <br>         /* Now that we are in protected mode jump to a 32-bit code segment. */<br>-       data32  ljmp    $0x10, $RELOCATED(1f)<br>+        ljmpl   $0x10, $RELOCATED(1f)<br> 1:<br>    .code32<br>       mov     $0x18, %ax<br>@@ -344,7 +344,7 @@<br>       movl    %eax, %cr0<br> <br>         /* ... and jump to a 32 bit code segment. */<br>- data32 ljmp    $0x10, $RELOCATED(1f)<br>+ ljmpl   $0x10, $RELOCATED(1f)<br> 1:<br>    .code32<br>       mov     $0x18, %ax<br></pre><p>To view, visit <a href="https://review.coreboot.org/21218">change 21218</a>. To unsubscribe, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/21218"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I6095d03d0289b418a49a10f135de5eb0e117cae0 </div>
<div style="display:none"> Gerrit-Change-Number: 21218 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Damien Zammit <damien@zamaudio.com> </div>