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

Icesugar-nano flash #66

Merged
merged 14 commits into from
Jun 16, 2023
Merged

Conversation

abderraouf-adjal
Copy link
Contributor

Fix issues in scripting, and add the support of running from flash for the dev board icesugar-nano.

details to run hello.c:

% make ICESUGAR_NANO
% cd FemtoRV/FIRMWARE/EXAMPLES
% make hello.prog

@abderraouf-adjal
Copy link
Contributor Author

@BrunoLevy Hello, this chain of improvements are working fine in my end for icesugar-nano (ice40lp1k). I don't know if anyone has the same board to test it too. The CPU is running from the SPI flash very well.

Maybe @wuxx the maker of that board can help testing?

@Chandler-Kluser
Copy link

@abderraouf-adjal, when I cloned your fork these files didn't come along with I don't know why.

But I have tested in my icesugar-nano board. The linebreaks didn't work as expected, but it works!!

uart-hello-icesugar-nano

@abderraouf-adjal
Copy link
Contributor Author

@Chandler-Kluser I didn't understand if you are saying the patch doesn't work or no.
If the only issue is the new line breaks maybe it's related to the UART application, try something else like the one from Arduino IDE or gtkterm, you can easily choose the new line char mode (unix, ms-windows, mac)

@Chandler-Kluser
Copy link

It does work!!

about the UART: tried putty and screen, I had the same outputs

@abderraouf-adjal
Copy link
Contributor Author

@Chandler-Kluser Use "gtkterm", select the right port, then from menu: "Configuration -> CR LF Auto". You have to tell the serial apps what line ending is being used.

@Chandler-Kluser
Copy link

In PuTTY you can solve this issue with Implicit CF on every LF in option:

https://documentation.help/PuTTY/config-crlf.html

@BrunoLevy BrunoLevy merged commit afda54b into BrunoLevy:master Jun 16, 2023
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.

None yet

3 participants