Commit 0bdba867 authored by Rob Herring's avatar Rob Herring Committed by Michael Ellerman

macintosh: Convert to using %pOFn instead of device_node.name

In preparation to remove the node name pointer from struct device_node,
convert printf users to use the %pOFn format specifier.
Signed-off-by: default avatarRob Herring <robh@kernel.org>
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
parent c417596d
...@@ -360,9 +360,10 @@ static struct macio_dev * macio_add_one_device(struct macio_chip *chip, ...@@ -360,9 +360,10 @@ static struct macio_dev * macio_add_one_device(struct macio_chip *chip,
struct macio_dev *in_bay, struct macio_dev *in_bay,
struct resource *parent_res) struct resource *parent_res)
{ {
char name[MAX_NODE_NAME_SIZE + 1];
struct macio_dev *dev; struct macio_dev *dev;
const u32 *reg; const u32 *reg;
if (np == NULL) if (np == NULL)
return NULL; return NULL;
...@@ -402,6 +403,7 @@ static struct macio_dev * macio_add_one_device(struct macio_chip *chip, ...@@ -402,6 +403,7 @@ static struct macio_dev * macio_add_one_device(struct macio_chip *chip,
#endif #endif
/* MacIO itself has a different reg, we use it's PCI base */ /* MacIO itself has a different reg, we use it's PCI base */
snprintf(name, sizeof(name), "%pOFn", np);
if (np == chip->of_node) { if (np == chip->of_node) {
dev_set_name(&dev->ofdev.dev, "%1d.%08x:%.*s", dev_set_name(&dev->ofdev.dev, "%1d.%08x:%.*s",
chip->lbus.index, chip->lbus.index,
...@@ -410,12 +412,12 @@ static struct macio_dev * macio_add_one_device(struct macio_chip *chip, ...@@ -410,12 +412,12 @@ static struct macio_dev * macio_add_one_device(struct macio_chip *chip,
#else #else
0, /* NuBus may want to do something better here */ 0, /* NuBus may want to do something better here */
#endif #endif
MAX_NODE_NAME_SIZE, np->name); MAX_NODE_NAME_SIZE, name);
} else { } else {
reg = of_get_property(np, "reg", NULL); reg = of_get_property(np, "reg", NULL);
dev_set_name(&dev->ofdev.dev, "%1d.%08x:%.*s", dev_set_name(&dev->ofdev.dev, "%1d.%08x:%.*s",
chip->lbus.index, chip->lbus.index,
reg ? *reg : 0, MAX_NODE_NAME_SIZE, np->name); reg ? *reg : 0, MAX_NODE_NAME_SIZE, name);
} }
/* Setup interrupts & resources */ /* Setup interrupts & resources */
......
...@@ -58,7 +58,13 @@ static ssize_t devspec_show(struct device *dev, ...@@ -58,7 +58,13 @@ static ssize_t devspec_show(struct device *dev,
static DEVICE_ATTR_RO(modalias); static DEVICE_ATTR_RO(modalias);
static DEVICE_ATTR_RO(devspec); static DEVICE_ATTR_RO(devspec);
macio_config_of_attr (name, "%s\n"); static ssize_t name_show(struct device *dev,
struct device_attribute *attr, char *buf)
{
return sprintf(buf, "%pOFn\n", dev->of_node);
}
static DEVICE_ATTR_RO(name);
macio_config_of_attr (type, "%s\n"); macio_config_of_attr (type, "%s\n");
static struct attribute *macio_dev_attrs[] = { static struct attribute *macio_dev_attrs[] = {
......
...@@ -277,7 +277,7 @@ static int __init smu_controls_init(void) ...@@ -277,7 +277,7 @@ static int __init smu_controls_init(void)
fct = smu_fan_create(fan, 0); fct = smu_fan_create(fan, 0);
if (fct == NULL) { if (fct == NULL) {
printk(KERN_WARNING "windfarm: Failed to create SMU " printk(KERN_WARNING "windfarm: Failed to create SMU "
"RPM fan %s\n", fan->name); "RPM fan %pOFn\n", fan);
continue; continue;
} }
list_add(&fct->link, &smu_fans); list_add(&fct->link, &smu_fans);
...@@ -296,7 +296,7 @@ static int __init smu_controls_init(void) ...@@ -296,7 +296,7 @@ static int __init smu_controls_init(void)
fct = smu_fan_create(fan, 1); fct = smu_fan_create(fan, 1);
if (fct == NULL) { if (fct == NULL) {
printk(KERN_WARNING "windfarm: Failed to create SMU " printk(KERN_WARNING "windfarm: Failed to create SMU "
"PWM fan %s\n", fan->name); "PWM fan %pOFn\n", fan);
continue; continue;
} }
list_add(&fct->link, &smu_fans); list_add(&fct->link, &smu_fans);
......
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