[POWERPC] Generalise device_node flag interface
authorMichael Ellerman <michael@ellerman.id.au>
Tue, 19 Jun 2007 06:07:58 +0000 (16:07 +1000)
committerPaul Mackerras <paulus@samba.org>
Tue, 10 Jul 2007 11:53:46 +0000 (21:53 +1000)
The struct device_node currently has a _flags variable, although
it's only used for one flag - OF_DYNAMIC.  Generalise the flag
accessors so we can use them with other flags in future.

Signed-off-by: Michael Ellerman <michael@ellerman.id.au>
Signed-off-by: Paul Mackerras <paulus@samba.org>
arch/powerpc/kernel/prom.c
arch/powerpc/platforms/pseries/reconfig.c
include/asm-powerpc/prom.h

index cc9632c7149ed3bc5f2c195d598f4a3d5678a906..bcd1c5ed44a35066be85372366f9d6240c409e88 100644 (file)
@@ -1375,7 +1375,7 @@ static void of_node_release(struct kref *kref)
        struct device_node *node = kref_to_device_node(kref);
        struct property *prop = node->properties;
 
-       if (!OF_IS_DYNAMIC(node))
+       if (!of_node_check_flag(node, OF_DYNAMIC))
                return;
        while (prop) {
                struct property *next = prop->next;
index 5aa97aff3391ee9a822997d6d4cd893798324725..c02f8742c54de5792650c2e641003637921cad03 100644 (file)
@@ -123,7 +123,7 @@ static int pSeries_reconfig_add_node(const char *path, struct property *proplist
        strcpy(np->full_name, path);
 
        np->properties = proplist;
-       OF_MARK_DYNAMIC(np);
+       of_node_set_flag(np, OF_DYNAMIC);
        kref_init(&np->kref);
 
        np->parent = derive_parent(path);
index 1122a9278afd86102e65ec3d66c0645dad5adb61..f1006b91bd1a40649f79a130ff728cabc9b389ef 100644 (file)
@@ -98,10 +98,18 @@ struct device_node {
 extern struct device_node *of_chosen;
 
 /* flag descriptions */
-#define OF_DYNAMIC 1 /* node and properties were allocated via kmalloc */
+#define OF_DYNAMIC     1 /* node and properties were allocated via kmalloc */
+
+static inline int of_node_check_flag(struct device_node *n, unsigned long flag)
+{
+       return test_bit(flag, &n->_flags);
+}
+
+static inline void of_node_set_flag(struct device_node *n, unsigned long flag)
+{
+       set_bit(flag, &n->_flags);
+}
 
-#define OF_IS_DYNAMIC(x) test_bit(OF_DYNAMIC, &x->_flags)
-#define OF_MARK_DYNAMIC(x) set_bit(OF_DYNAMIC, &x->_flags)
 
 #define HAVE_ARCH_DEVTREE_FIXUPS