Changeset 443695e in mainline for uspace/drv/block

Timestamp:

2024-05-21T11:33:56Z (17 months ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

master

Children:

3526f4f3

Parents:

1801005

git-author:

Jiri Svoboda <jiri@…> (2024-05-20 17:33:43)

git-committer:

Jiri Svoboda <jiri@…> (2024-05-21 11:33:56)

Message:

Basic PCI-IDE driver (no DMA support)

Also, make sure we avoid attaching ISA IDE and PCI IDE
at the same time. For simplicity, use ISA IDE on ISA systems
and PCI IDE on PCI-based systems.

Location:

uspace/drv/block

Files:

• isa-ide/isa-ide.c (modified) (1 diff)
• isa-ide/main.c (modified) (5 diffs)
• pci-ide/doc/doxygroups.h (added)
• pci-ide/main.c (added)
• pci-ide/main.h (added)
• pci-ide/meson.build (added)
• pci-ide/pci-ide.c (added)
• pci-ide/pci-ide.h (added)
• pci-ide/pci-ide.ma (added)
• pci-ide/pci-ide_hw.h (added)

uspace/drv/block/isa-ide/isa-ide.c

@@ -34 +34 @@
  * @file
  * @brief ISA IDE driver
- *
- * The driver services a single IDE channel.
  */
 

uspace/drv/block/isa-ide/main.c

@@ -66 +66 @@
 };
 
-static errno_t ata_get_res(ddf_dev_t *dev, isa_ide_hwres_t *ata_res)
+static errno_t isa_ide_get_res(ddf_dev_t *dev, isa_ide_hwres_t *res)
 {
         async_sess_t *parent_sess;
         hw_res_list_parsed_t hw_res;
+        hw_res_flags_t flags;
         errno_t rc;
 
@@ -75 +76 @@
         if (parent_sess == NULL)
                 return ENOMEM;
+
+        rc = hw_res_get_flags(parent_sess, &flags);
+        if (rc != EOK)
+                return rc;
+
+        /*
+         * Prevent attaching to the legacy ISA IDE register block
+         * on a system with PCI not to conflict with PCI IDE.
+         *
+         * XXX This is a simplification. If we had a PCI-based system without
+         * PCI-IDE or with PCI-IDE disabled and would still like to use
+         * an ISA IDE controller, this would prevent us from doing so.
+         */
+        if (flags & hwf_isa_bridge) {
+                ddf_msg(LVL_NOTE, "Will not attach to PCI/ISA bridge.");
+                return EIO;
+        }
 
         hw_res_list_parsed_init(&hw_res);
@@ -92 +110 @@
         addr_range_t *cmd2_rng = &hw_res.io_ranges.ranges[2];
         addr_range_t *ctl2_rng = &hw_res.io_ranges.ranges[3];
-        ata_res->cmd1 = RNGABS(*cmd1_rng);
-        ata_res->ctl1 = RNGABS(*ctl1_rng);
-        ata_res->cmd2 = RNGABS(*cmd2_rng);
-        ata_res->ctl2 = RNGABS(*ctl2_rng);
+        res->cmd1 = RNGABS(*cmd1_rng);
+        res->ctl1 = RNGABS(*ctl1_rng);
+        res->cmd2 = RNGABS(*cmd2_rng);
+        res->ctl2 = RNGABS(*ctl2_rng);
 
         if (RNGSZ(*ctl1_rng) < sizeof(ata_ctl_t)) {
@@ -119 +137 @@
         /* IRQ */
         if (hw_res.irqs.count > 0) {
-                ata_res->irq1 = hw_res.irqs.irqs[0];
+                res->irq1 = hw_res.irqs.irqs[0];
         } else {
-                ata_res->irq1 = -1;
+                res->irq1 = -1;
         }
 
         if (hw_res.irqs.count > 1) {
-                ata_res->irq2 = hw_res.irqs.irqs[1];
+                res->irq2 = hw_res.irqs.irqs[1];
         } else {
-                ata_res->irq2 = -1;
+                res->irq2 = -1;
         }
 
@@ -147 +165 @@
         errno_t rc;
 
-        rc = ata_get_res(dev, &res);
+        rc = isa_ide_get_res(dev, &res);
         if (rc != EOK) {
                 ddf_msg(LVL_ERROR, "Invalid HW resource configuration.");