powerpc: mpc83xx: Allow including initreg.h into multiple files
authorJ. Neuschäfer <j.ne@posteo.net>
Fri, 20 Dec 2024 10:37:52 +0000 (11:37 +0100)
committerTom Rini <trini@konsulko.com>
Mon, 30 Dec 2024 21:55:27 +0000 (15:55 -0600)
Globals defined in headers can result in multiple-definition errors
while linking, if they are visible beyond the current translation unit.

This hasn't been a problem for initreg.h so far, but would become a
problem in the next patch, where I use a constant from initreg.h in a
second C file.

Reviewed-by: Sinan Akman <sinan@writeme.com>
Signed-off-by: J. Neuschäfer <j.ne@posteo.net>
arch/powerpc/cpu/mpc83xx/initreg/initreg.h

index 63aa5c9..ea1176e 100644 (file)
@@ -13,7 +13,7 @@
 #define SPCR_TSECBDP_MASK      0x00000C00
 #define SPCR_TSECEP_MASK       0x00000300
 
-       const __be32 spcr_mask =
+       static const __be32 spcr_mask =
 #if defined(CONFIG_SPCR_OPT) && !defined(CONFIG_SPCR_OPT_UNSET)
                SPCR_OPT_MASK |
 #endif
@@ -27,7 +27,7 @@
                SPCR_TSEC2EP_MASK |
 #endif
                0;
-       const __be32 spcr_val =
+       static const __be32 spcr_val =
 #if defined(CONFIG_SPCR_OPT) && !defined(CONFIG_SPCR_OPT_UNSET)
                CONFIG_SPCR_OPT |
 #endif
@@ -42,7 +42,7 @@
 #endif
                0;
 
-       const __be32 lcrr_mask =
+       static const __be32 lcrr_mask =
 #if defined(CONFIG_LCRR_DBYP) && !defined(CONFIG_LCRR_DBYP_UNSET)
                LCRR_DBYP |
 #endif
@@ -60,7 +60,7 @@
 #endif
                0;
 
-       const __be32 lcrr_val =
+       static const __be32 lcrr_val =
 #if defined(CONFIG_LCRR_DBYP) && !defined(CONFIG_LCRR_DBYP_UNSET)
                CONFIG_LCRR_DBYP |
 #endif