Messages in this thread Patch in this message | | | From | Dmitry Torokhov <> | Subject | [RFC/PATCH 2/5] sysfs: (driver/base) if show/store is missing return -ENOSYS | Date | Thu, 28 Apr 2005 00:41:41 -0500 |
| |
sysfs: fix drivers/base so if an attribute doesn't implement show or store method read/write will return -ENOSYS instead of 0.
Signed-off-by: Dmitry Torokhov <dtor@mail.ru> ---
bus.c | 4 ++-- class.c | 4 ++-- core.c | 4 ++-- sys.c | 4 ++-- 4 files changed, 8 insertions(+), 8 deletions(-)
Index: dtor/drivers/base/class.c =================================================================== --- dtor.orig/drivers/base/class.c +++ dtor/drivers/base/class.c @@ -26,7 +26,7 @@ class_attr_show(struct kobject * kobj, s { struct class_attribute * class_attr = to_class_attr(attr); struct class * dc = to_class(kobj); - ssize_t ret = 0; + ssize_t ret = -ENOSYS; if (class_attr->show) ret = class_attr->show(dc, buf); @@ -39,7 +39,7 @@ class_attr_store(struct kobject * kobj, { struct class_attribute * class_attr = to_class_attr(attr); struct class * dc = to_class(kobj); - ssize_t ret = 0; + ssize_t ret = -ENOSYS; if (class_attr->store) ret = class_attr->store(dc, buf, count); Index: dtor/drivers/base/sys.c =================================================================== --- dtor.orig/drivers/base/sys.c +++ dtor/drivers/base/sys.c @@ -37,7 +37,7 @@ sysdev_show(struct kobject * kobj, struc if (sysdev_attr->show) return sysdev_attr->show(sysdev, buffer); - return 0; + return -ENOSYS; } @@ -50,7 +50,7 @@ sysdev_store(struct kobject * kobj, stru if (sysdev_attr->store) return sysdev_attr->store(sysdev, buffer, count); - return 0; + return -ENOSYS; } static struct sysfs_ops sysfs_ops = { Index: dtor/drivers/base/bus.c =================================================================== --- dtor.orig/drivers/base/bus.c +++ dtor/drivers/base/bus.c @@ -36,7 +36,7 @@ drv_attr_show(struct kobject * kobj, str { struct driver_attribute * drv_attr = to_drv_attr(attr); struct device_driver * drv = to_driver(kobj); - ssize_t ret = 0; + ssize_t ret = -ENOSYS; if (drv_attr->show) ret = drv_attr->show(drv, buf); @@ -49,7 +49,7 @@ drv_attr_store(struct kobject * kobj, st { struct driver_attribute * drv_attr = to_drv_attr(attr); struct device_driver * drv = to_driver(kobj); - ssize_t ret = 0; + ssize_t ret = -ENOSYS; if (drv_attr->store) ret = drv_attr->store(drv, buf, count); Index: dtor/drivers/base/core.c =================================================================== --- dtor.orig/drivers/base/core.c +++ dtor/drivers/base/core.c @@ -38,7 +38,7 @@ dev_attr_show(struct kobject * kobj, str { struct device_attribute * dev_attr = to_dev_attr(attr); struct device * dev = to_dev(kobj); - ssize_t ret = 0; + ssize_t ret = -ENOSYS; if (dev_attr->show) ret = dev_attr->show(dev, buf); @@ -51,7 +51,7 @@ dev_attr_store(struct kobject * kobj, st { struct device_attribute * dev_attr = to_dev_attr(attr); struct device * dev = to_dev(kobj); - ssize_t ret = 0; + ssize_t ret = -ENOSYS; if (dev_attr->store) ret = dev_attr->store(dev, buf, count); - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |