Icebram errors when no BRAM instances are replaced about tillitis-key1

[0xMihir]

It seems like this commit was added specifically to cause that error when nothing was replaced. I'm not sure if this is the intended behavior from the TKey build code.
YosysHQ/icestorm@a4d32c9

[0xMihir]

Actually, this might be a general build issue on macOS. I tried building the .bin file on a clean linux install and it did replace some portions of memory.

[quite]

Oh, that's a good change. When icebram fails to replace anything, that is most likely a fatal error. It is also for us.

But can you verify that you're building yosys itself from 06ef3f264afaa3eaeab45cc0404d8006c15f02b1 according to https://github.com/tillitis/tillitis-key1/blob/main/doc/toolchain_setup.md on both Macos and a Linux? We're sitting on that commit to avoid precisely some issue which causes icebram to fail to replace anything.

[0xMihir]

Sorry! It seems like brew overrode the path for yosys. I deleted the brew one, and it works now.

[quite]

@0xMihir oh, so you had a yosys installed from brew, which took precedence over the one installed in /usr/local/bin as part of Tillitis toolchain? I guess brew as their own entry in PATH, before /usr/local/bin then? But I don't know or have access to any Macos system. Did you just uninstall yosys from brew? What is the best way to avoid the conflict? Perhaps we should document something...

[0xMihir]

Yes, I ran brew uninstall yosys and reran sudo make install after