search

multigcs / riocore

riocore
GNU General Public License v2.0
22 stars 7 forks source link

Fix COMP/SITE that were provided with incorrect arguments #13

Closed festlv closed 4 months ago

festlv commented 4 months ago

While trying to test riocore on ecp5 I got an error about sysclk_in not being constrained in pins file. I don't know much about specifying constraints, but comparing pins.lpf generated by LinuxCNC-RIO and riocore, it became apparent to me COMP contained physical pin and SITE contained net-name (which should be the other way around).

This PR fixes this (at least my basic ECP5 config builds, haven't tested it on FPGA yet).

multigcs commented 4 months ago

many thanks, yes, forget to fix the lpf pin generator. now i know what i'm going to do today, a little test with an ecp5 :)

multigcs commented 4 months ago

colorlite_i5 works :)