Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unifying interface with rp2pio.StateMachine #161

Open
MiLaak opened this issue Feb 26, 2025 · 1 comment
Open

Unifying interface with rp2pio.StateMachine #161

MiLaak opened this issue Feb 26, 2025 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@MiLaak
Copy link

MiLaak commented Feb 26, 2025

Some of the pio_kwargs provided by adafruit_pioasm.Program() and used by rp2pio.StateMachine() needs to be renamed for pioemu.emulate().
Could pioemu be updated to use same kwargs?

For example in my test application, following modifications is needed to be done for pioemu.emulate():

program.pio_kwargs['side_set_count'] = program.pio_kwargs['sideset_pin_count']
program.pio_kwargs['wrap_top'] = program.pio_kwargs['wrap']

del program.pio_kwargs['sideset_enable']
del program.pio_kwargs['wrap']
del program.pio_kwargs['sideset_pin_count']
@NathanY3G
Copy link
Owner

Hi @MiLaak 👋

Thanks for the good suggestion - I can certainly see the advantage of aligning the names with CircuitPython (or MicroPython). Please note however, that some changes may still be required even if the names are aligned. For example, CircuitPython's wait_for_txstall is unlikely to be applicable so it would still need to be removed/commented-out if used. Would that match your expectations?

NB: If anybody else is reading this and has an opinion then please feel free to share it by adding a comment.

@NathanY3G NathanY3G self-assigned this Feb 27, 2025
@NathanY3G NathanY3G added the enhancement New feature or request label Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants