proppy / tiny_user_project

Apache License 2.0
10 stars 8 forks source link

make i/o configurable #7

Closed proppy closed 1 year ago

proppy commented 1 year ago
proppy commented 1 year ago

@shaos @mattvenn @jeffdi, do you think that's useful?

mattvenn commented 1 year ago

I would avoid pins < 8 as they are dual use

image

proppy commented 1 year ago

@mattvenn Done with df33cd5!

proppy commented 1 year ago

also exclude io marked as none with 7b3f48a

shaos commented 1 year ago

I tested these changes - looks like working properly with more number of inputs/outputs UPDATE: Not exactly - Wokwi returns hardcoded input [7:0] io_in, output [7:0] io_out so it has to be fixed... UPDATE2: Wokwi was fixed - NOW it's good to go!

shaos commented 1 year ago

also exclude io marked as none with 7b3f48a

not sure if it's a good change - those "none" inputs are actually defined and expected to be generated I think - if you skip them then verilog generated on Wokwi side may not be properly connected to tiny_user_project

proppy commented 1 year ago

not sure if it's a good change - those "none" inputs are actually defined and expected to be generated I think - if you skip them then verilog generated on Wokwi side may not be properly connected to tiny_user_project

Oh I see, I didn't realize that this was potentially mapping to what wowki generated.

proppy commented 1 year ago

reverted 7b3f48a and merged!