tgingold-cern / cheby

GNU General Public License v3.0
8 stars 4 forks source link

Vivado IPI compatibility #42

Open iljabek opened 5 months ago

iljabek commented 5 months ago

if using the cheby-generated AXI4-Lite interface in Vivado block design without a wrapper following issues arise:

minimal yaml example:

memory-map:
  name: info
  bus: axi4-lite-32
  x-hdl:
    bus-attribute: Xilinx

Current IP in BD + post-autoconnect:

cheby_default_clk2axi cheby_default_ipi cheby_default_ipi_conn

minimal "valid" IP in BD + post-autoconnect:

cheby_xattr_clk2axi cheby_xattr_ipi_conn

image

iljabek commented 5 months ago

@tgingold-cern I have to look into the failing test, but is this https://github.com/iljabek/cheby/commit/095f911874880a79eeb2a5248dca565f75e972bb going in good general direction? would you prefer to split the minimal and customization commits?

tgingold-cern commented 5 months ago

Yes, I think this is OK. I would use 'clock-name' and 'reset-name' instead of 'bus-clock-name' and 'bus-reset-name'

Not sure why you need 'use-bus-name-in-ports'. There is a 'reg-prefix' which might do what you'd like.

iljabek commented 5 months ago

There is a 'reg-prefix' which might do what you'd like

can the bus name consist only out of reg-prefix, i.e. name be empty?

tgingold-cern commented 5 months ago

I think that reg-prefix has no influence on bus name, only on ports (and internal names).