add burst converter tests, fix downconverting burst

enjoy-digital / litedram

Small footprint and configurable DRAM core

Other

372 stars 120 forks source link

add burst converter tests, fix downconverting burst #341

Closed hansfbaier closed 1 year ago

hansfbaier commented 1 year ago

This backports the tests from https://github.com/enjoy-digital/litedram/pull/340 to current master, to compare test results for up/downconverting bursts. After fixes of issues uncovered by the tests, the downconverter shows similar garbled memory contents in the RAM of the simulator:

0x00000000: 0xee00
0x00000001: 0xc0ff
0x00000002: 0x3210
0x00000003: 0x7654
0x00000004: 0xba98
0x00000005: 0xfedc

hansfbaier commented 1 year ago

Ah I found out why: I need to supply enough memory cells in mem_expected otherwise the addresses wrap around

hansfbaier commented 1 year ago

OK tests now all pass, and it has been verified on hardware too with this design: https://github.com/MiSTeX-devel/MiSTeX-boards/blob/main/mistex_boards/qmtech_xc7a100t_daughterboard.py Which should be a downconverting one (only one SDRAM chip).

hansfbaier commented 1 year ago

Closing this PR in favor of https://github.com/enjoy-digital/litedram/pull/340