Re: [PATCH] attribute_container: update to use the group interface
On Wed, Jan 02, 2008 at 06:48:47PM -0600, James Bottomley wrote: > This patch is the beginning of moving the attribute_containers to use > attribute groups exclusively. The attr element is now deprecated and > will eventually be removed (along with all the hand rolled code for > doing exactly what attribute groups do) when all the consumers are > converted to attribute groups. > > Signed-off-by: James Bottomley <[EMAIL PROTECTED]> Feel free to add: Signed-off-by: Greg Kroah-Hartman <[EMAIL PROTECTED]> and take it through your tree. thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] attribute_container: update to use the group interface
On Wed, Jan 02, 2008 at 06:48:47PM -0600, James Bottomley wrote: This patch is the beginning of moving the attribute_containers to use attribute groups exclusively. The attr element is now deprecated and will eventually be removed (along with all the hand rolled code for doing exactly what attribute groups do) when all the consumers are converted to attribute groups. Signed-off-by: James Bottomley [EMAIL PROTECTED] Feel free to add: Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] and take it through your tree. thanks, greg k-h -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] attribute_container: update to use the group interface
This patch is the beginning of moving the attribute_containers to use attribute groups exclusively. The attr element is now deprecated and will eventually be removed (along with all the hand rolled code for doing exactly what attribute groups do) when all the consumers are converted to attribute groups. Signed-off-by: James Bottomley <[EMAIL PROTECTED]> --- Would you believe this is another one I'd like to send via the SCSI tree? James drivers/base/attribute_container.c | 14 -- include/linux/attribute_container.h |1 + 2 files changed, 13 insertions(+), 2 deletions(-) diff --git a/drivers/base/attribute_container.c b/drivers/base/attribute_container.c index 7370d7c..5bf25c6 100644 --- a/drivers/base/attribute_container.c +++ b/drivers/base/attribute_container.c @@ -320,9 +320,14 @@ attribute_container_add_attrs(struct class_device *classdev) struct class_device_attribute **attrs = cont->attrs; int i, error; - if (!attrs) + BUG_ON(attrs && cont->grp); + + if (!attrs && !cont->grp) return 0; + if (cont->grp) + return sysfs_create_group(>kobj, cont->grp); + for (i = 0; attrs[i]; i++) { error = class_device_create_file(classdev, attrs[i]); if (error) @@ -378,9 +383,14 @@ attribute_container_remove_attrs(struct class_device *classdev) struct class_device_attribute **attrs = cont->attrs; int i; - if (!attrs) + if (!attrs && !cont->grp) return; + if (cont->grp) { + sysfs_remove_group(>kobj, cont->grp); + return ; + } + for (i = 0; attrs[i]; i++) class_device_remove_file(classdev, attrs[i]); } diff --git a/include/linux/attribute_container.h b/include/linux/attribute_container.h index 8ff2749..f558233 100644 --- a/include/linux/attribute_container.h +++ b/include/linux/attribute_container.h @@ -17,6 +17,7 @@ struct attribute_container { struct list_headnode; struct klistcontainers; struct class*class; + struct attribute_group *grp; struct class_device_attribute **attrs; int (*match)(struct attribute_container *, struct device *); #defineATTRIBUTE_CONTAINER_NO_CLASSDEVS0x01 -- 1.5.3.7 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] attribute_container: update to use the group interface
This patch is the beginning of moving the attribute_containers to use attribute groups exclusively. The attr element is now deprecated and will eventually be removed (along with all the hand rolled code for doing exactly what attribute groups do) when all the consumers are converted to attribute groups. Signed-off-by: James Bottomley [EMAIL PROTECTED] --- Would you believe this is another one I'd like to send via the SCSI tree? James drivers/base/attribute_container.c | 14 -- include/linux/attribute_container.h |1 + 2 files changed, 13 insertions(+), 2 deletions(-) diff --git a/drivers/base/attribute_container.c b/drivers/base/attribute_container.c index 7370d7c..5bf25c6 100644 --- a/drivers/base/attribute_container.c +++ b/drivers/base/attribute_container.c @@ -320,9 +320,14 @@ attribute_container_add_attrs(struct class_device *classdev) struct class_device_attribute **attrs = cont-attrs; int i, error; - if (!attrs) + BUG_ON(attrs cont-grp); + + if (!attrs !cont-grp) return 0; + if (cont-grp) + return sysfs_create_group(classdev-kobj, cont-grp); + for (i = 0; attrs[i]; i++) { error = class_device_create_file(classdev, attrs[i]); if (error) @@ -378,9 +383,14 @@ attribute_container_remove_attrs(struct class_device *classdev) struct class_device_attribute **attrs = cont-attrs; int i; - if (!attrs) + if (!attrs !cont-grp) return; + if (cont-grp) { + sysfs_remove_group(classdev-kobj, cont-grp); + return ; + } + for (i = 0; attrs[i]; i++) class_device_remove_file(classdev, attrs[i]); } diff --git a/include/linux/attribute_container.h b/include/linux/attribute_container.h index 8ff2749..f558233 100644 --- a/include/linux/attribute_container.h +++ b/include/linux/attribute_container.h @@ -17,6 +17,7 @@ struct attribute_container { struct list_headnode; struct klistcontainers; struct class*class; + struct attribute_group *grp; struct class_device_attribute **attrs; int (*match)(struct attribute_container *, struct device *); #defineATTRIBUTE_CONTAINER_NO_CLASSDEVS0x01 -- 1.5.3.7 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/