On Thu Oct 29, 2026 at 6:35 AM +08, E Shattow wrote: > Hi Anirudh, CC: Troy Mitchell, Guodong Xu > > On 10/28/26 15:01, Anirudh Srinivasan wrote: >> On Wed, Oct 28, 2026 at 11:45:05AM -0700, E Shattow wrote: >>> Linux boot stalls when probing peripherals unless OpenSBI has the >>> enabled mimsic and maplic nodes. Override these nodes so Linux can boot. >>> >>> Reported-by: Yixun Lan >>> Signed-off-by: E Shattow >>> --- >>> arch/riscv/dts/spacemit-k3-u-boot.dtsi | 4 ++++ >>> 1 file changed, 4 insertions(+) >>> >>> diff --git a/arch/riscv/dts/spacemit-k3-u-boot.dtsi b/arch/riscv/dts/spacemit-k3-u-boot.dtsi >>> index 352e85e7de87..278b36acc79f 100644 >>> --- a/arch/riscv/dts/spacemit-k3-u-boot.dtsi >>> +++ b/arch/riscv/dts/spacemit-k3-u-boot.dtsi >>> @@ -3,4 +3,8 @@ >>> * Copyright (C) 2026 Yixun Lan >>> */ >>> >>> +/* Override reserved status of mimsic and maplic for OpenSBI */ >>> +&mimsic { status = "okay"; }; >>> +&maplic { status = "okay"; }; >> >> This should be fixed in the DT in linux. I guess you could carry this >> patch till it is fixed in linux. I think status = "reserved" in k3.dtsi is correct and should stay. The DT spec defines "reserved" as a device that is operational but should not be used, typically because it is controlled by another software component such as platform firmware. That is exactly the case for the M-level IMSIC and APLIC. >> >>> + >>> #include "spacemit-k3-binman.dtsi" >>> -- >>> 2.53.0 >>> > > The upstream commit that adds these nodes as reserved status is: > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=56f37e391a626 > > Quote, "...Also add M-mode APLIC (maplic) and IMSIC (mimsic) nodes to > represent the hardware topology and ready for potential firmware usage." > > When the mainline OpenSBI series lands adding SpacemiT K3 support: > > https://lore.kernel.org/opensbi/20261028-spacemit-k3-v6-0-01d984f60941@linux.spacemit.com/ > > ...then I think we could send this upstream? Acked-by: Troy Mitchell > > Thanks for the review, -E -- Troy Mitchell