Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1703116 > unrolled thread

[PATCH 0/3] constify firmware attribute_group structures.

Back to article view | Back to linux.kernel

Contents

  [PATCH 0/3] constify firmware attribute_group structures. Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-08-03 15:40 +0200
    [PATCH 2/3] firmware: efi: constify attribute_group structures. Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-08-03 15:40 +0200
    [PATCH 1/3] firmware: dcdbas: constify attribute_group structures. Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-08-03 15:40 +0200
    [PATCH 3/3] firmware: efi: esrt: constify attribute_group structures. Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-08-03 15:40 +0200

#1703116 — [PATCH 0/3] constify firmware attribute_group structures.

attribute_group are not supposed to change at runtime. All functions
working with attribute_group provided by <linux/sysfs.h> work with
const attribute_group. So mark the non-const structs as const.

Arvind Yadav (3):
  [PATCH 1/3] firmware: dcdbas: constify attribute_group structures.
  [PATCH 2/3] firmware: efi: constify attribute_group structures.
  [PATCH 3/3] firmware: efi: esrt: constify attribute_group structures.

 drivers/firmware/dcdbas.c   | 2 +-
 drivers/firmware/efi/efi.c  | 2 +-
 drivers/firmware/efi/esrt.c | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

-- 
1.9.1

[toc] | [next] | [standalone]

#1703117 — [PATCH 2/3] firmware: efi: constify attribute_group structures.

attribute_group are not supposed to change at runtime. All functions
working with attribute_group provided by <linux/sysfs.h> work with
const attribute_group. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/firmware/efi/efi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/firmware/efi/efi.c b/drivers/firmware/efi/efi.c
index 045d6d3..6519be44 100644
--- a/drivers/firmware/efi/efi.c
+++ b/drivers/firmware/efi/efi.c
@@ -179,7 +179,7 @@ static umode_t efi_attr_is_visible(struct kobject *kobj,
 	return attr->mode;
 }
 
-static struct attribute_group efi_subsys_attr_group = {
+static const struct attribute_group efi_subsys_attr_group = {
 	.attrs = efi_subsys_attrs,
 	.is_visible = efi_attr_is_visible,
 };
-- 
1.9.1

[toc] | [prev] | [next] | [standalone]

#1703124 — [PATCH 1/3] firmware: dcdbas: constify attribute_group structures.

attribute_group are not supposed to change at runtime. All functions
working with attribute_group provided by <linux/sysfs.h> work with
const attribute_group. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/firmware/dcdbas.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/firmware/dcdbas.c b/drivers/firmware/dcdbas.c
index 2fe1a13..c16600f 100644
--- a/drivers/firmware/dcdbas.c
+++ b/drivers/firmware/dcdbas.c
@@ -534,7 +534,7 @@ static int dcdbas_reboot_notify(struct notifier_block *nb, unsigned long code,
 	NULL
 };
 
-static struct attribute_group dcdbas_attr_group = {
+static const struct attribute_group dcdbas_attr_group = {
 	.attrs = dcdbas_dev_attrs,
 	.bin_attrs = dcdbas_bin_attrs,
 };
-- 
1.9.1

[toc] | [prev] | [next] | [standalone]

#1703127 — [PATCH 3/3] firmware: efi: esrt: constify attribute_group structures.

attribute_group are not supposed to change at runtime. All functions
working with attribute_group provided by <linux/sysfs.h> work with
const attribute_group. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/firmware/efi/esrt.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/firmware/efi/esrt.c b/drivers/firmware/efi/esrt.c
index 8554d7a..bd7ed3c 100644
--- a/drivers/firmware/efi/esrt.c
+++ b/drivers/firmware/efi/esrt.c
@@ -230,7 +230,7 @@ static umode_t esrt_attr_is_visible(struct kobject *kobj,
 	return attr->mode;
 }
 
-static struct attribute_group esrt_attr_group = {
+static const struct attribute_group esrt_attr_group = {
 	.attrs = esrt_attrs,
 	.is_visible = esrt_attr_is_visible,
 };
-- 
1.9.1

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web