Changeset 8049b79 in mainline

Timestamp:

2013-09-12T19:30:25Z (11 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

7de1988c

Parents:

46eb2c4

Message:

Apply PIO window when parsing HW resources.

• Each parsed IO and memory range will have both absolute and PIO window relative addresses computed.
• Provide pio_enable_range() variant of pio_enable() working with addr_range_t.

Location:

uspace/lib/c

Files:

• generic/ddi.c (modified) (2 diffs)
• generic/device/hw_res_parsed.c (modified) (8 diffs)
• include/ddi.h (modified) (2 diffs)
• include/device/hw_res_parsed.h (modified) (3 diffs)

uspace/lib/c/generic/ddi.c

@@ -42 +42 @@
 #include <ddi.h>
 #include <libarch/ddi.h>
+#include <device/hw_res.h>
+#include <device/hw_res_parsed.h>
+#include <device/pio_window.h>
 #include <libc.h>
 #include <task.h>
@@ -136 +139 @@
 }
 
-/** Enable PIO for specified HW resource.
+/** Enable PIO for specified address range.
+ *
+ * @param range I/O range to be enable.
+ * @param virt  Virtual address for application's PIO operations. 
+ */
+int pio_enable_range(addr_range_t *range, void **virt)
+{
+        return pio_enable(RNGABSPTR(*range), RNGSZ(*range), virt);
+}
+
+/** Enable PIO for specified HW resource wrt. to the PIO window.
  *
  * @param win      PIO window. May be NULL if the resources are known to be

uspace/lib/c/generic/device/hw_res_parsed.c

@@ -80 +80 @@
 }
 
+static uint64_t absolutize(uint64_t addr, bool relative, uint64_t base)
+{
+        if (!relative)
+                return addr;
+        else
+                return addr + base;
+}
+
+static uint64_t relativize(uint64_t addr, bool relative, uint64_t base)
+{
+        if (relative)
+                return addr;
+        else
+                return addr - base;
+}
+
 static void hw_res_parse_add_io_range(hw_res_list_parsed_t *out,
-    const hw_resource_t *res, int flags)
-{
+    const pio_window_t *win, const hw_resource_t *res, int flags)
+{
+        endianness_t endianness;
+        uint64_t absolute;
+        uint64_t relative;
+        size_t size;
+
         assert(res && (res->type == IO_RANGE));
         
-        uint64_t address = res->res.io_range.address;
-        endianness_t endianness = res->res.io_range.endianness;
-        size_t size = res->res.io_range.size;
+        absolute = absolutize(res->res.io_range.address,
+            res->res.io_range.relative, win->io.base);
+        relative = relativize(res->res.io_range.address,
+            res->res.io_range.relative, win->io.base);
+        size = res->res.io_range.size;
+        endianness = res->res.io_range.endianness;
         
         if ((size == 0) && (!(flags & HW_RES_KEEP_ZERO_AREA)))
@@ -97 +121 @@
         if (!keep_duplicit) {
                 for (size_t i = 0; i < count; i++) {
-                        uint64_t s_address = out->io_ranges.ranges[i].address;
-                        size_t s_size = out->io_ranges.ranges[i].size;
+                        uint64_t s_address;
+                        size_t s_size;
+
+                        s_address = RNGABS(out->io_ranges.ranges[i]);
+                        s_size = RNGSZ(out->io_ranges.ranges[i]);
                         
-                        if ((address == s_address) && (size == s_size))
-                                return;
-                }
-        }
-        
-        out->io_ranges.ranges[count].address = address;
+                        if ((absolute == s_address) && (size == s_size))
+                                return;
+                }
+        }
+        
+        RNGABS(out->io_ranges.ranges[count]) = absolute;
+        RNGREL(out->io_ranges.ranges[count]) = relative;
+        RNGSZ(out->io_ranges.ranges[count]) = size;
         out->io_ranges.ranges[count].endianness = endianness;
-        out->io_ranges.ranges[count].size = size;
         out->io_ranges.count++;
 }
 
 static void hw_res_parse_add_mem_range(hw_res_list_parsed_t *out,
-    const hw_resource_t *res, int flags)
-{
+    const pio_window_t *win, const hw_resource_t *res, int flags)
+{
+        endianness_t endianness;
+        uint64_t absolute;
+        uint64_t relative;
+        size_t size;
+        
         assert(res && (res->type == MEM_RANGE));
         
-        uint64_t address = res->res.mem_range.address;
-        endianness_t endianness = res->res.mem_range.endianness;
-        size_t size = res->res.mem_range.size;
+        absolute = absolutize(res->res.mem_range.address,
+            res->res.mem_range.relative, win->mem.base);
+        relative = relativize(res->res.mem_range.address,
+            res->res.mem_range.relative, win->mem.base);
+        size = res->res.mem_range.size;
+        endianness = res->res.mem_range.endianness;
         
         if ((size == 0) && (!(flags & HW_RES_KEEP_ZERO_AREA)))
@@ -128 +164 @@
         if (!keep_duplicit) {
                 for (size_t i = 0; i < count; ++i) {
-                        uint64_t s_address = out->mem_ranges.ranges[i].address;
-                        size_t s_size = out->mem_ranges.ranges[i].size;
+                        uint64_t s_address;
+                        size_t s_size;
+
+                        s_address = RNGABS(out->mem_ranges.ranges[i]);;
+                        s_size = RNGSZ(out->mem_ranges.ranges[i]);
                         
-                        if ((address == s_address) && (size == s_size))
-                                return;
-                }
-        }
-        
-        out->mem_ranges.ranges[count].address = address;
+                        if ((absolute == s_address) && (size == s_size))
+                                return;
+                }
+        }
+        
+        RNGABS(out->mem_ranges.ranges[count]) = absolute;
+        RNGREL(out->mem_ranges.ranges[count]) = relative;
+        RNGSZ(out->mem_ranges.ranges[count]) = size;
         out->mem_ranges.ranges[count].endianness = endianness;
-        out->mem_ranges.ranges[count].size = size;
         out->mem_ranges.count++;
 }
@@ -144 +184 @@
 /** Parse list of hardware resources
  *
- * @param      hw_resources Original structure resource
- * @param[out] out          Output parsed resources
- * @param      flags        Flags of the parsing.
- *                          HW_RES_KEEP_ZERO_AREA for keeping
- *                          zero-size areas, HW_RES_KEEP_DUPLICITIES
- *                          for keep duplicit areas
+ * @param      win          PIO window.
+ * @param      res          Original structure resource.
+ * @param[out] out          Output parsed resources.
+ * @param      flags        Flags of the parsing:
+ *                          HW_RES_KEEP_ZERO_AREA for keeping zero-size areas,
+ *                          HW_RES_KEEP_DUPLICITIES to keep duplicit areas.
  *
  * @return EOK if succeed, error code otherwise.
  *
  */
