]> nv-tegra.nvidia Code Review - linux-2.6.git/blobdiff - drivers/ieee1394/csr1212.h
RDS: Implement masked atomic operations
[linux-2.6.git] / drivers / ieee1394 / csr1212.h
index e6734263a1d3c6c16375d52fba8c7455a48ac392..a892d922dbc9ed05240db633a3eff794b604b53a 100644 (file)
 #ifndef __CSR1212_H__
 #define __CSR1212_H__
 
-
-/* Compatibility layer */
-#ifdef __KERNEL__
-
 #include <linux/types.h>
 #include <linux/slab.h>
-#include <linux/interrupt.h>
-#include <linux/sched.h>
-#include <linux/vmalloc.h>
-#include <asm/pgalloc.h>
-
-#define CSR1212_MALLOC(size)           vmalloc((size))
-#define CSR1212_FREE(ptr)              vfree(ptr)
-#define CSR1212_BE16_TO_CPU(quad)      be16_to_cpu(quad)
-#define CSR1212_CPU_TO_BE16(quad)      cpu_to_be16(quad)
-#define CSR1212_BE32_TO_CPU(quad)      be32_to_cpu(quad)
-#define CSR1212_CPU_TO_BE32(quad)      cpu_to_be32(quad)
-#define CSR1212_BE64_TO_CPU(quad)      be64_to_cpu(quad)
-#define CSR1212_CPU_TO_BE64(quad)      cpu_to_be64(quad)
-
-#define CSR1212_LE16_TO_CPU(quad)      le16_to_cpu(quad)
-#define CSR1212_CPU_TO_LE16(quad)      cpu_to_le16(quad)
-#define CSR1212_LE32_TO_CPU(quad)      le32_to_cpu(quad)
-#define CSR1212_CPU_TO_LE32(quad)      cpu_to_le32(quad)
-#define CSR1212_LE64_TO_CPU(quad)      le64_to_cpu(quad)
-#define CSR1212_CPU_TO_LE64(quad)      cpu_to_le64(quad)
-
-#include <linux/errno.h>
-#define CSR1212_SUCCESS (0)
-#define CSR1212_EINVAL (-EINVAL)
-#define CSR1212_ENOMEM (-ENOMEM)
-#define CSR1212_ENOENT (-ENOENT)
-#define CSR1212_EIO    (-EIO)
-#define CSR1212_EBUSY  (-EBUSY)
-
-#else  /* Userspace */
-
-#include <sys/types.h>
-#include <malloc.h>
-#define CSR1212_MALLOC(size)           malloc(size)
-#define CSR1212_FREE(ptr)              free(ptr)
-#include <endian.h>
-#if __BYTE_ORDER == __LITTLE_ENDIAN
-#include <byteswap.h>
-#define CSR1212_BE16_TO_CPU(quad)      bswap_16(quad)
-#define CSR1212_CPU_TO_BE16(quad)      bswap_16(quad)
-#define CSR1212_BE32_TO_CPU(quad)      bswap_32(quad)
-#define CSR1212_CPU_TO_BE32(quad)      bswap_32(quad)
-#define CSR1212_BE64_TO_CPU(quad)      bswap_64(quad)
-#define CSR1212_CPU_TO_BE64(quad)      bswap_64(quad)
-
-#define CSR1212_LE16_TO_CPU(quad)      (quad)
-#define CSR1212_CPU_TO_LE16(quad)      (quad)
-#define CSR1212_LE32_TO_CPU(quad)      (quad)
-#define CSR1212_CPU_TO_LE32(quad)      (quad)
-#define CSR1212_LE64_TO_CPU(quad)      (quad)
-#define CSR1212_CPU_TO_LE64(quad)      (quad)
-#else
-#define CSR1212_BE16_TO_CPU(quad)      (quad)
-#define CSR1212_CPU_TO_BE16(quad)      (quad)
-#define CSR1212_BE32_TO_CPU(quad)      (quad)
-#define CSR1212_CPU_TO_BE32(quad)      (quad)
-#define CSR1212_BE64_TO_CPU(quad)      (quad)
-#define CSR1212_CPU_TO_BE64(quad)      (quad)
-
-#define CSR1212_LE16_TO_CPU(quad)      bswap_16(quad)
-#define CSR1212_CPU_TO_LE16(quad)      bswap_16(quad)
-#define CSR1212_LE32_TO_CPU(quad)      bswap_32(quad)
-#define CSR1212_CPU_TO_LE32(quad)      bswap_32(quad)
-#define CSR1212_LE64_TO_CPU(quad)      bswap_64(quad)
-#define CSR1212_CPU_TO_LE64(quad)      bswap_64(quad)
-#endif
-
-#include <errno.h>
-#define CSR1212_SUCCESS (0)
-#define CSR1212_EINVAL (EINVAL)
-#define CSR1212_ENOMEM (ENOMEM)
-#define CSR1212_ENOENT (ENOENT)
-#define CSR1212_EIO    (EIO)
-#define CSR1212_EBUSY  (EBUSY)
-
-#endif
+#include <asm/atomic.h>
 
+#define CSR1212_MALLOC(size)   kmalloc((size), GFP_KERNEL)
+#define CSR1212_FREE(ptr)      kfree(ptr)
 
