From 9ea13e27ba8210da9661a7514e93aa8bbd0be265 Mon Sep 17 00:00:00 2001 From: Shaun Ruffell Date: Wed, 21 Mar 2012 19:12:10 +0000 Subject: dahdi_dynamic: Since dynamic devices are 'parentless' we must name them. This in conjunction with r10449 "A parent-less device should not crash dahdi", this allows dahdi_dynamic spans to work post the dahdi_devices changes in 2.6.0. The full address of the device is not used since kernels prior to 2.6.31 limit the length of a devicename to 20 characters. The full address of the device can be pulled out of the "hardware_id" and "type" fields of the span. This patch is just to get things working again. dahdi_dynamic devices *may* still have issues if the auto_assign_spans module parameter is 0. Internal-Issue-ID: DAHLIN-280 Reported-by: Pavel Selivanov Signed-off-by: Shaun Ruffell Origin: http://svnview.digium.com/svn/dahdi?view=rev&rev=10563 git-svn-id: http://svn.asterisk.org/svn/dahdi/linux/branches/2.6@10571 a0bf4364-ded3-4de4-8d8a-66a801d63aff --- drivers/dahdi/dahdi_dynamic.c | 10 +++++++--- include/dahdi/kernel.h | 3 +++ 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/drivers/dahdi/dahdi_dynamic.c b/drivers/dahdi/dahdi_dynamic.c index cbcdce7..aec4a7d 100644 --- a/drivers/dahdi/dahdi_dynamic.c +++ b/drivers/dahdi/dahdi_dynamic.c @@ -605,9 +605,10 @@ static int _create_dynamic(struct dahdi_dynamic_span *dds) strlcpy(d->dname, dds->driver, sizeof(d->dname)); strlcpy(d->addr, dds->addr, sizeof(d->addr)); d->timing = dds->timing; - sprintf(d->span.name, "DYN/%s/%s", dds->driver, dds->addr); - sprintf(d->span.desc, "Dynamic '%s' span at '%s'", - dds->driver, dds->addr); + snprintf(d->span.name, sizeof(d->span.name), "DYN/%s/%s", + dds->driver, dds->addr); + snprintf(d->span.desc, sizeof(d->span.desc), + "Dynamic '%s' span at '%s'", dds->driver, dds->addr); d->span.deflaw = DAHDI_LAW_MULAW; d->span.flags |= DAHDI_FLAG_RBS; d->span.chans = d->chans; @@ -658,6 +659,9 @@ static int _create_dynamic(struct dahdi_dynamic_span *dds) return res; } + d->ddev->devicetype = d->span.name; + d->ddev->hardware_id = d->span.name; + dev_set_name(&d->ddev->dev, "dynamic:%s:%d", dds->driver, dtd->id++); list_add_tail(&d->span.device_node, &d->ddev->spans); /* Whee! We're created. Now register the span */ if (dahdi_register_device(d->ddev, d->dev)) { diff --git a/include/dahdi/kernel.h b/include/dahdi/kernel.h index 9607fd2..c3a8dd5 100644 --- a/include/dahdi/kernel.h +++ b/include/dahdi/kernel.h @@ -1116,6 +1116,9 @@ struct dahdi_dynamic_driver { struct list_head list; struct module *owner; + + /*! Numberic id of next device created by this driver. */ + unsigned int id; }; /*! \brief Receive a dynamic span message */ -- cgit v1.2.3