-int hw_res_list_parse(const hw_resource_list_t *hw_resources,
-    hw_res_list_parsed_t *out, int flags)
-{
-        if ((!hw_resources) || (!out))
+int hw_res_list_parse(const pio_window_t *win,
+    const hw_resource_list_t *res, hw_res_list_parsed_t *out, int flags)
+{
+        if (!res || !out)
                 return EINVAL;
         
-        size_t res_count = hw_resources->count;
+        size_t res_count = res->count;
         hw_res_list_parsed_clean(out);
         
@@ -174 +214 @@
         
         for (size_t i = 0; i < res_count; ++i) {
-                const hw_resource_t *resource = &(hw_resources->resources[i]);
+                const hw_resource_t *resource = &res->resources[i];
                 
                 switch (resource->type) {
@@ -181 +221 @@
                         break;
                 case IO_RANGE:
-                        hw_res_parse_add_io_range(out, resource, flags);
+                        hw_res_parse_add_io_range(out, win, resource, flags);
                         break;
                 case MEM_RANGE:
-                        hw_res_parse_add_mem_range(out, resource, flags);
+                        hw_res_parse_add_mem_range(out, win, resource, flags);
                         break;
                 case DMA_CHANNEL_8:
@@ -216 +256 @@
     hw_res_list_parsed_t *hw_res_parsed, int flags)
 {
+        pio_window_t pio_window;
+        int rc;
+
         if (!hw_res_parsed)
                 return EBADMEM;
@@ -222 +265 @@
         hw_res_list_parsed_clean(hw_res_parsed);
         memset(&hw_resources, 0, sizeof(hw_resource_list_t));
-        
-        int rc = hw_res_get_resource_list(sess, &hw_resources);
+
+        rc = pio_window_get(sess, &pio_window);
+        if (rc != EOK)
+                return rc; 
+        
+        rc = hw_res_get_resource_list(sess, &hw_resources);
         if (rc != EOK)
                 return rc;
-        
-        rc = hw_res_list_parse(&hw_resources, hw_res_parsed, flags);
+
+        rc = hw_res_list_parse(&pio_window, &hw_resources, hw_res_parsed,
+            flags);
         hw_res_clean_resource_list(&hw_resources);
         

uspace/lib/c/include/ddi.h

@@ -41 +41 @@
 #include <abi/ddi/irq.h>
 #include <device/hw_res.h>
+#include <device/hw_res_parsed.h>
 #include <device/pio_window.h>
 #include <task.h>
@@ -57 +58 @@
 extern int dmamem_unmap_anonymous(void *);
 
+extern int pio_enable_range(addr_range_t *, void **);
 extern int pio_enable_resource(pio_window_t *, hw_resource_t *, void **);
 extern int pio_enable(void *, size_t, void **);

uspace/lib/c/include/device/hw_res_parsed.h

@@ -37 +37 @@
 
 #include <device/hw_res.h>
+#include <device/pio_window.h>
 #include <str.h>
 
@@ -45 +46 @@
 #define HW_RES_KEEP_DUPLICIT   0x2
 
+
+#define RNGABS(rng)     (rng).address.absolute
+#define RNGREL(rng)     (rng).address.relative
+#define RNGSZ(rng)      (rng).size
+
+#define RNGABSPTR(rng)  ((void *) ((uintptr_t) RNGABS((rng))))
+
+typedef struct address64 {
+        /** Aboslute address. */ 
+        uint64_t absolute;
+        /** PIO window base relative address. */
+        uint64_t relative; 
+} address64_t;
+
 /** Address range structure */
 typedef struct addr_range {
         /** Start address */
-        uint64_t address;
-        
-        /** Endianness */
-        endianness_t endianness;
+        address64_t address;
         
         /** Area size */
         size_t size;
+
+        /** Endianness */
+        endianness_t endianness;
 } addr_range_t;
 
@@ -139 +154 @@
 }
 
-extern int hw_res_list_parse(const hw_resource_list_t *,
+extern int hw_res_list_parse(const pio_window_t *, const hw_resource_list_t *,
     hw_res_list_parsed_t *, int);
 extern int hw_res_get_list_parsed(async_sess_t *, hw_res_list_parsed_t *, int);