Commit 9309180f authored by Benjamin Herrenschmidt's avatar Benjamin Herrenschmidt Committed by Paul Mackerras

[POWERPC] powerpc: Workaround for of_platform without "reg" nor "dcr-reg"

Devices with no "reg" nor "dcr-reg" property are given a bus_id which
is the node name alone. This means that if more than one such device
with the same names are present in the system, sysfs will have
collisions when creating the symlinks and will fail registering the
devices.

This works around that problem by assigning successive numbers to such
devices.
Signed-off-by: default avatarBenjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: default avatarPaul Mackerras <paulus@samba.org>
parent 974a76f5
...@@ -27,6 +27,8 @@ ...@@ -27,6 +27,8 @@
#include <asm/topology.h> #include <asm/topology.h>
#include <asm/pci-bridge.h> #include <asm/pci-bridge.h>
#include <asm/ppc-pci.h> #include <asm/ppc-pci.h>
#include <asm/atomic.h>
/* /*
* The list of OF IDs below is used for matching bus types in the * The list of OF IDs below is used for matching bus types in the
...@@ -51,6 +53,8 @@ static struct of_device_id of_default_bus_ids[] = { ...@@ -51,6 +53,8 @@ static struct of_device_id of_default_bus_ids[] = {
{}, {},
}; };
static atomic_t bus_no_reg_magic;
/* /*
* *
* OF platform device type definition & base infrastructure * OF platform device type definition & base infrastructure
...@@ -165,6 +169,7 @@ static void of_platform_make_bus_id(struct of_device *dev) ...@@ -165,6 +169,7 @@ static void of_platform_make_bus_id(struct of_device *dev)
char *name = dev->dev.bus_id; char *name = dev->dev.bus_id;
const u32 *reg; const u32 *reg;
u64 addr; u64 addr;
long magic;
/* /*
* If it's a DCR based device, use 'd' for native DCRs * If it's a DCR based device, use 'd' for native DCRs
...@@ -203,9 +208,11 @@ static void of_platform_make_bus_id(struct of_device *dev) ...@@ -203,9 +208,11 @@ static void of_platform_make_bus_id(struct of_device *dev)
} }
/* /*
* No BusID, use the node name and pray * No BusID, use the node name and add a globally incremented
* counter (and pray...)
*/ */
snprintf(name, BUS_ID_SIZE, "%s", node->name); magic = atomic_add_return(1, &bus_no_reg_magic);
snprintf(name, BUS_ID_SIZE, "%s.%d", node->name, magic - 1);
} }
struct of_device* of_platform_device_create(struct device_node *np, struct of_device* of_platform_device_create(struct device_node *np,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment