diff options
| author | James Bottomley <James.Bottomley@steeleye.com> | 2005-08-15 16:13:19 -0500 |
|---|---|---|
| committer | James Bottomley <jejb@mulgrave.(none)> | 2005-08-28 11:14:06 -0500 |
| commit | ebd8bb7647e908e8654e565fa289b0300f9f8fa7 (patch) | |
| tree | a2afebb1cdcb6309e2d1bc5e8c39e7895ade9236 | |
| parent | 975f24bdc7d3833875309509abbc7da2b2a28234 (diff) | |
| download | op-kernel-dev-ebd8bb7647e908e8654e565fa289b0300f9f8fa7.zip op-kernel-dev-ebd8bb7647e908e8654e565fa289b0300f9f8fa7.tar.gz | |
[SCSI] fix transport class corner case after rework
If your transport class sets the ATTRIBUTE_CONTAINER_NO_CLASSDEVS flag,
then its configure method never gets called. This patch fixes that so
that the configure method is called with a NULL classdev.
Also remove a spurious inverted comma in the transport_class comments.
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
| -rw-r--r-- | drivers/base/attribute_container.c | 5 | ||||
| -rw-r--r-- | drivers/base/transport_class.c | 2 |
2 files changed, 6 insertions, 1 deletions
diff --git a/drivers/base/attribute_container.c b/drivers/base/attribute_container.c index 62c093d..ebcae5c 100644 --- a/drivers/base/attribute_container.c +++ b/drivers/base/attribute_container.c @@ -237,6 +237,11 @@ attribute_container_device_trigger(struct device *dev, if (!cont->match(cont, dev)) continue; + if (attribute_container_no_classdevs(cont)) { + fn(cont, dev, NULL); + continue; + } + spin_lock(&cont->containers_lock); list_for_each_entry_safe(ic, tmp, &cont->containers, node) { if (dev == ic->classdev.dev) diff --git a/drivers/base/transport_class.c b/drivers/base/transport_class.c index 4fb4c5d..f25e7c6 100644 --- a/drivers/base/transport_class.c +++ b/drivers/base/transport_class.c @@ -7,7 +7,7 @@ * This file is licensed under GPLv2 * * The basic idea here is to allow any "device controller" (which - * would most often be a Host Bus Adapter" to use the services of one + * would most often be a Host Bus Adapter to use the services of one * or more tranport classes for performing transport specific * services. Transport specific services are things that the generic * command layer doesn't want to know about (speed settings, line |
