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

Can we avoid the special case for tinyprog on darwin? #529

Closed
zapta opened this issue Jan 10, 2025 · 8 comments
Closed

Can we avoid the special case for tinyprog on darwin? #529

zapta opened this issue Jan 10, 2025 · 8 comments

Comments

@zapta
Copy link
Collaborator

zapta commented Jan 10, 2025

@cavearr, this is a question for you.

The general command line for the tinyprog progammer that apio uses is

tinyprog --pyserial -c ${SERIAL_PORT} --program $SOURCE

However there is a special hardcoded case for tinyprog on darwin which sets the command

tinyprog --libusb --program $SOURCE

Is this special case necessary? Can we merge the cases together into a single command line?

@cavearr
Copy link
Member

cavearr commented Jan 10, 2025

i'll try it . In osx always i use --libusb, the other command line i'm not use never.
Tomorrow afternoon i'll play with it in different OSs and try to find if is it possible reduce to one command.

Thanks @zapta!

@zapta
Copy link
Collaborator Author

zapta commented Jan 10, 2025 via email

@cavearr
Copy link
Member

cavearr commented Jan 10, 2025

Hi @zapta , TinyFPGA-BX but be careful because the last batches has been solded with bad firmware. And the project appears very abandoned or unatended.

I buyed several years ago, now i don't know where you can buy. I know, people that i had been helped in summer to work with it, buy in the past Spring at Mouser.

I'm working in a patched version of tinyprog to work with old and new boards i hope have for testing during the next week.

@zapta
Copy link
Collaborator Author

zapta commented Jan 17, 2025

Hi @cavearr, can we remove the special case of tinyprog on darwin that sets the command line to tinyprog --libusb --program $SOURCE instead of the command tinyprog --pyserial -c ${SERIAL_PORT} --program $SOURCE that is used on the other platforms.

@zapta
Copy link
Collaborator Author

zapta commented Feb 5, 2025

Hi @cavearr , can we remove this tinyprog special case? (more details in the other posts in this issue)

if "tinyprog" in board_info and apio_ctx.is_darwin:

@cavearr
Copy link
Member

cavearr commented Feb 5, 2025

yes @zapta remove it, i'll work around tinyfpga the next week, i have been planned a deep work on it to fix the programmers , i'm telling you around this the next week because i think we need to use our own programmer to have all tinybx models working.

@zapta
Copy link
Collaborator Author

zapta commented Feb 5, 2025

Thanks @cavearr, I will remove it.

zapta added a commit to zapta/apio_dev that referenced this issue Feb 5, 2025
@zapta
Copy link
Collaborator Author

zapta commented Feb 5, 2025

Removed. Closing.

@zapta zapta closed this as completed Feb 5, 2025
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

2 participants