Changeset fb78ae72 in mainline for uspace/drv/uhci-hcd

Timestamp:

2011-02-19T23:34:48Z (15 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

423e8c81, deece2f

Parents:

b9d910f

Message:

Move interrupt enabling code to pciintel driver

Location:

uspace/drv/uhci-hcd

Files:

• main.c (modified) (3 diffs)
• pci.c (modified) (1 diff)
• pci.h (modified) (1 diff)
• uhci.h (modified) (1 diff)

uspace/drv/uhci-hcd/main.c

@@ -34 +34 @@
 #include <driver.h>
 #include <usb_iface.h>
-#include <ipc/irc.h>
-#include <ipc/ns.h>
-#include <ipc/services.h>
-#include <sysinfo.h>
+#include <device/hw_res.h>
 
 #include <errno.h>
@@ -84 +81 @@
         assert(device);
         uhci_t *hc = dev_to_uhci(device);
-        usb_log_info("LOL HARDWARE INTERRUPT: %p.\n", hc);
         uint16_t status = IPC_GET_ARG1(*call);
         assert(hc);
@@ -111 +107 @@
 
         CHECK_RET_RETURN(ret,
-            "Failed(%d) to get I/O registers addresses for device:.\n",
-            ret, device->handle);
+            "Failed(%d) to get I/O addresses:.\n", ret, device->handle);
         usb_log_info("I/O regs at 0x%X (size %zu), IRQ %d.\n",
             io_reg_base, io_reg_size, irq);
 
-
-  sysarg_t apic;
-  sysarg_t i8259;
-        int irc_phone = -1;
-        int irc_service = 0;
-
-  if ((sysinfo_get_value("apic", &apic) == EOK) && (apic)) {
-    irc_service = SERVICE_APIC;
-                usb_log_debug("SERVICE_APIC\n");
-        } else if ((sysinfo_get_value("i8259", &i8259) == EOK) && (i8259)) {
-    irc_service = SERVICE_I8259;
-                usb_log_debug("SERVICE_I8259\n");
-        }
-
-  if (irc_service) {
-    while (irc_phone < 0)
-      irc_phone = service_connect_blocking(irc_service, 0, 0);
-  }
-        usb_log_debug("Interrupt conttroller phone: %d\n", irc_phone);
-
-        async_msg_1(irc_phone, IRC_ENABLE_INTERRUPT, irq);
-        async_hangup(irc_phone);
-
+        ret = pci_enable_interrupts(device);
+        CHECK_RET_RETURN(ret, "Failed(%d) to get enable interrupts:\n", ret);
 
         uhci_t *uhci_hc = malloc(sizeof(uhci_t));

uspace/drv/uhci-hcd/pci.c

@@ -121 +121 @@
         return rc;
 }
-
+/*----------------------------------------------------------------------------*/
+int pci_enable_interrupts(device_t *device)
+{
+        int parent_phone = devman_parent_device_connect(device->handle,
+            IPC_FLAG_BLOCKING);
+        bool enabled = hw_res_enable_interrupt(parent_phone);
+        return enabled ? EOK : EIO;
+}
 /**
  * @}

uspace/drv/uhci-hcd/pci.h

@@ -39 +39 @@
 
 int pci_get_my_registers(device_t *, uintptr_t *, size_t *, int *);
+int pci_enable_interrupts(device_t *device);
 
 #endif

uspace/drv/uhci-hcd/uhci.h

@@ -104 +104 @@
 static inline void uhci_fini(uhci_t *instance) {};
 
-int uhci_transfer(
-  uhci_t *instance,
-  device_t *dev,
-  usb_target_t target,
-  usb_transfer_type_t transfer_type,
-        bool toggle,
-  usb_packet_id pid,
-        bool low_speed,
-  void *buffer, size_t size,
-  usbhc_iface_transfer_out_callback_t callback_out,
-  usbhc_iface_transfer_in_callback_t callback_in,
-  void *arg );
-
 int uhci_schedule(uhci_t *instance, batch_t *batch);