Commit 376539fa authored by Behan Webster's avatar Behan Webster Committed by Jiri Slaby

x86: LLVMLinux: Fix "incomplete type const struct x86cpu_device_id"

commit c4586256 upstream.

Similar to the fix in 40413dcb

MODULE_DEVICE_TABLE(x86cpu, ...) expects the struct to be called struct
x86cpu_device_id, and not struct x86_cpu_id which is what is used in the rest
of the kernel code.  Although gcc seems to ignore this error, clang fails
without this define to fix the name.

Code from drivers/thermal/x86_pkg_temp_thermal.c
static const struct x86_cpu_id __initconst pkg_temp_thermal_ids[] = { ... };
MODULE_DEVICE_TABLE(x86cpu, pkg_temp_thermal_ids);

Error from clang:
drivers/thermal/x86_pkg_temp_thermal.c:577:1: error: variable has
      incomplete type 'const struct x86cpu_device_id'
MODULE_DEVICE_TABLE(x86cpu, pkg_temp_thermal_ids);
^
include/linux/module.h:145:3: note: expanded from macro
      'MODULE_DEVICE_TABLE'
  MODULE_GENERIC_TABLE(type##_device, name)
  ^
include/linux/module.h:87:32: note: expanded from macro
      'MODULE_GENERIC_TABLE'
extern const struct gtype##_id __mod_##gtype##_table            \
                               ^
<scratch space>:143:1: note: expanded from here
__mod_x86cpu_device_table
^
drivers/thermal/x86_pkg_temp_thermal.c:577:1: note: forward declaration of
      'struct x86cpu_device_id'
include/linux/module.h:145:3: note: expanded from macro
      'MODULE_DEVICE_TABLE'
  MODULE_GENERIC_TABLE(type##_device, name)
  ^
include/linux/module.h:87:21: note: expanded from macro
      'MODULE_GENERIC_TABLE'
extern const struct gtype##_id __mod_##gtype##_table            \
                    ^
<scratch space>:141:1: note: expanded from here
x86cpu_device_id
^
1 error generated.
Signed-off-by: default avatarBehan Webster <behanw@converseincode.com>
Signed-off-by: default avatarJan-Simon Möller <dl9pf@gmx.de>
Acked-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
[added vmbus, mei, and rapdio #defines, needed for 3.14 - gregkh]
Signed-off-by: default avatarJiri Slaby <jslaby@suse.cz>
parent 0a0ff4eb
...@@ -398,6 +398,7 @@ struct virtio_device_id { ...@@ -398,6 +398,7 @@ struct virtio_device_id {
/* /*
* For Hyper-V devices we use the device guid as the id. * For Hyper-V devices we use the device guid as the id.
*/ */
#define vmbus_device_id hv_vmbus_device_id
struct hv_vmbus_device_id { struct hv_vmbus_device_id {
__u8 guid[16]; __u8 guid[16];
kernel_ulong_t driver_data; /* Data private to the driver */ kernel_ulong_t driver_data; /* Data private to the driver */
...@@ -548,6 +549,11 @@ struct amba_id { ...@@ -548,6 +549,11 @@ struct amba_id {
* See documentation of "x86_match_cpu" for details. * See documentation of "x86_match_cpu" for details.
*/ */
/*
* MODULE_DEVICE_TABLE expects this struct to be called x86cpu_device_id.
* Although gcc seems to ignore this error, clang fails without this define.
*/
#define x86cpu_device_id x86_cpu_id
struct x86_cpu_id { struct x86_cpu_id {
__u16 vendor; __u16 vendor;
__u16 family; __u16 family;
...@@ -575,6 +581,7 @@ struct ipack_device_id { ...@@ -575,6 +581,7 @@ struct ipack_device_id {
#define MEI_CL_MODULE_PREFIX "mei:" #define MEI_CL_MODULE_PREFIX "mei:"
#define MEI_CL_NAME_SIZE 32 #define MEI_CL_NAME_SIZE 32
#define mei_device_id mei_cl_device_id
struct mei_cl_device_id { struct mei_cl_device_id {
char name[MEI_CL_NAME_SIZE]; char name[MEI_CL_NAME_SIZE];
kernel_ulong_t driver_info; kernel_ulong_t driver_info;
...@@ -594,6 +601,7 @@ struct mei_cl_device_id { ...@@ -594,6 +601,7 @@ struct mei_cl_device_id {
* Identifies a RapidIO device based on both the device/vendor IDs and * Identifies a RapidIO device based on both the device/vendor IDs and
* the assembly device/vendor IDs. * the assembly device/vendor IDs.
*/ */
#define rapidio_device_id rio_device_id
struct rio_device_id { struct rio_device_id {
__u16 did, vid; __u16 did, vid;
__u16 asm_did, asm_vid; __u16 asm_did, asm_vid;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment