On Thu Oct 29, 2026 at 2:34 PM +08, Troy Mitchell wrote: > On Thu Oct 29, 2026 at 6:35 AM +08, E Shattow wrote: >> Hi Anirudh, CC: Troy Mitchell, Guodong Xu I realized after sending that you CC'd the wrong Guodong email. I've corrected it here.
>> >> 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 <[email protected]> >>>> Signed-off-by: E Shattow <[email protected]> >>>> --- >>>> 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 <[email protected]> >>>> */ >>>> >>>> +/* 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/[email protected]/ >> >> ...then I think we could send this upstream? > Acked-by: Troy Mitchell <[email protected]> > >> >> Thanks for the review, -E -- Troy Mitchell
signature.asc
Description: PGP signature
