search

casper-astro / mlib_devel

http://casper-toolflow.readthedocs.io/en/latest/jasper_documentation.html
56 stars 86 forks source link

[WIP] Fix ublaze tcl to use 4MB memmap #183

Open kiranshila opened 2 years ago

kiranshila commented 2 years ago

Not working yet

kiranshila commented 2 years ago

This did not seem to fix it

Here is some output from the toofow run, not sure if it's helpful

Slave segment '/axi_slave_wishbone_classic_master_0/S_AXI/reg0' is being assigned into address space '/microblaze_0/Data' at <0x44C0_0000 [ 4M ]>.
Slave segment '/axi_timebase_wdt_0/S_AXI/Reg' is being assigned into address space '/microblaze_0/Data' at <0x41A0_0000 [ 64K ]>.
Slave segment '/axi_timer_0/S_AXI/Reg' is being assigned into address space '/microblaze_0/Data' at <0x41C0_0000 [ 64K ]>.
Slave segment '/axi_uartlite_0/S_AXI/Reg' is being assigned into address space '/microblaze_0/Data' at <0x4060_0000 [ 64K ]>.
Slave segment '/microblaze_0_local_memory/dlmb_bram_if_cntlr/SLMB/Mem' is being assigned into address space '/microblaze_0/Data' at <0x0000_0000 [ 128K ]>.
Slave segment '/microblaze_0_local_memory/ilmb_bram_if_cntlr/SLMB/Mem' is being assigned into address space '/microblaze_0/Instruction' at <0x0000_0000 [ 128K ]>.
Slave segment '/microblaze_0_axi_intc/S_AXI/Reg' is being assigned into address space '/microblaze_0/Data' at <0x4120_0000 [ 64K ]>.
WARNING: [BD 41-2670] Found an incomplete address path from address space '/microblaze_0/Data' to master interface '/microblaze_0_axi_periph/M05_AXI'. Please either complete or remove this path to resolve.
WARNING: [BD 41-2670] Found an incomplete address path from address space '/microblaze_0/Data' to master interface '/microblaze_0_axi_periph/M06_AXI'. Please either complete or remove this path to resolve.
WARNING: [BD 5-699] No address segments matched 'get_bd_addr_segs -addressables -of_objects /microblaze_0_axi_periph/xbar/M05_AXI'
WARNING: [BD 5-699] No address segments matched 'get_bd_addr_segs -addressables -of_objects /microblaze_0_axi_periph/xbar/M06_AXI'
jack-h commented 1 year ago

Did this get completed in the end?

jkocz commented 1 year ago

@jack-h As I understand it, this is not yet ready for merge.

@kiranshila can you confirm this is still WIP / convert to draft?

kiranshila commented 1 year ago

Yeah iirc I still had issues after all these changes - I was reasonably sure I needed to rebuild the JAM image but ran into too many problems even to start.

jkocz commented 1 year ago

This should now be addressed in the m2021a-dev branch. @liuweiseu made similar changes and found it to work.

Leaving this open for the moment until we're sure all issues are addressed, but hopefully we can now close it.