diff mbox

[v2] pwm: sysfs: Convert to use ATTRIBUTE_GROUPS macro

Message ID 1386152993.22135.0.camel@phoenix
State Accepted
Headers show

Commit Message

Axel Lin Dec. 4, 2013, 10:29 a.m. UTC 
Use new ATTRIBUTE_GROUPS macro to reduce the number of lines of code.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
---
v2: update commit log
 drivers/pwm/sysfs.c | 12 ++----------
 1 file changed, 2 insertions(+), 10 deletions(-)

Comments

Thierry Reding Dec. 4, 2013, 10:36 a.m. UTC | #1 
On Wed, Dec 04, 2013 at 06:29:53PM +0800, Axel Lin wrote:
> Use new ATTRIBUTE_GROUPS macro to reduce the number of lines of code.
> 
> Signed-off-by: Axel Lin <axel.lin@ingics.com>
> ---
> v2: update commit log
>  drivers/pwm/sysfs.c | 12 ++----------
>  1 file changed, 2 insertions(+), 10 deletions(-)

Applied, thanks.

Thierry
diff mbox

Patch

diff --git a/drivers/pwm/sysfs.c b/drivers/pwm/sysfs.c
index 8c20332..4bd0c63 100644
--- a/drivers/pwm/sysfs.c
+++ b/drivers/pwm/sysfs.c
@@ -169,15 +169,7 @@  static struct attribute *pwm_attrs[] = {
 	&dev_attr_polarity.attr,
 	NULL
 };
-
-static const struct attribute_group pwm_attr_group = {
-	.attrs		= pwm_attrs,
-};
-
-static const struct attribute_group *pwm_attr_groups[] = {
-	&pwm_attr_group,
-	NULL,
-};
+ATTRIBUTE_GROUPS(pwm);
 
 static void pwm_export_release(struct device *child)
 {
@@ -205,7 +197,7 @@  static int pwm_export_child(struct device *parent, struct pwm_device *pwm)
 	export->child.release = pwm_export_release;
 	export->child.parent = parent;
 	export->child.devt = MKDEV(0, 0);
-	export->child.groups = pwm_attr_groups;
+	export->child.groups = pwm_groups;
 	dev_set_name(&export->child, "pwm%u", pwm->hwpwm);
 
 	ret = device_register(&export->child);