Bootloader fails to synthesize with latest arachne-pnr

[elfmimi]

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]

@elfmimi Please reopen if this still happening for you.

[elfmimi]

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]

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

[elfmimi]

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]

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]

@elfmimi Thank you for confirming.