2
0
mirror of https://github.com/edk2-porting/linux-next.git synced 2024-12-25 05:34:00 +08:00

of: Ensure unique names without sacrificing determinism

The way the driver core is implemented, every device using the same bus
type is required to have a unique name because a symlink to each device
is created in the appropriate /sys/bus/*/devices directory, and two
identical names causes a collision.

The current code handles the requirement by using an globally
incremented counter that is appended to the device name. It works, but
it means any change to device registration will change the assigned
numbers. Instead, if we build up the name by using information from the
parent nodes, then it can be guaranteed to be unique without adding a
random number to the end of it.

Signed-off-by: Grant Likely <grant.likely@linaro.org>
Cc: Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>
Cc: Rob Herring <robh@kernel.org>
This commit is contained in:
Grant Likely 2014-05-21 15:40:31 +09:00
parent eafd370dfe
commit 07e461cd7e

View File

@ -68,17 +68,15 @@ EXPORT_SYMBOL(of_find_device_by_node);
* of_device_make_bus_id - Use the device node data to assign a unique name * of_device_make_bus_id - Use the device node data to assign a unique name
* @dev: pointer to device structure that is linked to a device tree node * @dev: pointer to device structure that is linked to a device tree node
* *
* This routine will first try using either the dcr-reg or the reg property * This routine will first try using the translated bus address to
* value to derive a unique name. As a last resort it will use the node * derive a unique name. If it cannot, then it will prepend names from
* name followed by a unique number. * parent nodes until a unique name can be derived.
*/ */
void of_device_make_bus_id(struct device *dev) void of_device_make_bus_id(struct device *dev)
{ {
static atomic_t bus_no_reg_magic;
struct device_node *node = dev->of_node; struct device_node *node = dev->of_node;
const __be32 *reg; const __be32 *reg;
u64 addr; u64 addr;
int magic;
#ifdef CONFIG_PPC_DCR #ifdef CONFIG_PPC_DCR
/* /*
@ -100,25 +98,25 @@ void of_device_make_bus_id(struct device *dev)
} }
#endif /* CONFIG_PPC_DCR */ #endif /* CONFIG_PPC_DCR */
/* /* Construct the name, using parent nodes if necessary to ensure uniqueness */
* For MMIO, get the physical address while (node->parent) {
*/ /*
reg = of_get_property(node, "reg", NULL); * If the address can be translated, then that is as much
if (reg) { * uniqueness as we need. Make it the first component and return
addr = of_translate_address(node, reg); */
if (addr != OF_BAD_ADDR) { reg = of_get_property(node, "reg", NULL);
dev_set_name(dev, "%llx.%s", if (reg && (addr = of_translate_address(node, reg)) != OF_BAD_ADDR) {
(unsigned long long)addr, node->name); dev_set_name(dev, dev_name(dev) ? "%llx.%s:%s" : "%llx.%s",
(unsigned long long)addr, node->name,
dev_name(dev));
return; return;
} }
}
/* /* format arguments only used if dev_name() resolves to NULL */
* No BusID, use the node name and add a globally incremented dev_set_name(dev, dev_name(dev) ? "%s:%s" : "%s",
* counter (and pray...) strrchr(node->full_name, '/') + 1, dev_name(dev));
*/ node = node->parent;
magic = atomic_add_return(1, &bus_no_reg_magic); }
dev_set_name(dev, "%s.%d", node->name, magic - 1);
} }
/** /**