diff options
| -rw-r--r-- | drivers/pci/hotplug/pci_hotplug_core.c | 4 | ||||
| -rw-r--r-- | drivers/pci/hotplug/rpadlpar_sysfs.c | 2 | ||||
| -rw-r--r-- | drivers/pci/pci-driver.c | 26 |
3 files changed, 17 insertions, 15 deletions
diff --git a/drivers/pci/hotplug/pci_hotplug_core.c b/drivers/pci/hotplug/pci_hotplug_core.c index c802f6270b89..c4282902cb52 100644 --- a/drivers/pci/hotplug/pci_hotplug_core.c +++ b/drivers/pci/hotplug/pci_hotplug_core.c | |||
| @@ -73,7 +73,7 @@ static ssize_t hotplug_slot_attr_show(struct kobject *kobj, | |||
| 73 | { | 73 | { |
| 74 | struct hotplug_slot *slot = to_hotplug_slot(kobj); | 74 | struct hotplug_slot *slot = to_hotplug_slot(kobj); |
| 75 | struct hotplug_slot_attribute *attribute = to_hotplug_attr(attr); | 75 | struct hotplug_slot_attribute *attribute = to_hotplug_attr(attr); |
| 76 | return attribute->show ? attribute->show(slot, buf) : 0; | 76 | return attribute->show ? attribute->show(slot, buf) : -EIO; |
| 77 | } | 77 | } |
| 78 | 78 | ||
| 79 | static ssize_t hotplug_slot_attr_store(struct kobject *kobj, | 79 | static ssize_t hotplug_slot_attr_store(struct kobject *kobj, |
| @@ -81,7 +81,7 @@ static ssize_t hotplug_slot_attr_store(struct kobject *kobj, | |||
| 81 | { | 81 | { |
| 82 | struct hotplug_slot *slot = to_hotplug_slot(kobj); | 82 | struct hotplug_slot *slot = to_hotplug_slot(kobj); |
| 83 | struct hotplug_slot_attribute *attribute = to_hotplug_attr(attr); | 83 | struct hotplug_slot_attribute *attribute = to_hotplug_attr(attr); |
| 84 | return attribute->store ? attribute->store(slot, buf, len) : 0; | 84 | return attribute->store ? attribute->store(slot, buf, len) : -EIO; |
| 85 | } | 85 | } |
| 86 | 86 | ||
| 87 | static struct sysfs_ops hotplug_slot_sysfs_ops = { | 87 | static struct sysfs_ops hotplug_slot_sysfs_ops = { |
diff --git a/drivers/pci/hotplug/rpadlpar_sysfs.c b/drivers/pci/hotplug/rpadlpar_sysfs.c index 3285b822478d..752e6513c447 100644 --- a/drivers/pci/hotplug/rpadlpar_sysfs.c +++ b/drivers/pci/hotplug/rpadlpar_sysfs.c | |||
| @@ -48,7 +48,7 @@ dlpar_attr_store(struct kobject * kobj, struct attribute * attr, | |||
| 48 | struct dlpar_io_attr *dlpar_attr = container_of(attr, | 48 | struct dlpar_io_attr *dlpar_attr = container_of(attr, |
| 49 | struct dlpar_io_attr, attr); | 49 | struct dlpar_io_attr, attr); |
| 50 | return dlpar_attr->store ? | 50 | return dlpar_attr->store ? |
| 51 | dlpar_attr->store(dlpar_attr, buf, nbytes) : 0; | 51 | dlpar_attr->store(dlpar_attr, buf, nbytes) : -EIO; |
| 52 | } | 52 | } |
| 53 | 53 | ||
| 54 | static struct sysfs_ops dlpar_attr_sysfs_ops = { | 54 | static struct sysfs_ops dlpar_attr_sysfs_ops = { |
diff --git a/drivers/pci/pci-driver.c b/drivers/pci/pci-driver.c index cf2cff7480f1..e65bf2b395aa 100644 --- a/drivers/pci/pci-driver.c +++ b/drivers/pci/pci-driver.c | |||
| @@ -335,13 +335,14 @@ pci_driver_attr_show(struct kobject * kobj, struct attribute *attr, char *buf) | |||
| 335 | { | 335 | { |
| 336 | struct device_driver *driver = kobj_to_pci_driver(kobj); | 336 | struct device_driver *driver = kobj_to_pci_driver(kobj); |
| 337 | struct driver_attribute *dattr = attr_to_driver_attribute(attr); | 337 | struct driver_attribute *dattr = attr_to_driver_attribute(attr); |
| 338 | ssize_t ret = 0; | 338 | ssize_t ret; |
| 339 | 339 | ||
| 340 | if (get_driver(driver)) { | 340 | if (!get_driver(driver)) |
| 341 | if (dattr->show) | 341 | return -ENODEV; |
| 342 | ret = dattr->show(driver, buf); | 342 | |
| 343 | put_driver(driver); | 343 | ret = dattr->show ? dattr->show(driver, buf) : -EIO; |
| 344 | } | 344 | |
| 345 | put_driver(driver); | ||
| 345 | return ret; | 346 | return ret; |
| 346 | } | 347 | } |
| 347 | 348 | ||
| @@ -351,13 +352,14 @@ pci_driver_attr_store(struct kobject * kobj, struct attribute *attr, | |||
| 351 | { | 352 | { |
| 352 | struct device_driver *driver = kobj_to_pci_driver(kobj); | 353 | struct device_driver *driver = kobj_to_pci_driver(kobj); |
| 353 | struct driver_attribute *dattr = attr_to_driver_attribute(attr); | 354 | struct driver_attribute *dattr = attr_to_driver_attribute(attr); |
| 354 | ssize_t ret = 0; | 355 | ssize_t ret; |
| 355 | 356 | ||
| 356 | if (get_driver(driver)) { | 357 | if (!get_driver(driver)) |
| 357 | if (dattr->store) | 358 | return -ENODEV; |
| 358 | ret = dattr->store(driver, buf, count); | 359 | |
| 359 | put_driver(driver); | 360 | ret = dattr->store ? dattr->store(driver, buf, count) : -EIO; |
| 360 | } | 361 | |
| 362 | put_driver(driver); | ||
| 361 | return ret; | 363 | return ret; |
| 362 | } | 364 | } |
| 363 | 365 | ||
