Re: [PATCH v2 0/3] MIPS: CI20: Enable SMP with two CPUs

[Paul Cercueil <paul@crapouillou.net>]

Hi Florian,

Le lundi 09 novembre 2026 à 20:31 -0800, Florian Fainelli a écrit :
> This patch series resurrects SMP support for the CI20 board. This was
> submitted back in 2020 by Zhou Yanjie.
> 
> Changes in v2:
> 
> • Moved SMP sources to arch/mips/ingenic/ and the header to
> arch/mips/include/asm/
> • Removed CONFIG_MIPS_INGENIC_SMP, using CONFIG_MACH_JZ4780 and CONFIG_SMP
> directly
> • Renamed JZ4780-specific SMP symbols to XBurst names
> • Changed CPU enumeration to assign consecutive logical IDs and removed the
> explicit two-core limit
> • Changed mailbox IRQ allocation failure from panic to logging and returning
> • Shortened commit descriptions, removed Codex attribution, and adjusted
> source credits and sign-offs
> • Kept the clock polling patch and CI20 defconfig changes unchanged,
>   added Paul's R-b for patch #1
> • Updated the kernel baseline to include additional upstream fixes

You should also have a changelog per-patch, in each patch's header.

Your patch [1/3] has a Fixes: tag but does not Cc stable@vger.kernel.org, so
you'll have to send a v3.

I reviewed [2/3] and it looks good to me, but I'm a bit concerned about
Nikolaus' reported issue.

Thomas, if you can wait a few days, I can test it on my CI20 and if it works
report back with a Tested-by: tag.

Cheers,
-Paul

> 
> Florian Fainelli (1):
>   clk: ingenic: Poll JZ4780 CPU1 power status atomically
> 
> 周琰杰 (Zhou Yanjie) (2):
>   MIPS: Ingenic: Add XBurst SMP support
>   MIPS: CI20: Enable SMP with two CPUs
> 
>  MAINTAINERS                           |   3 +-
>  arch/mips/Kbuild                      |   1 +
>  arch/mips/configs/ci20_defconfig      |   2 +
>  arch/mips/generic/init.c              |   3 +
>  arch/mips/include/asm/ingenic-smp.h   |  20 +++
>  arch/mips/ingenic/Kconfig             |   2 +
>  arch/mips/ingenic/Makefile            |   5 +
>  arch/mips/ingenic/ingenic-smp-entry.S |  54 ++++++
>  arch/mips/ingenic/ingenic-smp.c       | 247 ++++++++++++++++++++++++++
>  arch/mips/kernel/idle.c               |  26 ++-
>  drivers/clk/ingenic/jz4780-cgu.c      |   2 +-
>  11 files changed, 362 insertions(+), 3 deletions(-)
>  create mode 100644 arch/mips/include/asm/ingenic-smp.h
>  create mode 100644 arch/mips/ingenic/Makefile
>  create mode 100644 arch/mips/ingenic/ingenic-smp-entry.S
>  create mode 100644 arch/mips/ingenic/ingenic-smp.c