diff mbox series

imx: common: Move prototype outside the SMP block

Message ID 20210617145415.37440-1-festevam@denx.de (mailing list archive)
State New, archived
Headers show
Series imx: common: Move prototype outside the SMP block | expand

Commit Message

Fabio Estevam June 17, 2021, 2:54 p.m. UTC
Currently the imx_gpcv2_set_core1_pdn_pup_by_software() prototype is
guarded by the CONFIG_SMP symbol. This causes W=1 build warnings when
CONFIG_SMP is not selected:

arch/arm/mach-imx/src.c:103:6: warning: no previous prototype for 'imx_gpcv2_set_core1_pdn_pup_by_software' [-Wmissing-prototypes]

Fix it by moving the imx_gpcv2_set_core1_pdn_pup_by_software() prototype
outside of the CONFIG_SMP block.

Fixes: e34645f45805 ("ARM: imx: add smp support for imx7d")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm/mach-imx/common.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Shawn Guo July 14, 2021, 7:30 a.m. UTC | #1
On Thu, Jun 17, 2021 at 11:54:15AM -0300, Fabio Estevam wrote:
> Currently the imx_gpcv2_set_core1_pdn_pup_by_software() prototype is
> guarded by the CONFIG_SMP symbol. This causes W=1 build warnings when
> CONFIG_SMP is not selected:
> 
> arch/arm/mach-imx/src.c:103:6: warning: no previous prototype for 'imx_gpcv2_set_core1_pdn_pup_by_software' [-Wmissing-prototypes]
> 
> Fix it by moving the imx_gpcv2_set_core1_pdn_pup_by_software() prototype
> outside of the CONFIG_SMP block.
> 
> Fixes: e34645f45805 ("ARM: imx: add smp support for imx7d")
> Reported-by: kernel test robot <lkp@intel.com>
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Updated the subject prefix like 'ARM: imx: ...', and applied.

Shawn
diff mbox series

Patch

diff --git a/arch/arm/mach-imx/common.h b/arch/arm/mach-imx/common.h
index f0a073a71401..13f3068e9845 100644
--- a/arch/arm/mach-imx/common.h
+++ b/arch/arm/mach-imx/common.h
@@ -68,7 +68,6 @@  void imx_set_cpu_arg(int cpu, u32 arg);
 void v7_secondary_startup(void);
 void imx_scu_map_io(void);
 void imx_smp_prepare(void);
-void imx_gpcv2_set_core1_pdn_pup_by_software(bool pdn);
 #else
 static inline void imx_scu_map_io(void) {}
 static inline void imx_smp_prepare(void) {}
@@ -81,6 +80,7 @@  void imx_gpc_mask_all(void);
 void imx_gpc_restore_all(void);
 void imx_gpc_hwirq_mask(unsigned int hwirq);
 void imx_gpc_hwirq_unmask(unsigned int hwirq);
+void imx_gpcv2_set_core1_pdn_pup_by_software(bool pdn);
 void imx_anatop_init(void);
 void imx_anatop_pre_suspend(void);
 void imx_anatop_post_resume(void);