* [PATCH] Board phycore_imx8mm: spl: add 1 and 4GB RAM timings
@ 2026-06-23 6:24 INgo Rah
2026-06-24 8:00 ` Peng Fan
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: INgo Rah @ 2026-06-23 6:24 UTC (permalink / raw)
To: u-boot
Cc: INgo Rah, Benedikt Spranger, Gregor Herburger, Benjamin Hahn,
Fabio Estevam, NXP i.MX U-Boot Team, Patrice Chotard, Peng Fan,
Stefano Babic, Teresa Remmet, Tom Rini, Yannic Moog, Yao Zi,
upstream
The Phytec Tauri L is sold with other memory configurations then 2 GB.
Gather the memory size from the EEPROM of the underlaying PhyCore module
and set timing information accordingly.
Signed-off-by: INgo Rah <ingo.rah@linutronix.de>
Reviewed-by: Benedikt Spranger <b.spranger@linutronix.de>
Reviewed-by: Gregor Herburger <gregor.herburger@linutronix.de>
---
Board phycore_imx8mm: spl: add 1 and 4GB RAM timings
This patch heavily borrowed code from git://git.phytec.de/u-boot-imx commit
00ac765d6032 ("BSPIMX8M-2893 board: phycore_imx8mm: spl: add 1 and 4GB RAM
timings") and commit 93962421d7b4 ("BSPIMX8M-2893 board: phycore_imx8mm:
spl: add 1 and 4GB RAM timings")
.../dts/imx8mm-phygate-tauri-l-u-boot.dtsi | 8 ++++++
board/phytec/common/imx8m_som_detection.h | 7 ++++++
board/phytec/phycore_imx8mm/lpddr4_timing.c | 25 +++++++++++++++++++
board/phytec/phycore_imx8mm/lpddr4_timing.h | 12 +++++++++
board/phytec/phycore_imx8mm/spl.c | 24 +++++++++++++++++-
configs/imx8mm-phygate-tauri-l_defconfig | 1 +
6 files changed, 76 insertions(+), 1 deletion(-)
create mode 100644 board/phytec/phycore_imx8mm/lpddr4_timing.h
diff --git a/arch/arm/dts/imx8mm-phygate-tauri-l-u-boot.dtsi b/arch/arm/dts/imx8mm-phygate-tauri-l-u-boot.dtsi
index 26361780c01..79ddb1ade44 100644
--- a/arch/arm/dts/imx8mm-phygate-tauri-l-u-boot.dtsi
+++ b/arch/arm/dts/imx8mm-phygate-tauri-l-u-boot.dtsi
@@ -29,6 +29,10 @@
};
};
+&pinctrl_i2c1 {
+ bootph-pre-ram;
+};
+
&pinctrl_uart3 {
bootph-pre-ram;
};
@@ -73,6 +77,10 @@
bootph-pre-ram;
};
+&i2c1 {
+ bootph-pre-ram;
+};
+
&usdhc2 {
bootph-pre-ram;
};
diff --git a/board/phytec/common/imx8m_som_detection.h b/board/phytec/common/imx8m_som_detection.h
index 0176347414f..794d3add0d5 100644
--- a/board/phytec/common/imx8m_som_detection.h
+++ b/board/phytec/common/imx8m_som_detection.h
@@ -13,6 +13,13 @@
#define PHYTEC_IMX8MM_SOM 69
#define PHYTEC_IMX8MP_SOM 70
+enum phytec_imx8mm_ddr_eeprom_code {
+ INVALID = PHYTEC_EEPROM_INVAL,
+ PHYTEC_IMX8MM_DDR_1GB = 1,
+ PHYTEC_IMX8MM_DDR_2GB = 3,
+ PHYTEC_IMX8MM_DDR_4GB = 5,
+};
+
int __maybe_unused phytec_imx8m_detect(struct phytec_eeprom_data *data);
u8 __maybe_unused phytec_get_imx8m_ddr_size(struct phytec_eeprom_data *data);
u8 __maybe_unused phytec_get_imx8mp_rtc(struct phytec_eeprom_data *data);
diff --git a/board/phytec/phycore_imx8mm/lpddr4_timing.c b/board/phytec/phycore_imx8mm/lpddr4_timing.c
index f5a2f3268b3..73e35c7c3b9 100644
--- a/board/phytec/phycore_imx8mm/lpddr4_timing.c
+++ b/board/phytec/phycore_imx8mm/lpddr4_timing.c
@@ -1842,3 +1842,28 @@ struct dram_timing_info dram_timing = {
.ddrphy_pie_num = ARRAY_SIZE(ddr_phy_pie),
.fsp_table = { 3000, 400, 100,},
};
+
+void set_dram_timings_1gb(void)
+{
+ dram_timing.ddrc_cfg[5].val = 0x2d0087;
+ dram_timing.ddrc_cfg[21].val = 0x8d;
+ dram_timing.ddrc_cfg[42].val = 0xf070707;
+ dram_timing.ddrc_cfg[58].val = 0x60012;
+ dram_timing.ddrc_cfg[73].val = 0x13;
+ dram_timing.ddrc_cfg[83].val = 0x30005;
+ dram_timing.ddrc_cfg[98].val = 0x5;
+}
+
+void set_dram_timings_4gb(void)
+{
+ dram_timing.ddrc_cfg[2].val = 0xa3080020;
+ dram_timing.ddrc_cfg[37].val = 0x17;
+ dram_timing.fsp_msg[0].fsp_cfg[8].val = 0x310;
+ dram_timing.fsp_msg[0].fsp_cfg[20].val = 0x3;
+ dram_timing.fsp_msg[1].fsp_cfg[9].val = 0x310;
+ dram_timing.fsp_msg[1].fsp_cfg[21].val = 0x3;
+ dram_timing.fsp_msg[2].fsp_cfg[9].val = 0x310;
+ dram_timing.fsp_msg[2].fsp_cfg[21].val = 0x3;
+ dram_timing.fsp_msg[3].fsp_cfg[10].val = 0x310;
+ dram_timing.fsp_msg[3].fsp_cfg[22].val = 0x3;
+}
diff --git a/board/phytec/phycore_imx8mm/lpddr4_timing.h b/board/phytec/phycore_imx8mm/lpddr4_timing.h
new file mode 100644
index 00000000000..5334fa5b4ee
--- /dev/null
+++ b/board/phytec/phycore_imx8mm/lpddr4_timing.h
@@ -0,0 +1,12 @@
+/* SPDX-License-Identifier: GPL-2.0-or-later */
+/*
+ * Copyright (C) 2025 PHYTEC Messtechnik GmbH
+ */
+
+#ifndef __LPDDR4_TIMING_H__
+#define __LPDDR4_TIMING_H__
+
+void set_dram_timings_1gb(void);
+void set_dram_timings_4gb(void);
+
+#endif /* __LPDDR4_TIMING_H__ */
diff --git a/board/phytec/phycore_imx8mm/spl.c b/board/phytec/phycore_imx8mm/spl.c
index e688793bc74..c9f2243c819 100644
--- a/board/phytec/phycore_imx8mm/spl.c
+++ b/board/phytec/phycore_imx8mm/spl.c
@@ -16,6 +16,7 @@
#include <log.h>
#include <spl.h>
+#include "lpddr4_timing.h"
#include "../common/imx8m_som_detection.h"
#define EEPROM_ADDR 0x51
@@ -42,6 +43,7 @@ int spl_board_boot_device(enum boot_device boot_dev_spl)
static void spl_dram_init(void)
{
int ret;
+ enum phytec_imx8mm_ddr_eeprom_code size = PHYTEC_EEPROM_INVAL;
ret = phytec_eeprom_data_setup_fallback(NULL, 0, EEPROM_ADDR,
EEPROM_ADDR_FALLBACK);
@@ -49,10 +51,30 @@ static void spl_dram_init(void)
goto out;
ret = phytec_imx8m_detect(NULL);
- if (!ret)
+ if (!ret) {
phytec_print_som_info(NULL);
+ size = phytec_get_imx8m_ddr_size(NULL);
+ }
+ switch (size) {
+ case PHYTEC_IMX8MM_DDR_1GB:
+ puts("1 GiB\n");
+ set_dram_timings_1gb();
+ break;
+ case PHYTEC_IMX8MM_DDR_2GB:
+ puts("2 GiB\n");
+ break;
+ case PHYTEC_IMX8MM_DDR_4GB:
+ puts("4 GiB\n");
+ set_dram_timings_4gb();
+ break;
+ default:
+ goto out;
+ }
+ ddr_init(&dram_timing);
+ return;
out:
+ puts("Could not detect correct RAM size. Fall back to default.\n");
ddr_init(&dram_timing);
}
diff --git a/configs/imx8mm-phygate-tauri-l_defconfig b/configs/imx8mm-phygate-tauri-l_defconfig
index e34f12cabf3..866e5d1dbb3 100644
--- a/configs/imx8mm-phygate-tauri-l_defconfig
+++ b/configs/imx8mm-phygate-tauri-l_defconfig
@@ -4,6 +4,7 @@ CONFIG_TEXT_BASE=0x40200000
CONFIG_SYS_MALLOC_LEN=0x2000000
CONFIG_SPL_GPIO=y
CONFIG_SPL_LIBCOMMON_SUPPORT=y
+CONFIG_PHYTEC_SOM_DETECTION=y
CONFIG_ENV_SIZE=0x10000
CONFIG_ENV_OFFSET=0x3C0000
CONFIG_DM_GPIO=y
---
base-commit: f072620dc9ffda00b010783da27c41231c3a439b
branch: feature/phygate
--
2.47.3
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] Board phycore_imx8mm: spl: add 1 and 4GB RAM timings
2026-06-23 6:24 [PATCH] Board phycore_imx8mm: spl: add 1 and 4GB RAM timings INgo Rah
@ 2026-06-24 8:00 ` Peng Fan
2026-06-24 8:16 ` Yannic Moog
2026-06-24 9:29 ` Teresa Remmet
2 siblings, 0 replies; 4+ messages in thread
From: Peng Fan @ 2026-06-24 8:00 UTC (permalink / raw)
To: INgo Rah
Cc: u-boot, Benedikt Spranger, Gregor Herburger, Benjamin Hahn,
Fabio Estevam, NXP i.MX U-Boot Team, Patrice Chotard, Peng Fan,
Stefano Babic, Teresa Remmet, Tom Rini, Yannic Moog, Yao Zi,
upstream
I would write subject:
"board: phycore_imx8mm: add 1GB and 4GB RAM timings in SPL"
On Tue, Jun 23, 2026 at 08:24:07AM +0200, INgo Rah wrote:
>The Phytec Tauri L is sold with other memory configurations then 2 GB.
then->than
>Gather the memory size from the EEPROM of the underlaying PhyCore module
>and set timing information accordingly.
>
>Signed-off-by: INgo Rah <ingo.rah@linutronix.de>
>Reviewed-by: Benedikt Spranger <b.spranger@linutronix.de>
>Reviewed-by: Gregor Herburger <gregor.herburger@linutronix.de>
>---
With above corrected:
Reviewed-by: Peng Fan <peng.fan@nxp.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] Board phycore_imx8mm: spl: add 1 and 4GB RAM timings
2026-06-23 6:24 [PATCH] Board phycore_imx8mm: spl: add 1 and 4GB RAM timings INgo Rah
2026-06-24 8:00 ` Peng Fan
@ 2026-06-24 8:16 ` Yannic Moog
2026-06-24 9:29 ` Teresa Remmet
2 siblings, 0 replies; 4+ messages in thread
From: Yannic Moog @ 2026-06-24 8:16 UTC (permalink / raw)
To: INgo Rah, u-boot@lists.denx.de
Cc: Benedikt Spranger, Gregor Herburger, Benjamin Hahn, Fabio Estevam,
NXP i.MX U-Boot Team, Patrice Chotard, Peng Fan, Stefano Babic,
Teresa Remmet, Tom Rini, Yao Zi, upstream@lists.phytec.de
On Tue, 2026-06-23 at 08:24 +0200, INgo Rah wrote:
> The Phytec Tauri L is sold with other memory configurations then 2 GB.
> Gather the memory size from the EEPROM of the underlaying PhyCore module
> and set timing information accordingly.
I tested on phyBOARD Polis and phyGATE Tauri. U-Boot proper does not display/print correct RAM
size, SPL does.
I'd imagine board_phys_sdram_size is needed. Can you add that?
Anyway, since the SoM detection works,
Tested-by: Yannic Moog <y.moog@phytec.de>
Tested on
- 1GiB phyBOARD-Polis
- 2GiB phyGATE-Tauri-L
- 4GiB phyGATE-Tauri-L
Yannic
>
> Signed-off-by: INgo Rah <ingo.rah@linutronix.de>
> Reviewed-by: Benedikt Spranger <b.spranger@linutronix.de>
> Reviewed-by: Gregor Herburger <gregor.herburger@linutronix.de>
> ---
> Board phycore_imx8mm: spl: add 1 and 4GB RAM timings
> This patch heavily borrowed code from git://git.phytec.de/u-boot-imx commit
> 00ac765d6032 ("BSPIMX8M-2893 board: phycore_imx8mm: spl: add 1 and 4GB RAM
> timings") and commit 93962421d7b4 ("BSPIMX8M-2893 board: phycore_imx8mm:
> spl: add 1 and 4GB RAM timings")
>
> .../dts/imx8mm-phygate-tauri-l-u-boot.dtsi | 8 ++++++
> board/phytec/common/imx8m_som_detection.h | 7 ++++++
> board/phytec/phycore_imx8mm/lpddr4_timing.c | 25 +++++++++++++++++++
> board/phytec/phycore_imx8mm/lpddr4_timing.h | 12 +++++++++
> board/phytec/phycore_imx8mm/spl.c | 24 +++++++++++++++++-
> configs/imx8mm-phygate-tauri-l_defconfig | 1 +
> 6 files changed, 76 insertions(+), 1 deletion(-)
> create mode 100644 board/phytec/phycore_imx8mm/lpddr4_timing.h
>
> diff --git a/arch/arm/dts/imx8mm-phygate-tauri-l-u-boot.dtsi b/arch/arm/dts/imx8mm-phygate-tauri-
> l-u-boot.dtsi
> index 26361780c01..79ddb1ade44 100644
> --- a/arch/arm/dts/imx8mm-phygate-tauri-l-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mm-phygate-tauri-l-u-boot.dtsi
> @@ -29,6 +29,10 @@
> };
> };
>
> +&pinctrl_i2c1 {
> + bootph-pre-ram;
> +};
> +
> &pinctrl_uart3 {
> bootph-pre-ram;
> };
> @@ -73,6 +77,10 @@
> bootph-pre-ram;
> };
>
> +&i2c1 {
> + bootph-pre-ram;
> +};
> +
> &usdhc2 {
> bootph-pre-ram;
> };
> diff --git a/board/phytec/common/imx8m_som_detection.h
> b/board/phytec/common/imx8m_som_detection.h
> index 0176347414f..794d3add0d5 100644
> --- a/board/phytec/common/imx8m_som_detection.h
> +++ b/board/phytec/common/imx8m_som_detection.h
> @@ -13,6 +13,13 @@
> #define PHYTEC_IMX8MM_SOM 69
> #define PHYTEC_IMX8MP_SOM 70
>
> +enum phytec_imx8mm_ddr_eeprom_code {
> + INVALID = PHYTEC_EEPROM_INVAL,
> + PHYTEC_IMX8MM_DDR_1GB = 1,
> + PHYTEC_IMX8MM_DDR_2GB = 3,
> + PHYTEC_IMX8MM_DDR_4GB = 5,
> +};
> +
> int __maybe_unused phytec_imx8m_detect(struct phytec_eeprom_data *data);
> u8 __maybe_unused phytec_get_imx8m_ddr_size(struct phytec_eeprom_data *data);
> u8 __maybe_unused phytec_get_imx8mp_rtc(struct phytec_eeprom_data *data);
> diff --git a/board/phytec/phycore_imx8mm/lpddr4_timing.c
> b/board/phytec/phycore_imx8mm/lpddr4_timing.c
> index f5a2f3268b3..73e35c7c3b9 100644
> --- a/board/phytec/phycore_imx8mm/lpddr4_timing.c
> +++ b/board/phytec/phycore_imx8mm/lpddr4_timing.c
> @@ -1842,3 +1842,28 @@ struct dram_timing_info dram_timing = {
> .ddrphy_pie_num = ARRAY_SIZE(ddr_phy_pie),
> .fsp_table = { 3000, 400, 100,},
> };
> +
> +void set_dram_timings_1gb(void)
> +{
> + dram_timing.ddrc_cfg[5].val = 0x2d0087;
> + dram_timing.ddrc_cfg[21].val = 0x8d;
> + dram_timing.ddrc_cfg[42].val = 0xf070707;
> + dram_timing.ddrc_cfg[58].val = 0x60012;
> + dram_timing.ddrc_cfg[73].val = 0x13;
> + dram_timing.ddrc_cfg[83].val = 0x30005;
> + dram_timing.ddrc_cfg[98].val = 0x5;
> +}
> +
> +void set_dram_timings_4gb(void)
> +{
> + dram_timing.ddrc_cfg[2].val = 0xa3080020;
> + dram_timing.ddrc_cfg[37].val = 0x17;
> + dram_timing.fsp_msg[0].fsp_cfg[8].val = 0x310;
> + dram_timing.fsp_msg[0].fsp_cfg[20].val = 0x3;
> + dram_timing.fsp_msg[1].fsp_cfg[9].val = 0x310;
> + dram_timing.fsp_msg[1].fsp_cfg[21].val = 0x3;
> + dram_timing.fsp_msg[2].fsp_cfg[9].val = 0x310;
> + dram_timing.fsp_msg[2].fsp_cfg[21].val = 0x3;
> + dram_timing.fsp_msg[3].fsp_cfg[10].val = 0x310;
> + dram_timing.fsp_msg[3].fsp_cfg[22].val = 0x3;
> +}
> diff --git a/board/phytec/phycore_imx8mm/lpddr4_timing.h
> b/board/phytec/phycore_imx8mm/lpddr4_timing.h
> new file mode 100644
> index 00000000000..5334fa5b4ee
> --- /dev/null
> +++ b/board/phytec/phycore_imx8mm/lpddr4_timing.h
> @@ -0,0 +1,12 @@
> +/* SPDX-License-Identifier: GPL-2.0-or-later */
> +/*
> + * Copyright (C) 2025 PHYTEC Messtechnik GmbH
> + */
> +
> +#ifndef __LPDDR4_TIMING_H__
> +#define __LPDDR4_TIMING_H__
> +
> +void set_dram_timings_1gb(void);
> +void set_dram_timings_4gb(void);
> +
> +#endif /* __LPDDR4_TIMING_H__ */
> diff --git a/board/phytec/phycore_imx8mm/spl.c b/board/phytec/phycore_imx8mm/spl.c
> index e688793bc74..c9f2243c819 100644
> --- a/board/phytec/phycore_imx8mm/spl.c
> +++ b/board/phytec/phycore_imx8mm/spl.c
> @@ -16,6 +16,7 @@
> #include <log.h>
> #include <spl.h>
>
> +#include "lpddr4_timing.h"
> #include "../common/imx8m_som_detection.h"
>
> #define EEPROM_ADDR 0x51
> @@ -42,6 +43,7 @@ int spl_board_boot_device(enum boot_device boot_dev_spl)
> static void spl_dram_init(void)
> {
> int ret;
> + enum phytec_imx8mm_ddr_eeprom_code size = PHYTEC_EEPROM_INVAL;
>
> ret = phytec_eeprom_data_setup_fallback(NULL, 0, EEPROM_ADDR,
> EEPROM_ADDR_FALLBACK);
> @@ -49,10 +51,30 @@ static void spl_dram_init(void)
> goto out;
>
> ret = phytec_imx8m_detect(NULL);
> - if (!ret)
> + if (!ret) {
> phytec_print_som_info(NULL);
> + size = phytec_get_imx8m_ddr_size(NULL);
> + }
>
> + switch (size) {
> + case PHYTEC_IMX8MM_DDR_1GB:
> + puts("1 GiB\n");
> + set_dram_timings_1gb();
> + break;
> + case PHYTEC_IMX8MM_DDR_2GB:
> + puts("2 GiB\n");
> + break;
> + case PHYTEC_IMX8MM_DDR_4GB:
> + puts("4 GiB\n");
> + set_dram_timings_4gb();
> + break;
> + default:
> + goto out;
> + }
> + ddr_init(&dram_timing);
> + return;
> out:
> + puts("Could not detect correct RAM size. Fall back to default.\n");
> ddr_init(&dram_timing);
> }
>
> diff --git a/configs/imx8mm-phygate-tauri-l_defconfig b/configs/imx8mm-phygate-tauri-l_defconfig
> index e34f12cabf3..866e5d1dbb3 100644
> --- a/configs/imx8mm-phygate-tauri-l_defconfig
> +++ b/configs/imx8mm-phygate-tauri-l_defconfig
> @@ -4,6 +4,7 @@ CONFIG_TEXT_BASE=0x40200000
> CONFIG_SYS_MALLOC_LEN=0x2000000
> CONFIG_SPL_GPIO=y
> CONFIG_SPL_LIBCOMMON_SUPPORT=y
> +CONFIG_PHYTEC_SOM_DETECTION=y
> CONFIG_ENV_SIZE=0x10000
> CONFIG_ENV_OFFSET=0x3C0000
> CONFIG_DM_GPIO=y
> ---
> base-commit: f072620dc9ffda00b010783da27c41231c3a439b
> branch: feature/phygate
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] Board phycore_imx8mm: spl: add 1 and 4GB RAM timings
2026-06-23 6:24 [PATCH] Board phycore_imx8mm: spl: add 1 and 4GB RAM timings INgo Rah
2026-06-24 8:00 ` Peng Fan
2026-06-24 8:16 ` Yannic Moog
@ 2026-06-24 9:29 ` Teresa Remmet
2 siblings, 0 replies; 4+ messages in thread
From: Teresa Remmet @ 2026-06-24 9:29 UTC (permalink / raw)
To: u-boot@lists.denx.de, ingo.rah@linutronix.de
Cc: sbabic@nabladev.com, Benjamin Hahn, b.spranger@linutronix.de,
trini@konsulko.com, festevam@gmail.com,
gregor.herburger@linutronix.de, peng.fan@nxp.com,
uboot-imx@nxp.com, upstream@lists.phytec.de, me@ziyao.cc,
patrice.chotard@foss.st.com, Yannic Moog
Hello INgo,
Am Dienstag, dem 23.06.2026 um 08:24 +0200 schrieb INgo Rah:
> The Phytec Tauri L is sold with other memory configurations then 2
> GB.
> Gather the memory size from the EEPROM of the underlaying PhyCore
> module
> and set timing information accordingly.
>
> Signed-off-by: INgo Rah <ingo.rah@linutronix.de>
> Reviewed-by: Benedikt Spranger <b.spranger@linutronix.de>
> Reviewed-by: Gregor Herburger <gregor.herburger@linutronix.de>
> ---
> Board phycore_imx8mm: spl: add 1 and 4GB RAM timings
> This patch heavily borrowed code from git://git.phytec.de/u-boot-
> imx commit
> 00ac765d6032 ("BSPIMX8M-2893 board: phycore_imx8mm: spl: add 1 and
> 4GB RAM
> timings") and commit 93962421d7b4 ("BSPIMX8M-2893 board:
> phycore_imx8mm:
> spl: add 1 and 4GB RAM timings")
there has been meanwhile an new update of the RAM Timings.
Can you checkout downstream commits:
f5e9c4e8cb74 ("board: phytec: phycore-imx8mm: Update RAM Timings")
a2aeeb82e1d9 ("BSPIMX8M-3891: board: phytec: phycore-imx8mm: Set
correct RAM timing fallback")
>
> .../dts/imx8mm-phygate-tauri-l-u-boot.dtsi | 8 ++++++
> board/phytec/common/imx8m_som_detection.h | 7 ++++++
> board/phytec/phycore_imx8mm/lpddr4_timing.c | 25
> +++++++++++++++++++
> board/phytec/phycore_imx8mm/lpddr4_timing.h | 12 +++++++++
> board/phytec/phycore_imx8mm/spl.c | 24
> +++++++++++++++++-
> configs/imx8mm-phygate-tauri-l_defconfig | 1 +
> 6 files changed, 76 insertions(+), 1 deletion(-)
> create mode 100644 board/phytec/phycore_imx8mm/lpddr4_timing.h
>
[...]
> +#endif /* __LPDDR4_TIMING_H__ */
> diff --git a/board/phytec/phycore_imx8mm/spl.c
> b/board/phytec/phycore_imx8mm/spl.c
> index e688793bc74..c9f2243c819 100644
> --- a/board/phytec/phycore_imx8mm/spl.c
> +++ b/board/phytec/phycore_imx8mm/spl.c
> @@ -16,6 +16,7 @@
> #include <log.h>
> #include <spl.h>
>
> +#include "lpddr4_timing.h"
> #include "../common/imx8m_som_detection.h"
>
> #define EEPROM_ADDR 0x51
> @@ -42,6 +43,7 @@ int spl_board_boot_device(enum boot_device
> boot_dev_spl)
> static void spl_dram_init(void)
> {
> int ret;
> + enum phytec_imx8mm_ddr_eeprom_code size =
> PHYTEC_EEPROM_INVAL;
>
> ret = phytec_eeprom_data_setup_fallback(NULL, 0,
> EEPROM_ADDR,
> EEPROM_ADDR_FALLBACK);
> @@ -49,10 +51,30 @@ static void spl_dram_init(void)
> goto out;
>
> ret = phytec_imx8m_detect(NULL);
> - if (!ret)
> + if (!ret) {
> phytec_print_som_info(NULL);
> + size = phytec_get_imx8m_ddr_size(NULL);
> + }
>
> + switch (size) {
> + case PHYTEC_IMX8MM_DDR_1GB:
> + puts("1 GiB\n");
I would prefer if you drop the output here and below
as u-boot proper prints the size, too.
Teresa
> + set_dram_timings_1gb();
> + break;
> + case PHYTEC_IMX8MM_DDR_2GB:
> + puts("2 GiB\n");
> + break;
> + case PHYTEC_IMX8MM_DDR_4GB:
> + puts("4 GiB\n");
> + set_dram_timings_4gb();
> + break;
> + default:
> + goto out;
> + }
> + ddr_init(&dram_timing);
> + return;
> out:
> + puts("Could not detect correct RAM size. Fall back to
> default.\n");
> ddr_init(&dram_timing);
> }
>
> diff --git a/configs/imx8mm-phygate-tauri-l_defconfig
> b/configs/imx8mm-phygate-tauri-l_defconfig
> index e34f12cabf3..866e5d1dbb3 100644
> --- a/configs/imx8mm-phygate-tauri-l_defconfig
> +++ b/configs/imx8mm-phygate-tauri-l_defconfig
> @@ -4,6 +4,7 @@ CONFIG_TEXT_BASE=0x40200000
> CONFIG_SYS_MALLOC_LEN=0x2000000
> CONFIG_SPL_GPIO=y
> CONFIG_SPL_LIBCOMMON_SUPPORT=y
> +CONFIG_PHYTEC_SOM_DETECTION=y
> CONFIG_ENV_SIZE=0x10000
> CONFIG_ENV_OFFSET=0x3C0000
> CONFIG_DM_GPIO=y
> ---
> base-commit: f072620dc9ffda00b010783da27c41231c3a439b
> branch: feature/phygate
>
--
PHYTEC Messtechnik GmbH | Barcelona-Allee 1 | 55129 Mainz, Germany
Geschäftsführer: Dipl.-Ing. Michael Mitezki, Dipl.-Ing. Bodo Huber,
Dipl.-Ing. (FH) Markus Lickes | Handelsregister Mainz HRB 4656 |
Finanzamt Mainz | St.Nr. 26/665/00608, DE 149059855
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-06-24 9:29 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-23 6:24 [PATCH] Board phycore_imx8mm: spl: add 1 and 4GB RAM timings INgo Rah
2026-06-24 8:00 ` Peng Fan
2026-06-24 8:16 ` Yannic Moog
2026-06-24 9:29 ` Teresa Remmet
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox