[U-Boot] [PATCH v2 15/47] x86: Tidy up a few minor issues with interrupts
Bin Meng
bmeng.cn at gmail.com
Sun Aug 2 15:19:55 CEST 2015
On Fri, Jul 31, 2015 at 11:31 PM, Simon Glass <sjg at chromium.org> wrote:
> Fix a typo, remove an unused field and make sure to use existing #define
> constants instead of open-coded values.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> Changes in v2: None
>
> arch/x86/cpu/interrupts.c | 9 ++++-----
> 1 file changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/arch/x86/cpu/interrupts.c b/arch/x86/cpu/interrupts.c
> index 853c82f..3a9c2d4 100644
> --- a/arch/x86/cpu/interrupts.c
> +++ b/arch/x86/cpu/interrupts.c
> @@ -19,6 +19,7 @@
> #include <asm/processor-flags.h>
> #include <linux/compiler.h>
> #include <asm/msr.h>
> +#include <asm/processor.h>
> #include <asm/u-boot-x86.h>
> #include <asm/i8259.h>
>
> @@ -46,7 +47,7 @@ static char *exceptions[] = {
> "Invalid TSS",
> "Segment Not Present",
> "Stack Segment Fault",
> - "Gerneral Protection",
> + "General Protection",
> "Page Fault",
> "Reserved",
> "x87 FPU Floating-Point Error",
> @@ -165,7 +166,6 @@ struct idt_entry {
> struct desc_ptr {
> unsigned short size;
> unsigned long address;
> - unsigned short segment;
> } __packed;
>
> struct idt_entry idt[256] __aligned(16);
> @@ -202,14 +202,13 @@ int cpu_init_interrupts(void)
> for (i = 0; i < 256; i++) {
> idt[i].access = 0x8e;
> idt[i].res = 0;
> - idt[i].selector = 0x10;
> + idt[i].selector = X86_GDT_ENTRY_32BIT_CS * X86_GDT_ENTRY_SIZE;
> set_vector(i, irq_entry);
> irq_entry += irq_entry_size;
> }
>
> - idt_ptr.size = 256 * 8;
> + idt_ptr.size = 256 * 8 - 1;
> idt_ptr.address = (unsigned long) idt;
> - idt_ptr.segment = 0x18;
>
> load_idt(&idt_ptr);
>
> --
Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
More information about the U-Boot
mailing list