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

add ULP support with uPython assembler #214

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ubIQio
Copy link

@ubIQio ubIQio commented Sep 13, 2018

Adds support for loading and running ULP code (taken from Andreas Valder's PR to Damien George's MicroPython) together with Thomas Waldmann's micropython ULP assembler.

ulp_count.py included as simple usage example
pulse_counter.py shows use of GPIO by ULP to debounce a noisy pulse stream

@mattytrentini
Copy link

Running a ULP blob should be integrated; it's necessary to be included in the firmware. But does it make sense to integrate Thomas Waldmann's assembler? It can just be deployed to a device and run.

Allowing it to be deployed to a device using upip would be very helpful though!

@ubIQio
Copy link
Author

ubIQio commented Sep 17, 2018

Not sure, but it certainly lowers the bar for developing code for the ULP, and ULP support is a essential feature not a nice to have, esp. if you want a battery powered device.

The code/debug/test cycle is much faster with the python assembler, as theres just one file to copy to the target, which given the lack of visibility of the ULP code, really helps development (esp. as the ULP binary is functionally very closely coupled to the python code that launches and talks to it)

Until/unless someone integrates the esp-idf ULP compiler into the build scripts, Thomas Waldmann's assembler seems like a great approach. Even then, the one file binary+python ULP update makes it preferable IMO.

@loboris
Copy link
Owner

loboris commented Dec 17, 2018

Thank you for the PR.

I'll work on ULP support next month, then I'll probably merge this PR or parts of it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants