FPGAwars / icestudio

:snowflake: Visual editor for open FPGA boards
https://icestudio.io
GNU General Public License v2.0
1.7k stars 245 forks source link

ERROR - Just test this with both FunICE and Bx #470

Open ph1lj-6321 opened 3 years ago

ph1lj-6321 commented 3 years ago

Use apio 0.5.6, I get the error as seen in the image - if I use 0.4.1 however on the the BX at least all is good RS error

A simple RS flip-flop - sorry just realized the tool chain dialog covers the other Nand

fuesika commented 3 years ago

I am not able to reproduce your error (with the latest Apio 0.6). Please provide more details or close this issue.

ph1lj-6321 commented 3 years ago

I will try it with Apio 0.6 - will let you know

ph1lj-6321 commented 3 years ago

Uhm - how do you force it use Apio 0.6.**. I have downloaded and installed through pip - it always default to the older Apio Any ideas ?

fuesika commented 3 years ago

Did you download Apio through pip? In that case you may need to check with your repository. You can force specific versions by appending "==0.6", e.g. pip install apio==0.6

Alternatively, you can try the icestudio AppImage which comes readily packaged with Apio. Downloads are available here: https://github.com/FPGAwars/icestudio/releases

ph1lj-6321 commented 3 years ago

OK - I'm trying to install the Toolchain now using the 5.1w210224 for win64 build - started OK - seems to get stuck around the 50% mark - showing that it's 'installing default apio...'

It's been an hour - still there.

I'm using a laptop with Win10 Pro

I guess the Win10 install issue hasn't been resolved

Obijuan commented 3 years ago

As a workaround, please, install the toolchain manually from outside icestudio, following this instructions:

https://github.com/FPGAwars/icestudio/wiki/Installing-the-Toolchains-from-outside-icestudio

Once it is working, it would be great if you could find more information about the bug and proceed as indicated in this issue:

https://github.com/FPGAwars/icestudio/issues/494

The developers usually work in Linux and it is difficult for us to reproduce the bugs in other platforms. Any help is very welcome

ph1lj-6321 commented 3 years ago

Ok - big step forward, with the above process 5.1w210224 was installed with apio 0.6.0 - Thank you.

However the original issue of not building the simple SR example, with the BX is still there - icestudio SR error with last build

ph1lj-6321 commented 3 years ago

By the way - tried to use the error log, but after the above, my error log folder is empty (yes I did set Preference's Logging enabled.)

ph1lj-6321 commented 3 years ago

Also - I just tried the supplied S-R latch (Examples. 3.Gate 05. SR latch) - same thing with both the BX and IceFUN boards

Obijuan commented 3 years ago

That particilar example (the S-R latch) does not work by default with the new toolchain (it is not a problem related to IceFun board). It worked ok with the old arachne-pnr tool, but when it was upgraded to nextpnr it stoped working. This new tool, by default, does not allow to have combinational loop. Of course, this can be overrided using the --force or --ignore-loops flags when invoking nextpnr:

https://github.com/YosysHQ/nextpnr/issues/224

But this option has not yet been incluided in apio/icestudio

The S-R latch example should be removed or at least we should provide comments explaining that it does not work with the current toolchain (the --ignore-loops flag should be used)

Obijuan commented 3 years ago

Insted of impleting the latches and flip-flops directly using logic gates it is better to implement them in verilog or using the provided blocks. The latest collection related with FF can be found here:

https://github.com/FPGAwars/iceFF

There is no a release yet, but you can download as zip and install it easily

https://github.com/FPGAwars/iceFF/archive/refs/heads/master.zip

It is a work in progress. We are adding new FFs as we need them

cavearr commented 3 years ago

By the way - tried to use the error log, but after the above, my error log folder is empty (yes I did set Preference's Logging enabled.)

Hi!,thanks for your feedback, Has you specify in the Preferences->Logging file the output file? Are you sure that Icestudio can write at this location?