enjoy-digital / litedram

Small footprint and configurable DRAM core
Other
365 stars 115 forks source link

Add MT46H128M16 and change bankmaschine to not use A10 for col addresses. #313

Closed cklarhorst closed 12 months ago

cklarhorst commented 1 year ago

For larger memories, the col encoding needs to skip A10 because that one is reserved for auto-precharge. The remaining col encoding bits are then mapped to A11 and higher.

I think that newer RAM generations are defining additional reserved address bits, e.g. for DDR4 A12 is used for burst-chop. But I ignored those higher bits, as we don't have any module with that many col bits used.

cklarhorst commented 1 year ago

(Thanks to the unit tests, I found an off by one error in my if)