pioasm: please keep the labels #803

Open
opened 2022-05-04 04:15:29 -05:00 by m-u-xyz · 3 comments
m-u-xyz commented 2022-05-04 04:15:29 -05:00 (Migrated from github.com)

pioasm replaces labels with line numbers. This is unfortunate. Please don't do that.

(a) Micropython can deal with textual labels, no problem at all.
(b) For C I'd like to be able to refer to a specific instruction so that I can send a jump command to PIO. Currently I need to look at the generated code and/or count instructions, and woe is me when that location changes.

pioasm replaces labels with line numbers. This is unfortunate. Please don't do that. (a) Micropython can deal with textual labels, no problem at all. (b) For C I'd like to be able to refer to a specific instruction so that I can send a jump command to PIO. Currently I need to look at the generated code and/or count instructions, and woe is me when that location changes.
lurch commented 2022-05-04 06:15:23 -05:00 (Migrated from github.com)

Section 3.3.5 of https://datasheets.raspberrypi.com/rp2040/rp2040-datasheet.pdf says "A PUBLIC label is exposed to the user code in the same way as a PUBLIC .define."
Is that what you're looking for, or are you requesting something different?

Section 3.3.5 of https://datasheets.raspberrypi.com/rp2040/rp2040-datasheet.pdf says "A PUBLIC label is exposed to the user code in the same way as a PUBLIC .define." Is that what you're looking for, or are you requesting something different?
kilograham commented 2022-07-11 08:58:43 -05:00 (Migrated from github.com)

This is quite a change to fix... is the PUBLIC not a workaround?

This is quite a change to fix... is the PUBLIC not a workaround?
magy00 commented 2025-03-11 21:08:00 -05:00 (Migrated from github.com)

For (a): Maybe just add an option to pioasm to dump all labels/defines and their values (either as text, code, or as comments included in the generated file to stdout, or as additional debug info to stderr).

For (a): Maybe just add an option to `pioasm` to dump all labels/defines and their values (either as text, code, or as comments included in the generated file to stdout, or as additional debug info to stderr).
Sign in to join this conversation.
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
gh-raspberrypi/pico-sdk#803
No description provided.