-
Notifications
You must be signed in to change notification settings - Fork 65
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
"SPI Flash not functional" #163
Comments
If I recall correctly, there isn't a simple SPIFlash on a VCUI118 board so the overlay would not have anything to connect to. |
Thanks for the reply! There's qspi flash that looks like it's usable for data storage. I'm still working my way through understanding overlays, but it looks like |
Yeah, I had tried the STARTUPE3 SPI flash as an experiment a while ago and wasn't able to get it working. I didn't spend too much time on it after that, so it might be a small fix, but I haven't had time to look back at it since then |
With pcie hooked up, I'm now revisiting the idea of hooking up flash. I'm assuming the targetted toplevel for the spiflash controller is val se3 = Module(new STARTUPE3())
se3.io.USRDONEO := true.B
se3.io.USRDONETS := false.B
se3.io.USRCCLKO := tlqspiSink.bundle.sck.asClock
se3.io.USRCCLKTS := false.B
se3.io.FCSBO := !tlqspiSink.bundle.cs(0) // I believe cs is active high in SPIFlash
se3.io.FCSBTS := false.B
se3.io.DO := Cat(tlqspiSink.bundle.dq.map(_.o))
se3.io.DTS := Cat(tlqspiSink.bundle.dq.map(_.oe))
tlqspiSink.bundle.dq(0).i := se3.io.DI(0)
tlqspiSink.bundle.dq(1).i := se3.io.DI(1)
tlqspiSink.bundle.dq(2).i := se3.io.DI(2)
tlqspiSink.bundle.dq(3).i := se3.io.DI(3)
se3.io.GSR := false.B
se3.io.GTS := false.B
se3.io.KEYCLEARB := true.B
se3.io.PACK := false.B I'll have to hook up the |
I'm trying to understand FPGA-shells so I can add a PCIe to my SoC in Chipyard, but I was also thinking of adding SPI flash. While digging through VCU118NewShell.scala I saw that
//SPI Flash not functional
withinVCU118ShellBasicOverlays()
Is the overlay for SPI broken in some way?The text was updated successfully, but these errors were encountered: