Merge tag 'for-v4.12-2' of git://git.kernel.org/pub/scm/linux/kernel/git/sre/linux...

[karo-tx-linux.git] / drivers / rapidio / rio-sysfs.c

diff --git a/drivers/rapidio/rio-sysfs.c b/drivers/rapidio/rio-sysfs.c
index eda41563d06d8a1f0e1fa65267fb9441bdf75b30..73e4b407f162fa0b5b9184736d17850e1d8c35c0 100644 (file)
--- a/drivers/rapidio/rio-sysfs.c
+++ b/drivers/rapidio/rio-sysfs.c
@@ -108,15 +108,11 @@ static struct attribute *rio_dev_attrs[] = {
        &dev_attr_lprev.attr,
        &dev_attr_destid.attr,
        &dev_attr_modalias.attr,
-       NULL,
-};
 
-static const struct attribute_group rio_dev_group = {
-       .attrs = rio_dev_attrs,
-};
-
-const struct attribute_group *rio_dev_groups[] = {
-       &rio_dev_group,
+       /* Switch-only attributes */
+       &dev_attr_routes.attr,
+       &dev_attr_lnext.attr,
+       &dev_attr_hopcount.attr,
        NULL,
 };
 
@@ -259,46 +255,40 @@ static struct bin_attribute rio_config_attr = {
        .write = rio_write_config,
 };
 
-/**
- * rio_create_sysfs_dev_files - create RIO specific sysfs files
- * @rdev: device whose entries should be created
- *
- * Create files when @rdev is added to sysfs.
- */
-int rio_create_sysfs_dev_files(struct rio_dev *rdev)
-{
-       int err = 0;
-
-       err = device_create_bin_file(&rdev->dev, &rio_config_attr);
+static struct bin_attribute *rio_dev_bin_attrs[] = {
+       &rio_config_attr,
+       NULL,
+};
 
-       if (!err && (rdev->pef & RIO_PEF_SWITCH)) {
-               err |= device_create_file(&rdev->dev, &dev_attr_routes);
-               err |= device_create_file(&rdev->dev, &dev_attr_lnext);
-               err |= device_create_file(&rdev->dev, &dev_attr_hopcount);
+static umode_t rio_dev_is_attr_visible(struct kobject *kobj,
+                                      struct attribute *attr, int n)
+{
+       struct rio_dev *rdev = to_rio_dev(kobj_to_dev(kobj));
+       umode_t mode = attr->mode;
+
+       if (!(rdev->pef & RIO_PEF_SWITCH) &&
+           (attr == &dev_attr_routes.attr ||
+            attr == &dev_attr_lnext.attr ||
+            attr == &dev_attr_hopcount.attr)) {
+               /*
+                * Hide switch-specific attributes for a non-switch device.
+                */
+               mode = 0;
        }
 
-       if (err)
-               pr_warning("RIO: Failed to create attribute file(s) for %s\n",
-                          rio_name(rdev));
-
-       return err;
+       return mode;
 }
 
-/**
- * rio_remove_sysfs_dev_files - cleanup RIO specific sysfs files
- * @rdev: device whose entries we should free
- *
- * Cleanup when @rdev is removed from sysfs.
- */
-void rio_remove_sysfs_dev_files(struct rio_dev *rdev)
-{
-       device_remove_bin_file(&rdev->dev, &rio_config_attr);
-       if (rdev->pef & RIO_PEF_SWITCH) {
-               device_remove_file(&rdev->dev, &dev_attr_routes);
-               device_remove_file(&rdev->dev, &dev_attr_lnext);
-               device_remove_file(&rdev->dev, &dev_attr_hopcount);
-       }
-}
+static const struct attribute_group rio_dev_group = {
+       .attrs          = rio_dev_attrs,
+       .is_visible     = rio_dev_is_attr_visible,
+       .bin_attrs      = rio_dev_bin_attrs,
+};
+
+const struct attribute_group *rio_dev_groups[] = {
+       &rio_dev_group,
+       NULL,
+};
 
 static ssize_t bus_scan_store(struct bus_type *bus, const char *buf,
                                size_t count)