On Mon, 29 Jan 2024 at 08:18, Thomas Huth <th...@redhat.com> wrote: > > If CONFIG_ARM_V7M is not set, we don't want to include the full-fledged > helper functions that require additional functions for linking. The > reduced set of the linux-user functions works fine as stubs in this > case, so change the #ifdef statement accordingly. > > Signed-off-by: Thomas Huth <th...@redhat.com> > --- > target/arm/tcg/m_helper.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/target/arm/tcg/m_helper.c b/target/arm/tcg/m_helper.c > index d1f1e02acc..a5a6e96fc3 100644 > --- a/target/arm/tcg/m_helper.c > +++ b/target/arm/tcg/m_helper.c > @@ -22,6 +22,7 @@ > #endif > #if !defined(CONFIG_USER_ONLY) > #include "hw/intc/armv7m_nvic.h" > +#include CONFIG_DEVICES > #endif > > static void v7m_msr_xpsr(CPUARMState *env, uint32_t mask, > @@ -69,7 +70,7 @@ uint32_t arm_v7m_mrs_control(CPUARMState *env, uint32_t > secure) > return value; > } > > -#ifdef CONFIG_USER_ONLY > +#if defined(CONFIG_USER_ONLY) || !defined(CONFIG_ARM_V7M)
This looks a bit odd. If we don't have CONFIG_ARM_V7M why are we compiling this file at all? > > void HELPER(v7m_msr)(CPUARMState *env, uint32_t maskreg, uint32_t val) > { thanks -- PMM