Messages in this thread | | | Date | Thu, 1 Mar 2018 17:23:30 +0800 | From | kbuild test robot <> | Subject | Re: [PATCH v2] drivers/virt: vm_gen_counter: initial driver implementation |
| |
Hi Or,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on char-misc/char-misc-testing] [also build test ERROR on v4.16-rc3 next-20180301] [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/Or-Idgar/drivers-virt-vm_gen_counter-initial-driver-implementation/20180227-050111 config: openrisc-allyesconfig (attached as .config) compiler: or1k-linux-gcc (GCC) 6.0.0 20160327 (experimental) reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree make.cross ARCH=openrisc
All errors (new ones prefixed by >>):
drivers/misc/vmgenid.c: In function 'acpi_vmgenid_add': drivers/misc/vmgenid.c:102:29: error: dereferencing pointer to incomplete type 'struct acpi_device' retval = get_vmgenid(device->handle); ^~ drivers/misc/vmgenid.c: At top level: drivers/misc/vmgenid.c:114:36: error: array type has incomplete element type 'struct acpi_device_id' static const struct acpi_device_id vmgenid_ids[] = { ^~~~~~~~~~~ drivers/misc/vmgenid.c:119:15: error: variable 'acpi_vmgenid_driver' has initializer but incomplete type static struct acpi_driver acpi_vmgenid_driver = { ^~~~~~~~~~~ >> drivers/misc/vmgenid.c:120:2: error: unknown field 'name' specified in initializer .name = "vm_gen_counter", ^ drivers/misc/vmgenid.c:120:10: warning: excess elements in struct initializer .name = "vm_gen_counter", ^~~~~~~~~~~~~~~~ drivers/misc/vmgenid.c:120:10: note: (near initialization for 'acpi_vmgenid_driver') >> drivers/misc/vmgenid.c:121:2: error: unknown field 'ids' specified in initializer .ids = vmgenid_ids, ^ drivers/misc/vmgenid.c:121:9: warning: excess elements in struct initializer .ids = vmgenid_ids, ^~~~~~~~~~~ drivers/misc/vmgenid.c:121:9: note: (near initialization for 'acpi_vmgenid_driver') >> drivers/misc/vmgenid.c:122:2: error: unknown field 'owner' specified in initializer .owner = THIS_MODULE, ^ In file included from include/linux/linkage.h:7:0, from include/linux/kernel.h:7, from include/linux/list.h:9, from include/linux/module.h:9, from drivers/misc/vmgenid.c:12: include/linux/export.h:37:21: warning: excess elements in struct initializer #define THIS_MODULE ((struct module *)0) ^ drivers/misc/vmgenid.c:122:11: note: in expansion of macro 'THIS_MODULE' .owner = THIS_MODULE, ^~~~~~~~~~~ include/linux/export.h:37:21: note: (near initialization for 'acpi_vmgenid_driver') #define THIS_MODULE ((struct module *)0) ^ drivers/misc/vmgenid.c:122:11: note: in expansion of macro 'THIS_MODULE' .owner = THIS_MODULE, ^~~~~~~~~~~ >> drivers/misc/vmgenid.c:123:2: error: unknown field 'ops' specified in initializer .ops = { ^ drivers/misc/vmgenid.c:123:9: error: extra brace group at end of initializer .ops = { ^ drivers/misc/vmgenid.c:123:9: note: (near initialization for 'acpi_vmgenid_driver') drivers/misc/vmgenid.c:123:9: warning: excess elements in struct initializer drivers/misc/vmgenid.c:123:9: note: (near initialization for 'acpi_vmgenid_driver') drivers/misc/vmgenid.c: In function 'vmgenid_init': >> drivers/misc/vmgenid.c:131:9: error: implicit declaration of function 'acpi_bus_register_driver' [-Werror=implicit-function-declaration] return acpi_bus_register_driver(&acpi_vmgenid_driver); ^~~~~~~~~~~~~~~~~~~~~~~~ drivers/misc/vmgenid.c: In function 'vmgenid_exit': >> drivers/misc/vmgenid.c:136:2: error: implicit declaration of function 'acpi_bus_unregister_driver' [-Werror=implicit-function-declaration] acpi_bus_unregister_driver(&acpi_vmgenid_driver); ^~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/misc/vmgenid.c: At top level: drivers/misc/vmgenid.c:119:27: error: storage size of 'acpi_vmgenid_driver' isn't known static struct acpi_driver acpi_vmgenid_driver = { ^~~~~~~~~~~~~~~~~~~ drivers/misc/vmgenid.c:114:36: warning: 'vmgenid_ids' defined but not used [-Wunused-variable] static const struct acpi_device_id vmgenid_ids[] = { ^~~~~~~~~~~ cc1: some warnings being treated as errors
vim +/name +120 drivers/misc/vmgenid.c
113 > 114 static const struct acpi_device_id vmgenid_ids[] = { 115 {"QEMUVGID", 0}, 116 {"", 0}, 117 }; 118 119 static struct acpi_driver acpi_vmgenid_driver = { > 120 .name = "vm_gen_counter", > 121 .ids = vmgenid_ids, > 122 .owner = THIS_MODULE, > 123 .ops = { 124 .add = acpi_vmgenid_add, 125 .remove = acpi_vmgenid_remove, 126 } 127 }; 128 129 static int __init vmgenid_init(void) 130 { > 131 return acpi_bus_register_driver(&acpi_vmgenid_driver); 132 } 133 134 static void __exit vmgenid_exit(void) 135 { > 136 acpi_bus_unregister_driver(&acpi_vmgenid_driver); 137 } 138
--- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation [unhandled content-type:application/gzip] | |