refactor: send the command with retry and error detection

[lunaix-os.git] / lunaix-os / hal / ioapic.c

diff --git a/lunaix-os/hal/ioapic.c b/lunaix-os/hal/ioapic.c
index 3ca3dee854047cbdd3a2ecc9d02fe01bb4029b5a..5a03eff11bd3809b6f752d733af7009700a1f12d 100644 (file)
--- a/lunaix-os/hal/ioapic.c
+++ b/lunaix-os/hal/ioapic.c
@@ -1,36 +1,45 @@
 #include <arch/x86/interrupts.h>
 #include <arch/x86/interrupts.h>

-#include <hal/ioapic.h>

 #include <hal/acpi/acpi.h>
 #include <hal/acpi/acpi.h>

+#include <hal/ioapic.h>
+#include <lunaix/common.h>
+#include <lunaix/mm/mmio.h>

 
 

+#define IOAPIC_REG_SEL *((volatile uint32_t*)(_ioapic_base + IOAPIC_IOREGSEL))
+#define IOAPIC_REG_WIN *((volatile uint32_t*)(_ioapic_base + IOAPIC_IOWIN))

 
 

-#define IOAPIC_REG_SEL     *((volatile uint32_t*)(IOAPIC_BASE_VADDR + IOAPIC_IOREGSEL))
-#define IOAPIC_REG_WIN     *((volatile uint32_t*)(IOAPIC_BASE_VADDR + IOAPIC_IOWIN))
+static volatile uintptr_t _ioapic_base;

 
 uint8_t
 ioapic_get_irq(acpi_context* acpi_ctx, uint8_t old_irq);
 
 void
 
 uint8_t
 ioapic_get_irq(acpi_context* acpi_ctx, uint8_t old_irq);
 
 void

-ioapic_init() {
+ioapic_init()
+{

     // Remapping the IRQs
     // Remapping the IRQs

-    
+

     acpi_context* acpi_ctx = acpi_get_context();
 
     acpi_context* acpi_ctx = acpi_get_context();
 

+    _ioapic_base = ioremap(acpi_ctx->madt.ioapic->ioapic_addr & ~0xfff, 4096);
+

     // Remap the IRQ 8 (rtc timer's vector) to RTC_TIMER_IV in ioapic
     //       (Remarks IRQ 8 is pin INTIN8)
     //       See IBM PC/AT Technical Reference 1-10 for old RTC IRQ
     // Remap the IRQ 8 (rtc timer's vector) to RTC_TIMER_IV in ioapic
     //       (Remarks IRQ 8 is pin INTIN8)
     //       See IBM PC/AT Technical Reference 1-10 for old RTC IRQ

-    //       See Intel's Multiprocessor Specification for IRQ - IOAPIC INTIN mapping config.
-    
+    //       See Intel's Multiprocessor Specification for IRQ - IOAPIC INTIN
+    //       mapping config.
+

     // The ioapic_get_irq is to make sure we capture those overriden IRQs
 
     // grab ourselves these irq numbers
     uint8_t irq_rtc = ioapic_get_irq(acpi_ctx, PC_AT_IRQ_RTC);
 
     // The ioapic_get_irq is to make sure we capture those overriden IRQs
 
     // grab ourselves these irq numbers
     uint8_t irq_rtc = ioapic_get_irq(acpi_ctx, PC_AT_IRQ_RTC);
 

-    // PC_AT_IRQ_RTC -> RTC_TIMER_IV, fixed, edge trigged, polarity=high, physical, APIC ID 0
+    // PC_AT_IRQ_RTC -> RTC_TIMER_IV, fixed, edge trigged, polarity=high,
+    // physical, APIC ID 0

     ioapic_redirect(irq_rtc, RTC_TIMER_IV, 0, IOAPIC_DELMOD_FIXED);
 }
 
 uint8_t
     ioapic_redirect(irq_rtc, RTC_TIMER_IV, 0, IOAPIC_DELMOD_FIXED);
 }
 
 uint8_t

-ioapic_get_irq(acpi_context* acpi_ctx, uint8_t old_irq) {
+ioapic_get_irq(acpi_context* acpi_ctx, uint8_t old_irq)
+{

     if (old_irq >= 24) {
         return old_irq;
     }
     if (old_irq >= 24) {
         return old_irq;
     }


@@ -39,19 +48,22 @@ ioapic_get_irq(acpi_context* acpi_ctx, uint8_t old_irq) {
 }
 
 void
 }
 
 void

-ioapic_write(uint8_t sel, uint32_t val) {
+ioapic_write(uint8_t sel, uint32_t val)
+{

     IOAPIC_REG_SEL = sel;
     IOAPIC_REG_WIN = val;
 }
 
 uint32_t
     IOAPIC_REG_SEL = sel;
     IOAPIC_REG_WIN = val;
 }
 
 uint32_t

-ioapic_read(uint8_t sel) {
+ioapic_read(uint8_t sel)
+{

     IOAPIC_REG_SEL = sel;
     return IOAPIC_REG_WIN;
 }
 
 void
     IOAPIC_REG_SEL = sel;
     return IOAPIC_REG_WIN;
 }
 
 void

-ioapic_redirect(uint8_t irq, uint8_t vector, uint8_t dest, uint32_t flags) {
+ioapic_redirect(uint8_t irq, uint8_t vector, uint8_t dest, uint32_t flags)
+{

     uint8_t reg_sel = IOAPIC_IOREDTBL_BASE + irq * 2;
 
     // Write low 32 bits
     uint8_t reg_sel = IOAPIC_IOREDTBL_BASE + irq * 2;
 
     // Write low 32 bits