Remove pre-existing magic and gate level verilog because they may be … #102
Conversation
There was a problem hiding this comment.
@d-m-bailey : Do you really want to remove the base files like gpio_defaults_block.mag or just the generated ones, which would be gpio_defaults_block_* rather than gpio_defaults_block*?
|
@RTimothyEdwards good catch. But we don't need or want any un-programmed default blocks in the user area, do we? |
|
@d-m-bailey : Is it located in the user project contents, or in the user project's copy of caravel? If the former, then it would be appropriate to remove it---it should not be there. If the latter, it's a source file. I know this doesn't go back into the user's repository, and I'm probably being way too pedantic about a use case that might not even exist, which is taking the contents of the tapeout directory and running a new local build with it. |
Only remove created copies. Do not remove originals (which shouldn't exist in the user area).
|
@RTimothyEdwards I was actually just thinking the same thing. If someone were to run the script in the caravel directory, it would wipeout the master copy. I made the change. Can you review? |
|
@a-omla @DavidRLindley -- this PR will require an update and verification of the tapeout flow on the platform. I can merge now or we can hold until you are ready to implement on the platform. |
…out-of-date.
Without this change, existing
gpio_defaults_block_*files will not be updated if changes are made to this script.