orpsoc-cores icon indicating copy to clipboard operation
orpsoc-cores copied to clipboard

wb_ack_o missing in wb_to_avalon_bridge for BURST_SUPPORT = 0

Open AakaFosfor opened this issue 8 years ago • 4 comments

The core wb_to_avalon_bridge doesn't work when the burst support is not selected - there is no assignment for the wb_ack_o signal.

AakaFosfor avatar Oct 28 '16 08:10 AakaFosfor

Hi,

Just wanted to let you know that I've seen the issue, but unfortunately I don't know enough about that core to give you an answer right away, and I haven't had time to look into it either

olofk avatar Nov 08 '16 20:11 olofk

Hi, thanks for letting me know. No direct problem for me, I have finally used another core (simplified version I found my colleague already did for the same situation I wanted to used it in). So the issue is just FYI. Theoretically, WB-Avalon crossing should be quite easy (so the patch), but I didn't work with Avalon before, so I don't dare to propose a correction.

AakaFosfor avatar Nov 09 '16 05:11 AakaFosfor

I'm not sure I understand, AFAICT 'wb_ack_o' is assigned at https://github.com/openrisc/orpsoc-cores/blob/master/cores/wb_avalon_bridge/verilog/wb_to_avalon_bridge.v#L231 when BURST_SUPPORT == 0. What am I missing?

skristiansson avatar Nov 10 '16 19:11 skristiansson

You are right, thank you for pointing it out. I missed "generate-else" at line 214. So the red X in my simulation had to be caused by something else... So, sorry for this issue, you can close it.

AakaFosfor avatar Nov 11 '16 11:11 AakaFosfor