diff options
| author | Philippe Schenker <philippe.schenker@toradex.com> | 2021-10-07 11:12:00 +0200 |
|---|---|---|
| committer | Max Krummenacher <max.krummenacher@toradex.com> | 2021-10-12 14:21:01 +0000 |
| commit | e4279843b222d170114a01eb4dca2b9f6515de61 (patch) | |
| tree | c2ef26537dc086aad0f1adeca4abd8d1ca245aac | |
| parent | 6532ee47b14a9dc3c1e9ce1ef0fcac0b516b0e99 (diff) | |
colibri-imx6ull: fix setup of nand gpmi clock
NXP used to setup the gpmi clock root from gpmi_clk in early versions in their
downstream BSP. [1]
However on mainline the gpmi clock root was always setup from enfc since the
beginning of the i.MX 6 series SoCs, which is still the same today. [2]
NXP followed the mainline approach at some point and changed setup_gpmi_io_clk
to setup gpmi clock root from enfc which left faulty code behind in our board
file. [3]
This commit follows the change of NXP as it improves the performance of the NAND
from ~1.2 MiB/s to ~12 MiB/s. [3]
This change was verified to work in recovery-mode and u-boot loaded from NAND on
all four Colibri iMX6ULL SKUs from Toradex.
The frequency used to read the NAND, measured on RE# (Read Enable):
before this patch: 1.4 MHz
after this patch: 22 MHz
in Linux Kernel: 50 MHz
[1] https://source.codeaurora.org/external/imx/uboot-imx/tree/arch/arm/cpu/armv7/mx6/clock.c?h=nxp/imx_v2016.03_4.1.15_2.0.0_ga#n62
[2] 23608e23fd65 (i.mx: add the initial support for freescale i.MX6Q processor)
[3] https://source.codeaurora.org/external/imx/uboot-imx/commit/?id=7a82a19ceabfb04bbc1591a67c99751748781c7d
Related-to: ELB-4176
Signed-off-by: Philippe Schenker <philippe.schenker@toradex.com>
| -rw-r--r-- | board/toradex/colibri-imx6ull/colibri-imx6ull.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/board/toradex/colibri-imx6ull/colibri-imx6ull.c b/board/toradex/colibri-imx6ull/colibri-imx6ull.c index 200a1447e0..192005aec2 100644 --- a/board/toradex/colibri-imx6ull/colibri-imx6ull.c +++ b/board/toradex/colibri-imx6ull/colibri-imx6ull.c @@ -60,8 +60,9 @@ int dram_init(void) #ifdef CONFIG_NAND_MXS static void setup_gpmi_nand(void) { - setup_gpmi_io_clk((3 << MXC_CCM_CSCDR1_BCH_PODF_OFFSET) | - (3 << MXC_CCM_CSCDR1_GPMI_PODF_OFFSET)); + setup_gpmi_io_clk((MXC_CCM_CS2CDR_ENFC_CLK_PODF(0) | + MXC_CCM_CS2CDR_ENFC_CLK_PRED(3) | + MXC_CCM_CS2CDR_ENFC_CLK_SEL(3))); } #endif /* CONFIG_NAND_MXS */ |