RE: [PATCH 4/4] x86/platform/uv: Constify static attribute_group struct

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> The only use of base_attr_group and hubless_base_attr_group is to pass
> their addresses to sysfs_create_group() and sysfs_remove_group(), both
> which takes pointers to const attribute_group structs. Make them const
> to allow the compiler to put them in read-only memory.
> 
> Signed-off-by: Rikard Falkeborn <rikard.falkeborn@xxxxxxxxx>

Thank you for the patch. 

Reviewed-by: Justin Ernst <justin.ernst@xxxxxxx>

> ---
>  drivers/platform/x86/uv_sysfs.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/platform/x86/uv_sysfs.c b/drivers/platform/x86/uv_sysfs.c
> index 7badcfa3f384..956a354b57c1 100644
> --- a/drivers/platform/x86/uv_sysfs.c
> +++ b/drivers/platform/x86/uv_sysfs.c
> @@ -778,7 +778,7 @@ static struct attribute *base_attrs[] = {
>  	NULL,
>  };
> 
> -static struct attribute_group base_attr_group = {
> +static const struct attribute_group base_attr_group = {
>  	.attrs = base_attrs
>  };
> 
> @@ -823,7 +823,7 @@ static struct attribute *hubless_base_attrs[] = {
>  	NULL,
>  };
> 
> -static struct attribute_group hubless_base_attr_group = {
> +static const struct attribute_group hubless_base_attr_group = {
>  	.attrs = hubless_base_attrs
>  };
> 
> --
> 2.31.1





[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux