litedram icon indicating copy to clipboard operation
litedram copied to clipboard

Published 20 hours ago • verified publisher icon enjoy-digital

Upconverter rewrite

Open mtdudek opened this issue 2 years ago • 4 comments

This series of commits improve frontend adapter bandwidth.

mtdudek avatar Sep 24 '21 13:09 mtdudek

@enjoy-digital what do you think?

mtdudek avatar Sep 24 '21 13:09 mtdudek

Thanks @mtdudek, I'm not able to review it now but will try to do so in the next days.

enjoy-digital avatar Sep 29 '21 06:09 enjoy-digital

So it failed on picolibc, and not on this upconverter, it states that meson is not installed

mtdudek avatar Sep 29 '21 08:09 mtdudek

Thanks @mtdudek, I've been able to have a first look. The new code is becoming really complex to follow and I'm not sure it will be possible to maintain it, so I'm not able to merge it now and will have to spend time understanding the previous limitations.

The primary issue is in fact coming from LiteVideo that should not require the use of an UpConverter and should directly handle access to the DRAM port with native data-width. This allows using a very simple specific UpConverter vs the generic (but complex) UpConverter here. The new VideoFramebuffer (now recommended vs LiteVideo VideoOut core) does not have this limitation and directly runs with native data-width: https://github.com/enjoy-digital/litex/blob/master/litex/soc/cores/video.py#L618.

enjoy-digital avatar Nov 01 '21 21:11 enjoy-digital