From: Segher Boessenkool <segher@kernel•crashing.org>
To: Paul Mackerras <paulus@ozlabs•org>
Cc: linuxppc-dev@ozlabs•org
Subject: Re: [PATCH v2 2/9] powerpc: Add Microwatt device tree
Date: Mon, 21 Jun 2021 08:54:07 -0500 [thread overview]
Message-ID: <20210621135407.GB5077@gate.crashing.org> (raw)
In-Reply-To: <YM8v2ricaCzGi2vv@thinks.paulus.ozlabs.org>
On Sun, Jun 20, 2021 at 10:08:58PM +1000, Paul Mackerras wrote:
> On Sat, Jun 19, 2021 at 09:26:16AM -0500, Segher Boessenkool wrote:
> > On Fri, Jun 18, 2021 at 01:44:16PM +1000, Paul Mackerras wrote:
> > > Microwatt currently runs with MSR[HV] = 0,
> >
> > That isn't compliant though? If your implementation does not have LPAR
> > it must set MSR[HV]=1 always.
>
> True - but if I actually do that, Linux starts trying to use hrfid
> (for example in masked_Hinterrupt), which Microwatt doesn't have.
> Something for Nick to fix. :)
That looks like it needs fixing, yes (it is hard to actually read). But
one thing you can do to make this Just Work is to make hrfid do exactly
the same as rfid, i.e. decode hrfid (01000 10010) as rfid (00000 10010).
That probably makes things run already, you don't even need to alias
to SPRs HSRRn (01001 1101n) to SRRn (00000 1101n) :-)
Segher
next prev parent reply other threads:[~2021-06-21 13:55 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-18 3:42 [PATCH v2 0/9] powerpc: Add support for Microwatt soft-core Paul Mackerras
2021-06-18 3:43 ` [PATCH v2 1/9] powerpc: Add Microwatt platform Paul Mackerras
2021-06-19 3:03 ` Nicholas Piggin
2021-06-18 3:44 ` [PATCH v2 2/9] powerpc: Add Microwatt device tree Paul Mackerras
2021-06-19 14:26 ` Segher Boessenkool
2021-06-20 12:08 ` Paul Mackerras
2021-06-21 13:54 ` Segher Boessenkool [this message]
2021-06-18 3:45 ` [PATCH v2 3/9] powerpc/microwatt: Populate platform bus from device-tree Paul Mackerras
2021-06-18 3:45 ` [PATCH v2 4/9] powerpc/xics: Add a native ICS backend for microwatt Paul Mackerras
2021-06-18 3:46 ` [PATCH v2 5/9] powerpc/microwatt: Use standard 16550 UART for console Paul Mackerras
2021-06-18 7:40 ` Nicholas Piggin
2021-06-18 12:12 ` Paul Mackerras
2021-06-19 2:58 ` Nicholas Piggin
2021-08-12 13:14 ` Christophe Leroy
2021-08-12 16:09 ` Segher Boessenkool
2021-06-18 3:47 ` [PATCH v2 6/9] powerpc/microwatt: Add support for hardware random number generator Paul Mackerras
2021-06-19 3:08 ` Nicholas Piggin
2021-06-19 14:36 ` Segher Boessenkool
2021-06-20 8:19 ` Nicholas Piggin
2021-06-18 3:48 ` [PATCH v2 7/9] powerpc/microwatt: Add microwatt_defconfig Paul Mackerras
2021-06-18 3:49 ` [PATCH v2 8/9] powerpc/boot: Fixup device-tree on little endian Paul Mackerras
2021-06-19 3:14 ` Nicholas Piggin
2021-06-18 3:49 ` [PATCH v2 9/9] powerpc/boot: Add a boot wrapper for Microwatt Paul Mackerras
2021-06-19 3:16 ` Nicholas Piggin
2021-06-19 14:45 ` [PATCH v2 0/9] powerpc: Add support for Microwatt soft-core Segher Boessenkool
2021-06-24 14:03 ` Michael Ellerman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210621135407.GB5077@gate.crashing.org \
--to=segher@kernel$(echo .)crashing.org \
--cc=linuxppc-dev@ozlabs$(echo .)org \
--cc=paulus@ozlabs$(echo .)org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox