tinyfpga/TinyFPGA-Bootloader

Bootloader fails to synthesize with latest arachne-pnr

elfmimi opened this issue · 6 comments

elfmimi commented

This is the error message it reports.

fatal error: Top level port 'pin_usbp' assigned to an IO pad 'Y' and internal nodes
Makefile:29: recipe for target 'bootloader.asc' failed

It seems to be a regression caused by this commit in arachne-pnr repository.
YosysHQ/arachne-pnr@c00a141

It completes successfully if you roll it back to previous commit. (commit id: 969836e)

Which side is responsible for fixing the issue?

mithro commented

@elfmimi Please reopen if this still happening for you.

elfmimi commented

Of course it still happens. Because the above PR referenced by microbuilder is not the one which aims to fix this. But it's the one which caused this. @mithro

mithro commented

So, I'm leaning towards dropping support for arachne-pnr and only supporting nextpnr here.

elfmimi commented

Sorry! My last report was wrong. I thought I had been pulling latest TinyFPGA-Bootloader, but I was failing to do so.
Both of anachne-pnr and nextpnr are processing latest source okay. > a77f828
Please clone this issue. @mithro

elfmimi commented

My personal note.

First attempt to get rid of tristate inference was introduced on 2018-09-02
into smunaut:master at af17343

boards/TinyFPGA_BX: Don't rely on tristate inference

Merged to tinyfpga:master at 858d1a9
with pull-request #21

And is there until 2019-07-06
https://github.com/tinyfpga/TinyFPGA-Bootloader/blob/09b70464fff5a8c9db221c583b15e8a6ac843e9b/boards/TinyFPGA_BX/bootloader.v

But replaced by merge of osresearch:tomu on 2019-07-06 at 704c8fc
with pull-request #34

Which is first introduced to osresearch:clock-crossing at 84959e7

TinyFPGA_BX: use nextpnr-ice40

and merged to osresearch:tomu at 22f8d62

mithro commented

@elfmimi Thank you for confirming.