On Tue, 17 Jan 2023 at 11:35, Peter Maydell <peter.mayd...@linaro.org> wrote: > > On Mon, 9 Jan 2023 at 23:18, ~dreiss-meta <dreiss-m...@git.sr.ht> wrote: > > > > From: David Reiss <dre...@meta.com> > > > > BASEPRI, FAULTMASK, and their _NS equivalents only exist on devices with > > the Main Extension. However, the MRS instruction did not check this, > > and the MSR instruction handled it inconsistently (warning BASEPRI, but > > silently ignoring writes to BASEPRI_NS). Unify this behavior and always > > warn when reading or writing any of these registers if the extension is > > not present. > > > > Signed-off-by: David Reiss <dre...@meta.com> > > --- > > Hi; it looks like you didn't send a cover letter for this patchset. > If you're making contributions in future, if you could send a cover > letter for a multi-patch submission, that's helpful because our > automated tooling tends to get confused by patchsets which don't > have one. (A single standalone patch doesn't need a cover letter.) > > Reviewed-by: Peter Maydell <peter.mayd...@linaro.org>
Since this patch is OK and not really related to 2 and 3, I'm going to apply this to target-arm.next. I've left review comments on the other two patches. thanks -- PMM
