Searched hist:cb58c277ff1a35432cd84a6cc9768c60ce4c2cad (Results 1 – 2 of 2) sorted by relevance
/openbmc/linux/drivers/platform/x86/dell/dell-wmi-sysman/ |
H A D | sysman.c | diff cb58c277ff1a35432cd84a6cc9768c60ce4c2cad Wed Jun 09 09:59:52 CDT 2021 Hans de Goede <hdegoede@redhat.com> platform/x86: dell-wmi-sysman/think-lmi: Make fw_attr_class global static
The dell-wmi-sysman and think-lmi kernel modules both have a global struct class *fw_attr_class variable, leading to the following compile errors when both are builtin:
ld: drivers/platform/x86/think-lmi.o:(.bss+0x0): multiple definition of `fw_attr_class'; drivers/platform/x86/dell/dell-wmi-sysman/sysman.o:(.bss+0x0): first defined here
In both cases the variable is only used in the file where it is declared. Make both declarations static to avoid the linker error.
Cc: Mark Pearson <markpearson@lenovo.com> Cc: Dell.Client.Kernel@dell.com Reported-by: Nathan Chancellor <nathan@kernel.org> Signed-off-by: Hans de Goede <hdegoede@redhat.com> Link: https://lore.kernel.org/r/20210609145952.113393-1-hdegoede@redhat.com
|
/openbmc/linux/drivers/platform/x86/ |
H A D | think-lmi.c | diff cb58c277ff1a35432cd84a6cc9768c60ce4c2cad Wed Jun 09 09:59:52 CDT 2021 Hans de Goede <hdegoede@redhat.com> platform/x86: dell-wmi-sysman/think-lmi: Make fw_attr_class global static
The dell-wmi-sysman and think-lmi kernel modules both have a global struct class *fw_attr_class variable, leading to the following compile errors when both are builtin:
ld: drivers/platform/x86/think-lmi.o:(.bss+0x0): multiple definition of `fw_attr_class'; drivers/platform/x86/dell/dell-wmi-sysman/sysman.o:(.bss+0x0): first defined here
In both cases the variable is only used in the file where it is declared. Make both declarations static to avoid the linker error.
Cc: Mark Pearson <markpearson@lenovo.com> Cc: Dell.Client.Kernel@dell.com Reported-by: Nathan Chancellor <nathan@kernel.org> Signed-off-by: Hans de Goede <hdegoede@redhat.com> Link: https://lore.kernel.org/r/20210609145952.113393-1-hdegoede@redhat.com
|