diff options
author | Arnd Bergmann <arnd@arndb.de> | 2024-05-29 11:51:11 +0200 |
---|---|---|
committer | Tony Luck <tony.luck@intel.com> | 2024-05-29 22:30:10 +0200 |
commit | 123b158635505c89ed0d3ef45c5845ff9030a466 (patch) | |
tree | 55f0ecadf09a02aaffce38a58caf521ec665e1fc /drivers/edac/skx_common.h | |
parent | EDAC/skx: Switch to new Intel CPU model defines (diff) | |
download | linux-123b158635505c89ed0d3ef45c5845ff9030a466.tar.xz linux-123b158635505c89ed0d3ef45c5845ff9030a466.zip |
EDAC, i10nm: make skx_common.o a separate module
Commit 598afa050403 ("kbuild: warn objects shared among multiple modules")
was added to track down cases where the same object is linked into
multiple modules. This can cause serious problems if some modules are
builtin while others are not.
That test triggers this warning:
scripts/Makefile.build:236: drivers/edac/Makefile: skx_common.o is added to multiple modules: i10nm_edac skx_edac
Make this a separate module instead.
[Tony: Added more background details to commit message]
Fixes: d4dc89d069aa ("EDAC, i10nm: Add a driver for Intel 10nm server processors")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Tony Luck <tony.luck@intel.com>
Link: https://lore.kernel.org/all/20240529095132.1929397-1-arnd@kernel.org/
Diffstat (limited to 'drivers/edac/skx_common.h')
-rw-r--r-- | drivers/edac/skx_common.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/edac/skx_common.h b/drivers/edac/skx_common.h index b6d3607dffe2..11faf1db4fa4 100644 --- a/drivers/edac/skx_common.h +++ b/drivers/edac/skx_common.h @@ -231,8 +231,8 @@ typedef int (*get_dimm_config_f)(struct mem_ctl_info *mci, typedef bool (*skx_decode_f)(struct decoded_addr *res); typedef void (*skx_show_retry_log_f)(struct decoded_addr *res, char *msg, int len, bool scrub_err); -int __init skx_adxl_get(void); -void __exit skx_adxl_put(void); +int skx_adxl_get(void); +void skx_adxl_put(void); void skx_set_decode(skx_decode_f decode, skx_show_retry_log_f show_retry_log); void skx_set_mem_cfg(bool mem_cfg_2lm); |