driver core: bus_type: add bus_groups

attribute groups are much more flexible than just a list of attributes,
due to their support for visibility of the attributes, and binary
attributes. Add bus_groups to struct bus_type which should be used
instead of bus_attrs.

bus_attrs will be removed from the structure soon.

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

1 files changed, 37 insertions, 0 deletions

diff --git a/drivers/base/bus.c b/drivers/base/bus.c
index de5ce22df306..5ee5d3c1d74b 100644
--- a/drivers/base/bus.c
+++ b/drivers/base/bus.c
@@ -881,6 +881,37 @@ static void bus_remove_attrs(struct bus_type *bus)
         }
 }
 
+static int bus_add_groups(struct bus_type *bus,
+                          const struct attribute_group **groups)
+{
+        int error = 0;
+        int i;
+
+        if (groups) {
+                for (i = 0; groups[i]; i++) {
+                        error = sysfs_create_group(&bus->p->subsys.kobj,
+                                                   groups[i]);
+                        if (error) {
+                                while (--i >= 0)
+                                        sysfs_remove_group(&bus->p->subsys.kobj,
+                                                           groups[i]);
+                                break;
+                        }
+                }
+        }
+        return error;
+}
+
+static void bus_remove_groups(struct bus_type *bus,
+                              const struct attribute_group **groups)
+{
+        int i;
+
+        if (groups)
+                for (i = 0; groups[i]; i++)
+                        sysfs_remove_group(&bus->p->subsys.kobj, groups[i]);
+}
+
 static void klist_devices_get(struct klist_node *n)
 {
         struct device_private *dev_prv = to_device_private_bus(n);
@@ -973,10 +1004,15 @@ int bus_register(struct bus_type *bus)
         retval = bus_add_attrs(bus);
         if (retval)
                 goto bus_attrs_fail;
+        retval = bus_add_groups(bus, bus->bus_groups);
+        if (retval)
+                goto bus_groups_fail;
 
         pr_debug("bus: '%s': registered\n", bus->name);
         return 0;
 
+bus_groups_fail:
+        bus_remove_attrs(bus);
 bus_attrs_fail:
         remove_probe_files(bus);
 bus_probe_files_fail:
@@ -1007,6 +1043,7 @@ void bus_unregister(struct bus_type *bus)
         if (bus->dev_root)
                 device_unregister(bus->dev_root);
         bus_remove_attrs(bus);
+        bus_remove_groups(bus, bus->bus_groups);
         remove_probe_files(bus);
         kset_unregister(bus->p->drivers_kset);
         kset_unregister(bus->p->devices_kset);