Hi Jiaxun, On 2026-10-29T05:07:18+11:00, Yao Zi <[email protected]> wrote: > LoongArch: Exception handling > > Add exception entry assembly code, import stackframe.h > from Linux, provide debug prints when exception happens. > > Signed-off-by: Jiaxun Yang <[email protected]> > Signed-off-by: Yao Zi <[email protected]> > > arch/loongarch/Kconfig | 19 ++++ > arch/loongarch/cpu/Makefile | 2 +- > arch/loongarch/cpu/genex.S | 21 ++++ > arch/loongarch/include/asm/stackframe.h | 175 ++++++++++++++++++++++++++++++ > arch/loongarch/lib/interrupts.c | 185 > ++++++++++++++++++++++++++++++++ > 5 files changed, 401 insertions(+), 1 deletion(-)
> diff --git a/arch/loongarch/lib/interrupts.c b/arch/loongarch/lib/interrupts.c > @@ -0,0 +1,185 @@ > +#if IS_ENABLED(CONFIG_SHOW_REGS) > + show_regs(regs); > +#endif show_regs() is already __maybe_unused, so please can you use a plain if (IS_ENABLED(CONFIG_SHOW_REGS)) here, as you do for FRAMEPOINTER just below? LoongArch has no SPL support yet, so it is fine to leave out SPL_SHOW_REGS. In any case: Reviewed-by: Simon Glass <[email protected]> Regards, Simon
