U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Georgi Vlaev <g-vlaev@ti.com>
To: Vignesh Raghavendra <vigneshr@ti.com>,
	Anand Gadiyar <gadiyar@ti.com>, Tom Rini <trini@konsulko.com>
Cc: <u-boot@lists.denx.de>, Georgi Vlaev <g-vlaev@ti.com>
Subject: [PATCH 0/5] board: ti: am62x: Add support for 1-bit inline DDRSS ECC
Date: Tue, 14 Jun 2022 17:45:29 +0300	[thread overview]
Message-ID: <20220614144534.317407-1-g-vlaev@ti.com> (raw)

Hi,

This series enables the 1-bit inline DDR ECC support in the
TI DDRSS bridge for AM62x. The base DDRSS ECC support was
merged for k3-ddrss in a previous series for AM64x [1].

The ECC data is stored together with the data, which will
reduce the total available memory with 1/9th. The k3-ddrss
driver enables the ECC support and primes the full memory
from the R5 SPL, so the the memory size changes must
propagate from the R5 SPL to A53 SPL and then to A53 u-boot.

The patches are similar to those we have for AM64x, with
one exception: "arm: mach-k3: common: Use ddr_init in
spl_enable_dcache". Since we've switched the boards to DT
to fetch the memory configuration, dram_init_banksize()
will no longer set the total memory size in the global data.
And spl_enable_dcache() uses dram_init_banksize() instead
of dram_init(), which actually sets the memory base/size.

Note this doesn't enable the 1-bit ECC on any platform by
default. This can be done by adding the "ti,ecc-enable"
property to the memorycontroller node in k3-am6*-ddr.dtsi.

[1] https://lore.kernel.org/u-boot/20220317170346.31162-1-d-gerlach@ti.com/

The patches depend on the base AM62 support, which was
recently merged in u-boot/next.

Georgi Vlaev (5):
  arm: mach-k3: common: Use ddr_init in spl_enable_dcache
  arm: dts: k3-am625-*: Mark memory with u-boot,dm-spl
  board: ti: am62x: Use fdt functions for ram and bank init
  board: ti: am62x: Account for DDR size fixups if ECC is enabled
  configs: am62x_evm_r5: Add CONFIG_NR_DRAM_BANKS as done in a53
    defconfig

 arch/arm/dts/k3-am625-r5-sk.dts      |  1 +
 arch/arm/dts/k3-am625-sk-u-boot.dtsi |  4 ++
 arch/arm/mach-k3/common.c            |  2 +-
 board/ti/am62x/evm.c                 | 62 ++++++++++++++++++++++++----
 configs/am62x_evm_r5_defconfig       |  1 +
 5 files changed, 61 insertions(+), 9 deletions(-)


base-commit: a87a6fcd20c0e29fe55bfbb6917c4aa1f1bbce74
-- 
2.30.2


             reply	other threads:[~2022-06-14 15:23 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-14 14:45 Georgi Vlaev [this message]
2022-06-14 14:45 ` [PATCH 1/5] arm: mach-k3: common: Use ddr_init in spl_enable_dcache Georgi Vlaev
2022-06-16 14:38   ` Tom Rini
2022-07-07  1:55   ` Tom Rini
2022-06-14 14:45 ` [PATCH 2/5] arm: dts: k3-am625-*: Mark memory with u-boot,dm-spl Georgi Vlaev
2022-06-16 14:38   ` Tom Rini
2022-07-07  1:55   ` Tom Rini
2022-06-14 14:45 ` [PATCH 3/5] board: ti: am62x: Use fdt functions for ram and bank init Georgi Vlaev
2022-06-16 14:38   ` Tom Rini
2022-07-07  1:55   ` Tom Rini
2022-06-14 14:45 ` [PATCH 4/5] board: ti: am62x: Account for DDR size fixups if ECC is enabled Georgi Vlaev
2022-06-16 14:39   ` Tom Rini
2022-07-07  1:55   ` Tom Rini
2022-06-14 14:45 ` [PATCH 5/5] configs: am62x_evm_r5: Add CONFIG_NR_DRAM_BANKS as done in a53 defconfig Georgi Vlaev
2022-06-16 14:39   ` Tom Rini
2022-07-07  1:55   ` Tom Rini

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=20220614144534.317407-1-g-vlaev@ti.com \
    --to=g-vlaev@ti.com \
    --cc=gadiyar@ti.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=vigneshr@ti.com \
    /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