Patch 1/3 was already accepted, but it seems is not in master yet.

Comments addressed in patches 2 and 3.
Let me know if you'd like me to split out a separate commit
for renaming arm_v7m_get_sp_ptr.

David Reiss (3):
  target/arm: Unify checking for M Main Extension in MRS/MSR
  target/arm/gdbstub: Support reading M system registers from GDB
  target/arm/gdbstub: Support reading M security extension registers
    from GDB

 target/arm/cpu.h      |  25 ++++-
 target/arm/gdbstub.c  | 241 ++++++++++++++++++++++++++++++++++++++++++
 target/arm/m_helper.c |  51 +++++----
 3 files changed, 296 insertions(+), 21 deletions(-)

-- 
2.34.5

On Tue, 17 Jan 2023 at 19:44, ~dreiss-meta <dreiss-meta@git.sr.ht> wrote:
>
> Patch 1/3 was already accepted, but it seems is not in master yet.
>
> Comments addressed in patches 2 and 3.
> Let me know if you'd like me to split out a separate commit
> for renaming arm_v7m_get_sp_ptr.

Hi; these patches look good to me. If you could respin
to fix the thing RTH pointed out about using NULL instead
of "" for "not present" then I'll take them.

thanks
-- PMM