1 files changed, 61 insertions, 30 deletions

diff --git a/arch/sparc64/kernel/prom.c b/arch/sparc64/kernel/prom.c
index 7151513f156e..dbba82f9b142 100644
--- a/arch/sparc64/kernel/prom.c
+++ b/arch/sparc64/kernel/prom.c
@@ -38,7 +38,7 @@ struct device_node *of_find_node_by_phandle(phandle handle)
 {
         struct device_node *np;
 
-        for (np = allnodes; np != 0; np = np->allnext)
+        for (np = allnodes; np; np = np->allnext)
                 if (np->node == handle)
                         break;
 
@@ -59,6 +59,9 @@ int of_getintprop_default(struct device_node *np, const char *name, int def)
 }
 EXPORT_SYMBOL(of_getintprop_default);
 
+DEFINE_MUTEX(of_set_property_mutex);
+EXPORT_SYMBOL(of_set_property_mutex);
+
 int of_set_property(struct device_node *dp, const char *name, void *val, int len)
 {
         struct property **prevp;
@@ -82,7 +85,10 @@ int of_set_property(struct device_node *dp, const char *name, void *val, int len
                         void *old_val = prop->value;
                         int ret;
 
+                        mutex_lock(&of_set_property_mutex);
                         ret = prom_setprop(dp->node, name, val, len);
+                        mutex_unlock(&of_set_property_mutex);
+
                         err = -EINVAL;
                         if (ret >= 0) {
                                 prop->value = new_val;
@@ -945,22 +951,30 @@ static void __init irq_trans_init(struct device_node *dp)
                 for (i = 0; i < ARRAY_SIZE(pci_irq_trans_table); i++) {
                         struct irq_trans *t = &pci_irq_trans_table[i];
 
-                        if (!strcmp(model, t->name))
-                                return t->init(dp);
+                        if (!strcmp(model, t->name)) {
+                                t->init(dp);
+                                return;
+                        }
                 }
         }
 #endif
 #ifdef CONFIG_SBUS
         if (!strcmp(dp->name, "sbus") ||
-            !strcmp(dp->name, "sbi"))
-                return sbus_irq_trans_init(dp);
+            !strcmp(dp->name, "sbi")) {
+                sbus_irq_trans_init(dp);
+                return;
+        }
 #endif
         if (!strcmp(dp->name, "fhc") &&
-            !strcmp(dp->parent->name, "central"))
-                return central_irq_trans_init(dp);
+            !strcmp(dp->parent->name, "central")) {
+                central_irq_trans_init(dp);
+                return;
+        }
         if (!strcmp(dp->name, "virtual-devices") ||
-            !strcmp(dp->name, "niu"))
-                return sun4v_vdev_irq_trans_init(dp);
+            !strcmp(dp->name, "niu")) {
+                sun4v_vdev_irq_trans_init(dp);
+                return;
+        }
 }
 
 static int is_root_node(const struct device_node *dp)
@@ -1231,32 +1245,49 @@ static void __init __build_path_component(struct device_node *dp, char *tmp_buf)
 
         if (parent != NULL) {
                 if (!strcmp(parent->type, "pci") ||
-                    !strcmp(parent->type, "pciex"))
-                        return pci_path_component(dp, tmp_buf);
-                if (!strcmp(parent->type, "sbus"))
-                        return sbus_path_component(dp, tmp_buf);
-                if (!strcmp(parent->type, "upa"))
-                        return upa_path_component(dp, tmp_buf);
-                if (!strcmp(parent->type, "ebus"))
-                        return ebus_path_component(dp, tmp_buf);
+                    !strcmp(parent->type, "pciex")) {
+                        pci_path_component(dp, tmp_buf);
+                        return;
+                }
+                if (!strcmp(parent->type, "sbus")) {
+                        sbus_path_component(dp, tmp_buf);
+                        return;
+                }
+                if (!strcmp(parent->type, "upa")) {
+                        upa_path_component(dp, tmp_buf);
+                        return;
+                }
+                if (!strcmp(parent->type, "ebus")) {
+                        ebus_path_component(dp, tmp_buf);
+                        return;
+                }
                 if (!strcmp(parent->name, "usb") ||
-                    !strcmp(parent->name, "hub"))
-                        return usb_path_component(dp, tmp_buf);
-                if (!strcmp(parent->type, "i2c"))
-                        return i2c_path_component(dp, tmp_buf);
-                if (!strcmp(parent->type, "firewire"))
-                        return ieee1394_path_component(dp, tmp_buf);
-                if (!strcmp(parent->type, "virtual-devices"))
-                        return vdev_path_component(dp, tmp_buf);
-
+                    !strcmp(parent->name, "hub")) {
+                        usb_path_component(dp, tmp_buf);
+                        return;
+                }
+                if (!strcmp(parent->type, "i2c")) {
+                        i2c_path_component(dp, tmp_buf);
+                        return;
+                }
+                if (!strcmp(parent->type, "firewire")) {
+                        ieee1394_path_component(dp, tmp_buf);
+                        return;
+                }
+                if (!strcmp(parent->type, "virtual-devices")) {
+                        vdev_path_component(dp, tmp_buf);
+                        return;
+                }
                 /* "isa" is handled with platform naming */
         }
 
         /* Use platform naming convention.  */
-        if (tlb_type == hypervisor)
-                return sun4v_path_component(dp, tmp_buf);
-        else
-                return sun4u_path_component(dp, tmp_buf);
+        if (tlb_type == hypervisor) {
+                sun4v_path_component(dp, tmp_buf);
+                return;
+        } else {
+                sun4u_path_component(dp, tmp_buf);
+        }
 }
 
 static char * __init build_path_component(struct device_node *dp)