On Fri, Dec 20, 2019 at 03:36:47PM +0000, Marc Zyngier wrote:
> On 2019-12-20 15:05, Mark Rutland wrote:
> > +static inline unsigned long host_spsr_to_spsr32(unsigned long spsr)
> > +{
> > + const unsigned long overlap = BIT(24) | BIT(21);
> > + unsigned long dit = !!(spsr & PSR_AA32_DIT_BIT);
> > +
> > + spsr &= overlap;
>
> Are you sure this is what you want to do? This doesn't leave
> that many bits set in SPSR... :-/
Oh, whoops. :(
> Apart from what I believe is a missing ~ above, this looks good.
I've added the missing '~' and pushed out the updated branch.
Thanks,
Mark.
