public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Icenowy Zheng <icenowy@aosc.xyz>
To: u-boot@lists.denx.de
Subject: [U-Boot] Some discussion about Allwinner DesignWare DRAM controller SPL code cleanup
Date: Tue, 10 Jan 2017 18:25:18 +0800	[thread overview]
Message-ID: <1484961484043918@web7h.yandex.ru> (raw)

Hi Jens, Andre, Chen-Yu and everyone,
As we know, allwinner start to refuse to provide code about DRAM initialization
after H3.
For A64, H5, V3s we have already did some work on reusing H3 code and it gives
good result.
For R40 the result is also usable (although half of the memory is lost)
I think we will need some refactor, which will makes the H3 driver finally a
SUNXI_DW_DRAM driver ;-)
The first thing I want to say is that the DDR timing parameters part needs a lot
refactor, for V3s' integrated DDR2, for SoPine's LPDDR3, etc...
What I think is to create some header files included by the DRAM driver which
defines the timing parameters, and make dram_sun8i_h3.c include them by the
choice in .config .
The second thing is that, for some alike controller from AW, we have at least
source files: dram_sun8i_{a33,h3}.c (A33 uses a similar DW DRAM).
Should we merge A33 into the H3 source code?
Opinions wanted ;-)

Thanks,
Icenowy

             reply	other threads:[~2017-01-10 10:25 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-10 10:25 Icenowy Zheng [this message]
2017-01-10 10:53 ` [U-Boot] Some discussion about Allwinner DesignWare DRAM controller SPL code cleanup Andre Przywara

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1484961484043918@web7h.yandex.ru \
    --to=icenowy@aosc.xyz \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox