efabless / caravel_user_project_analog

Apache License 2.0
43 stars 90 forks source link

user_defines needs to uncomment initial values #35

Closed jeffdi closed 1 year ago

jeffdi commented 1 year ago

commented values are causing gpio_defaults to fail in tapeout

RTimothyEdwards commented 1 year ago

Need to swap pullup and pulldown definitions

RTimothyEdwards commented 1 year ago

@jeffdi : The intent was that the configuration definitions should be included from another file out of the user's control, like you split "caravel.h" from "defs.h". But to be out of the user's control, that file would need to be in the caravel repository. I think there would need to be a change to "caravel_netlists.v", in that case?

RTimothyEdwards commented 1 year ago

Fixed by PR #37