chipyard icon indicating copy to clipboard operation
chipyard copied to clipboard

Arty GPIO & Freedom E SDK Improvements

Open alexdlukens opened this issue 3 years ago • 3 comments

Related issue:

Type of change: other enhancement

Impact: other

Release Notes

Added GPIO functionality to Arty FPGA prototyping flow, used Freedom E SDK to verify functionality. Added script to enable multilib compiling flag & add additional env variables to RISCV GNU Toolchain build script.

alexdlukens avatar Jun 25 '21 06:06 alexdlukens

This looks good to me, but I'm not sure what the status of the arty-spi-flash branch is @jamesdunn should decide what to do here.

jerryz123 avatar Jun 26 '21 01:06 jerryz123

Hi @jerryz123 and @alexdlukens , I'm very sorry for being so late on this. The GPIO enhancements look good to me. I think that it would be good to merge the GPIO work into arty-spi-flash. The branch contains fixes to UART as well, so I think GPIO is in line with this branch making IO fixes/improvements (UART, GPIO, SPI flash). My only suggestion is that the GPIO peripherals reside in WithArtyTweaks rather than its own config, since it is useful in the default case.

Once I resolve the arty-sim issues, I will simulate these IOs before submitting a merge request of both arty-spi-flash and arty-sim to dev.

I believe that the freedom-e-sdk/multilib work may be better suited for a different branch, which I will create after merging in just the GPIO work here. My comment in the other PR has more details.

Thanks, James

jamesdunn avatar Jul 28 '21 00:07 jamesdunn

Thank you for the response, this all sounds great. I will move the GPIO into the WithArtyTweaks fragment when I get a chance to look at the code.

alexdlukens avatar Jul 29 '21 14:07 alexdlukens

Closing stale PR

joonho3020 avatar Feb 04 '24 20:02 joonho3020