-
Notifications
You must be signed in to change notification settings - Fork 142
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
Programmers definitions 'openfpgaloader', 'ujprog' are not used. #536
Comments
Please, do not remove the programmers by now. Some boards can be programmed using different programmers. Buy default one is used, but the others can also be used. It is good for testing. For example, the Alhambra-II board uses iceprog, but it can also use openFPGALoader, which is faster (I am planning to switch to it) Before removing things I think it is better to mark them as deprecated, and give more time to other uses to test. The current apio development branch has too many changes. I fear that when releasing it, many projects will be broken |
@Obijuan , in Icestudio i'm evolving the compiler to add verilator exceptions for some things that broken a lot of projects at verify process. The big "projects conflicts" comes from verify with verilator, but this is good in medium term because the problems are really bad practice implementations. I'm requesting at fpgawars community to test with projects and send me each of this that verify fails to add exceptions or evolve the error handling. Don't worry with that, i think we don't needed of release nothing that has not been tested for months and all of the problems will be solved before. The migration to openfpgaloader i thinks is a very good idea, i use a lof and thinks is better than iceprog, if you need help with this, tell us. I'm ok with no delete this programmers, in fact we could do in near future that in apio.ini some boards could be configure tu use one or other, what do you think @zapta ? Setup a preferrer programmer for board, but do the capability of setup an alternative programmer per board, for example Alhambra-ii, default programmer iceprog but alternative could be openfpgaloader defined in apio.ini. |
@Obijuan, sounds good, I will leave it. The 'openfpgaloader' is in use now for the gowin/tang boards so As for breaking existing project, they are supposed to break only were we had concuss decisions such as requiring apio.ini or dropping support for custom SConstruct.
|
@cavearr, regarding allowing the user to switch the programmer a board uses, there are a few options:
apio.ini
Any thoughts? |
i like a lot all of the options, the board.json in the project folder for custom boards is very interesting, this is very aligned with a new feature to add boards that i'm developing at icestudio. For me and for the momment with this should be enought, maybe in apio.ini needs and "args" parameter for the programmer. |
BTW, the programmers.json is also customizable and users can add them to
their projects as well.
https://github.com/FPGAwars/apio/blob/884eabd14858967295cb7ba538128914d6c68c8f/apio/apio_context.py#L185
…On Mon, Jan 13, 2025 at 12:01 AM Carlos Venegas Arrabé < ***@***.***> wrote:
i like a lot all of the options, the board.json in the project folder for
custom boards is very interesting, this is very aligned with a new feature
to add boards that i'm developing at icestudio.
For me and for the momment with this should be enought, maybe in apio.ini
needs and "args" parameter for the programmer.
—
Reply to this email directly, view it on GitHub
<#536 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAQVMQP2FYSOQT3EPQLHC532KNXF7AVCNFSM6AAAAABU7XBHUWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKOBWGQZTEOJXGU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I created an issue to discuss the board and programmer replication issue. https://github.com/FPGAwars/apio/issues/new Closing this one. The unused programmer will say. |
@Obijuan, the (There is another programmer definition called
|
Closing. |
The two programmers 'openfpgaloader', 'ujprog' are defined but not used.
@juan, @cavearr, OK to delete? (we can also add back when needed).
Motivation: I am adding an automatic test that validates the boards, fpgas, and programmers and this breaks the test.
https://github.com/FPGAwars/apio/blob/develop/apio/resources/programmers.json
The text was updated successfully, but these errors were encountered: