Skip to content

Commit

Permalink
[S390] cio: remove ccw_device init_name
Browse files Browse the repository at this point in the history
We used the init_name to set the console ccw_device's name early
at the boot stage. This patch moves the name setting (for all ccw
devices) to the point where we actually register the device. At this
time we can do dynamic allocations and therefore use dev_set_name.

Signed-off-by: Sebastian Ott <sebott@linux.vnet.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
  • Loading branch information
Sebastian Ott authored and Martin Schwidefsky committed Sep 11, 2009
1 parent 3b554a1 commit 3ac276f
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 19 deletions.
2 changes: 0 additions & 2 deletions drivers/s390/cio/cio.h
Original file line number Diff line number Diff line change
Expand Up @@ -133,13 +133,11 @@ extern int cio_is_console(struct subchannel_id);
extern struct subchannel *cio_get_console_subchannel(void);
extern spinlock_t * cio_get_console_lock(void);
extern void *cio_get_console_priv(void);
extern const char *cio_get_console_cdev_name(struct subchannel *sch);
#else
#define cio_is_console(schid) 0
#define cio_get_console_subchannel() NULL
#define cio_get_console_lock() NULL
#define cio_get_console_priv() NULL
#define cio_get_console_cdev_name(sch) NULL
#endif

#endif
23 changes: 6 additions & 17 deletions drivers/s390/cio/device.c
Original file line number Diff line number Diff line change
Expand Up @@ -669,8 +669,12 @@ static int ccw_device_register(struct ccw_device *cdev)
int ret;

dev->bus = &ccw_bus_type;

if ((ret = device_add(dev)))
ret = dev_set_name(&cdev->dev, "0.%x.%04x", cdev->private->dev_id.ssid,
cdev->private->dev_id.devno);
if (ret)
return ret;
ret = device_add(dev);
if (ret)
return ret;

set_bit(1, &cdev->private->registered);
Expand Down Expand Up @@ -1124,13 +1128,6 @@ io_subchannel_recog(struct ccw_device *cdev, struct subchannel *sch)
init_waitqueue_head(&priv->wait_q);
init_timer(&priv->timer);

/* Set an initial name for the device. */
if (cio_is_console(sch->schid))
cdev->dev.init_name = cio_get_console_cdev_name(sch);
else
dev_set_name(&cdev->dev, "0.%x.%04x",
sch->schid.ssid, sch->schib.pmcw.dev);

/* Increase counter of devices currently in recognition. */
atomic_inc(&ccw_device_init_count);

Expand Down Expand Up @@ -1731,7 +1728,6 @@ static int io_subchannel_sch_event(struct subchannel *sch, int slow)

#ifdef CONFIG_CCW_CONSOLE
static struct ccw_device console_cdev;
static char console_cdev_name[10] = "0.x.xxxx";
static struct ccw_device_private console_private;
static int console_cdev_in_use;

Expand Down Expand Up @@ -1815,13 +1811,6 @@ int ccw_device_force_console(void)
return ccw_device_pm_restore(&console_cdev.dev);
}
EXPORT_SYMBOL_GPL(ccw_device_force_console);

const char *cio_get_console_cdev_name(struct subchannel *sch)
{
snprintf(console_cdev_name, 10, "0.%x.%04x",
sch->schid.ssid, sch->schib.pmcw.dev);
return (const char *)console_cdev_name;
}
#endif

/*
Expand Down

0 comments on commit 3ac276f

Please sign in to comment.