From 776da0872efcc58543afbe32fc55202521ef0014 Mon Sep 17 00:00:00 2001
From: Patrick Rudolph <siro@das-labor.org>
Date: Fri, 25 Oct 2019 08:09:33 +0200
Subject: arch/x86/*.S: use defines instead of hardcoded values

As preparation for x86_64 clean the assembly code and introduce
arch/ram_segs.h similar to existing arch/rom_segs.h.

Replace open coded segment values with the defines from the new
header.

Change-Id: Ib006cd4df59951335506b8153e9347450ec3403e
Signed-off-by: Patrick Rudolph <siro@das-labor.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/36321
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
 src/cpu/x86/lapic/secondary.S | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

(limited to 'src/cpu/x86/lapic/secondary.S')

diff --git a/src/cpu/x86/lapic/secondary.S b/src/cpu/x86/lapic/secondary.S
index 48360ad532..09cd6f7c30 100644
--- a/src/cpu/x86/lapic/secondary.S
+++ b/src/cpu/x86/lapic/secondary.S
@@ -13,6 +13,7 @@
 
 #include <cpu/x86/mtrr.h>
 #include <cpu/x86/lapic_def.h>
+#include <arch/ram_segs.h>
 
 	.text
 	.globl _secondary_start, _secondary_start_end, _secondary_gdt_addr
@@ -38,7 +39,7 @@ _secondary_start:
 	orl	$0x60000001, %eax /* CD, NW, PE = 1 */
 	movl	%eax, %cr0
 
-	ljmpl	$0x10, $__ap_protected_start
+	ljmpl	$RAM_CODE_SEG, $__ap_protected_start
 
 	/* This will get filled in by C code. */
 _secondary_gdt_addr:
@@ -51,11 +52,11 @@ _secondary_start_end:
 ap_protected_start:
 	.code32
 	lgdt	gdtaddr
-	ljmpl	$0x10, $__ap_protected_start
+	ljmpl	$RAM_CODE_SEG, $__ap_protected_start
 
 __ap_protected_start:
 
-	movw	$0x18, %ax
+	movw	$RAM_DATA_SEG, %ax
 	movw	%ax, %ds
 	movw	%ax, %es
 	movw	%ax, %ss
-- 
cgit v1.2.3