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
next 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