-#define CSR1212_KV_VAL_MASK                    0xffffff
-#define CSR1212_KV_KEY_SHIFT                   24
-#define CSR1212_KV_KEY_TYPE_SHIFT              6
-#define CSR1212_KV_KEY_ID_MASK                 0x3f
-#define CSR1212_KV_KEY_TYPE_MASK               0x3             /* After shift */
+#define CSR1212_SUCCESS (0)
 
 
 /* CSR 1212 key types */
 #define  CSR1212_UNITS_SPACE_END               (CSR1212_UNITS_SPACE_BASE + CSR1212_UNITS_SPACE_SIZE)
 #define  CSR1212_UNITS_SPACE_OFFSET            (CSR1212_UNITS_SPACE_BASE - CSR1212_REGISTER_SPACE_BASE)
 
-#define  CSR1212_EXTENDED_ROM_SIZE             (0x10000 * sizeof(u_int32_t))
+#define  CSR1212_INVALID_ADDR_SPACE            -1
 
 
 /* Config ROM image structures */
 struct csr1212_bus_info_block_img {
-       u_int8_t length;
-       u_int8_t crc_length;
-       u_int16_t crc;
-
-       /* Must be last */
-       u_int32_t data[0];      /* older gcc can't handle [] which is standard */
-};
-
-#define CSR1212_KV_KEY(quad)           (CSR1212_BE32_TO_CPU(quad) >> CSR1212_KV_KEY_SHIFT)
-#define CSR1212_KV_KEY_TYPE(quad)      (CSR1212_KV_KEY(quad) >> CSR1212_KV_KEY_TYPE_SHIFT)
-#define CSR1212_KV_KEY_ID(quad)                (CSR1212_KV_KEY(quad) & CSR1212_KV_KEY_ID_MASK)
-#define CSR1212_KV_VAL(quad)           (CSR1212_BE32_TO_CPU(quad) & CSR1212_KV_VAL_MASK)
-
-#define CSR1212_SET_KV_KEY(quad, key)  ((quad) = \
-       CSR1212_CPU_TO_BE32(CSR1212_KV_VAL(quad) | ((key) << CSR1212_KV_KEY_SHIFT)))
-#define CSR1212_SET_KV_VAL(quad, val)  ((quad) = \
-       CSR1212_CPU_TO_BE32((CSR1212_KV_KEY(quad) << CSR1212_KV_KEY_SHIFT) | (val)))
-#define CSR1212_SET_KV_TYPEID(quad, type, id)  ((quad) = \
-       CSR1212_CPU_TO_BE32(CSR1212_KV_VAL(quad) | \
-       (((((type) & CSR1212_KV_KEY_TYPE_MASK) << CSR1212_KV_KEY_TYPE_SHIFT) | \
-         ((id) & CSR1212_KV_KEY_ID_MASK)) << CSR1212_KV_KEY_SHIFT)))
-
-typedef u_int32_t csr1212_quad_t;
-
-
-struct csr1212_keyval_img {
-       u_int16_t length;
-       u_int16_t crc;
+       u8 length;
+       u8 crc_length;
+       u16 crc;
 
        /* Must be last */
-       csr1212_quad_t data[0]; /* older gcc can't handle [] which is standard */
+       u32 data[0];    /* older gcc can't handle [] which is standard */
 };
 
 struct csr1212_leaf {
        int len;
-       u_int32_t *data;
+       u32 *data;
 };
 
 struct csr1212_dentry {
@@ -246,29 +140,29 @@ struct csr1212_directory {
 
 struct csr1212_keyval {
        struct {
-               u_int8_t type;
-               u_int8_t id;
+               u8 type;
+               u8 id;
        } key;
        union {
-               u_int32_t immediate;
-               u_int32_t csr_offset;
+               u32 immediate;
+               u32 csr_offset;
                struct csr1212_leaf leaf;
                struct csr1212_directory directory;
        } value;
        struct csr1212_keyval *associate;
-       int refcnt;
+       atomic_t refcnt;
 
        /* used in generating and/or parsing CSR image */
        struct csr1212_keyval *next, *prev;     /* flat list of CSR elements */
-       u_int32_t offset;       /* position in CSR from 0xffff f000 0000 */
-       u_int8_t valid;         /* flag indicating keyval has valid data*/
+       u32 offset;     /* position in CSR from 0xffff f000 0000 */
+       u8 valid;       /* flag indicating keyval has valid data*/
 };
 
 
 struct csr1212_cache_region {
        struct csr1212_cache_region *next, *prev;
-       u_int32_t offset_start;         /* inclusive */
-       u_int32_t offset_end;           /* exclusive */
+       u32 offset_start;       /* inclusive */
+       u32 offset_end;         /* exclusive */
 };
 
 struct csr1212_csr_rom_cache {
@@ -276,18 +170,18 @@ struct csr1212_csr_rom_cache {
        struct csr1212_cache_region *filled_head, *filled_tail;
        struct csr1212_keyval *layout_head, *layout_tail;
        size_t size;
-       u_int32_t offset;
+       u32 offset;
        struct csr1212_keyval *ext_rom;
        size_t len;
 
        /* Must be last */
-       u_int32_t data[0];      /* older gcc can't handle [] which is standard */
+       u32 data[0];    /* older gcc can't handle [] which is standard */
 };
 
 struct csr1212_csr {
        size_t bus_info_len;    /* bus info block length in bytes */
        size_t crc_len;         /* crc length in bytes */
-       u_int32_t *bus_info_data;       /* bus info data incl bus name and EUI */
+       __be32 *bus_info_data;  /* bus info data incl bus name and EUI */
 
        void *private;          /* private, bus specific data */
        struct csr1212_bus_ops *ops;
@@ -305,52 +199,33 @@ struct csr1212_bus_ops {
         * from remote nodes when parsing a Config ROM (i.e., read Config ROM
         * entries located in the Units Space.  Must return 0 on success
         * anything else indicates an error. */
-       int (*bus_read) (struct csr1212_csr *csr, u_int64_t addr,
-                        u_int16_t length, void *buffer, void *private);
+       int (*bus_read) (struct csr1212_csr *csr, u64 addr,
+                        void *buffer, void *private);
 
        /* This function is used by csr1212 to allocate a region in units space
         * in the event that Config ROM entries don't all fit in the predefined
         * 1K region.  The void *private parameter is private member of struct
         * csr1212_csr. */
-       u_int64_t (*allocate_addr_range) (u_int64_t size, u_int32_t alignment,
-                                         void *private);
-
+       u64 (*allocate_addr_range) (u64 size, u32 alignment, void *private);
 
        /* This function is used by csr1212 to release a region in units space
         * that is no longer needed. */
-       void (*release_addr) (u_int64_t addr, void *private);
-
-       /* This function is used by csr1212 to determine the max read request
-        * supported by a remote node when reading the ConfigROM space.  Must
-        * return 0, 1, or 2 per IEEE 1212.  */
-       int (*get_max_rom) (u_int32_t *bus_info, void *private);
+       void (*release_addr) (u64 addr, void *private);
 };
 
 
-
-
 /* Descriptor Leaf manipulation macros */
 #define CSR1212_DESCRIPTOR_LEAF_TYPE_SHIFT 24
 #define CSR1212_DESCRIPTOR_LEAF_SPECIFIER_ID_MASK 0xffffff
-#define CSR1212_DESCRIPTOR_LEAF_OVERHEAD (1 * sizeof(u_int32_t))
+#define CSR1212_DESCRIPTOR_LEAF_OVERHEAD (1 * sizeof(u32))
 
 #define CSR1212_DESCRIPTOR_LEAF_TYPE(kv) \
-       (CSR1212_BE32_TO_CPU((kv)->value.leaf.data[0]) >> CSR1212_DESCRIPTOR_LEAF_TYPE_SHIFT)
+       (be32_to_cpu((kv)->value.leaf.data[0]) >> \
+        CSR1212_DESCRIPTOR_LEAF_TYPE_SHIFT)
 #define CSR1212_DESCRIPTOR_LEAF_SPECIFIER_ID(kv) \
-       (CSR1212_BE32_TO_CPU((kv)->value.leaf.data[0]) & \
+       (be32_to_cpu((kv)->value.leaf.data[0]) & \
         CSR1212_DESCRIPTOR_LEAF_SPECIFIER_ID_MASK)
-#define CSR1212_DESCRIPTOR_LEAF_DATA(kv) \
-       (&((kv)->value.leaf.data[1]))
-
-#define CSR1212_DESCRIPTOR_LEAF_SET_TYPE(kv, type) \
-       ((kv)->value.leaf.data[0] = \
-        CSR1212_CPU_TO_BE32(CSR1212_DESCRIPTOR_LEAF_SPECIFIER_ID(kv) | \
-                            ((type) << CSR1212_DESCRIPTOR_LEAF_TYPE_SHIFT)))
-#define CSR1212_DESCRIPTOR_LEAF_SET_SPECIFIER_ID(kv, spec_id) \
-       ((kv)->value.leaf.data[0] = \
-        CSR1212_CPU_TO_BE32((CSR1212_DESCRIPTOR_LEAF_TYPE(kv) << \
-                             CSR1212_DESCRIPTOR_LEAF_TYPE_SHIFT) | \
-                            ((spec_id) & CSR1212_DESCRIPTOR_LEAF_SPECIFIER_ID_MASK)))
+
 
 /* Text Descriptor Leaf manipulation macros */
 #define CSR1212_TEXTUAL_DESCRIPTOR_LEAF_WIDTH_SHIFT 28
@@ -358,182 +233,21 @@ struct csr1212_bus_ops {
 #define CSR1212_TEXTUAL_DESCRIPTOR_LEAF_CHAR_SET_SHIFT 16
 #define CSR1212_TEXTUAL_DESCRIPTOR_LEAF_CHAR_SET_MASK 0xfff  /* after shift */
 #define CSR1212_TEXTUAL_DESCRIPTOR_LEAF_LANGUAGE_MASK 0xffff
-#define CSR1212_TEXTUAL_DESCRIPTOR_LEAF_OVERHEAD (1 * sizeof(u_int32_t))
+#define CSR1212_TEXTUAL_DESCRIPTOR_LEAF_OVERHEAD (1 * sizeof(u32))
 
 #define CSR1212_TEXTUAL_DESCRIPTOR_LEAF_WIDTH(kv) \
-       (CSR1212_BE32_TO_CPU((kv)->value.leaf.data[1]) >> \
+       (be32_to_cpu((kv)->value.leaf.data[1]) >> \
         CSR1212_TEXTUAL_DESCRIPTOR_LEAF_WIDTH_SHIFT)
 #define CSR1212_TEXTUAL_DESCRIPTOR_LEAF_CHAR_SET(kv) \
-       ((CSR1212_BE32_TO_CPU((kv)->value.leaf.data[1]) >> \
-                            CSR1212_TEXTUAL_DESCRIPTOR_LEAF_CHAR_SET_SHIFT) & \
-                           CSR1212_TEXTUAL_DESCRIPTOR_LEAF_CHAR_SET_MASK)
+       ((be32_to_cpu((kv)->value.leaf.data[1]) >> \
+         CSR1212_TEXTUAL_DESCRIPTOR_LEAF_CHAR_SET_SHIFT) & \
+        CSR1212_TEXTUAL_DESCRIPTOR_LEAF_CHAR_SET_MASK)
 #define CSR1212_TEXTUAL_DESCRIPTOR_LEAF_LANGUAGE(kv) \
-       (CSR1212_BE32_TO_CPU((kv)->value.leaf.data[1]) & \
+       (be32_to_cpu((kv)->value.leaf.data[1]) & \
         CSR1212_TEXTUAL_DESCRIPTOR_LEAF_LANGUAGE_MASK)
 #define CSR1212_TEXTUAL_DESCRIPTOR_LEAF_DATA(kv) \
        (&((kv)->value.leaf.data[2]))
 
-#define CSR1212_TEXTUAL_DESCRIPTOR_LEAF_SET_WIDTH(kv, width) \
-       ((kv)->value.leaf.data[1] = \
-        ((kv)->value.leaf.data[1] & \
-         CSR1212_CPU_TO_BE32(~(CSR1212_TEXTUAL_DESCRIPTOR_LEAF_WIDTH_MASK << \
-                               CSR1212_TEXTUAL_DESCRIPTOR_LEAF_WIDTH_SHIFT))) | \
-        CSR1212_CPU_TO_BE32(((width) & \
-                             CSR1212_TEXTUAL_DESCRIPTOR_LEAF_WIDTH_MASK) << \
-                            CSR1212_TEXTUAL_DESCRIPTOR_LEAF_WIDTH_SHIFT))
-#define CSR1212_TEXTUAL_DESCRIPTOR_LEAF_SET_CHAR_SET(kv, char_set) \
-       ((kv)->value.leaf.data[1] = \
-        ((kv)->value.leaf.data[1] & \
-         CSR1212_CPU_TO_BE32(~(CSR1212_TEXTUAL_DESCRIPTOR_LEAF_CHAR_SET_MASK << \
-                               CSR1212_TEXTUAL_DESCRIPTOR_LEAF_CHAR_SET_SHIFT))) | \
-        CSR1212_CPU_TO_BE32(((char_set) & \
-                             CSR1212_TEXTUAL_DESCRIPTOR_LEAF_CHAR_SET_MASK) << \
-                            CSR1212_TEXTUAL_DESCRIPTOR_LEAF_CHAR_SET_SHIFT))
-#define CSR1212_TEXTUAL_DESCRIPTOR_LEAF_SET_LANGUAGE(kv, language) \
-       ((kv)->value.leaf.data[1] = \
-        ((kv)->value.leaf.data[1] & \
-         CSR1212_CPU_TO_BE32(~(CSR1212_TEXTUAL_DESCRIPTOR_LEAF_LANGUAGE_MASK))) | \
-        CSR1212_CPU_TO_BE32(((language) & \
-                             CSR1212_TEXTUAL_DESCRIPTOR_LEAF_LANGUAGE_MASK)))
-
-
-/* Icon Descriptor Leaf manipulation macros */
-#define CSR1212_ICON_DESCRIPTOR_LEAF_VERSION_MASK 0xffffff
-#define CSR1212_ICON_DESCRIPTOR_LEAF_PALETTE_DEPTH_SHIFT 30
-#define CSR1212_ICON_DESCRIPTOR_LEAF_PALETTE_DEPTH_MASK 0x3 /* after shift */
-#define CSR1212_ICON_DESCRIPTOR_LEAF_COLOR_SPACE_SHIFT 16
-#define CSR1212_ICON_DESCRIPTOR_LEAF_COLOR_SPACE_MASK 0xf /* after shift */
-#define CSR1212_ICON_DESCRIPTOR_LEAF_LANGUAGE_MASK 0xffff
-#define CSR1212_ICON_DESCRIPTOR_LEAF_HSCAN_SHIFT 16
-#define CSR1212_ICON_DESCRIPTOR_LEAF_HSCAN_MASK 0xffff /* after shift */
-#define CSR1212_ICON_DESCRIPTOR_LEAF_VSCAN_MASK 0xffff
-#define CSR1212_ICON_DESCRIPTOR_LEAF_OVERHEAD (3 * sizeof(u_int32_t))
-
-#define CSR1212_ICON_DESCRIPTOR_LEAF_VERSION(kv) \
-       (CSR1212_BE32_TO_CPU((kv)->value.leaf.data[2]) & \
-        CSR1212_ICON_DESCRIPTOR_LEAF_VERSION_MASK)
-
-#define CSR1212_ICON_DESCRIPTOR_LEAF_PALETTE_DEPTH(kv) \
-       (CSR1212_BE32_TO_CPU((kv)->value.leaf.data[3]) >> \
-        CSR1212_ICON_DESCRIPTOR_LEAF_PALETTE_DEPTH_SHIFT)
-
-#define CSR1212_ICON_DESCRIPTOR_LEAF_COLOR_SPACE(kv) \
-       ((CSR1212_BE32_TO_CPU((kv)->value.leaf.data[3]) >> \
-         CSR1212_ICON_DESCRIPTOR_LEAF_COLOR_SPACE_SHIFT) & \
-        CSR1212_ICON_DESCRIPTOR_LEAF_COLOR_SPACE_MASK)
-
-#define CSR1212_ICON_DESCRIPTOR_LEAF_LANGUAGE(kv) \
-       (CSR1212_BE32_TO_CPU((kv)->value.leaf.data[3]) & \
-        CSR1212_ICON_DESCRIPTOR_LEAF_LANGUAGE_MASK)
-
-#define CSR1212_ICON_DESCRIPTOR_LEAF_HSCAN(kv) \
-       ((CSR1212_BE32_TO_CPU((kv)->value.leaf.data[4]) >> \
-         CSR1212_ICON_DESCRIPTOR_LEAF_COLOR_HSCAN_SHIFT) & \
-        CSR1212_ICON_DESCRIPTOR_LEAF_COLOR_HSCAN_MASK)
-
-#define CSR1212_ICON_DESCRIPTOR_LEAF_VSCAN(kv) \
-       (CSR1212_BE32_TO_CPU((kv)->value.leaf.data[4]) & \
-        CSR1212_ICON_DESCRIPTOR_LEAF_VSCAN_MASK)
-
-#define CSR1212_ICON_DESCRIPTOR_LEAF_PALETTE(kv) \
-       (&((kv)->value.leaf.data[5]))
-
-static inline u_int32_t *CSR1212_ICON_DESCRIPTOR_LEAF_PIXELS(struct csr1212_keyval *kv)
-{
-       static const int pd[4] = { 0, 4, 16, 256 };
-       static const int cs[16] = { 4, 2 };
-       int ps = pd[CSR1212_ICON_DESCRIPTOR_LEAF_PALETTE_DEPTH(kv)];
-
-       return &kv->value.leaf.data[5 +
-                                   (ps * cs[CSR1212_ICON_DESCRIPTOR_LEAF_COLOR_SPACE(kv)]) /
-                          sizeof(u_int32_t)];
-}
-
-#define CSR1212_ICON_DESCRIPTOR_LEAF_SET_VERSION(kv, version) \
-       ((kv)->value.leaf.data[2] = \
-        ((kv)->value.leaf.data[2] & \
-         CSR1212_CPU_TO_BE32(~(CSR1212_ICON_DESCRIPTOR_LEAF_VERSION_MASK))) | \
-        CSR1212_CPU_TO_BE32(((version) & \
-                             CSR1212_ICON_DESCRIPTOR_LEAF_VERSION_MASK)))
-
-#define CSR1212_ICON_DESCRIPTOR_LEAF_SET_PALETTE_DEPTH(kv, palette_depth) \
-       ((kv)->value.leaf.data[3] = \
-        ((kv)->value.leaf.data[3] & \
-         CSR1212_CPU_TO_BE32(~(CSR1212_ICON_DESCRIPTOR_LEAF_PALETTE_DEPTH_MASK << \
-                               CSR1212_ICON_DESCRIPTOR_LEAF_PALETTE_DEPTH_SHIFT))) | \
-        CSR1212_CPU_TO_BE32(((palette_depth) & \
-                             CSR1212_ICON_DESCRIPTOR_LEAF_PALETTE_DEPTH_MASK) << \
-                            CSR1212_ICON_DESCRIPTOR_LEAF_PALETTE_DEPTH_SHIFT))
-
-#define CSR1212_ICON_DESCRIPTOR_LEAF_SET_COLOR_SPACE(kv, color_space) \
-       ((kv)->value.leaf.data[3] = \
-        ((kv)->value.leaf.data[3] & \
-         CSR1212_CPU_TO_BE32(~(CSR1212_ICON_DESCRIPTOR_LEAF_COLOR_SPACE_MASK << \
-                               CSR1212_ICON_DESCRIPTOR_LEAF_COLOR_SPACE_SHIFT))) | \
-        CSR1212_CPU_TO_BE32(((color_space) & \
-                             CSR1212_ICON_DESCRIPTOR_LEAF_COLOR_SPACE_MASK) << \
-                            CSR1212_ICON_DESCRIPTOR_LEAF_COLOR_SPACE_SHIFT))
-
-#define CSR1212_ICON_DESCRIPTOR_LEAF_SET_LANGUAGE(kv, language) \
-       ((kv)->value.leaf.data[3] = \
-        ((kv)->value.leaf.data[3] & \
-         CSR1212_CPU_TO_BE32(~(CSR1212_ICON_DESCRIPTOR_LEAF_LANGUAGE_MASK))) | \
-        CSR1212_CPU_TO_BE32(((language) & \
-                             CSR1212_ICON_DESCRIPTOR_LEAF_LANGUAGE_MASK)))
-
-#define CSR1212_ICON_DESCRIPTOR_LEAF_SET_HSCAN(kv, hscan) \
-       ((kv)->value.leaf.data[4] = \
-        ((kv)->value.leaf.data[4] & \
-         CSR1212_CPU_TO_BE32(~(CSR1212_ICON_DESCRIPTOR_LEAF_HSCAN_MASK << \
-                               CSR1212_ICON_DESCRIPTOR_LEAF_HSCAN_SHIFT))) | \
-        CSR1212_CPU_TO_BE32(((hscan) & \
-                             CSR1212_ICON_DESCRIPTOR_LEAF_HSCAN_MASK) << \
-                            CSR1212_ICON_DESCRIPTOR_LEAF_HSCAN_SHIFT))
-
-#define CSR1212_ICON_DESCRIPTOR_LEAF_SET_VSCAN(kv, vscan) \
-       ((kv)->value.leaf.data[4] = \
-        (((kv)->value.leaf.data[4] & \
-         CSR1212_CPU_TO_BE32(~CSR1212_ICON_DESCRIPTOR_LEAF_VSCAN_MASK))) | \
-        CSR1212_CPU_TO_BE32(((vscan) & \
-                             CSR1212_ICON_DESCRIPTOR_LEAF_VSCAN_MASK)))
-
-
-/* Modifiable Descriptor Leaf manipulation macros */
-#define CSR1212_MODIFIABLE_DESCRIPTOR_LEAF_MAX_SIZE_SHIFT 16
-#define CSR1212_MODIFIABLE_DESCRIPTOR_LEAF_MAX_SIZE_MASK 0xffff
-#define CSR1212_MODIFIABLE_DESCRIPTOR_LEAF_ADDR_HI_SHIFT 32
-#define CSR1212_MODIFIABLE_DESCRIPTOR_LEAF_ADDR_HI_MASK 0xffff
-#define CSR1212_MODIFIABLE_DESCRIPTOR_LEAF_ADDR_LO_MASK 0xffffffffULL
-
-#define CSR1212_MODIFIABLE_DESCRIPTOR_MAX_SIZE(kv) \
-       CSR1212_BE16_TO_CPU((kv)->value.leaf.data[0] >> CSR1212_MODIFIABLE_DESCRIPTOR_MAX_SIZE_SHIFT)
-
-#define CSR1212_MODIFIABLE_DESCRIPTOR_ADDRESS(kv) \
-       (CSR1212_BE16_TO_CPU(((u_int64_t)((kv)->value.leaf.data[0])) << \
-                            CSR1212_MODIFIABLE_DESCRIPTOR_ADDR_HI_SHIFT) | \
-        CSR1212_BE32_TO_CPU((kv)->value.leaf.data[1]))
-
-#define CSR1212_MODIFIABLE_DESCRIPTOR_SET_MAX_SIZE(kv, size) \
-       ((kv)->value.leaf.data[0] = \
-        ((kv)->value.leaf.data[0] & \
-         CSR1212_CPU_TO_BE32(~(CSR1212_MODIFIABLE_DESCRIPTOR_LEAF_MAX_SIZE_MASK << \
-                               CSR1212_MODIFIABLE_DESCRIPTOR_LEAF_MAX_SIZE_SHIFT))) | \
-        CSR1212_CPU_TO_BE32(((size) & \
-                             CSR1212_MODIFIABLE_DESCRIPTOR_LEAF_MAX_SIZE_MASK) << \
-                            CSR1212_MODIFIABLE_DESCRIPTOR_LEAF_MAX_SIZE_SHIFT))
-
-#define CSR1212_MODIFIABLE_DESCRIPTOR_SET_ADDRESS_HI(kv, addr) \
-       ((kv)->value.leaf.data[0] = \
-        ((kv)->value.leaf.data[0] & \
-         CSR1212_CPU_TO_BE32(~(CSR1212_MODIFIABLE_DESCRIPTOR_LEAF_ADDR_HI_MASK))) | \
-         CSR1212_CPU_TO_BE32(((addr) & \
-                              CSR1212_MODIFIABLE_DESCRIPTOR_LEAF_ADDR_HI_MASK)))
-
-#define CSR1212_MODIFIABLE_DESCRIPTOR_SET_ADDRESS_LO(kv, addr) \
-       ((kv)->value.leaf.data[1] = \
-        CSR1212_CPU_TO_BE32(addr & CSR1212_MODIFIABLE_DESCRIPTOR_LEAF_ADDR_LO_MASK))
-
-
 
 /* The following 2 function are for creating new Configuration ROM trees.  The
  * first function is used for both creating local trees and parsing remote
@@ -543,11 +257,10 @@ extern struct csr1212_csr *csr1212_create_csr(struct csr1212_bus_ops *ops,
                                              size_t bus_info_size,
                                              void *private);
 extern void csr1212_init_local_csr(struct csr1212_csr *csr,
-                                  const u_int32_t *bus_info_data, int max_rom);
+                                  const u32 *bus_info_data, int max_rom);
 
 
-/* The following function destroys a Configuration ROM tree and release all
- * memory taken by the tree. */
+/* Destroy a Configuration ROM tree and release all memory taken by the tree. */
 extern void csr1212_destroy_csr(struct csr1212_csr *csr);
 
 
@@ -555,50 +268,20 @@ extern void csr1212_destroy_csr(struct csr1212_csr *csr);
  * a Configuration ROM tree.  Code that creates new keyvals with these functions
  * must release those keyvals with csr1212_release_keyval() when they are no
  * longer needed. */
-extern struct csr1212_keyval *csr1212_new_immediate(u_int8_t key, u_int32_t value);
-extern struct csr1212_keyval *csr1212_new_leaf(u_int8_t key, const void *data,
-                                              size_t data_len);
-extern struct csr1212_keyval *csr1212_new_csr_offset(u_int8_t key,
-                                                    u_int32_t csr_offset);
-extern struct csr1212_keyval *csr1212_new_directory(u_int8_t key);
-extern struct csr1212_keyval *csr1212_new_extended_immediate(u_int32_t spec,
-                                                            u_int32_t key,
-                                                            u_int32_t value);
-extern struct csr1212_keyval *csr1212_new_extended_leaf(u_int32_t spec,
-                                                       u_int32_t key,
-                                                       const void *data,
-                                                       size_t data_len);
-extern struct csr1212_keyval *csr1212_new_descriptor_leaf(u_int8_t dtype,
-                                                         u_int32_t specifier_id,
-                                                         const void *data,
-                                                         size_t data_len);
-extern struct csr1212_keyval *csr1212_new_textual_descriptor_leaf(u_int8_t cwidth,
-                                                                 u_int16_t cset,
-                                                                 u_int16_t language,
-                                                                 const void *data,
-                                                                 size_t data_len);
+extern struct csr1212_keyval *csr1212_new_immediate(u8 key, u32 value);
+extern struct csr1212_keyval *csr1212_new_directory(u8 key);
 extern struct csr1212_keyval *csr1212_new_string_descriptor_leaf(const char *s);
-extern struct csr1212_keyval *csr1212_new_icon_descriptor_leaf(u_int32_t version,
-                                                              u_int8_t palette_depth,
-                                                              u_int8_t color_space,
-                                                              u_int16_t language,
-                                                              u_int16_t hscan,
-                                                              u_int16_t vscan,
-                                                              u_int32_t *palette,
-                                                              u_int32_t *pixels);
-extern struct csr1212_keyval *csr1212_new_modifiable_descriptor_leaf(u_int16_t max_size,
-                                                                    u_int64_t address);
-extern struct csr1212_keyval *csr1212_new_keyword_leaf(int strc,
-                                                      const char *strv[]);
-
-
-/* The following functions manage association between keyvals.  Typically,
+
+
+/* The following function manages association between keyvals.  Typically,
  * Descriptor Leaves and Directories will be associated with another keyval and
  * it is desirable for the Descriptor keyval to be place immediately after the
- * keyval that it is associated with.*/
-extern int csr1212_associate_keyval(struct csr1212_keyval *kv,
-                                   struct csr1212_keyval *associate);
-extern void csr1212_disassociate_keyval(struct csr1212_keyval *kv);
+ * keyval that it is associated with.
+ * Take care with subsequent ROM modifications:  There is no function to remove
+ * previously specified associations.
+ */
+extern void csr1212_associate_keyval(struct csr1212_keyval *kv,
+                                    struct csr1212_keyval *associate);
 
 
 /* The following functions manage the association of a keyval and directories.
@@ -609,23 +292,15 @@ extern void csr1212_detach_keyval_from_directory(struct csr1212_keyval *dir,
                                                 struct csr1212_keyval *kv);
 
 
-/* The following functions create a Configuration ROM image from the tree of
- * keyvals provided.  csr1212_generate_csr_image() creates a complete image in
- * the list of caches available via csr->cache_head.  The other functions are
- * provided should there be a need to create a flat image without restrictions
- * placed by IEEE 1212. */
-extern struct csr1212_keyval *csr1212_generate_positions(struct csr1212_csr_rom_cache *cache,
-                                                        struct csr1212_keyval *start_kv,
-                                                        int start_pos);
-extern size_t csr1212_generate_layout_order(struct csr1212_keyval *kv);
-extern void csr1212_fill_cache(struct csr1212_csr_rom_cache *cache);
+/* Creates a complete Configuration ROM image in the list of caches available
+ * via csr->cache_head. */
 extern int csr1212_generate_csr_image(struct csr1212_csr *csr);
 
 
 /* This is a convience function for reading a block of data out of one of the
  * caches in the csr->cache_head list. */
-extern int csr1212_read(struct csr1212_csr *csr, u_int32_t offset, void *buffer,
-                       u_int32_t len);
+extern int csr1212_read(struct csr1212_csr *csr, u32 offset, void *buffer,
+                       u32 len);
 
 
 /* The following functions are in place for parsing Configuration ROM images.
@@ -635,19 +310,15 @@ extern int csr1212_parse_keyval(struct csr1212_keyval *kv,
                                struct csr1212_csr_rom_cache *cache);
 extern int csr1212_parse_csr(struct csr1212_csr *csr);
 
-/* These are internal functions referenced by inline functions below. */
-extern int _csr1212_read_keyval(struct csr1212_csr *csr, struct csr1212_keyval *kv);
-extern void _csr1212_destroy_keyval(struct csr1212_keyval *kv);
-
 
 /* This function allocates a new cache which may be used for either parsing or
  * generating sub-sets of Configuration ROM images. */
-static inline struct csr1212_csr_rom_cache *csr1212_rom_cache_malloc(u_int32_t offset,
-                                                                    size_t size)
+static inline struct csr1212_csr_rom_cache *
+csr1212_rom_cache_malloc(u32 offset, size_t size)
 {
        struct csr1212_csr_rom_cache *cache;
 
-       cache = CSR1212_MALLOC(sizeof(struct csr1212_csr_rom_cache) + size);
+       cache = CSR1212_MALLOC(sizeof(*cache) + size);
        if (!cache)
                return NULL;
 
@@ -667,23 +338,16 @@ static inline struct csr1212_csr_rom_cache *csr1212_rom_cache_malloc(u_int32_t o
 
 /* This function ensures that a keyval contains data when referencing a keyval
  * created by parsing a Configuration ROM. */
-static inline struct csr1212_keyval *csr1212_get_keyval(struct csr1212_csr *csr,
-                                                       struct csr1212_keyval *kv)
-{
-       if (!kv)
-               return NULL;
-       if (!kv->valid)
-               if (_csr1212_read_keyval(csr, kv) != CSR1212_SUCCESS)
-                       return NULL;
-       return kv;
-}
+extern struct csr1212_keyval *
+csr1212_get_keyval(struct csr1212_csr *csr, struct csr1212_keyval *kv);
 
 
 /* This function increments the reference count for a keyval should there be a
  * need for code to retain a keyval that has been parsed. */
 static inline void csr1212_keep_keyval(struct csr1212_keyval *kv)
 {
-       kv->refcnt++;
+       atomic_inc(&kv->refcnt);
+       smp_mb__after_atomic_inc();
 }
 
 
@@ -691,37 +355,29 @@ static inline void csr1212_keep_keyval(struct csr1212_keyval *kv)
  * keyval when there are no more users of the keyval.  This should be called by
  * any code that calls csr1212_keep_keyval() or any of the keyval creation
  * routines csr1212_new_*(). */
-static inline void csr1212_release_keyval(struct csr1212_keyval *kv)
-{
-       if (kv->refcnt > 1)
-               kv->refcnt--;
-       else
-               _csr1212_destroy_keyval(kv);
-}
+extern void csr1212_release_keyval(struct csr1212_keyval *kv);
 
 
 /*
  * This macro allows for looping over the keyval entries in a directory and it
  * ensures that keyvals from remote ConfigROMs are parsed properly.
  *
- * _csr is a struct csr1212_csr * that points to CSR associated with dir.
- * _kv is a struct csr1212_keyval * that'll point to the current keyval (loop index).
- * _dir is a struct csr1212_keyval * that points to the directory to be looped.
- * _pos is a struct csr1212_dentry * that is used internally for indexing.
+ * struct csr1212_csr *_csr points to the CSR associated with dir.
+ * struct csr1212_keyval *_kv points to the current keyval (loop index).
+ * struct csr1212_keyval *_dir points to the directory to be looped.
+ * struct csr1212_dentry *_pos is used internally for indexing.
  *
  * kv will be NULL upon exit of the loop.
  */
-#define csr1212_for_each_dir_entry(_csr, _kv, _dir, _pos)                      \
-       for (csr1212_get_keyval((_csr), (_dir)),                                \
-            _pos = (_dir)->value.directory.dentries_head,                      \
-            _kv = (_pos) ? csr1212_get_keyval((_csr), _pos->kv) : NULL;        \
-            (_kv) && (_pos);                                                   \
-            (_kv->associate == NULL) ?                                         \
-                    ((_pos = _pos->next),                                      \
-                     (_kv = (_pos) ? csr1212_get_keyval((_csr), _pos->kv) :    \
-                          NULL)) :                                             \
+#define csr1212_for_each_dir_entry(_csr, _kv, _dir, _pos)              \
+       for (csr1212_get_keyval((_csr), (_dir)),                        \
+            _pos = (_dir)->value.directory.dentries_head,              \
+            _kv = (_pos) ? csr1212_get_keyval((_csr), _pos->kv) : NULL;\
+            (_kv) && (_pos);                                           \
+            (_kv->associate == NULL) ?                                 \
+                    ((_pos = _pos->next), (_kv = (_pos) ?              \
+                               csr1212_get_keyval((_csr), _pos->kv) :  \
+                               NULL)) :                                \
                     (_kv = csr1212_get_keyval((_csr), _kv->associate)))
 
-
-
 #endif /* __CSR1212_H__ */