}
 static DEVICE_ATTR_RW(consistency_check);
 
-static struct device_attribute *myrs_sdev_attrs[] = {
-       &dev_attr_consistency_check,
-       &dev_attr_rebuild,
-       &dev_attr_raid_state,
-       &dev_attr_raid_level,
+static struct attribute *myrs_sdev_attrs[] = {
+       &dev_attr_consistency_check.attr,
+       &dev_attr_rebuild.attr,
+       &dev_attr_raid_state.attr,
+       &dev_attr_raid_level.attr,
        NULL,
 };
 
+ATTRIBUTE_GROUPS(myrs_sdev);
+
 static ssize_t serial_show(struct device *dev,
                struct device_attribute *attr, char *buf)
 {
 }
 static DEVICE_ATTR_RW(disable_enclosure_messages);
 
-static struct device_attribute *myrs_shost_attrs[] = {
-       &dev_attr_serial,
-       &dev_attr_ctlr_num,
-       &dev_attr_processor,
-       &dev_attr_model,
-       &dev_attr_ctlr_type,
-       &dev_attr_cache_size,
-       &dev_attr_firmware,
-       &dev_attr_discovery,
-       &dev_attr_flush_cache,
-       &dev_attr_disable_enclosure_messages,
+static struct attribute *myrs_shost_attrs[] = {
+       &dev_attr_serial.attr,
+       &dev_attr_ctlr_num.attr,
+       &dev_attr_processor.attr,
+       &dev_attr_model.attr,
+       &dev_attr_ctlr_type.attr,
+       &dev_attr_cache_size.attr,
+       &dev_attr_firmware.attr,
+       &dev_attr_discovery.attr,
+       &dev_attr_flush_cache.attr,
+       &dev_attr_disable_enclosure_messages.attr,
        NULL,
 };
 
+ATTRIBUTE_GROUPS(myrs_shost);
+
 /*
  * SCSI midlayer interface
  */
        .slave_configure        = myrs_slave_configure,
        .slave_destroy          = myrs_slave_destroy,
        .cmd_size               = sizeof(struct myrs_cmdblk),
-       .shost_attrs            = myrs_shost_attrs,
-       .sdev_attrs             = myrs_sdev_attrs,
+       .shost_groups           = myrs_shost_groups,
+       .sdev_groups            = myrs_sdev_groups,
        .this_id                = -1,
 };