"amdgpu/vega20_mec2.bin" did not exist on "fea4c0a37389bf6af54763ca03e8c2dbec979484"
Commit 80bb6afd authored by Juan Castillo's avatar Juan Castillo
Browse files

Fix build error with optimizations disabled (-O0)

If Trusted Firmware is built with optimizations disabled (-O0), the
linker throws the following error:

    undefined reference to 'xxx'

Where 'xxx' is a raw inline function defined in a header file. The
reason is that, with optimizations disabled, GCC may decide to skip
the inlining. If that is the case, an external definition to the
compilation unit must be provided. Because no external definition
is present, the linker throws the error.

This patch fixes the problem by declaring the following inline
functions static, so the internal definition is used:

    inline void soc_css_security_setup(void)
    inline const arm_config_t *get_arm_config(void)

Change-Id: Id650d6be1b1396bdb48af1ac8a4c7900d212e95f
parent 53d069c2
...@@ -55,7 +55,7 @@ typedef struct arm_config { ...@@ -55,7 +55,7 @@ typedef struct arm_config {
/* If used, arm_config must be defined and populated in the platform port */ /* If used, arm_config must be defined and populated in the platform port */
extern arm_config_t arm_config; extern arm_config_t arm_config;
inline const arm_config_t *get_arm_config(void) static inline const arm_config_t *get_arm_config(void)
{ {
return &arm_config; return &arm_config;
} }
......
...@@ -37,7 +37,7 @@ ...@@ -37,7 +37,7 @@
void soc_css_init_nic400(void); void soc_css_init_nic400(void);
void soc_css_init_pcie(void); void soc_css_init_pcie(void);
inline void soc_css_security_setup(void) static inline void soc_css_security_setup(void)
{ {
soc_css_init_nic400(); soc_css_init_nic400();
soc_css_init_pcie(); soc_css_init_pcie();
......
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