[U-Boot] [PATCH v2 15/47] x86: Tidy up a few minor issues with interrupts

Simon Glass sjg at chromium.org
Mon Aug 3 01:37:34 CEST 2015


On 2 August 2015 at 07:19, Bin Meng <bmeng.cn at gmail.com> wrote:
> 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>

Applied to u-boot-x86.


More information about the U-Boot mailing list