* [PATCH] ACPI: configfs: make config_item_type const
@ 2017-10-12 10:02 Bhumika Goyal
2017-10-12 15:05 ` Bhumika Goyal
2017-10-15 8:08 ` kbuild test robot
0 siblings, 2 replies; 3+ messages in thread
From: Bhumika Goyal @ 2017-10-12 10:02 UTC (permalink / raw)
To: julia.lawall, rjw, lenb, linux-acpi, linux-kernel; +Cc: Bhumika Goyal
This is a followup patch for: https://lkml.org/lkml/2017/10/11/375 and
https://patchwork.kernel.org/patch/9999649/
Make these structures const as they are either passed to the functions
having the argument as const or stored as a reference in the "ci_type"
const field of a config_item structure.
Done using Coccienlle.
Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
drivers/acpi/acpi_configfs.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/acpi/acpi_configfs.c b/drivers/acpi/acpi_configfs.c
index 853bc7f..b588503 100644
--- a/drivers/acpi/acpi_configfs.c
+++ b/drivers/acpi/acpi_configfs.c
@@ -204,7 +204,7 @@ struct configfs_attribute *acpi_table_attrs[] = {
NULL,
};
-static struct config_item_type acpi_table_type = {
+static const struct config_item_type acpi_table_type = {
.ct_owner = THIS_MODULE,
.ct_bin_attrs = acpi_table_bin_attrs,
.ct_attrs = acpi_table_attrs,
@@ -237,12 +237,12 @@ struct configfs_group_operations acpi_table_group_ops = {
.drop_item = acpi_table_drop_item,
};
-static struct config_item_type acpi_tables_type = {
+static const struct config_item_type acpi_tables_type = {
.ct_owner = THIS_MODULE,
.ct_group_ops = &acpi_table_group_ops,
};
-static struct config_item_type acpi_root_group_type = {
+static const struct config_item_type acpi_root_group_type = {
.ct_owner = THIS_MODULE,
};
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ACPI: configfs: make config_item_type const
2017-10-12 10:02 [PATCH] ACPI: configfs: make config_item_type const Bhumika Goyal
@ 2017-10-12 15:05 ` Bhumika Goyal
2017-10-15 8:08 ` kbuild test robot
1 sibling, 0 replies; 3+ messages in thread
From: Bhumika Goyal @ 2017-10-12 15:05 UTC (permalink / raw)
To: Julia Lawall, Rafael J. Wysocki, Len Brown,
ACPI Devel Maling List, Linux Kernel Mailing List
Cc: Bhumika Goyal
On Thu, Oct 12, 2017 at 12:02 PM, Bhumika Goyal <bhumirks@gmail.com> wrote:
> This is a followup patch for: https://lkml.org/lkml/2017/10/11/375 and
> https://patchwork.kernel.org/patch/9999649/
>
> Make these structures const as they are either passed to the functions
> having the argument as const or stored as a reference in the "ci_type"
> const field of a config_item structure.
>
> Done using Coccienlle.
>
Actually, this patch is dependent on the patches in the links
https://lkml.org/lkml/2017/10/11/375 and
https://patchwork.kernel.org/patch/9999649/. Therefore, this patch
won't be correct unless the patches in these links gets applied.
> Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
> ---
> drivers/acpi/acpi_configfs.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/acpi/acpi_configfs.c b/drivers/acpi/acpi_configfs.c
> index 853bc7f..b588503 100644
> --- a/drivers/acpi/acpi_configfs.c
> +++ b/drivers/acpi/acpi_configfs.c
> @@ -204,7 +204,7 @@ struct configfs_attribute *acpi_table_attrs[] = {
> NULL,
> };
>
> -static struct config_item_type acpi_table_type = {
> +static const struct config_item_type acpi_table_type = {
> .ct_owner = THIS_MODULE,
> .ct_bin_attrs = acpi_table_bin_attrs,
> .ct_attrs = acpi_table_attrs,
> @@ -237,12 +237,12 @@ struct configfs_group_operations acpi_table_group_ops = {
> .drop_item = acpi_table_drop_item,
> };
>
> -static struct config_item_type acpi_tables_type = {
> +static const struct config_item_type acpi_tables_type = {
> .ct_owner = THIS_MODULE,
> .ct_group_ops = &acpi_table_group_ops,
> };
>
> -static struct config_item_type acpi_root_group_type = {
> +static const struct config_item_type acpi_root_group_type = {
> .ct_owner = THIS_MODULE,
> };
>
> --
> 1.9.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ACPI: configfs: make config_item_type const
2017-10-12 10:02 [PATCH] ACPI: configfs: make config_item_type const Bhumika Goyal
2017-10-12 15:05 ` Bhumika Goyal
@ 2017-10-15 8:08 ` kbuild test robot
1 sibling, 0 replies; 3+ messages in thread
From: kbuild test robot @ 2017-10-15 8:08 UTC (permalink / raw)
Cc: kbuild-all, julia.lawall, rjw, lenb, linux-acpi, linux-kernel,
Bhumika Goyal
[-- Attachment #1: Type: text/plain, Size: 8798 bytes --]
Hi Bhumika,
[auto build test WARNING on pm/linux-next]
[also build test WARNING on v4.14-rc4 next-20171013]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Bhumika-Goyal/ACPI-configfs-make-config_item_type-const/20171015-153321
base: https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git linux-next
config: x86_64-randconfig-x018-201742 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64
All warnings (new ones prefixed by >>):
drivers/acpi/acpi_configfs.c: In function 'acpi_table_make_item':
>> drivers/acpi/acpi_configfs.c:222:48: warning: passing argument 3 of 'config_item_init_type_name' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
config_item_init_type_name(&table->cfg, name, &acpi_table_type);
^
In file included from drivers/acpi/acpi_configfs.c:15:0:
include/linux/configfs.h:73:13: note: expected 'struct config_item_type *' but argument is of type 'const struct config_item_type *'
extern void config_item_init_type_name(struct config_item *item,
^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/acpi/acpi_configfs.c: At top level:
>> drivers/acpi/acpi_configfs.c:253:15: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
.ci_type = &acpi_root_group_type,
^
drivers/acpi/acpi_configfs.c: In function 'acpi_configfs_init':
>> drivers/acpi/acpi_configfs.c:271:11: warning: passing argument 3 of 'configfs_register_default_group' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
&acpi_tables_type);
^
In file included from drivers/acpi/acpi_configfs.c:15:0:
include/linux/configfs.h:262:1: note: expected 'struct config_item_type *' but argument is of type 'const struct config_item_type *'
configfs_register_default_group(struct config_group *parent_group,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
vim +222 drivers/acpi/acpi_configfs.c
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 212
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 213 static struct config_item *acpi_table_make_item(struct config_group *group,
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 214 const char *name)
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 215 {
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 216 struct acpi_table *table;
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 217
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 218 table = kzalloc(sizeof(*table), GFP_KERNEL);
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 219 if (!table)
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 220 return ERR_PTR(-ENOMEM);
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 221
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 @222 config_item_init_type_name(&table->cfg, name, &acpi_table_type);
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 223 return &table->cfg;
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 224 }
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 225
772bf1e2 drivers/acpi/acpi_configfs.c Jan Kiszka 2017-06-09 226 static void acpi_table_drop_item(struct config_group *group,
772bf1e2 drivers/acpi/acpi_configfs.c Jan Kiszka 2017-06-09 227 struct config_item *cfg)
772bf1e2 drivers/acpi/acpi_configfs.c Jan Kiszka 2017-06-09 228 {
772bf1e2 drivers/acpi/acpi_configfs.c Jan Kiszka 2017-06-09 229 struct acpi_table *table = container_of(cfg, struct acpi_table, cfg);
772bf1e2 drivers/acpi/acpi_configfs.c Jan Kiszka 2017-06-09 230
772bf1e2 drivers/acpi/acpi_configfs.c Jan Kiszka 2017-06-09 231 ACPI_INFO(("Host-directed Dynamic ACPI Table Unload"));
772bf1e2 drivers/acpi/acpi_configfs.c Jan Kiszka 2017-06-09 232 acpi_tb_unload_table(table->index);
772bf1e2 drivers/acpi/acpi_configfs.c Jan Kiszka 2017-06-09 233 }
772bf1e2 drivers/acpi/acpi_configfs.c Jan Kiszka 2017-06-09 234
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 235 struct configfs_group_operations acpi_table_group_ops = {
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 236 .make_item = acpi_table_make_item,
772bf1e2 drivers/acpi/acpi_configfs.c Jan Kiszka 2017-06-09 237 .drop_item = acpi_table_drop_item,
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 238 };
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 239
39f59af3 drivers/acpi/acpi_configfs.c Bhumika Goyal 2017-10-12 240 static const struct config_item_type acpi_tables_type = {
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 241 .ct_owner = THIS_MODULE,
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 242 .ct_group_ops = &acpi_table_group_ops,
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 243 };
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 244
39f59af3 drivers/acpi/acpi_configfs.c Bhumika Goyal 2017-10-12 245 static const struct config_item_type acpi_root_group_type = {
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 246 .ct_owner = THIS_MODULE,
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 247 };
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 248
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 249 static struct configfs_subsystem acpi_configfs = {
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 250 .su_group = {
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 251 .cg_item = {
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 252 .ci_namebuf = "acpi",
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 @253 .ci_type = &acpi_root_group_type,
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 254 },
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 255 },
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 256 .su_mutex = __MUTEX_INITIALIZER(acpi_configfs.su_mutex),
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 257 };
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 258
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 259 static int __init acpi_configfs_init(void)
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 260 {
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 261 int ret;
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 262 struct config_group *root = &acpi_configfs.su_group;
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 263
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 264 config_group_init(root);
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 265
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 266 ret = configfs_register_subsystem(&acpi_configfs);
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 267 if (ret)
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 268 return ret;
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 269
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 270 acpi_table_group = configfs_register_default_group(root, "table",
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 @271 &acpi_tables_type);
612bd01f drivers/acpi/configfs.c Octavian Purdila 2016-07-08 272 return PTR_ERR_OR_ZERO(acpi_table_group);
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 273 }
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 274 module_init(acpi_configfs_init);
0bf54fcd drivers/acpi/configfs.c Octavian Purdila 2016-07-08 275
:::::: The code at line 222 was first introduced by commit
:::::: 612bd01fc6e04c3ce9eb59587b4a7e4ebd6aff35 ACPI: add support for loading SSDTs via configfs
:::::: TO: Octavian Purdila <octavian.purdila@intel.com>
:::::: CC: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 28026 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-10-15 8:08 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-10-12 10:02 [PATCH] ACPI: configfs: make config_item_type const Bhumika Goyal
2017-10-12 15:05 ` Bhumika Goyal
2017-10-15 8:08 ` kbuild test robot
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox