lkml.org 
[lkml]   [2008]   [Nov]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: sparc: struct device - replace bus_id with dev_name(), dev_set_name()
From
From: Kay Sievers <kay.sievers@vrfy.org>
Date: Fri, 07 Nov 2008 01:43:59 +0100

> --- a/arch/sparc/kernel/of_device.c
> +++ b/arch/sparc/kernel/of_device.c
> @@ -563,9 +563,9 @@ build_resources:
> op->dev.parent = parent;
> op->dev.bus = &of_platform_bus_type;
> if (!parent)
> - strcpy(op->dev.bus_id, "root");
> + dev_set_name(&op->dev, "root");
> else
> - sprintf(op->dev.bus_id, "%08x", dp->node);
> + dev_set_name(&op->dev, "%08x", dp->node);
>
> if (of_device_register(op)) {
> printk("%s: Could not register of device.\n",

This part is OK.

> --- a/arch/sparc64/kernel/vio.c
> +++ b/arch/sparc64/kernel/vio.c
> @@ -224,7 +224,7 @@ static struct vio_dev *vio_create_one(st
> if (!strcmp(type, "domain-services-port"))
> bus_id_name = "ds";
>
> - if (strlen(bus_id_name) >= BUS_ID_SIZE - 4) {
> + if (strlen(bus_id_name) >= 20 - 4) {
> printk(KERN_ERR "VIO: bus_id_name [%s] is too long.\n",
> bus_id_name);
> return NULL;
>
>

But I don't like this.

Could you please keep the macro around until everything is converted?
Then you can remove the test entirely.

Leaving it with just constants there is inviting confusion, no matter
how short amount of time it will be there.


\
 
 \ /
  Last update: 2008-11-07 08:39    [W:0.089 / U:0.196 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site