orbtop: Exception number format, 0..N vs -16..N about orbuculum

[zyp]

I'd argue that when only displaying a number, it makes most sense to use the same range as the IPSR register for consistency, i.e. exception numbers with IRQ0 showing as exception 16. Having to add 16 to a negative number to find the exception number seems to me more inconvenient and error prone than having to subtract 16 to find the IRQ number from an exception number.

In longer form, it makes good sense to split the ranges and distinguish between core exceptions and IRQs, or even display both; e.g: Exception 3 (HardFault) and Exception 18 (IRQ 2).

[Novakov]

I would say that depends in what context you are looking, I see arguments for for schemes.

I really like the idea of displaying both - exception number with name and IRQ number. It would be even better to supply IRQ names from file and display them as well but that's just my wishful thinking ;)

[mubes]

Well, it's trivial to add a command line option to emit negative numbers if you want them. I have to say I do like the idea of 'named' exceptions and interrupts. We can obviously do that for the standardised ones, but will need SVDs to do that meaningfully, which is another tedious configuration requirement. It's going to be a low priority thing though, unless someone wants to contribute a patch... Dave

…

On Sat, 2 Jul 2022, 21:34 Maciej T. Nowak, ***@***.***> wrote: I would say that depends in what context you are looking, I see arguments for for schemes. I really like the idea of displaying both - exception number with name and IRQ number. It would be even better to supply IRQ names from file and display them as well but that's just my wishful thinking ;) — Reply to this email directly, view it on GitHub <#69 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABJTBDZKHXGE46BJK3EIZETVSCRWFANCNFSM52PFL2JQ> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[Novakov]

#70 - only standard exception names, no SVD parsing

[mubes]

Hey, waddya know, a patch. Thanks! This is fixed via 3db5900.