Re: [PATCH] kernel/params.c: remove confusing cast
Rasmus Villemoes writes: > Both sides of the assignment are const char*, so this cast is > unnecessary and confusing. > > Signed-off-by: Rasmus Villemoes Acked-by: Rusty Russell Thanks, Rusty. > kernel/params.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/kernel/params.c b/kernel/params.c > index b6554aa71094..0b433fdb45eb 100644 > --- a/kernel/params.c > +++ b/kernel/params.c > @@ -709,7 +709,7 @@ static __modinit int add_sysfs_param(struct > module_kobject *mk, > mk->mp->attrs[mk->mp->num].mattr.store = param_attr_store; > else > mk->mp->attrs[mk->mp->num].mattr.store = NULL; > - mk->mp->attrs[mk->mp->num].mattr.attr.name = (char *)name; > + mk->mp->attrs[mk->mp->num].mattr.attr.name = name; > mk->mp->attrs[mk->mp->num].mattr.attr.mode = kp->perm; > mk->mp->num++; > > -- > 2.1.3 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] kernel/params.c: remove confusing cast
Both sides of the assignment are const char*, so this cast is unnecessary and confusing. Signed-off-by: Rasmus Villemoes --- kernel/params.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/params.c b/kernel/params.c index b6554aa71094..0b433fdb45eb 100644 --- a/kernel/params.c +++ b/kernel/params.c @@ -709,7 +709,7 @@ static __modinit int add_sysfs_param(struct module_kobject *mk, mk->mp->attrs[mk->mp->num].mattr.store = param_attr_store; else mk->mp->attrs[mk->mp->num].mattr.store = NULL; - mk->mp->attrs[mk->mp->num].mattr.attr.name = (char *)name; + mk->mp->attrs[mk->mp->num].mattr.attr.name = name; mk->mp->attrs[mk->mp->num].mattr.attr.mode = kp->perm; mk->mp->num++; -- 2.1.3 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] kernel/params.c: remove confusing cast
Both sides of the assignment are const char*, so this cast is unnecessary and confusing. Signed-off-by: Rasmus Villemoes--- kernel/params.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/params.c b/kernel/params.c index b6554aa71094..0b433fdb45eb 100644 --- a/kernel/params.c +++ b/kernel/params.c @@ -709,7 +709,7 @@ static __modinit int add_sysfs_param(struct module_kobject *mk, mk->mp->attrs[mk->mp->num].mattr.store = param_attr_store; else mk->mp->attrs[mk->mp->num].mattr.store = NULL; - mk->mp->attrs[mk->mp->num].mattr.attr.name = (char *)name; + mk->mp->attrs[mk->mp->num].mattr.attr.name = name; mk->mp->attrs[mk->mp->num].mattr.attr.mode = kp->perm; mk->mp->num++; -- 2.1.3 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] kernel/params.c: remove confusing cast
Rasmus Villemoeswrites: > Both sides of the assignment are const char*, so this cast is > unnecessary and confusing. > > Signed-off-by: Rasmus Villemoes Acked-by: Rusty Russell Thanks, Rusty. > kernel/params.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/kernel/params.c b/kernel/params.c > index b6554aa71094..0b433fdb45eb 100644 > --- a/kernel/params.c > +++ b/kernel/params.c > @@ -709,7 +709,7 @@ static __modinit int add_sysfs_param(struct > module_kobject *mk, > mk->mp->attrs[mk->mp->num].mattr.store = param_attr_store; > else > mk->mp->attrs[mk->mp->num].mattr.store = NULL; > - mk->mp->attrs[mk->mp->num].mattr.attr.name = (char *)name; > + mk->mp->attrs[mk->mp->num].mattr.attr.name = name; > mk->mp->attrs[mk->mp->num].mattr.attr.mode = kp->perm; > mk->mp->num++; > > -- > 2.1.3 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/