attrib##_show(struct device *dev, struct device_attribute *attr, char *buf) \
 { \
        return sprintf(buf, format_string, dev_to_ssb_dev(dev)->field); \
-}
+} \
+static DEVICE_ATTR_RO(attrib);
 
 ssb_config_attr(core_num, core_index, "%u\n")
 ssb_config_attr(coreid, id.coreid, "0x%04x\n")
        return sprintf(buf, "%s\n",
                       ssb_core_name(dev_to_ssb_dev(dev)->id.coreid));
 }
-
-static struct device_attribute ssb_device_attrs[] = {
-       __ATTR_RO(name),
-       __ATTR_RO(core_num),
-       __ATTR_RO(coreid),
-       __ATTR_RO(vendor),
-       __ATTR_RO(revision),
-       __ATTR_RO(irq),
-       __ATTR_NULL,
+static DEVICE_ATTR_RO(name);
+
+static struct attribute *ssb_device_attrs[] = {
+       &dev_attr_name.attr,
+       &dev_attr_core_num.attr,
+       &dev_attr_coreid.attr,
+       &dev_attr_vendor.attr,
+       &dev_attr_revision.attr,
+       &dev_attr_irq.attr,
+       NULL,
 };
+ATTRIBUTE_GROUPS(ssb_device);
 
 static struct bus_type ssb_bustype = {
        .name           = "ssb",
        .suspend        = ssb_device_suspend,
        .resume         = ssb_device_resume,
        .uevent         = ssb_device_uevent,
-       .dev_attrs      = ssb_device_attrs,
+       .dev_groups     = ssb_device_groups,
 };
 
 static void ssb_buses_lock(void)