fupy / micropython

MicroPython port to litex FPGA platforms
https://micropython.org
MIT License
40 stars 14 forks source link

Include Guard Improvement #53

Closed cr1901 closed 6 years ago

cr1901 commented 6 years ago

While fupy has an include guard testing for the existence of ControlAndStatus, some boards may not include all of the components of the ControlAndStatus module. This PR improves the include guard to enable the stub read and write functions for boards lacking one or more components of ControlAndStatus. Prior to this commit, the fupy build would error out with an unresolved symbol for csr read/write functions during linking.

mithro commented 6 years ago

LGTM!