Suggested fixes

[hstarmans]

Dear Matt Venn,

This is a really nice board.

I have made the following changes;

• Add missing qspi to the Pi to be able to program bitstream faster.
  Done

• used IS25LP016D-JBLE as flash, but
  it has problems with enabling QE for QSPI.
  Relace with the model used on icebreaker

  This is already done, comment should be removed

• ecp5 SPI clock pin can't be used as an input
  so have to add a bodge wire
  for SPI peripheral comms with Pi.
  See the test-spi-clk-in branch for a workaround.

Worked around by connecting this pin to a second FPGA pin

• connected I2C pins are just GPIOs. I have found the Pi's I2C hardware unusable, so always use the slower bitbanged version.

Connected I2C pins to official pins. I have not had this issue.
Removed pull resistors as this is not needed

In case you wonder. I plan to use this board for my laser scanner. A video can be found here https://youtu.be/6PEZtsm7FvE
Future boards are made so that you can simply connect you FPGA base board to the laser scanner (using say 2 pmods) and
to the stepper motor board using say 1 PMOD.

Looking forward to your feedback.

Regards,

Rik

[mattvenn]

HI thanks for your PR, I didn't see it until now for some reason!

[hstarmans]

I did order your board but in the end never build it.. Currently still use UP5K and my interest is now more focused on PCB motors (https://youtu.be/r3wv7XpXPck) might get back to this later..