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

ULX3S: Invalid pin ID #249

Closed
mole99 opened this issue Aug 23, 2022 · 8 comments
Closed

ULX3S: Invalid pin ID #249

mole99 opened this issue Aug 23, 2022 · 8 comments

Comments

@mole99
Copy link

mole99 commented Aug 23, 2022

Previously I could upload to the ULX3S without problems:

> openFPGALoader --board=ulx3s ulx3s.bit
Jtag probe limited to 3MHz
Jtag frequency : requested 6000000Hz -> real 3000000Hz
ret 0
Open file: DONE
Parse file: DONE
Enable configuration: DONE
SRAM erase: DONE
Loading: [==================================================] 100.00%
Done
Disable configuration: DONE

But after updating to the latest release, I get:

> openFPGALoader --board=ulx3s ulx3s.bit
Invalid pin ID
JTAG init failed with: std::exception

My system is: Debian bookworm

I don't have the exact commit where it broke, but I can look for it if needed.

@trabucayre
Copy link
Owner

This regression was introduces when libgpiod support was added. pins configuration are now described by it's ID instead of shift (4 instead of 1 << 4).
I have pushed a fix: works with my ulx3s but could you confirm?
Thanks

@mole99
Copy link
Author

mole99 commented Aug 23, 2022

Yes, I can confirm that it fixed the issue. Thank you very much :)

@trabucayre
Copy link
Owner

Thanks for your feedback!

@carlosedp
Copy link

carlosedp commented Sep 14, 2022

Hi @trabucayre ... any news about cutting a new release with this fix so we can have MacOS Homebrew updated too.
Thanks!

@trabucayre
Copy link
Owner

I did not have planned to create a new release soon, but it's true with this regression now fixed and GW2A's MSPI mode added it make sense to create at leas a minor release (0.9.1).
I will doing that ASAP.
Thanks!

@trabucayre
Copy link
Owner

@carlosedp with a bit of late: done!
Thanks!

@carlosedp
Copy link

carlosedp commented Sep 19, 2022

Thanks, sent a PR bumping Homebrew.
Homebrew/homebrew-core#111142

@trabucayre
Copy link
Owner

Thanks!

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

No branches or pull requests

3 participants