* [PATCH 1/3] stm32mp1: ram: add read valid training support
@ 2021-11-15 14:32 Patrick Delaunay
2021-11-15 14:32 ` [PATCH 2/3] stm32mp1: ram: remove the support of calibration result Patrick Delaunay
` (3 more replies)
0 siblings, 4 replies; 7+ messages in thread
From: Patrick Delaunay @ 2021-11-15 14:32 UTC (permalink / raw)
To: u-boot; +Cc: Patrick Delaunay, Patrice Chotard, U-Boot STM32
Add the read data eye training = training for optimal read valid placement
(RVTRN) when the built-in calibration is executed for LPDDR2 and LPDDR3.
This training is supported on the PUBL integrated in the STM32MP15x
DDR subsystem and it is not required for DDR3.
Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
---
drivers/ram/stm32mp1/stm32mp1_ddr.c | 8 ++++++--
drivers/ram/stm32mp1/stm32mp1_ddr_regs.h | 1 +
2 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr.c b/drivers/ram/stm32mp1/stm32mp1_ddr.c
index 0457166b12..1f8422518b 100644
--- a/drivers/ram/stm32mp1/stm32mp1_ddr.c
+++ b/drivers/ram/stm32mp1/stm32mp1_ddr.c
@@ -826,8 +826,12 @@ start:
*/
/* 10. configure PUBL PIR register to specify which training step to run */
- /* warning : RVTRN is not supported by this PUBL */
- stm32mp1_ddrphy_init(priv->phy, DDRPHYC_PIR_QSTRN);
+ /* RVTRN is excuted only on LPDDR2/LPDDR3 */
+ if (config->c_reg.mstr & DDRCTRL_MSTR_DDR3)
+ pir = DDRPHYC_PIR_QSTRN;
+ else
+ pir = DDRPHYC_PIR_QSTRN | DDRPHYC_PIR_RVTRN;
+ stm32mp1_ddrphy_init(priv->phy, pir);
/* 11. monitor PUB PGSR.IDONE to poll cpmpletion of training sequence */
ddrphy_idone_wait(priv->phy);
diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h b/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h
index 3c8885a965..ada3087328 100644
--- a/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h
+++ b/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h
@@ -309,6 +309,7 @@ struct stm32mp1_ddrphy {
#define DDRPHYC_PIR_DRAMRST BIT(5)
#define DDRPHYC_PIR_DRAMINIT BIT(6)
#define DDRPHYC_PIR_QSTRN BIT(7)
+#define DDRPHYC_PIR_RVTRN BIT(8)
#define DDRPHYC_PIR_ICPC BIT(16)
#define DDRPHYC_PIR_ZCALBYP BIT(30)
#define DDRPHYC_PIR_INITSTEPS_MASK GENMASK(31, 7)
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/3] stm32mp1: ram: remove the support of calibration result
2021-11-15 14:32 [PATCH 1/3] stm32mp1: ram: add read valid training support Patrick Delaunay
@ 2021-11-15 14:32 ` Patrick Delaunay
2021-11-22 7:49 ` Patrice CHOTARD
2021-11-15 14:32 ` [PATCH 3/3] stm32mp1: ram: remove tuning support Patrick Delaunay
` (2 subsequent siblings)
3 siblings, 1 reply; 7+ messages in thread
From: Patrick Delaunay @ 2021-11-15 14:32 UTC (permalink / raw)
To: u-boot
Cc: Patrick Delaunay, Jagan Teki, Matteo Lisi, Patrice Chotard, U-Boot STM32
The support of a predefined DDR PHY tuning result is removed for
STM32MP1 driver because it is not needed at the supported frequency
when built-in calibration is executed.
The calibration parameters were provided in the device tree by the
optional node "st,phy-cal", activated in ddr helper file by the
compilation flag DDR_PHY_CAL_SKIP and filled with values generated
by the CubeMX DDR utilities.
This patch
- updates the binding file to remove "st,phy-cal" support
- updates the device trees and remove the associated defines
- simplifies the STM32MP1 DDR driver and remove the support of
the optional parameter "st,phy-cal"
After this patch, the built-in calibration is always executed
and the calibration registers are moved in the phy dynamic part;
that allows manual tests.
Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
---
arch/arm/dts/stm32mp15-ddr.dtsi | 30 ---------
.../dts/stm32mp15-ddr3-1x4Gb-1066-binG.dtsi | 12 ----
.../dts/stm32mp15-ddr3-2x4Gb-1066-binG.dtsi | 12 ----
.../stm32mp15-ddr3-dhsom-2x1Gb-1066-binG.dtsi | 12 ----
.../stm32mp15-ddr3-dhsom-2x2Gb-1066-binG.dtsi | 12 ----
.../stm32mp15-ddr3-dhsom-2x4Gb-1066-binG.dtsi | 12 ----
.../stm32mp15-ddr3-icore-1x4Gb-1066-binG.dtsi | 12 ----
.../memory-controllers/st,stm32mp1-ddr.txt | 32 ---------
drivers/ram/stm32mp1/stm32mp1_ddr.c | 65 +++++++------------
drivers/ram/stm32mp1/stm32mp1_ddr.h | 17 -----
drivers/ram/stm32mp1/stm32mp1_interactive.c | 15 +----
drivers/ram/stm32mp1/stm32mp1_ram.c | 22 +------
12 files changed, 27 insertions(+), 226 deletions(-)
diff --git a/arch/arm/dts/stm32mp15-ddr.dtsi b/arch/arm/dts/stm32mp15-ddr.dtsi
index 2a139c54e9..0aac9131a6 100644
--- a/arch/arm/dts/stm32mp15-ddr.dtsi
+++ b/arch/arm/dts/stm32mp15-ddr.dtsi
@@ -116,24 +116,6 @@
DDR_MR3
>;
-#ifdef DDR_PHY_CAL_SKIP
- st,phy-cal = <
- DDR_DX0DLLCR
- DDR_DX0DQTR
- DDR_DX0DQSTR
- DDR_DX1DLLCR
- DDR_DX1DQTR
- DDR_DX1DQSTR
- DDR_DX2DLLCR
- DDR_DX2DQTR
- DDR_DX2DQSTR
- DDR_DX3DLLCR
- DDR_DX3DQTR
- DDR_DX3DQSTR
- >;
-
-#endif
-
status = "okay";
};
};
@@ -224,18 +206,6 @@
#undef DDR_ODTCR
#undef DDR_ZQ0CR1
#undef DDR_DX0GCR
-#undef DDR_DX0DLLCR
-#undef DDR_DX0DQTR
-#undef DDR_DX0DQSTR
#undef DDR_DX1GCR
-#undef DDR_DX1DLLCR
-#undef DDR_DX1DQTR
-#undef DDR_DX1DQSTR
#undef DDR_DX2GCR
-#undef DDR_DX2DLLCR
-#undef DDR_DX2DQTR
-#undef DDR_DX2DQSTR
#undef DDR_DX3GCR
-#undef DDR_DX3DLLCR
-#undef DDR_DX3DQTR
-#undef DDR_DX3DQSTR
diff --git a/arch/arm/dts/stm32mp15-ddr3-1x4Gb-1066-binG.dtsi b/arch/arm/dts/stm32mp15-ddr3-1x4Gb-1066-binG.dtsi
index 978331b279..e60d0ae606 100644
--- a/arch/arm/dts/stm32mp15-ddr3-1x4Gb-1066-binG.dtsi
+++ b/arch/arm/dts/stm32mp15-ddr3-1x4Gb-1066-binG.dtsi
@@ -100,20 +100,8 @@
#define DDR_ODTCR 0x00010000
#define DDR_ZQ0CR1 0x00000038
#define DDR_DX0GCR 0x0000CE81
-#define DDR_DX0DLLCR 0x40000000
-#define DDR_DX0DQTR 0xFFFFFFFF
-#define DDR_DX0DQSTR 0x3DB02000
#define DDR_DX1GCR 0x0000CE81
-#define DDR_DX1DLLCR 0x40000000
-#define DDR_DX1DQTR 0xFFFFFFFF
-#define DDR_DX1DQSTR 0x3DB02000
#define DDR_DX2GCR 0x0000CE80
-#define DDR_DX2DLLCR 0x40000000
-#define DDR_DX2DQTR 0xFFFFFFFF
-#define DDR_DX2DQSTR 0x3DB02000
#define DDR_DX3GCR 0x0000CE80
-#define DDR_DX3DLLCR 0x40000000
-#define DDR_DX3DQTR 0xFFFFFFFF
-#define DDR_DX3DQSTR 0x3DB02000
#include "stm32mp15-ddr.dtsi"
diff --git a/arch/arm/dts/stm32mp15-ddr3-2x4Gb-1066-binG.dtsi b/arch/arm/dts/stm32mp15-ddr3-2x4Gb-1066-binG.dtsi
index 426be21f42..1a6fa80edf 100644
--- a/arch/arm/dts/stm32mp15-ddr3-2x4Gb-1066-binG.dtsi
+++ b/arch/arm/dts/stm32mp15-ddr3-2x4Gb-1066-binG.dtsi
@@ -100,20 +100,8 @@
#define DDR_ODTCR 0x00010000
#define DDR_ZQ0CR1 0x00000038
#define DDR_DX0GCR 0x0000CE81
-#define DDR_DX0DLLCR 0x40000000
-#define DDR_DX0DQTR 0xFFFFFFFF
-#define DDR_DX0DQSTR 0x3DB02000
#define DDR_DX1GCR 0x0000CE81
-#define DDR_DX1DLLCR 0x40000000
-#define DDR_DX1DQTR 0xFFFFFFFF
-#define DDR_DX1DQSTR 0x3DB02000
#define DDR_DX2GCR 0x0000CE81
-#define DDR_DX2DLLCR 0x40000000
-#define DDR_DX2DQTR 0xFFFFFFFF
-#define DDR_DX2DQSTR 0x3DB02000
#define DDR_DX3GCR 0x0000CE81
-#define DDR_DX3DLLCR 0x40000000
-#define DDR_DX3DQTR 0xFFFFFFFF
-#define DDR_DX3DQSTR 0x3DB02000
#include "stm32mp15-ddr.dtsi"
diff --git a/arch/arm/dts/stm32mp15-ddr3-dhsom-2x1Gb-1066-binG.dtsi b/arch/arm/dts/stm32mp15-ddr3-dhsom-2x1Gb-1066-binG.dtsi
index b3eb280f96..0a277cd675 100644
--- a/arch/arm/dts/stm32mp15-ddr3-dhsom-2x1Gb-1066-binG.dtsi
+++ b/arch/arm/dts/stm32mp15-ddr3-dhsom-2x1Gb-1066-binG.dtsi
@@ -101,20 +101,8 @@
#define DDR_ODTCR 0x00010000
#define DDR_ZQ0CR1 0x00000038
#define DDR_DX0GCR 0x0000CE81
-#define DDR_DX0DLLCR 0x40000000
-#define DDR_DX0DQTR 0xFFFFFFFF
-#define DDR_DX0DQSTR 0x3DB02000
#define DDR_DX1GCR 0x0000CE81
-#define DDR_DX1DLLCR 0x40000000
-#define DDR_DX1DQTR 0xFFFFFFFF
-#define DDR_DX1DQSTR 0x3DB02000
#define DDR_DX2GCR 0x0000CE81
-#define DDR_DX2DLLCR 0x40000000
-#define DDR_DX2DQTR 0xFFFFFFFF
-#define DDR_DX2DQSTR 0x3DB02000
#define DDR_DX3GCR 0x0000CE81
-#define DDR_DX3DLLCR 0x40000000
-#define DDR_DX3DQTR 0xFFFFFFFF
-#define DDR_DX3DQSTR 0x3DB02000
#include "stm32mp15-ddr.dtsi"
diff --git a/arch/arm/dts/stm32mp15-ddr3-dhsom-2x2Gb-1066-binG.dtsi b/arch/arm/dts/stm32mp15-ddr3-dhsom-2x2Gb-1066-binG.dtsi
index ed3a5248f8..92774fffb9 100644
--- a/arch/arm/dts/stm32mp15-ddr3-dhsom-2x2Gb-1066-binG.dtsi
+++ b/arch/arm/dts/stm32mp15-ddr3-dhsom-2x2Gb-1066-binG.dtsi
@@ -101,20 +101,8 @@
#define DDR_ODTCR 0x00010000
#define DDR_ZQ0CR1 0x00000038
#define DDR_DX0GCR 0x0000CE81
-#define DDR_DX0DLLCR 0x40000000
-#define DDR_DX0DQTR 0xFFFFFFFF
-#define DDR_DX0DQSTR 0x3DB02000
#define DDR_DX1GCR 0x0000CE81
-#define DDR_DX1DLLCR 0x40000000
-#define DDR_DX1DQTR 0xFFFFFFFF
-#define DDR_DX1DQSTR 0x3DB02000
#define DDR_DX2GCR 0x0000CE81
-#define DDR_DX2DLLCR 0x40000000
-#define DDR_DX2DQTR 0xFFFFFFFF
-#define DDR_DX2DQSTR 0x3DB02000
#define DDR_DX3GCR 0x0000CE81
-#define DDR_DX3DLLCR 0x40000000
-#define DDR_DX3DQTR 0xFFFFFFFF
-#define DDR_DX3DQSTR 0x3DB02000
#include "stm32mp15-ddr.dtsi"
diff --git a/arch/arm/dts/stm32mp15-ddr3-dhsom-2x4Gb-1066-binG.dtsi b/arch/arm/dts/stm32mp15-ddr3-dhsom-2x4Gb-1066-binG.dtsi
index d5813d64b0..e53ab18a69 100644
--- a/arch/arm/dts/stm32mp15-ddr3-dhsom-2x4Gb-1066-binG.dtsi
+++ b/arch/arm/dts/stm32mp15-ddr3-dhsom-2x4Gb-1066-binG.dtsi
@@ -101,20 +101,8 @@
#define DDR_ODTCR 0x00010000
#define DDR_ZQ0CR1 0x00000038
#define DDR_DX0GCR 0x0000CE81
-#define DDR_DX0DLLCR 0x40000000
-#define DDR_DX0DQTR 0xFFFFFFFF
-#define DDR_DX0DQSTR 0x3DB02000
#define DDR_DX1GCR 0x0000CE81
-#define DDR_DX1DLLCR 0x40000000
-#define DDR_DX1DQTR 0xFFFFFFFF
-#define DDR_DX1DQSTR 0x3DB02000
#define DDR_DX2GCR 0x0000CE81
-#define DDR_DX2DLLCR 0x40000000
-#define DDR_DX2DQTR 0xFFFFFFFF
-#define DDR_DX2DQSTR 0x3DB02000
#define DDR_DX3GCR 0x0000CE81
-#define DDR_DX3DLLCR 0x40000000
-#define DDR_DX3DQTR 0xFFFFFFFF
-#define DDR_DX3DQSTR 0x3DB02000
#include "stm32mp15-ddr.dtsi"
diff --git a/arch/arm/dts/stm32mp15-ddr3-icore-1x4Gb-1066-binG.dtsi b/arch/arm/dts/stm32mp15-ddr3-icore-1x4Gb-1066-binG.dtsi
index 24c81269b0..ff582ac6af 100644
--- a/arch/arm/dts/stm32mp15-ddr3-icore-1x4Gb-1066-binG.dtsi
+++ b/arch/arm/dts/stm32mp15-ddr3-icore-1x4Gb-1066-binG.dtsi
@@ -100,20 +100,8 @@
#define DDR_ODTCR 0x00010000
#define DDR_ZQ0CR1 0x00000038
#define DDR_DX0GCR 0x0000CE81
-#define DDR_DX0DLLCR 0x40000000
-#define DDR_DX0DQTR 0xFFFFFFFF
-#define DDR_DX0DQSTR 0x3DB02000
#define DDR_DX1GCR 0x0000CE81
-#define DDR_DX1DLLCR 0x40000000
-#define DDR_DX1DQTR 0xFFFFFFFF
-#define DDR_DX1DQSTR 0x3DB02000
#define DDR_DX2GCR 0x0000CE81
-#define DDR_DX2DLLCR 0x40000000
-#define DDR_DX2DQTR 0xFFFFFFFF
-#define DDR_DX2DQSTR 0x3DB02000
#define DDR_DX3GCR 0x0000CE81
-#define DDR_DX3DLLCR 0x40000000
-#define DDR_DX3DQTR 0xFFFFFFFF
-#define DDR_DX3DQSTR 0x3DB02000
#include "stm32mp15-ddr.dtsi"
diff --git a/doc/device-tree-bindings/memory-controllers/st,stm32mp1-ddr.txt b/doc/device-tree-bindings/memory-controllers/st,stm32mp1-ddr.txt
index ac6a7df432..926e3e83b3 100644
--- a/doc/device-tree-bindings/memory-controllers/st,stm32mp1-ddr.txt
+++ b/doc/device-tree-bindings/memory-controllers/st,stm32mp1-ddr.txt
@@ -128,23 +128,6 @@ phyc attributes:
MR2
MR3
-- st,phy-cal : phy cal depending of calibration or tuning of DDR
- This parameter is optional; when it is absent the built-in PHY
- calibration is done.
- for STM32MP15x: 12 values are requested in this order
- DX0DLLCR
- DX0DQTR
- DX0DQSTR
- DX1DLLCR
- DX1DQTR
- DX1DQSTR
- DX2DLLCR
- DX2DQTR
- DX2DQSTR
- DX3DLLCR
- DX3DQTR
- DX3DQSTR
-
Example:
/ {
@@ -280,21 +263,6 @@ Example:
0x00000000 /*MR3*/
>;
- st,phy-cal = <
- 0x40000000 /*DX0DLLCR*/
- 0xFFFFFFFF /*DX0DQTR*/
- 0x3DB02000 /*DX0DQSTR*/
- 0x40000000 /*DX1DLLCR*/
- 0xFFFFFFFF /*DX1DQTR*/
- 0x3DB02000 /*DX1DQSTR*/
- 0x40000000 /*DX2DLLCR*/
- 0xFFFFFFFF /*DX2DQTR*/
- 0x3DB02000 /*DX2DQSTR*/
- 0x40000000 /*DX3DLLCR*/
- 0xFFFFFFFF /*DX3DQTR*/
- 0x3DB02000 /*DX3DQSTR*/
- >;
-
status = "okay";
};
};
diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr.c b/drivers/ram/stm32mp1/stm32mp1_ddr.c
index 1f8422518b..9d086601a4 100644
--- a/drivers/ram/stm32mp1/stm32mp1_ddr.c
+++ b/drivers/ram/stm32mp1/stm32mp1_ddr.c
@@ -68,7 +68,6 @@ struct reg_desc {
#define DDRPHY_REG_REG_SIZE 11 /* st,phy-reg */
#define DDRPHY_REG_TIMING_SIZE 10 /* st,phy-timing */
-#define DDRPHY_REG_CAL_SIZE 12 /* st,phy-cal */
#define DDRCTL_REG_REG(x) DDRCTL_REG(x, stm32mp1_ddrctrl_reg)
static const struct reg_desc ddr_reg[DDRCTL_REG_REG_SIZE] = {
@@ -178,22 +177,6 @@ static const struct reg_desc ddrphy_timing[DDRPHY_REG_TIMING_SIZE] = {
DDRPHY_REG_TIMING(mr3),
};
-#define DDRPHY_REG_CAL(x) DDRPHY_REG(x, stm32mp1_ddrphy_cal)
-static const struct reg_desc ddrphy_cal[DDRPHY_REG_CAL_SIZE] = {
- DDRPHY_REG_CAL(dx0dllcr),
- DDRPHY_REG_CAL(dx0dqtr),
- DDRPHY_REG_CAL(dx0dqstr),
- DDRPHY_REG_CAL(dx1dllcr),
- DDRPHY_REG_CAL(dx1dqtr),
- DDRPHY_REG_CAL(dx1dqstr),
- DDRPHY_REG_CAL(dx2dllcr),
- DDRPHY_REG_CAL(dx2dqtr),
- DDRPHY_REG_CAL(dx2dqstr),
- DDRPHY_REG_CAL(dx3dllcr),
- DDRPHY_REG_CAL(dx3dqtr),
- DDRPHY_REG_CAL(dx3dqstr),
-};
-
/**************************************************************
* DYNAMIC REGISTERS: only used for debug purpose (read/modify)
**************************************************************/
@@ -218,12 +201,24 @@ static const struct reg_desc ddrphy_dyn[] = {
DDRPHY_REG_DYN(zq0sr1),
DDRPHY_REG_DYN(dx0gsr0),
DDRPHY_REG_DYN(dx0gsr1),
+ DDRPHY_REG_DYN(dx0dllcr),
+ DDRPHY_REG_DYN(dx0dqtr),
+ DDRPHY_REG_DYN(dx0dqstr),
DDRPHY_REG_DYN(dx1gsr0),
DDRPHY_REG_DYN(dx1gsr1),
+ DDRPHY_REG_DYN(dx1dllcr),
+ DDRPHY_REG_DYN(dx1dqtr),
+ DDRPHY_REG_DYN(dx1dqstr),
DDRPHY_REG_DYN(dx2gsr0),
DDRPHY_REG_DYN(dx2gsr1),
+ DDRPHY_REG_DYN(dx2dllcr),
+ DDRPHY_REG_DYN(dx2dqtr),
+ DDRPHY_REG_DYN(dx2dqstr),
DDRPHY_REG_DYN(dx3gsr0),
DDRPHY_REG_DYN(dx3gsr1),
+ DDRPHY_REG_DYN(dx3dllcr),
+ DDRPHY_REG_DYN(dx3dqtr),
+ DDRPHY_REG_DYN(dx3dqstr),
};
#define DDRPHY_REG_DYN_SIZE ARRAY_SIZE(ddrphy_dyn)
@@ -240,7 +235,6 @@ enum reg_type {
REG_MAP,
REGPHY_REG,
REGPHY_TIMING,
- REGPHY_CAL,
#ifdef CONFIG_STM32MP1_DDR_INTERACTIVE
/* dynamic registers => managed in driver or not changed,
* can be dumped in interactive mode
@@ -264,8 +258,6 @@ struct ddr_reg_info {
enum base_type base;
};
-#define DDRPHY_REG_CAL(x) DDRPHY_REG(x, stm32mp1_ddrphy_cal)
-
const struct ddr_reg_info ddr_registers[REG_TYPE_NB] = {
[REG_REG] = {
"static", ddr_reg, DDRCTL_REG_REG_SIZE, DDR_BASE},
@@ -279,8 +271,6 @@ const struct ddr_reg_info ddr_registers[REG_TYPE_NB] = {
"static", ddrphy_reg, DDRPHY_REG_REG_SIZE, DDRPHY_BASE},
[REGPHY_TIMING] = {
"timing", ddrphy_timing, DDRPHY_REG_TIMING_SIZE, DDRPHY_BASE},
-[REGPHY_CAL] = {
- "cal", ddrphy_cal, DDRPHY_REG_CAL_SIZE, DDRPHY_BASE},
#ifdef CONFIG_STM32MP1_DDR_INTERACTIVE
[REG_DYN] = {
"dyn", ddr_dyn, DDR_REG_DYN_SIZE, DDR_BASE},
@@ -456,9 +446,6 @@ static u32 get_par_addr(const struct stm32mp1_ddr_config *config,
case REGPHY_TIMING:
par_addr = (u32)&config->p_timing;
break;
- case REGPHY_CAL:
- par_addr = (u32)&config->p_cal;
- break;
case REG_DYN:
case REGPHY_DYN:
case REG_TYPE_NB:
@@ -774,8 +761,6 @@ start:
*/
set_reg(priv, REGPHY_REG, &config->p_reg);
set_reg(priv, REGPHY_TIMING, &config->p_timing);
- if (config->p_cal_present)
- set_reg(priv, REGPHY_CAL, &config->p_cal);
if (INTERACTIVE(STEP_PHY_INIT))
goto start;
@@ -810,36 +795,32 @@ start:
wait_operating_mode(priv, DDRCTRL_STAT_OPERATING_MODE_NORMAL);
- if (config->p_cal_present) {
- log_debug("DDR DQS training skipped.\n");
- } else {
- log_debug("DDR DQS training : ");
+ log_debug("DDR DQS training : ");
/* 8. Disable Auto refresh and power down by setting
* - RFSHCTL3.dis_au_refresh = 1
* - PWRCTL.powerdown_en = 0
* - DFIMISC.dfiinit_complete_en = 0
*/
- stm32mp1_refresh_disable(priv->ctl);
+ stm32mp1_refresh_disable(priv->ctl);
/* 9. Program PUBL PGCR to enable refresh during training and rank to train
* not done => keep the programed value in PGCR
*/
/* 10. configure PUBL PIR register to specify which training step to run */
- /* RVTRN is excuted only on LPDDR2/LPDDR3 */
- if (config->c_reg.mstr & DDRCTRL_MSTR_DDR3)
- pir = DDRPHYC_PIR_QSTRN;
- else
- pir = DDRPHYC_PIR_QSTRN | DDRPHYC_PIR_RVTRN;
- stm32mp1_ddrphy_init(priv->phy, pir);
+ /* RVTRN is excuted only on LPDDR2/LPDDR3 */
+ if (config->c_reg.mstr & DDRCTRL_MSTR_DDR3)
+ pir = DDRPHYC_PIR_QSTRN;
+ else
+ pir = DDRPHYC_PIR_QSTRN | DDRPHYC_PIR_RVTRN;
+ stm32mp1_ddrphy_init(priv->phy, pir);
/* 11. monitor PUB PGSR.IDONE to poll cpmpletion of training sequence */
- ddrphy_idone_wait(priv->phy);
+ ddrphy_idone_wait(priv->phy);
/* 12. set back registers in step 8 to the orginal values if desidered */
- stm32mp1_refresh_restore(priv->ctl, config->c_reg.rfshctl3,
- config->c_reg.pwrctl);
- } /* if (config->p_cal_present) */
+ stm32mp1_refresh_restore(priv->ctl, config->c_reg.rfshctl3,
+ config->c_reg.pwrctl);
/* enable uMCTL2 AXI port 0 and 1 */
setbits_le32(&priv->ctl->pctrl_0, DDRCTRL_PCTRL_N_PORT_EN);
diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr.h b/drivers/ram/stm32mp1/stm32mp1_ddr.h
index 4998f04439..3bfcb85a8f 100644
--- a/drivers/ram/stm32mp1/stm32mp1_ddr.h
+++ b/drivers/ram/stm32mp1/stm32mp1_ddr.h
@@ -140,21 +140,6 @@ struct stm32mp1_ddrphy_timing {
u32 mr3;
};
-struct stm32mp1_ddrphy_cal {
- u32 dx0dllcr;
- u32 dx0dqtr;
- u32 dx0dqstr;
- u32 dx1dllcr;
- u32 dx1dqtr;
- u32 dx1dqstr;
- u32 dx2dllcr;
- u32 dx2dqtr;
- u32 dx2dqstr;
- u32 dx3dllcr;
- u32 dx3dqtr;
- u32 dx3dqstr;
-};
-
struct stm32mp1_ddr_info {
const char *name;
u32 speed; /* in kHZ */
@@ -169,8 +154,6 @@ struct stm32mp1_ddr_config {
struct stm32mp1_ddrctrl_perf c_perf;
struct stm32mp1_ddrphy_reg p_reg;
struct stm32mp1_ddrphy_timing p_timing;
- struct stm32mp1_ddrphy_cal p_cal;
- bool p_cal_present;
};
int stm32mp1_ddr_clk_enable(struct ddr_info *priv, u32 mem_speed);
diff --git a/drivers/ram/stm32mp1/stm32mp1_interactive.c b/drivers/ram/stm32mp1/stm32mp1_interactive.c
index 8c2310ac90..a667d49cff 100644
--- a/drivers/ram/stm32mp1/stm32mp1_interactive.c
+++ b/drivers/ram/stm32mp1/stm32mp1_interactive.c
@@ -111,7 +111,7 @@ static void stm32mp1_do_usage(void)
"help displays help\n"
"info displays DDR information\n"
"info <param> <val> changes DDR information\n"
- " with <param> = step, name, size, speed or cal\n"
+ " with <param> = step, name, size or speed\n"
"freq displays the DDR PHY frequency in kHz\n"
"freq <freq> changes the DDR PHY frequency\n"
"param [type|reg] prints input parameters\n"
@@ -132,7 +132,7 @@ static void stm32mp1_do_usage(void)
"\nwith for [type|reg]:\n"
" all registers if absent\n"
" <type> = ctl, phy\n"
- " or one category (static, timing, map, perf, cal, dyn)\n"
+ " or one category (static, timing, map, perf, dyn)\n"
" <reg> = name of the register\n"
};
@@ -165,7 +165,6 @@ static void stm32mp1_do_info(struct ddr_info *priv,
printf("name = %s\n", config->info.name);
printf("size = 0x%x\n", config->info.size);
printf("speed = %d kHz\n", config->info.speed);
- printf("cal = %d\n", config->p_cal_present);
return;
}
@@ -214,16 +213,6 @@ static void stm32mp1_do_info(struct ddr_info *priv,
}
return;
}
- if (!strcmp(argv[1], "cal")) {
- if (strict_strtoul(argv[2], 10, &value) < 0 ||
- (value != 0 && value != 1)) {
- printf("invalid value %s\n", argv[2]);
- } else {
- config->p_cal_present = value;
- printf("cal = %d\n", config->p_cal_present);
- }
- return;
- }
printf("argument %s invalid\n", argv[1]);
}
diff --git a/drivers/ram/stm32mp1/stm32mp1_ram.c b/drivers/ram/stm32mp1/stm32mp1_ram.c
index 98fa1f4f11..3b65269b98 100644
--- a/drivers/ram/stm32mp1/stm32mp1_ram.c
+++ b/drivers/ram/stm32mp1/stm32mp1_ram.c
@@ -95,26 +95,22 @@ static __maybe_unused int stm32mp1_ddr_setup(struct udevice *dev)
{ .name = x, \
.offset = offsetof(struct stm32mp1_ddr_config, y), \
.size = sizeof(config.y) / sizeof(u32), \
- .present = z, \
}
#define CTL_PARAM(x) PARAM("st,ctl-"#x, c_##x, NULL)
#define PHY_PARAM(x) PARAM("st,phy-"#x, p_##x, NULL)
-#define PHY_PARAM_OPT(x) PARAM("st,phy-"#x, p_##x, &config.p_##x##_present)
const struct {
const char *name; /* name in DT */
const u32 offset; /* offset in config struct */
const u32 size; /* size of parameters */
- bool * const present; /* presence indication for opt */
} param[] = {
CTL_PARAM(reg),
CTL_PARAM(timing),
CTL_PARAM(map),
CTL_PARAM(perf),
PHY_PARAM(reg),
- PHY_PARAM(timing),
- PHY_PARAM_OPT(cal)
+ PHY_PARAM(timing)
};
config.info.speed = ofnode_read_u32_default(node, "st,mem-speed", 0);
@@ -133,25 +129,11 @@ static __maybe_unused int stm32mp1_ddr_setup(struct udevice *dev)
param[idx].size);
dev_dbg(dev, "%s: %s[0x%x] = %d\n", __func__,
param[idx].name, param[idx].size, ret);
- if (ret &&
- (ret != -FDT_ERR_NOTFOUND || !param[idx].present)) {
+ if (ret) {
dev_err(dev, "Cannot read %s, error=%d\n",
param[idx].name, ret);
return -EINVAL;
}
- if (param[idx].present) {
- /* save presence of optional parameters */
- *param[idx].present = true;
- if (ret == -FDT_ERR_NOTFOUND) {
- *param[idx].present = false;
-#ifdef CONFIG_STM32MP1_DDR_INTERACTIVE
- /* reset values if used later */
- memset((void *)((u32)&config +
- param[idx].offset),
- 0, param[idx].size * sizeof(u32));
-#endif
- }
- }
}
ret = clk_get_by_name(dev, "axidcg", &axidcg);
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 3/3] stm32mp1: ram: remove tuning support
2021-11-15 14:32 [PATCH 1/3] stm32mp1: ram: add read valid training support Patrick Delaunay
2021-11-15 14:32 ` [PATCH 2/3] stm32mp1: ram: remove the support of calibration result Patrick Delaunay
@ 2021-11-15 14:32 ` Patrick Delaunay
2021-11-22 7:51 ` Patrice CHOTARD
2021-11-22 7:48 ` [PATCH 1/3] stm32mp1: ram: add read valid training support Patrice CHOTARD
2021-11-30 10:36 ` Patrice CHOTARD
3 siblings, 1 reply; 7+ messages in thread
From: Patrick Delaunay @ 2021-11-15 14:32 UTC (permalink / raw)
To: u-boot; +Cc: Patrick Delaunay, Patrice Chotard, U-Boot STM32
Remove the DDR interactive command tuning, as the support of a predefined
DDR PHY tuning is removed for STM32MP1 driver in SPL and in TF-A
and the result of this tuning will be never used.
Moreover this SW tuning procedure can failed on some hardware
configuration (to many BIST errors and no convergence); it will be no
more supported in the next delivery of the DDR utilities included in
the CubeMX tool of STMicroelectronics.
Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
---
drivers/ram/stm32mp1/Makefile | 1 -
drivers/ram/stm32mp1/stm32mp1_ddr.c | 8 +-
drivers/ram/stm32mp1/stm32mp1_ddr.h | 5 -
drivers/ram/stm32mp1/stm32mp1_ddr_regs.h | 64 +-
drivers/ram/stm32mp1/stm32mp1_interactive.c | 22 +-
drivers/ram/stm32mp1/stm32mp1_tests.h | 3 -
drivers/ram/stm32mp1/stm32mp1_tuning.c | 1540 -------------------
7 files changed, 7 insertions(+), 1636 deletions(-)
delete mode 100644 drivers/ram/stm32mp1/stm32mp1_tuning.c
diff --git a/drivers/ram/stm32mp1/Makefile b/drivers/ram/stm32mp1/Makefile
index e1e9135603..71ded6bed4 100644
--- a/drivers/ram/stm32mp1/Makefile
+++ b/drivers/ram/stm32mp1/Makefile
@@ -8,7 +8,6 @@ obj-y += stm32mp1_ddr.o
obj-$(CONFIG_STM32MP1_DDR_INTERACTIVE) += stm32mp1_interactive.o
obj-$(CONFIG_STM32MP1_DDR_TESTS) += stm32mp1_tests.o
-obj-$(CONFIG_STM32MP1_DDR_TUNING) += stm32mp1_tuning.o
ifneq ($(DDR_INTERACTIVE),)
CFLAGS_stm32mp1_interactive.o += -DCONFIG_STM32MP1_DDR_INTERACTIVE_FORCE=y
diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr.c b/drivers/ram/stm32mp1/stm32mp1_ddr.c
index 9d086601a4..4d78aa5cb1 100644
--- a/drivers/ram/stm32mp1/stm32mp1_ddr.c
+++ b/drivers/ram/stm32mp1/stm32mp1_ddr.c
@@ -557,7 +557,7 @@ static void ddrphy_idone_wait(struct stm32mp1_ddrphy *phy)
(u32)&phy->pgsr, pgsr, ret);
}
-void stm32mp1_ddrphy_init(struct stm32mp1_ddrphy *phy, u32 pir)
+static void stm32mp1_ddrphy_init(struct stm32mp1_ddrphy *phy, u32 pir)
{
pir |= DDRPHYC_PIR_INIT;
writel(pir, &phy->pir);
@@ -626,7 +626,7 @@ static void wait_operating_mode(struct ddr_info *priv, int mode)
log_debug("[0x%08x] stat = 0x%08x\n", (u32)&priv->ctl->stat, stat);
}
-void stm32mp1_refresh_disable(struct stm32mp1_ddrctl *ctl)
+static void stm32mp1_refresh_disable(struct stm32mp1_ddrctl *ctl)
{
start_sw_done(ctl);
/* quasi-dynamic register update*/
@@ -637,8 +637,8 @@ void stm32mp1_refresh_disable(struct stm32mp1_ddrctl *ctl)
wait_sw_done_ack(ctl);
}
-void stm32mp1_refresh_restore(struct stm32mp1_ddrctl *ctl,
- u32 rfshctl3, u32 pwrctl)
+static void stm32mp1_refresh_restore(struct stm32mp1_ddrctl *ctl,
+ u32 rfshctl3, u32 pwrctl)
{
start_sw_done(ctl);
if (!(rfshctl3 & DDRCTRL_RFSHCTL3_DIS_AUTO_REFRESH))
diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr.h b/drivers/ram/stm32mp1/stm32mp1_ddr.h
index 3bfcb85a8f..861efff92b 100644
--- a/drivers/ram/stm32mp1/stm32mp1_ddr.h
+++ b/drivers/ram/stm32mp1/stm32mp1_ddr.h
@@ -157,11 +157,6 @@ struct stm32mp1_ddr_config {
};
int stm32mp1_ddr_clk_enable(struct ddr_info *priv, u32 mem_speed);
-void stm32mp1_ddrphy_init(struct stm32mp1_ddrphy *phy, u32 pir);
-void stm32mp1_refresh_disable(struct stm32mp1_ddrctl *ctl);
-void stm32mp1_refresh_restore(struct stm32mp1_ddrctl *ctl,
- u32 rfshctl3,
- u32 pwrctl);
void stm32mp1_ddr_init(
struct ddr_info *priv,
diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h b/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h
index ada3087328..be89d81018 100644
--- a/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h
+++ b/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h
@@ -6,8 +6,9 @@
#ifndef _RAM_STM32MP1_DDR_REGS_H
#define _RAM_STM32MP1_DDR_REGS_H
-/* DDR3/LPDDR2/LPDDR3 Controller (DDRCTRL) registers */
#include <linux/bitops.h>
+
+/* DDR3/LPDDR2/LPDDR3 Controller (DDRCTRL) registers */
struct stm32mp1_ddrctl {
u32 mstr ; /* 0x0 Master*/
u32 stat; /* 0x4 Operating Mode Status*/
@@ -275,25 +276,6 @@ struct stm32mp1_ddrphy {
#define DDRCTRL_DFIMISC_DFI_INIT_COMPLETE_EN BIT(0)
-#define DDRCTRL_DBG1_DIS_HIF BIT(1)
-
-#define DDRCTRL_DBGCAM_WR_DATA_PIPELINE_EMPTY BIT(29)
-#define DDRCTRL_DBGCAM_RD_DATA_PIPELINE_EMPTY BIT(28)
-#define DDRCTRL_DBGCAM_DBG_WR_Q_EMPTY BIT(26)
-#define DDRCTRL_DBGCAM_DBG_LPR_Q_DEPTH GENMASK(12, 8)
-#define DDRCTRL_DBGCAM_DBG_HPR_Q_DEPTH GENMASK(4, 0)
-#define DDRCTRL_DBGCAM_DATA_PIPELINE_EMPTY \
- (DDRCTRL_DBGCAM_WR_DATA_PIPELINE_EMPTY | \
- DDRCTRL_DBGCAM_RD_DATA_PIPELINE_EMPTY)
-#define DDRCTRL_DBGCAM_DBG_Q_DEPTH \
- (DDRCTRL_DBGCAM_DBG_WR_Q_EMPTY | \
- DDRCTRL_DBGCAM_DBG_LPR_Q_DEPTH | \
- DDRCTRL_DBGCAM_DBG_HPR_Q_DEPTH)
-
-#define DDRCTRL_DBGCMD_RANK0_REFRESH BIT(0)
-
-#define DDRCTRL_DBGSTAT_RANK0_REFRESH_BUSY BIT(0)
-
#define DDRCTRL_SWCTL_SW_DONE BIT(0)
#define DDRCTRL_SWSTAT_SW_DONE_ACK BIT(0)
@@ -312,11 +294,6 @@ struct stm32mp1_ddrphy {
#define DDRPHYC_PIR_RVTRN BIT(8)
#define DDRPHYC_PIR_ICPC BIT(16)
#define DDRPHYC_PIR_ZCALBYP BIT(30)
-#define DDRPHYC_PIR_INITSTEPS_MASK GENMASK(31, 7)
-
-#define DDRPHYC_PGCR_DFTCMP BIT(2)
-#define DDRPHYC_PGCR_PDDISDX BIT(24)
-#define DDRPHYC_PGCR_RFSHDT_MASK GENMASK(28, 25)
#define DDRPHYC_PGSR_IDONE BIT(0)
#define DDRPHYC_PGSR_DTERR BIT(5)
@@ -325,43 +302,6 @@ struct stm32mp1_ddrphy {
#define DDRPHYC_PGSR_RVERR BIT(8)
#define DDRPHYC_PGSR_RVEIRR BIT(9)
-#define DDRPHYC_DLLGCR_BPS200 BIT(23)
-
-#define DDRPHYC_ACDLLCR_DLLDIS BIT(31)
-
-#define DDRPHYC_ZQ0CRN_ZDATA_MASK GENMASK(27, 0)
-#define DDRPHYC_ZQ0CRN_ZDATA_SHIFT 0
-#define DDRPHYC_ZQ0CRN_ZDEN BIT(28)
-
-#define DDRPHYC_DXNGCR_DXEN BIT(0)
-
-#define DDRPHYC_DXNDLLCR_DLLSRST BIT(30)
-#define DDRPHYC_DXNDLLCR_DLLDIS BIT(31)
-#define DDRPHYC_DXNDLLCR_SDPHASE_MASK GENMASK(17, 14)
-#define DDRPHYC_DXNDLLCR_SDPHASE_SHIFT 14
-
-#define DDRPHYC_DXNDQTR_DQDLY_SHIFT(bit) (4 * (bit))
-#define DDRPHYC_DXNDQTR_DQDLY_MASK GENMASK(3, 0)
-#define DDRPHYC_DXNDQTR_DQDLY_LOW_MASK GENMASK(1, 0)
-#define DDRPHYC_DXNDQTR_DQDLY_HIGH_MASK GENMASK(3, 2)
-
-#define DDRPHYC_DXNDQSTR_DQSDLY_MASK GENMASK(22, 20)
-#define DDRPHYC_DXNDQSTR_DQSDLY_SHIFT 20
-#define DDRPHYC_DXNDQSTR_DQSNDLY_MASK GENMASK(25, 23)
-#define DDRPHYC_DXNDQSTR_DQSNDLY_SHIFT 23
-#define DDRPHYC_DXNDQSTR_R0DGSL_MASK GENMASK(2, 0)
-#define DDRPHYC_DXNDQSTR_R0DGSL_SHIFT 0
-#define DDRPHYC_DXNDQSTR_R0DGPS_MASK GENMASK(13, 12)
-#define DDRPHYC_DXNDQSTR_R0DGPS_SHIFT 12
-
-#define DDRPHYC_BISTRR_BDXSEL_MASK GENMASK(22, 19)
-#define DDRPHYC_BISTRR_BDXSEL_SHIFT 19
-
-#define DDRPHYC_BISTGSR_BDDONE BIT(0)
-#define DDRPHYC_BISTGSR_BDXERR BIT(2)
-
-#define DDRPHYC_BISTWCSR_DXWCNT_SHIFT 16
-
/* PWR registers */
#define PWR_CR3 0x00C
#define PWR_CR3_DDRSRDIS BIT(11)
diff --git a/drivers/ram/stm32mp1/stm32mp1_interactive.c b/drivers/ram/stm32mp1/stm32mp1_interactive.c
index a667d49cff..f0fe7e61e3 100644
--- a/drivers/ram/stm32mp1/stm32mp1_interactive.c
+++ b/drivers/ram/stm32mp1/stm32mp1_interactive.c
@@ -32,7 +32,6 @@ enum ddr_command {
DDR_CMD_NEXT,
DDR_CMD_GO,
DDR_CMD_TEST,
- DDR_CMD_TUNING,
DDR_CMD_UNKNOWN,
};
@@ -59,9 +58,6 @@ enum ddr_command stm32mp1_get_command(char *cmd, int argc)
[DDR_CMD_GO] = "go",
#ifdef CONFIG_STM32MP1_DDR_TESTS
[DDR_CMD_TEST] = "test",
-#endif
-#ifdef CONFIG_STM32MP1_DDR_TUNING
- [DDR_CMD_TUNING] = "tuning",
#endif
};
/* min and max number of argument */
@@ -78,9 +74,6 @@ enum ddr_command stm32mp1_get_command(char *cmd, int argc)
[DDR_CMD_GO] = { 0, 0 },
#ifdef CONFIG_STM32MP1_DDR_TESTS
[DDR_CMD_TEST] = { 0, 255 },
-#endif
-#ifdef CONFIG_STM32MP1_DDR_TUNING
- [DDR_CMD_TUNING] = { 0, 255 },
#endif
};
int i;
@@ -125,9 +118,6 @@ static void stm32mp1_do_usage(void)
"reset reboots machine\n"
#ifdef CONFIG_STM32MP1_DDR_TESTS
"test [help] | <n> [...] lists (with help) or executes test <n>\n"
-#endif
-#ifdef CONFIG_STM32MP1_DDR_TUNING
- "tuning [help] | <n> [...] lists (with help) or execute tuning <n>\n"
#endif
"\nwith for [type|reg]:\n"
" all registers if absent\n"
@@ -311,7 +301,7 @@ end:
return step;
}
-#if defined(CONFIG_STM32MP1_DDR_TESTS) || defined(CONFIG_STM32MP1_DDR_TUNING)
+#if defined(CONFIG_STM32MP1_DDR_TESTS)
static const char * const s_result[] = {
[TEST_PASSED] = "Pass",
[TEST_FAILED] = "Failed",
@@ -468,16 +458,6 @@ bool stm32mp1_ddr_interactive(void *priv,
stm32mp1_ddr_subcmd(priv, argc, argv, test, test_nb);
break;
#endif
-
-#ifdef CONFIG_STM32MP1_DDR_TUNING
- case DDR_CMD_TUNING:
- if (!stm32mp1_check_step(step, STEP_DDR_READY))
- continue;
- stm32mp1_ddr_subcmd(priv, argc, argv,
- tuning, tuning_nb);
- break;
-#endif
-
default:
break;
}
diff --git a/drivers/ram/stm32mp1/stm32mp1_tests.h b/drivers/ram/stm32mp1/stm32mp1_tests.h
index 55f5d6d93b..8436780790 100644
--- a/drivers/ram/stm32mp1/stm32mp1_tests.h
+++ b/drivers/ram/stm32mp1/stm32mp1_tests.h
@@ -28,7 +28,4 @@ struct test_desc {
extern const struct test_desc test[];
extern const int test_nb;
-extern const struct test_desc tuning[];
-extern const int tuning_nb;
-
#endif
diff --git a/drivers/ram/stm32mp1/stm32mp1_tuning.c b/drivers/ram/stm32mp1/stm32mp1_tuning.c
deleted file mode 100644
index c8cd7c3cea..0000000000
--- a/drivers/ram/stm32mp1/stm32mp1_tuning.c
+++ /dev/null
@@ -1,1540 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0+ OR BSD-3-Clause
-/*
- * Copyright (C) 2019, STMicroelectronics - All Rights Reserved
- */
-
-#define LOG_CATEGORY UCLASS_RAM
-
-#include <common.h>
-#include <console.h>
-#include <clk.h>
-#include <log.h>
-#include <ram.h>
-#include <rand.h>
-#include <reset.h>
-#include <asm/io.h>
-#include <linux/bitops.h>
-#include <linux/delay.h>
-#include <linux/iopoll.h>
-
-#include "stm32mp1_ddr_regs.h"
-#include "stm32mp1_ddr.h"
-#include "stm32mp1_tests.h"
-
-#define MAX_DQS_PHASE_IDX _144deg
-#define MAX_DQS_UNIT_IDX 7
-#define MAX_GSL_IDX 5
-#define MAX_GPS_IDX 3
-
-/* Number of bytes used in this SW. ( min 1--> max 4). */
-#define NUM_BYTES 4
-
-enum dqs_phase_enum {
- _36deg = 0,
- _54deg = 1,
- _72deg = 2,
- _90deg = 3,
- _108deg = 4,
- _126deg = 5,
- _144deg = 6
-};
-
-/* BIST Result struct */
-struct BIST_result {
- /* Overall test result:
- * 0 Fail (any bit failed) ,
- * 1 Success (All bits success)
- */
- bool test_result;
- /* 1: true, all fail / 0: False, not all bits fail */
- bool all_bits_fail;
- bool bit_i_test_result[8]; /* 0 fail / 1 success */
-};
-
-/* a struct that defines tuning parameters of a byte. */
-struct tuning_position {
- u8 phase; /* DQS phase */
- u8 unit; /* DQS unit delay */
- u32 bits_delay; /* Bits deskew in this byte */
-};
-
-/* 36deg, 54deg, 72deg, 90deg, 108deg, 126deg, 144deg */
-const u8 dx_dll_phase[7] = {3, 2, 1, 0, 14, 13, 12};
-
-static u8 BIST_error_max = 1;
-static u32 BIST_seed = 0x1234ABCD;
-
-static u8 get_nb_bytes(struct stm32mp1_ddrctl *ctl)
-{
- u32 data_bus = readl(&ctl->mstr) & DDRCTRL_MSTR_DATA_BUS_WIDTH_MASK;
- u8 nb_bytes = NUM_BYTES;
-
- switch (data_bus) {
- case DDRCTRL_MSTR_DATA_BUS_WIDTH_HALF:
- nb_bytes /= 2;
- break;
- case DDRCTRL_MSTR_DATA_BUS_WIDTH_QUARTER:
- nb_bytes /= 4;
- break;
- default:
- break;
- }
-
- return nb_bytes;
-}
-
-static u8 get_nb_bank(struct stm32mp1_ddrctl *ctl)
-{
- /* Count bank address bits */
- u8 bits = 0;
- u32 reg, val;
-
- reg = readl(&ctl->addrmap1);
- /* addrmap1.addrmap_bank_b1 */
- val = (reg & GENMASK(5, 0)) >> 0;
- if (val <= 31)
- bits++;
- /* addrmap1.addrmap_bank_b2 */
- val = (reg & GENMASK(13, 8)) >> 8;
- if (val <= 31)
- bits++;
- /* addrmap1.addrmap_bank_b3 */
- val = (reg & GENMASK(21, 16)) >> 16;
- if (val <= 31)
- bits++;
-
- return bits;
-}
-
-static u8 get_nb_col(struct stm32mp1_ddrctl *ctl)
-{
- u8 bits;
- u32 reg, val;
-
- /* Count column address bits, start at 2 for b0 and b1 (fixed) */
- bits = 2;
-
- reg = readl(&ctl->addrmap2);
- /* addrmap2.addrmap_col_b2 */
- val = (reg & GENMASK(3, 0)) >> 0;
- if (val <= 7)
- bits++;
- /* addrmap2.addrmap_col_b3 */
- val = (reg & GENMASK(11, 8)) >> 8;
- if (val <= 7)
- bits++;
- /* addrmap2.addrmap_col_b4 */
- val = (reg & GENMASK(19, 16)) >> 16;
- if (val <= 7)
- bits++;
- /* addrmap2.addrmap_col_b5 */
- val = (reg & GENMASK(27, 24)) >> 24;
- if (val <= 7)
- bits++;
-
- reg = readl(&ctl->addrmap3);
- /* addrmap3.addrmap_col_b6 */
- val = (reg & GENMASK(3, 0)) >> 0;
- if (val <= 7)
- bits++;
- /* addrmap3.addrmap_col_b7 */
- val = (reg & GENMASK(11, 8)) >> 8;
- if (val <= 7)
- bits++;
- /* addrmap3.addrmap_col_b8 */
- val = (reg & GENMASK(19, 16)) >> 16;
- if (val <= 7)
- bits++;
- /* addrmap3.addrmap_col_b9 */
- val = (reg & GENMASK(27, 24)) >> 24;
- if (val <= 7)
- bits++;
-
- reg = readl(&ctl->addrmap4);
- /* addrmap4.addrmap_col_b10 */
- val = (reg & GENMASK(3, 0)) >> 0;
- if (val <= 7)
- bits++;
- /* addrmap4.addrmap_col_b11 */
- val = (reg & GENMASK(11, 8)) >> 8;
- if (val <= 7)
- bits++;
-
- return bits;
-}
-
-static u8 get_nb_row(struct stm32mp1_ddrctl *ctl)
-{
- /* Count row address bits */
- u8 bits = 0;
- u32 reg, val;
-
- reg = readl(&ctl->addrmap5);
- /* addrmap5.addrmap_row_b0 */
- val = (reg & GENMASK(3, 0)) >> 0;
- if (val <= 11)
- bits++;
- /* addrmap5.addrmap_row_b1 */
- val = (reg & GENMASK(11, 8)) >> 8;
- if (val <= 11)
- bits++;
- /* addrmap5.addrmap_row_b2_10 */
- val = (reg & GENMASK(19, 16)) >> 16;
- if (val <= 11)
- bits += 9;
- else
- printf("warning: addrmap5.addrmap_row_b2_10 not supported\n");
- /* addrmap5.addrmap_row_b11 */
- val = (reg & GENMASK(27, 24)) >> 24;
- if (val <= 11)
- bits++;
-
- reg = readl(&ctl->addrmap6);
- /* addrmap6.addrmap_row_b12 */
- val = (reg & GENMASK(3, 0)) >> 0;
- if (val <= 7)
- bits++;
- /* addrmap6.addrmap_row_b13 */
- val = (reg & GENMASK(11, 8)) >> 8;
- if (val <= 7)
- bits++;
- /* addrmap6.addrmap_row_b14 */
- val = (reg & GENMASK(19, 16)) >> 16;
- if (val <= 7)
- bits++;
- /* addrmap6.addrmap_row_b15 */
- val = (reg & GENMASK(27, 24)) >> 24;
- if (val <= 7)
- bits++;
-
- return bits;
-}
-
-static void itm_soft_reset(struct stm32mp1_ddrphy *phy)
-{
- stm32mp1_ddrphy_init(phy, DDRPHYC_PIR_ITMSRST);
-}
-
-/* Read DQ unit delay register and provides the retrieved value for DQS
- * We are assuming that we have the same delay when clocking
- * by DQS and when clocking by DQSN
- */
-static u8 DQ_unit_index(struct stm32mp1_ddrphy *phy, u8 byte, u8 bit)
-{
- u32 index;
- u32 addr = DXNDQTR(phy, byte);
-
- /* We are assuming that we have the same delay when clocking by DQS
- * and when clocking by DQSN : use only the low bits
- */
- index = (readl(addr) >> DDRPHYC_DXNDQTR_DQDLY_SHIFT(bit))
- & DDRPHYC_DXNDQTR_DQDLY_LOW_MASK;
-
- log_debug("[%x]: %x => DQ unit index = %x\n", addr, readl(addr), index);
-
- return index;
-}
-
-/* Sets the DQS phase delay for a byte lane.
- *phase delay is specified by giving the index of the desired delay
- * in the dx_dll_phase array.
- */
-static void DQS_phase_delay(struct stm32mp1_ddrphy *phy, u8 byte, u8 phase_idx)
-{
- u8 sdphase_val = 0;
-
- /* Write DXNDLLCR.SDPHASE = dx_dll_phase(phase_index); */
- sdphase_val = dx_dll_phase[phase_idx];
- clrsetbits_le32(DXNDLLCR(phy, byte),
- DDRPHYC_DXNDLLCR_SDPHASE_MASK,
- sdphase_val << DDRPHYC_DXNDLLCR_SDPHASE_SHIFT);
-}
-
-/* Sets the DQS unit delay for a byte lane.
- * unit delay is specified by giving the index of the desired delay
- * for dgsdly and dqsndly (same value).
- */
-static void DQS_unit_delay(struct stm32mp1_ddrphy *phy,
- u8 byte, u8 unit_dly_idx)
-{
- /* Write the same value in DXNDQSTR.DQSDLY and DXNDQSTR.DQSNDLY */
- clrsetbits_le32(DXNDQSTR(phy, byte),
- DDRPHYC_DXNDQSTR_DQSDLY_MASK |
- DDRPHYC_DXNDQSTR_DQSNDLY_MASK,
- (unit_dly_idx << DDRPHYC_DXNDQSTR_DQSDLY_SHIFT) |
- (unit_dly_idx << DDRPHYC_DXNDQSTR_DQSNDLY_SHIFT));
-
- /* After changing this value, an ITM soft reset (PIR.ITMSRST=1,
- * plus PIR.INIT=1) must be issued.
- */
- stm32mp1_ddrphy_init(phy, DDRPHYC_PIR_ITMSRST);
-}
-
-/* Sets the DQ unit delay for a bit line in particular byte lane.
- * unit delay is specified by giving the desired delay
- */
-static void set_DQ_unit_delay(struct stm32mp1_ddrphy *phy,
- u8 byte, u8 bit,
- u8 dq_delay_index)
-{
- u8 dq_bit_delay_val = dq_delay_index | (dq_delay_index << 2);
-
- /* same value on delay for clock DQ an DQS_b */
- clrsetbits_le32(DXNDQTR(phy, byte),
- DDRPHYC_DXNDQTR_DQDLY_MASK
- << DDRPHYC_DXNDQTR_DQDLY_SHIFT(bit),
- dq_bit_delay_val << DDRPHYC_DXNDQTR_DQDLY_SHIFT(bit));
-}
-
-static void set_r0dgsl_delay(struct stm32mp1_ddrphy *phy,
- u8 byte, u8 r0dgsl_idx)
-{
- clrsetbits_le32(DXNDQSTR(phy, byte),
- DDRPHYC_DXNDQSTR_R0DGSL_MASK,
- r0dgsl_idx << DDRPHYC_DXNDQSTR_R0DGSL_SHIFT);
-}
-
-static void set_r0dgps_delay(struct stm32mp1_ddrphy *phy,
- u8 byte, u8 r0dgps_idx)
-{
- clrsetbits_le32(DXNDQSTR(phy, byte),
- DDRPHYC_DXNDQSTR_R0DGPS_MASK,
- r0dgps_idx << DDRPHYC_DXNDQSTR_R0DGPS_SHIFT);
-}
-
-/* Basic BIST configuration for data lane tests. */
-static void config_BIST(struct stm32mp1_ddrctl *ctl,
- struct stm32mp1_ddrphy *phy)
-{
- u8 nb_bank = get_nb_bank(ctl);
- u8 nb_row = get_nb_row(ctl);
- u8 nb_col = get_nb_col(ctl);
-
- /* Selects the SDRAM bank address to be used during BIST. */
- u32 bbank = 0;
- /* Selects the SDRAM row address to be used during BIST. */
- u32 brow = 0;
- /* Selects the SDRAM column address to be used during BIST. */
- u32 bcol = 0;
- /* Selects the value by which the SDRAM address is incremented
- * for each write/read access.
- */
- u32 bainc = 0x00000008;
- /* Specifies the maximum SDRAM rank to be used during BIST.
- * The default value is set to maximum ranks minus 1.
- * must be 0 with single rank
- */
- u32 bmrank = 0;
- /* Selects the SDRAM rank to be used during BIST.
- * must be 0 with single rank
- */
- u32 brank = 0;
-
- /* Specifies the maximum SDRAM bank address to be used during
- * BIST before the address & increments to the next rank.
- */
- u32 bmbank = (1 << nb_bank) - 1;
- /* Specifies the maximum SDRAM row address to be used during
- * BIST before the address & increments to the next bank.
- */
- u32 bmrow = (1 << nb_row) - 1;
- /* Specifies the maximum SDRAM column address to be used during
- * BIST before the address & increments to the next row.
- */
- u32 bmcol = (1 << nb_col) - 1;
-
- u32 bmode_conf = 0x00000001; /* DRam mode */
- u32 bdxen_conf = 0x00000001; /* BIST on Data byte */
- u32 bdpat_conf = 0x00000002; /* Select LFSR pattern */
-
- /*Setup BIST for DRAM mode, and LFSR-random data pattern.*/
- /*Write BISTRR.BMODE = 1?b1;*/
- /*Write BISTRR.BDXEN = 1?b1;*/
- /*Write BISTRR.BDPAT = 2?b10;*/
-
- /* reset BIST */
- writel(0x3, &phy->bistrr);
-
- writel((bmode_conf << 3) | (bdxen_conf << 14) | (bdpat_conf << 17),
- &phy->bistrr);
-
- /*Setup BIST Word Count*/
- /*Write BISTWCR.BWCNT = 16?b0008;*/
- writel(0x00000200, &phy->bistwcr); /* A multiple of BL/2 */
-
- writel(bcol | (brow << 12) | (bbank << 28), &phy->bistar0);
- writel(brank | (bmrank << 2) | (bainc << 4), &phy->bistar1);
- writel(bmcol | (bmrow << 12) | (bmbank << 28), &phy->bistar2);
-}
-
-/* Select the Byte lane to be tested by BIST. */
-static void BIST_datx8_sel(struct stm32mp1_ddrphy *phy, u8 datx8)
-{
- clrsetbits_le32(&phy->bistrr,
- DDRPHYC_BISTRR_BDXSEL_MASK,
- datx8 << DDRPHYC_BISTRR_BDXSEL_SHIFT);
-
- /*(For example, selecting Byte Lane 3, BISTRR.BDXSEL = 4?b0011)*/
- /* Write BISTRR.BDXSEL = datx8; */
-}
-
-/* Perform BIST Write_Read test on a byte lane and return test result. */
-static void BIST_test(struct stm32mp1_ddrphy *phy, u8 byte,
- struct BIST_result *bist)
-{
- bool result = true; /* BIST_SUCCESS */
- u32 cnt = 0;
- u32 error = 0;
- u32 val;
- int ret;
-
- bist->test_result = true;
-
-run:
- itm_soft_reset(phy);
-
- /*Perform BIST Reset*/
- /* Write BISTRR.BINST = 3?b011; */
- clrsetbits_le32(&phy->bistrr,
- 0x00000007,
- 0x00000003);
-
- /*Re-seed LFSR*/
- /* Write BISTLSR.SEED = 32'h1234ABCD; */
- if (BIST_seed)
- writel(BIST_seed, &phy->bistlsr);
- else
- writel(rand(), &phy->bistlsr);
-
- /* some delay to reset BIST */
- udelay(10);
-
- /*Perform BIST Run*/
- clrsetbits_le32(&phy->bistrr,
- 0x00000007,
- 0x00000001);
- /* Write BISTRR.BINST = 3?b001; */
-
- /* poll on BISTGSR.BDONE and wait max 1000 us */
- ret = readl_poll_timeout(&phy->bistgsr, val,
- val & DDRPHYC_BISTGSR_BDDONE, 1000);
-
- if (ret < 0) {
- printf("warning: BIST timeout\n");
- result = false; /* BIST_FAIL; */
- /*Perform BIST Stop */
- clrsetbits_le32(&phy->bistrr, 0x00000007, 0x00000002);
- } else {
- /*Check if received correct number of words*/
- /* if (Read BISTWCSR.DXWCNT = Read BISTWCR.BWCNT) */
- if (((readl(&phy->bistwcsr)) >> DDRPHYC_BISTWCSR_DXWCNT_SHIFT)
- == readl(&phy->bistwcr)) {
- /*Determine if there is a data comparison error*/
- /* if (Read BISTGSR.BDXERR = 1?b0) */
- if (readl(&phy->bistgsr) & DDRPHYC_BISTGSR_BDXERR)
- result = false; /* BIST_FAIL; */
- else
- result = true; /* BIST_SUCCESS; */
- } else {
- result = false; /* BIST_FAIL; */
- }
- }
-
- /* loop while success */
- cnt++;
- if (result && cnt != 1000)
- goto run;
-
- if (!result)
- error++;
-
- if (error < BIST_error_max) {
- if (cnt != 1000)
- goto run;
- bist->test_result = true;
- } else {
- bist->test_result = false;
- }
-}
-
-/* After running the deskew algo, this function applies the new DQ delays
- * by reading them from the array "deskew_delay"and writing in PHY registers.
- * The bits that are not deskewed parfectly (too much skew on them,
- * or data eye very wide) are marked in the array deskew_non_converge.
- */
-static void apply_deskew_results(struct stm32mp1_ddrphy *phy, u8 byte,
- u8 deskew_delay[NUM_BYTES][8],
- u8 deskew_non_converge[NUM_BYTES][8])
-{
- u8 bit_i;
- u8 index;
-
- for (bit_i = 0; bit_i < 8; bit_i++) {
- set_DQ_unit_delay(phy, byte, bit_i, deskew_delay[byte][bit_i]);
- index = DQ_unit_index(phy, byte, bit_i);
- log_debug("Byte %d ; bit %d : The new DQ delay (%d) index=%d [delta=%d, 3 is the default]",
- byte, bit_i, deskew_delay[byte][bit_i],
- index, index - 3);
- printf("Byte %d, bit %d, DQ delay = %d",
- byte, bit_i, deskew_delay[byte][bit_i]);
- if (deskew_non_converge[byte][bit_i] == 1)
- log_debug(" - not converged : still more skew");
- printf("\n");
- }
-}
-
-/* DQ Bit de-skew algorithm.
- * Deskews data lines as much as possible.
- * 1. Add delay to DQS line until finding the failure
- * (normally a hold time violation)
- * 2. Reduce DQS line by small steps until finding the very first time
- * we go back to "Pass" condition.
- * 3. For each DQ line, Reduce DQ delay until finding the very first failure
- * (normally a hold time fail)
- * 4. When all bits are at their first failure delay, we can consider them
- * aligned.
- * Handle conrer situation (Can't find Pass-fail, or fail-pass transitions
- * at any step)
- * TODO Provide a return Status. Improve doc
- */
-static enum test_result bit_deskew(struct stm32mp1_ddrctl *ctl,
- struct stm32mp1_ddrphy *phy, char *string)
-{
- /* New DQ delay value (index), set during Deskew algo */
- u8 deskew_delay[NUM_BYTES][8];
- /*If there is still skew on a bit, mark this bit. */
- u8 deskew_non_converge[NUM_BYTES][8];
- struct BIST_result result;
- s8 dqs_unit_delay_index = 0;
- u8 datx8 = 0;
- u8 bit_i = 0;
- s8 phase_idx = 0;
- s8 bit_i_delay_index = 0;
- u8 success = 0;
- struct tuning_position last_right_ok;
- u8 force_stop = 0;
- u8 fail_found;
- u8 error = 0;
- u8 nb_bytes = get_nb_bytes(ctl);
- /* u8 last_pass_dqs_unit = 0; */
-
- memset(deskew_delay, 0, sizeof(deskew_delay));
- memset(deskew_non_converge, 0, sizeof(deskew_non_converge));
-
- /*Disable DQS Drift Compensation*/
- clrbits_le32(&phy->pgcr, DDRPHYC_PGCR_DFTCMP);
- /*Disable all bytes*/
- /* Disable automatic power down of DLL and IOs when disabling
- * a byte (To avoid having to add programming and delay
- * for a DLL re-lock when later re-enabling a disabled Byte Lane)
- */
- clrbits_le32(&phy->pgcr, DDRPHYC_PGCR_PDDISDX);
-
- /* Disable all data bytes */
- clrbits_le32(&phy->dx0gcr, DDRPHYC_DXNGCR_DXEN);
- clrbits_le32(&phy->dx1gcr, DDRPHYC_DXNGCR_DXEN);
- clrbits_le32(&phy->dx2gcr, DDRPHYC_DXNGCR_DXEN);
- clrbits_le32(&phy->dx3gcr, DDRPHYC_DXNGCR_DXEN);
-
- /* Config the BIST block */
- config_BIST(ctl, phy);
- log_debug("BIST Config done.\n");
-
- /* Train each byte */
- for (datx8 = 0; datx8 < nb_bytes; datx8++) {
- if (ctrlc()) {
- sprintf(string, "interrupted at byte %d/%d, error=%d",
- datx8 + 1, nb_bytes, error);
- return TEST_FAILED;
- }
- log_debug("\n======================\n");
- log_debug("Start deskew byte %d .\n", datx8);
- log_debug("======================\n");
- /* Enable Byte (DXNGCR, bit DXEN) */
- setbits_le32(DXNGCR(phy, datx8), DDRPHYC_DXNGCR_DXEN);
-
- /* Select the byte lane for comparison of read data */
- BIST_datx8_sel(phy, datx8);
-
- /* Set all DQDLYn to maximum value. All bits within the byte
- * will be delayed with DQSTR = 2 instead of max = 3
- * to avoid inter bits fail influence
- */
- writel(0xAAAAAAAA, DXNDQTR(phy, datx8));
-
- /* Set the DQS phase delay to 90 DEG (default).
- * What is defined here is the index of the desired config
- * in the PHASE array.
- */
- phase_idx = _90deg;
-
- /* Set DQS unit delay to the max value. */
- dqs_unit_delay_index = MAX_DQS_UNIT_IDX;
- DQS_unit_delay(phy, datx8, dqs_unit_delay_index);
- DQS_phase_delay(phy, datx8, phase_idx);
-
- /* Issue a DLL soft reset */
- clrbits_le32(DXNDLLCR(phy, datx8), DDRPHYC_DXNDLLCR_DLLSRST);
- setbits_le32(DXNDLLCR(phy, datx8), DDRPHYC_DXNDLLCR_DLLSRST);
-
- /* Test this typical init condition */
- BIST_test(phy, datx8, &result);
- success = result.test_result;
-
- /* If the test pass in this typical condition,
- * start the algo with it.
- * Else, look for Pass init condition
- */
- if (!success) {
- log_debug("Fail at init condtion. Let's look for a good init condition.\n");
- success = 0; /* init */
- /* Make sure we start with a PASS condition before
- * looking for a fail condition.
- * Find the first PASS PHASE condition
- */
-
- /* escape if we find a PASS */
- log_debug("increase Phase idx\n");
- while (!success && (phase_idx <= MAX_DQS_PHASE_IDX)) {
- DQS_phase_delay(phy, datx8, phase_idx);
- BIST_test(phy, datx8, &result);
- success = result.test_result;
- phase_idx++;
- }
- /* if ended with success
- * ==>> Restore the fist success condition
- */
- if (success)
- phase_idx--; /* because it ended with ++ */
- }
- if (ctrlc()) {
- sprintf(string, "interrupted at byte %d/%d, error=%d",
- datx8 + 1, nb_bytes, error);
- return TEST_FAILED;
- }
- /* We couldn't find a successful condition, its seems
- * we have hold violation, lets try reduce DQS_unit Delay
- */
- if (!success) {
- /* We couldn't find a successful condition, its seems
- * we have hold violation, lets try reduce DQS_unit
- * Delay
- */
- log_debug("Still fail. Try decrease DQS Unit delay\n");
-
- phase_idx = 0;
- dqs_unit_delay_index = 0;
- DQS_phase_delay(phy, datx8, phase_idx);
-
- /* escape if we find a PASS */
- while (!success &&
- (dqs_unit_delay_index <=
- MAX_DQS_UNIT_IDX)) {
- DQS_unit_delay(phy, datx8,
- dqs_unit_delay_index);
- BIST_test(phy, datx8, &result);
- success = result.test_result;
- dqs_unit_delay_index++;
- }
- if (success) {
- /* Restore the first success condition*/
- dqs_unit_delay_index--;
- /* last_pass_dqs_unit = dqs_unit_delay_index;*/
- DQS_unit_delay(phy, datx8,
- dqs_unit_delay_index);
- } else {
- /* No need to continue,
- * there is no pass region.
- */
- force_stop = 1;
- }
- }
-
- /* There is an initial PASS condition
- * Look for the first failing condition by PHASE stepping.
- * This part of the algo can finish without converging.
- */
- if (force_stop) {
- printf("Result: Failed ");
- printf("[Cannot Deskew lines, ");
- printf("there is no PASS region]\n");
- error++;
- continue;
- }
- if (ctrlc()) {
- sprintf(string, "interrupted at byte %d/%d, error=%d",
- datx8 + 1, nb_bytes, error);
- return TEST_FAILED;
- }
-
- log_debug("there is a pass region for phase idx %d\n",
- phase_idx);
- log_debug("Step1: Find the first failing condition\n");
- /* Look for the first failing condition by PHASE stepping.
- * This part of the algo can finish without converging.
- */
-
- /* escape if we find a fail (hold time violation)
- * condition at any bit or if out of delay range.
- */
- while (success && (phase_idx <= MAX_DQS_PHASE_IDX)) {
- DQS_phase_delay(phy, datx8, phase_idx);
- BIST_test(phy, datx8, &result);
- success = result.test_result;
- phase_idx++;
- }
- if (ctrlc()) {
- sprintf(string, "interrupted at byte %d/%d, error=%d",
- datx8 + 1, nb_bytes, error);
- return TEST_FAILED;
- }
-
- /* if the loop ended with a failing condition at any bit,
- * lets look for the first previous success condition by unit
- * stepping (minimal delay)
- */
- if (!success) {
- log_debug("Fail region (PHASE) found phase idx %d\n",
- phase_idx);
- log_debug("Let's look for first success by DQS Unit steps\n");
- /* This part, the algo always converge */
- phase_idx--;
-
- /* escape if we find a success condition
- * or if out of delay range.
- */
- while (!success && dqs_unit_delay_index >= 0) {
- DQS_unit_delay(phy, datx8,
- dqs_unit_delay_index);
- BIST_test(phy, datx8, &result);
- success = result.test_result;
- dqs_unit_delay_index--;
- }
- /* if the loop ended with a success condition,
- * the last delay Right OK (before hold violation)
- * condition is then defined as following:
- */
- if (success) {
- /* Hold the dely parameters of the the last
- * delay Right OK condition.
- * -1 to get back to current condition
- */
- last_right_ok.phase = phase_idx;
- /*+1 to get back to current condition */
- last_right_ok.unit = dqs_unit_delay_index + 1;
- last_right_ok.bits_delay = 0xFFFFFFFF;
- log_debug("Found %d\n", dqs_unit_delay_index);
- } else {
- /* the last OK condition is then with the
- * previous phase_idx.
- * -2 instead of -1 because at the last
- * iteration of the while(),
- * we incremented phase_idx
- */
- last_right_ok.phase = phase_idx - 1;
- /* Nominal+1. Because we want the previous
- * delay after reducing the phase delay.
- */
- last_right_ok.unit = 1;
- last_right_ok.bits_delay = 0xFFFFFFFF;
- log_debug("Not Found : try previous phase %d\n",
- phase_idx - 1);
-
- DQS_phase_delay(phy, datx8, phase_idx - 1);
- dqs_unit_delay_index = 0;
- success = true;
- while (success &&
- (dqs_unit_delay_index <
- MAX_DQS_UNIT_IDX)) {
- DQS_unit_delay(phy, datx8,
- dqs_unit_delay_index);
- BIST_test(phy, datx8, &result);
- success = result.test_result;
- dqs_unit_delay_index++;
- log_debug("dqs_unit_delay_index = %d, result = %d\n",
- dqs_unit_delay_index, success);
- }
-
- if (!success) {
- last_right_ok.unit =
- dqs_unit_delay_index - 1;
- } else {
- last_right_ok.unit = 0;
- log_debug("ERROR: failed region not FOUND");
- }
- }
- } else {
- /* we can't find a failing condition at all bits
- * ==> Just hold the last test condition
- * (the max DQS delay)
- * which is the most likely,
- * the closest to a hold violation
- * If we can't find a Fail condition after
- * the Pass region, stick at this position
- * In order to have max chances to find a fail
- * when reducing DQ delays.
- */
- last_right_ok.phase = MAX_DQS_PHASE_IDX;
- last_right_ok.unit = MAX_DQS_UNIT_IDX;
- last_right_ok.bits_delay = 0xFFFFFFFF;
- log_debug("Can't find the a fail condition\n");
- }
-
- /* step 2:
- * if we arrive at this stage, it means that we found the last
- * Right OK condition (by tweeking the DQS delay). Or we simply
- * pushed DQS delay to the max
- * This means that by reducing the delay on some DQ bits,
- * we should find a failing condition.
- */
- printf("Byte %d, DQS unit = %d, phase = %d\n",
- datx8, last_right_ok.unit, last_right_ok.phase);
- log_debug("Step2, unit = %d, phase = %d, bits delay=%x\n",
- last_right_ok.unit, last_right_ok.phase,
- last_right_ok.bits_delay);
-
- /* Restore the last_right_ok condtion. */
- DQS_unit_delay(phy, datx8, last_right_ok.unit);
- DQS_phase_delay(phy, datx8, last_right_ok.phase);
- writel(last_right_ok.bits_delay, DXNDQTR(phy, datx8));
-
- /* train each bit
- * reduce delay on each bit, and perform a write/read test
- * and stop at the very first time it fails.
- * the goal is the find the first failing condition
- * for each bit.
- * When we achieve this condition< for all the bits,
- * we are sure they are aligned (+/- step resolution)
- */
- fail_found = 0;
- for (bit_i = 0; bit_i < 8; bit_i++) {
- if (ctrlc()) {
- sprintf(string,
- "interrupted at byte %d/%d, error=%d",
- datx8 + 1, nb_bytes, error);
- return error;
- }
- log_debug("deskewing bit %d:\n", bit_i);
- success = 1; /* init */
- /* Set all DQDLYn to maximum value.
- * Only bit_i will be down-delayed
- * ==> if we have a fail, it will be definitely
- * from bit_i
- */
- writel(0xFFFFFFFF, DXNDQTR(phy, datx8));
- /* Arriving at this stage,
- * we have a success condition with delay = 3;
- */
- bit_i_delay_index = 3;
-
- /* escape if bit delay is out of range or
- * if a fatil occurs
- */
- while ((bit_i_delay_index >= 0) && success) {
- set_DQ_unit_delay(phy, datx8,
- bit_i,
- bit_i_delay_index);
- BIST_test(phy, datx8, &result);
- success = result.test_result;
- bit_i_delay_index--;
- }
-
- /* if escape with a fail condition
- * ==> save this position for bit_i
- */
- if (!success) {
- /* save the delay position.
- * Add 1 because the while loop ended with a --,
- * and that we need to hold the last success
- * delay
- */
- deskew_delay[datx8][bit_i] =
- bit_i_delay_index + 2;
- if (deskew_delay[datx8][bit_i] > 3)
- deskew_delay[datx8][bit_i] = 3;
-
- /* A flag that states we found at least a fail
- * at one bit.
- */
- fail_found = 1;
- log_debug("Fail found on bit %d, for delay = %d => deskew[%d][%d] = %d\n",
- bit_i, bit_i_delay_index + 1,
- datx8, bit_i,
- deskew_delay[datx8][bit_i]);
- } else {
- /* if we can find a success condition by
- * back-delaying this bit, just set the delay
- * to 0 (the best deskew
- * possible) and mark the bit.
- */
- deskew_delay[datx8][bit_i] = 0;
- /* set a flag that will be used later
- * in the report.
- */
- deskew_non_converge[datx8][bit_i] = 1;
- log_debug("Fail not found on bit %d => deskew[%d][%d] = %d\n",
- bit_i, datx8, bit_i,
- deskew_delay[datx8][bit_i]);
- }
- }
- log_debug("**********byte %d tuning complete************\n",
- datx8);
- /* If we can't find any failure by back delaying DQ lines,
- * hold the default values
- */
- if (!fail_found) {
- for (bit_i = 0; bit_i < 8; bit_i++)
- deskew_delay[datx8][bit_i] = 0;
- log_debug("The Deskew algorithm can't converge, there is too much margin in your design. Good job!\n");
- }
-
- apply_deskew_results(phy, datx8, deskew_delay,
- deskew_non_converge);
- /* Restore nominal value for DQS delay */
- DQS_phase_delay(phy, datx8, 3);
- DQS_unit_delay(phy, datx8, 3);
- /* disable byte after byte bits deskew */
- clrbits_le32(DXNGCR(phy, datx8), DDRPHYC_DXNGCR_DXEN);
- } /* end of byte deskew */
-
- /* re-enable all data bytes */
- setbits_le32(&phy->dx0gcr, DDRPHYC_DXNGCR_DXEN);
- setbits_le32(&phy->dx1gcr, DDRPHYC_DXNGCR_DXEN);
- setbits_le32(&phy->dx2gcr, DDRPHYC_DXNGCR_DXEN);
- setbits_le32(&phy->dx3gcr, DDRPHYC_DXNGCR_DXEN);
-
- if (error) {
- sprintf(string, "error = %d", error);
- return TEST_FAILED;
- }
-
- return TEST_PASSED;
-} /* end function */
-
-/* Trim DQS timings and set it in the centre of data eye.
- * Look for a PPPPF region, then look for a FPPP region and finally select
- * the mid of the FPPPPPF region
- */
-static enum test_result eye_training(struct stm32mp1_ddrctl *ctl,
- struct stm32mp1_ddrphy *phy, char *string)
-{
- /*Stores the DQS trim values (PHASE index, unit index) */
- u8 eye_training_val[NUM_BYTES][2];
- u8 byte = 0;
- struct BIST_result result;
- s8 dqs_unit_delay_index = 0;
- s8 phase_idx = 0;
- s8 dqs_unit_delay_index_pass = 0;
- s8 phase_idx_pass = 0;
- u8 success = 0;
- u8 left_phase_bound_found, right_phase_bound_found;
- u8 left_unit_bound_found, right_unit_bound_found;
- u8 left_bound_found, right_bound_found;
- struct tuning_position left_bound, right_bound;
- u8 error = 0;
- u8 nb_bytes = get_nb_bytes(ctl);
-
- /*Disable DQS Drift Compensation*/
- clrbits_le32(&phy->pgcr, DDRPHYC_PGCR_DFTCMP);
- /*Disable all bytes*/
- /* Disable automatic power down of DLL and IOs when disabling a byte
- * (To avoid having to add programming and delay
- * for a DLL re-lock when later re-enabling a disabled Byte Lane)
- */
- clrbits_le32(&phy->pgcr, DDRPHYC_PGCR_PDDISDX);
-
- /*Disable all data bytes */
- clrbits_le32(&phy->dx0gcr, DDRPHYC_DXNGCR_DXEN);
- clrbits_le32(&phy->dx1gcr, DDRPHYC_DXNGCR_DXEN);
- clrbits_le32(&phy->dx2gcr, DDRPHYC_DXNGCR_DXEN);
- clrbits_le32(&phy->dx3gcr, DDRPHYC_DXNGCR_DXEN);
-
- /* Config the BIST block */
- config_BIST(ctl, phy);
-
- for (byte = 0; byte < nb_bytes; byte++) {
- if (ctrlc()) {
- sprintf(string, "interrupted at byte %d/%d, error=%d",
- byte + 1, nb_bytes, error);
- return TEST_FAILED;
- }
- right_bound.phase = 0;
- right_bound.unit = 0;
-
- left_bound.phase = 0;
- left_bound.unit = 0;
-
- left_phase_bound_found = 0;
- right_phase_bound_found = 0;
-
- left_unit_bound_found = 0;
- right_unit_bound_found = 0;
-
- left_bound_found = 0;
- right_bound_found = 0;
-
- /* Enable Byte (DXNGCR, bit DXEN) */
- setbits_le32(DXNGCR(phy, byte), DDRPHYC_DXNGCR_DXEN);
-
- /* Select the byte lane for comparison of read data */
- BIST_datx8_sel(phy, byte);
-
- /* Set DQS phase delay to the nominal value. */
- phase_idx = _90deg;
- phase_idx_pass = phase_idx;
-
- /* Set DQS unit delay to the nominal value. */
- dqs_unit_delay_index = 3;
- dqs_unit_delay_index_pass = dqs_unit_delay_index;
- success = 0;
-
- log_debug("STEP0: Find Init delay\n");
- /* STEP0: Find Init delay: a delay that put the system
- * in a "Pass" condition then (TODO) update
- * dqs_unit_delay_index_pass & phase_idx_pass
- */
- DQS_unit_delay(phy, byte, dqs_unit_delay_index);
- DQS_phase_delay(phy, byte, phase_idx);
- BIST_test(phy, byte, &result);
- success = result.test_result;
- /* If we have a fail in the nominal condition */
- if (!success) {
- /* Look at the left */
- while (phase_idx >= 0 && !success) {
- phase_idx--;
- DQS_phase_delay(phy, byte, phase_idx);
- BIST_test(phy, byte, &result);
- success = result.test_result;
- }
- }
- if (!success) {
- /* if we can't find pass condition,
- * then look at the right
- */
- phase_idx = _90deg;
- while (phase_idx <= MAX_DQS_PHASE_IDX &&
- !success) {
- phase_idx++;
- DQS_phase_delay(phy, byte,
- phase_idx);
- BIST_test(phy, byte, &result);
- success = result.test_result;
- }
- }
- /* save the pass condition */
- if (success) {
- phase_idx_pass = phase_idx;
- } else {
- printf("Result: Failed ");
- printf("[Cannot DQS timings, ");
- printf("there is no PASS region]\n");
- error++;
- continue;
- }
-
- if (ctrlc()) {
- sprintf(string, "interrupted at byte %d/%d, error=%d",
- byte + 1, nb_bytes, error);
- return TEST_FAILED;
- }
- log_debug("STEP1: Find LEFT PHASE DQS Bound\n");
- /* STEP1: Find LEFT PHASE DQS Bound */
- while ((phase_idx >= 0) &&
- (phase_idx <= MAX_DQS_PHASE_IDX) &&
- !left_phase_bound_found) {
- DQS_unit_delay(phy, byte,
- dqs_unit_delay_index);
- DQS_phase_delay(phy, byte,
- phase_idx);
- BIST_test(phy, byte, &result);
- success = result.test_result;
-
- /*TODO: Manage the case were at the beginning
- * there is already a fail
- */
- if (!success) {
- /* the last pass condition */
- left_bound.phase = ++phase_idx;
- left_phase_bound_found = 1;
- } else if (success) {
- phase_idx--;
- }
- }
- if (!left_phase_bound_found) {
- left_bound.phase = 0;
- phase_idx = 0;
- }
- /* If not found, lets take 0 */
-
- if (ctrlc()) {
- sprintf(string, "interrupted at byte %d/%d, error=%d",
- byte + 1, nb_bytes, error);
- return TEST_FAILED;
- }
- log_debug("STEP2: Find UNIT left bound\n");
- /* STEP2: Find UNIT left bound */
- while ((dqs_unit_delay_index >= 0) &&
- !left_unit_bound_found) {
- DQS_unit_delay(phy, byte,
- dqs_unit_delay_index);
- DQS_phase_delay(phy, byte, phase_idx);
- BIST_test(phy, byte, &result);
- success = result.test_result;
- if (!success) {
- left_bound.unit =
- ++dqs_unit_delay_index;
- left_unit_bound_found = 1;
- left_bound_found = 1;
- } else if (success) {
- dqs_unit_delay_index--;
- }
- }
-
- /* If not found, lets take 0 */
- if (!left_unit_bound_found)
- left_bound.unit = 0;
-
- if (ctrlc()) {
- sprintf(string, "interrupted at byte %d/%d, error=%d",
- byte + 1, nb_bytes, error);
- return TEST_FAILED;
- }
- log_debug("STEP3: Find PHase right bound\n");
- /* STEP3: Find PHase right bound, start with "pass"
- * condition
- */
-
- /* Set DQS phase delay to the pass value. */
- phase_idx = phase_idx_pass;
-
- /* Set DQS unit delay to the pass value. */
- dqs_unit_delay_index = dqs_unit_delay_index_pass;
-
- while ((phase_idx <= MAX_DQS_PHASE_IDX) &&
- !right_phase_bound_found) {
- DQS_unit_delay(phy, byte,
- dqs_unit_delay_index);
- DQS_phase_delay(phy, byte, phase_idx);
- BIST_test(phy, byte, &result);
- success = result.test_result;
- if (!success) {
- /* the last pass condition */
- right_bound.phase = --phase_idx;
- right_phase_bound_found = 1;
- } else if (success) {
- phase_idx++;
- }
- }
-
- /* If not found, lets take the max value */
- if (!right_phase_bound_found) {
- right_bound.phase = MAX_DQS_PHASE_IDX;
- phase_idx = MAX_DQS_PHASE_IDX;
- }
-
- if (ctrlc()) {
- sprintf(string, "interrupted at byte %d/%d, error=%d",
- byte + 1, nb_bytes, error);
- return TEST_FAILED;
- }
- log_debug("STEP4: Find UNIT right bound\n");
- /* STEP4: Find UNIT right bound */
- while ((dqs_unit_delay_index <= MAX_DQS_UNIT_IDX) &&
- !right_unit_bound_found) {
- DQS_unit_delay(phy, byte,
- dqs_unit_delay_index);
- DQS_phase_delay(phy, byte, phase_idx);
- BIST_test(phy, byte, &result);
- success = result.test_result;
- if (!success) {
- right_bound.unit =
- --dqs_unit_delay_index;
- right_unit_bound_found = 1;
- right_bound_found = 1;
- } else if (success) {
- dqs_unit_delay_index++;
- }
- }
- /* If not found, lets take the max value */
- if (!right_unit_bound_found)
- right_bound.unit = MAX_DQS_UNIT_IDX;
-
- /* If we found a regular FAil Pass FAil pattern
- * FFPPPPPPFF
- * OR PPPPPFF Or FFPPPPP
- */
-
- if (left_bound_found || right_bound_found) {
- eye_training_val[byte][0] = (right_bound.phase +
- left_bound.phase) / 2;
- eye_training_val[byte][1] = (right_bound.unit +
- left_bound.unit) / 2;
-
- /* If we already lost 1/2PHASE Tuning,
- * let's try to recover by ++ on unit
- */
- if (((right_bound.phase + left_bound.phase) % 2 == 1) &&
- eye_training_val[byte][1] != MAX_DQS_UNIT_IDX)
- eye_training_val[byte][1]++;
- log_debug("** found phase : %d - %d & unit %d - %d\n",
- right_bound.phase, left_bound.phase,
- right_bound.unit, left_bound.unit);
- log_debug("** calculating mid region: phase: %d unit: %d (nominal is 3)\n",
- eye_training_val[byte][0],
- eye_training_val[byte][1]);
- } else {
- /* PPPPPPPPPP, we're already good.
- * Set nominal values.
- */
- eye_training_val[byte][0] = 3;
- eye_training_val[byte][1] = 3;
- }
- DQS_phase_delay(phy, byte, eye_training_val[byte][0]);
- DQS_unit_delay(phy, byte, eye_training_val[byte][1]);
-
- printf("Byte %d, DQS unit = %d, phase = %d\n",
- byte,
- eye_training_val[byte][1],
- eye_training_val[byte][0]);
- }
-
- if (error) {
- sprintf(string, "error = %d", error);
- return TEST_FAILED;
- }
-
- return TEST_PASSED;
-}
-
-static void display_reg_results(struct stm32mp1_ddrphy *phy, u8 byte)
-{
- u8 i = 0;
-
- printf("Byte %d Dekew result, bit0 delay, bit1 delay...bit8 delay\n ",
- byte);
-
- for (i = 0; i < 8; i++)
- printf("%d ", DQ_unit_index(phy, byte, i));
- printf("\n");
-
- printf("dxndllcr: [%08x] val:%08x\n",
- DXNDLLCR(phy, byte),
- readl(DXNDLLCR(phy, byte)));
- printf("dxnqdstr: [%08x] val:%08x\n",
- DXNDQSTR(phy, byte),
- readl(DXNDQSTR(phy, byte)));
- printf("dxndqtr: [%08x] val:%08x\n",
- DXNDQTR(phy, byte),
- readl(DXNDQTR(phy, byte)));
-}
-
-/* analyse the dgs gating log table, and determine the midpoint.*/
-static u8 set_midpoint_read_dqs_gating(struct stm32mp1_ddrphy *phy, u8 byte,
- u8 dqs_gating[NUM_BYTES]
- [MAX_GSL_IDX + 1]
- [MAX_GPS_IDX + 1])
-{
- /* stores the dqs gate values (gsl index, gps index) */
- u8 dqs_gate_values[NUM_BYTES][2];
- u8 gsl_idx, gps_idx = 0;
- u8 left_bound_idx[2] = {0, 0};
- u8 right_bound_idx[2] = {0, 0};
- u8 left_bound_found = 0;
- u8 right_bound_found = 0;
- u8 intermittent = 0;
- u8 value;
-
- for (gsl_idx = 0; gsl_idx <= MAX_GSL_IDX; gsl_idx++) {
- for (gps_idx = 0; gps_idx <= MAX_GPS_IDX; gps_idx++) {
- value = dqs_gating[byte][gsl_idx][gps_idx];
- if (value == 1 && left_bound_found == 0) {
- left_bound_idx[0] = gsl_idx;
- left_bound_idx[1] = gps_idx;
- left_bound_found = 1;
- } else if (value == 0 &&
- left_bound_found == 1 &&
- !right_bound_found) {
- if (gps_idx == 0) {
- right_bound_idx[0] = gsl_idx - 1;
- right_bound_idx[1] = MAX_GPS_IDX;
- } else {
- right_bound_idx[0] = gsl_idx;
- right_bound_idx[1] = gps_idx - 1;
- }
- right_bound_found = 1;
- } else if (value == 1 &&
- right_bound_found == 1) {
- intermittent = 1;
- }
- }
- }
-
- /* if only ppppppp is found, there is no mid region. */
- if (left_bound_idx[0] == 0 && left_bound_idx[1] == 0 &&
- right_bound_idx[0] == 0 && right_bound_idx[1] == 0)
- intermittent = 1;
-
- /*if we found a regular fail pass fail pattern ffppppppff
- * or pppppff or ffppppp
- */
- if (!intermittent) {
- /*if we found a regular fail pass fail pattern ffppppppff
- * or pppppff or ffppppp
- */
- if (left_bound_found || right_bound_found) {
- log_debug("idx0(%d): %d %d idx1(%d) : %d %d\n",
- left_bound_found,
- right_bound_idx[0], left_bound_idx[0],
- right_bound_found,
- right_bound_idx[1], left_bound_idx[1]);
- dqs_gate_values[byte][0] =
- (right_bound_idx[0] + left_bound_idx[0]) / 2;
- dqs_gate_values[byte][1] =
- (right_bound_idx[1] + left_bound_idx[1]) / 2;
- /* if we already lost 1/2gsl tuning,
- * let's try to recover by ++ on gps
- */
- if (((right_bound_idx[0] +
- left_bound_idx[0]) % 2 == 1) &&
- dqs_gate_values[byte][1] != MAX_GPS_IDX)
- dqs_gate_values[byte][1]++;
- /* if we already lost 1/2gsl tuning and gps is on max*/
- else if (((right_bound_idx[0] +
- left_bound_idx[0]) % 2 == 1) &&
- dqs_gate_values[byte][1] == MAX_GPS_IDX) {
- dqs_gate_values[byte][1] = 0;
- dqs_gate_values[byte][0]++;
- }
- /* if we have gsl left and write limit too close
- * (difference=1)
- */
- if (((right_bound_idx[0] - left_bound_idx[0]) == 1)) {
- dqs_gate_values[byte][1] = (left_bound_idx[1] +
- right_bound_idx[1] +
- 4) / 2;
- if (dqs_gate_values[byte][1] >= 4) {
- dqs_gate_values[byte][0] =
- right_bound_idx[0];
- dqs_gate_values[byte][1] -= 4;
- } else {
- dqs_gate_values[byte][0] =
- left_bound_idx[0];
- }
- }
- log_debug("*******calculating mid region: system latency: %d phase: %d********\n",
- dqs_gate_values[byte][0],
- dqs_gate_values[byte][1]);
- log_debug("*******the nominal values were system latency: 0 phase: 2*******\n");
- }
- } else {
- /* if intermitant, restore defaut values */
- log_debug("dqs gating:no regular fail/pass/fail found. defaults values restored.\n");
- dqs_gate_values[byte][0] = 0;
- dqs_gate_values[byte][1] = 2;
- }
- set_r0dgsl_delay(phy, byte, dqs_gate_values[byte][0]);
- set_r0dgps_delay(phy, byte, dqs_gate_values[byte][1]);
- printf("Byte %d, R0DGSL = %d, R0DGPS = %d\n",
- byte, dqs_gate_values[byte][0], dqs_gate_values[byte][1]);
-
- /* return 0 if intermittent or if both left_bound
- * and right_bound are not found
- */
- return !(intermittent || (left_bound_found && right_bound_found));
-}
-
-static enum test_result read_dqs_gating(struct stm32mp1_ddrctl *ctl,
- struct stm32mp1_ddrphy *phy,
- char *string)
-{
- /* stores the log of pass/fail */
- u8 dqs_gating[NUM_BYTES][MAX_GSL_IDX + 1][MAX_GPS_IDX + 1];
- u8 byte, gsl_idx, gps_idx = 0;
- struct BIST_result result;
- u8 success = 0;
- u8 nb_bytes = get_nb_bytes(ctl);
-
- memset(dqs_gating, 0x0, sizeof(dqs_gating));
-
- /*disable dqs drift compensation*/
- clrbits_le32(&phy->pgcr, DDRPHYC_PGCR_DFTCMP);
- /*disable all bytes*/
- /* disable automatic power down of dll and ios when disabling a byte
- * (to avoid having to add programming and delay
- * for a dll re-lock when later re-enabling a disabled byte lane)
- */
- clrbits_le32(&phy->pgcr, DDRPHYC_PGCR_PDDISDX);
-
- /* disable all data bytes */
- clrbits_le32(&phy->dx0gcr, DDRPHYC_DXNGCR_DXEN);
- clrbits_le32(&phy->dx1gcr, DDRPHYC_DXNGCR_DXEN);
- clrbits_le32(&phy->dx2gcr, DDRPHYC_DXNGCR_DXEN);
- clrbits_le32(&phy->dx3gcr, DDRPHYC_DXNGCR_DXEN);
-
- /* config the bist block */
- config_BIST(ctl, phy);
-
- for (byte = 0; byte < nb_bytes; byte++) {
- if (ctrlc()) {
- sprintf(string, "interrupted at byte %d/%d",
- byte + 1, nb_bytes);
- return TEST_FAILED;
- }
- /* enable byte x (dxngcr, bit dxen) */
- setbits_le32(DXNGCR(phy, byte), DDRPHYC_DXNGCR_DXEN);
-
- /* select the byte lane for comparison of read data */
- BIST_datx8_sel(phy, byte);
- for (gsl_idx = 0; gsl_idx <= MAX_GSL_IDX; gsl_idx++) {
- for (gps_idx = 0; gps_idx <= MAX_GPS_IDX; gps_idx++) {
- if (ctrlc()) {
- sprintf(string,
- "interrupted at byte %d/%d",
- byte + 1, nb_bytes);
- return TEST_FAILED;
- }
- /* write cfg to dxndqstr */
- set_r0dgsl_delay(phy, byte, gsl_idx);
- set_r0dgps_delay(phy, byte, gps_idx);
-
- BIST_test(phy, byte, &result);
- success = result.test_result;
- if (success)
- dqs_gating[byte][gsl_idx][gps_idx] = 1;
- itm_soft_reset(phy);
- }
- }
- set_midpoint_read_dqs_gating(phy, byte, dqs_gating);
- /* dummy reads */
- readl(0xc0000000);
- readl(0xc0000000);
- }
-
- /* re-enable drift compensation */
- /* setbits_le32(&phy->pgcr, DDRPHYC_PGCR_DFTCMP); */
- return TEST_PASSED;
-}
-
-/****************************************************************
- * TEST
- ****************************************************************
- */
-static enum test_result do_read_dqs_gating(struct stm32mp1_ddrctl *ctl,
- struct stm32mp1_ddrphy *phy,
- char *string, int argc,
- char *argv[])
-{
- u32 rfshctl3 = readl(&ctl->rfshctl3);
- u32 pwrctl = readl(&ctl->pwrctl);
- u32 derateen = readl(&ctl->derateen);
- enum test_result res;
-
- writel(0x0, &ctl->derateen);
- stm32mp1_refresh_disable(ctl);
-
- res = read_dqs_gating(ctl, phy, string);
-
- stm32mp1_refresh_restore(ctl, rfshctl3, pwrctl);
- writel(derateen, &ctl->derateen);
-
- return res;
-}
-
-static enum test_result do_bit_deskew(struct stm32mp1_ddrctl *ctl,
- struct stm32mp1_ddrphy *phy,
- char *string, int argc, char *argv[])
-{
- u32 rfshctl3 = readl(&ctl->rfshctl3);
- u32 pwrctl = readl(&ctl->pwrctl);
- u32 derateen = readl(&ctl->derateen);
- enum test_result res;
-
- writel(0x0, &ctl->derateen);
- stm32mp1_refresh_disable(ctl);
-
- res = bit_deskew(ctl, phy, string);
-
- stm32mp1_refresh_restore(ctl, rfshctl3, pwrctl);
- writel(derateen, &ctl->derateen);
-
- return res;
-}
-
-static enum test_result do_eye_training(struct stm32mp1_ddrctl *ctl,
- struct stm32mp1_ddrphy *phy,
- char *string, int argc, char *argv[])
-{
- u32 rfshctl3 = readl(&ctl->rfshctl3);
- u32 pwrctl = readl(&ctl->pwrctl);
- u32 derateen = readl(&ctl->derateen);
- enum test_result res;
-
- writel(0x0, &ctl->derateen);
- stm32mp1_refresh_disable(ctl);
-
- res = eye_training(ctl, phy, string);
-
- stm32mp1_refresh_restore(ctl, rfshctl3, pwrctl);
- writel(derateen, &ctl->derateen);
-
- return res;
-}
-
-static enum test_result do_display(struct stm32mp1_ddrctl *ctl,
- struct stm32mp1_ddrphy *phy,
- char *string, int argc, char *argv[])
-{
- int byte;
- u8 nb_bytes = get_nb_bytes(ctl);
-
- for (byte = 0; byte < nb_bytes; byte++)
- display_reg_results(phy, byte);
-
- return TEST_PASSED;
-}
-
-static enum test_result do_bist_config(struct stm32mp1_ddrctl *ctl,
- struct stm32mp1_ddrphy *phy,
- char *string, int argc, char *argv[])
-{
- unsigned long value;
-
- if (argc > 0) {
- if (strict_strtoul(argv[0], 0, &value) < 0) {
- sprintf(string, "invalid nbErr %s", argv[0]);
- return TEST_FAILED;
- }
- BIST_error_max = value;
- }
- if (argc > 1) {
- if (strict_strtoul(argv[1], 0, &value) < 0) {
- sprintf(string, "invalid Seed %s", argv[1]);
- return TEST_FAILED;
- }
- BIST_seed = value;
- }
- printf("Bist.nbErr = %d\n", BIST_error_max);
- if (BIST_seed)
- printf("Bist.Seed = 0x%x\n", BIST_seed);
- else
- printf("Bist.Seed = random\n");
-
- return TEST_PASSED;
-}
-
-/****************************************************************
- * TEST Description
- ****************************************************************
- */
-
-const struct test_desc tuning[] = {
- {do_read_dqs_gating, "Read DQS gating",
- "software read DQS Gating", "", 0 },
- {do_bit_deskew, "Bit de-skew", "", "", 0 },
- {do_eye_training, "Eye Training", "or DQS training", "", 0 },
- {do_display, "Display registers", "", "", 0 },
- {do_bist_config, "Bist config", "[nbErr] [seed]",
- "configure Bist test", 2},
-};
-
-const int tuning_nb = ARRAY_SIZE(tuning);
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] stm32mp1: ram: add read valid training support
2021-11-15 14:32 [PATCH 1/3] stm32mp1: ram: add read valid training support Patrick Delaunay
2021-11-15 14:32 ` [PATCH 2/3] stm32mp1: ram: remove the support of calibration result Patrick Delaunay
2021-11-15 14:32 ` [PATCH 3/3] stm32mp1: ram: remove tuning support Patrick Delaunay
@ 2021-11-22 7:48 ` Patrice CHOTARD
2021-11-30 10:36 ` Patrice CHOTARD
3 siblings, 0 replies; 7+ messages in thread
From: Patrice CHOTARD @ 2021-11-22 7:48 UTC (permalink / raw)
To: Patrick Delaunay, u-boot; +Cc: U-Boot STM32
Hi Patrick
On 11/15/21 3:32 PM, Patrick Delaunay wrote:
> Add the read data eye training = training for optimal read valid placement
> (RVTRN) when the built-in calibration is executed for LPDDR2 and LPDDR3.
>
> This training is supported on the PUBL integrated in the STM32MP15x
> DDR subsystem and it is not required for DDR3.
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
> ---
>
> drivers/ram/stm32mp1/stm32mp1_ddr.c | 8 ++++++--
> drivers/ram/stm32mp1/stm32mp1_ddr_regs.h | 1 +
> 2 files changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr.c b/drivers/ram/stm32mp1/stm32mp1_ddr.c
> index 0457166b12..1f8422518b 100644
> --- a/drivers/ram/stm32mp1/stm32mp1_ddr.c
> +++ b/drivers/ram/stm32mp1/stm32mp1_ddr.c
> @@ -826,8 +826,12 @@ start:
> */
>
> /* 10. configure PUBL PIR register to specify which training step to run */
> - /* warning : RVTRN is not supported by this PUBL */
> - stm32mp1_ddrphy_init(priv->phy, DDRPHYC_PIR_QSTRN);
> + /* RVTRN is excuted only on LPDDR2/LPDDR3 */
> + if (config->c_reg.mstr & DDRCTRL_MSTR_DDR3)
> + pir = DDRPHYC_PIR_QSTRN;
> + else
> + pir = DDRPHYC_PIR_QSTRN | DDRPHYC_PIR_RVTRN;
> + stm32mp1_ddrphy_init(priv->phy, pir);
>
> /* 11. monitor PUB PGSR.IDONE to poll cpmpletion of training sequence */
> ddrphy_idone_wait(priv->phy);
> diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h b/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h
> index 3c8885a965..ada3087328 100644
> --- a/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h
> +++ b/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h
> @@ -309,6 +309,7 @@ struct stm32mp1_ddrphy {
> #define DDRPHYC_PIR_DRAMRST BIT(5)
> #define DDRPHYC_PIR_DRAMINIT BIT(6)
> #define DDRPHYC_PIR_QSTRN BIT(7)
> +#define DDRPHYC_PIR_RVTRN BIT(8)
> #define DDRPHYC_PIR_ICPC BIT(16)
> #define DDRPHYC_PIR_ZCALBYP BIT(30)
> #define DDRPHYC_PIR_INITSTEPS_MASK GENMASK(31, 7)
>
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
Thanks
Patrice
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/3] stm32mp1: ram: remove the support of calibration result
2021-11-15 14:32 ` [PATCH 2/3] stm32mp1: ram: remove the support of calibration result Patrick Delaunay
@ 2021-11-22 7:49 ` Patrice CHOTARD
0 siblings, 0 replies; 7+ messages in thread
From: Patrice CHOTARD @ 2021-11-22 7:49 UTC (permalink / raw)
To: Patrick Delaunay, u-boot; +Cc: Jagan Teki, Matteo Lisi, U-Boot STM32
Hi Patrick
On 11/15/21 3:32 PM, Patrick Delaunay wrote:
> The support of a predefined DDR PHY tuning result is removed for
> STM32MP1 driver because it is not needed at the supported frequency
> when built-in calibration is executed.
>
> The calibration parameters were provided in the device tree by the
> optional node "st,phy-cal", activated in ddr helper file by the
> compilation flag DDR_PHY_CAL_SKIP and filled with values generated
> by the CubeMX DDR utilities.
>
> This patch
> - updates the binding file to remove "st,phy-cal" support
> - updates the device trees and remove the associated defines
> - simplifies the STM32MP1 DDR driver and remove the support of
> the optional parameter "st,phy-cal"
>
> After this patch, the built-in calibration is always executed
> and the calibration registers are moved in the phy dynamic part;
> that allows manual tests.
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp15-ddr.dtsi | 30 ---------
> .../dts/stm32mp15-ddr3-1x4Gb-1066-binG.dtsi | 12 ----
> .../dts/stm32mp15-ddr3-2x4Gb-1066-binG.dtsi | 12 ----
> .../stm32mp15-ddr3-dhsom-2x1Gb-1066-binG.dtsi | 12 ----
> .../stm32mp15-ddr3-dhsom-2x2Gb-1066-binG.dtsi | 12 ----
> .../stm32mp15-ddr3-dhsom-2x4Gb-1066-binG.dtsi | 12 ----
> .../stm32mp15-ddr3-icore-1x4Gb-1066-binG.dtsi | 12 ----
> .../memory-controllers/st,stm32mp1-ddr.txt | 32 ---------
> drivers/ram/stm32mp1/stm32mp1_ddr.c | 65 +++++++------------
> drivers/ram/stm32mp1/stm32mp1_ddr.h | 17 -----
> drivers/ram/stm32mp1/stm32mp1_interactive.c | 15 +----
> drivers/ram/stm32mp1/stm32mp1_ram.c | 22 +------
> 12 files changed, 27 insertions(+), 226 deletions(-)
>
> diff --git a/arch/arm/dts/stm32mp15-ddr.dtsi b/arch/arm/dts/stm32mp15-ddr.dtsi
> index 2a139c54e9..0aac9131a6 100644
> --- a/arch/arm/dts/stm32mp15-ddr.dtsi
> +++ b/arch/arm/dts/stm32mp15-ddr.dtsi
> @@ -116,24 +116,6 @@
> DDR_MR3
> >;
>
> -#ifdef DDR_PHY_CAL_SKIP
> - st,phy-cal = <
> - DDR_DX0DLLCR
> - DDR_DX0DQTR
> - DDR_DX0DQSTR
> - DDR_DX1DLLCR
> - DDR_DX1DQTR
> - DDR_DX1DQSTR
> - DDR_DX2DLLCR
> - DDR_DX2DQTR
> - DDR_DX2DQSTR
> - DDR_DX3DLLCR
> - DDR_DX3DQTR
> - DDR_DX3DQSTR
> - >;
> -
> -#endif
> -
> status = "okay";
> };
> };
> @@ -224,18 +206,6 @@
> #undef DDR_ODTCR
> #undef DDR_ZQ0CR1
> #undef DDR_DX0GCR
> -#undef DDR_DX0DLLCR
> -#undef DDR_DX0DQTR
> -#undef DDR_DX0DQSTR
> #undef DDR_DX1GCR
> -#undef DDR_DX1DLLCR
> -#undef DDR_DX1DQTR
> -#undef DDR_DX1DQSTR
> #undef DDR_DX2GCR
> -#undef DDR_DX2DLLCR
> -#undef DDR_DX2DQTR
> -#undef DDR_DX2DQSTR
> #undef DDR_DX3GCR
> -#undef DDR_DX3DLLCR
> -#undef DDR_DX3DQTR
> -#undef DDR_DX3DQSTR
> diff --git a/arch/arm/dts/stm32mp15-ddr3-1x4Gb-1066-binG.dtsi b/arch/arm/dts/stm32mp15-ddr3-1x4Gb-1066-binG.dtsi
> index 978331b279..e60d0ae606 100644
> --- a/arch/arm/dts/stm32mp15-ddr3-1x4Gb-1066-binG.dtsi
> +++ b/arch/arm/dts/stm32mp15-ddr3-1x4Gb-1066-binG.dtsi
> @@ -100,20 +100,8 @@
> #define DDR_ODTCR 0x00010000
> #define DDR_ZQ0CR1 0x00000038
> #define DDR_DX0GCR 0x0000CE81
> -#define DDR_DX0DLLCR 0x40000000
> -#define DDR_DX0DQTR 0xFFFFFFFF
> -#define DDR_DX0DQSTR 0x3DB02000
> #define DDR_DX1GCR 0x0000CE81
> -#define DDR_DX1DLLCR 0x40000000
> -#define DDR_DX1DQTR 0xFFFFFFFF
> -#define DDR_DX1DQSTR 0x3DB02000
> #define DDR_DX2GCR 0x0000CE80
> -#define DDR_DX2DLLCR 0x40000000
> -#define DDR_DX2DQTR 0xFFFFFFFF
> -#define DDR_DX2DQSTR 0x3DB02000
> #define DDR_DX3GCR 0x0000CE80
> -#define DDR_DX3DLLCR 0x40000000
> -#define DDR_DX3DQTR 0xFFFFFFFF
> -#define DDR_DX3DQSTR 0x3DB02000
>
> #include "stm32mp15-ddr.dtsi"
> diff --git a/arch/arm/dts/stm32mp15-ddr3-2x4Gb-1066-binG.dtsi b/arch/arm/dts/stm32mp15-ddr3-2x4Gb-1066-binG.dtsi
> index 426be21f42..1a6fa80edf 100644
> --- a/arch/arm/dts/stm32mp15-ddr3-2x4Gb-1066-binG.dtsi
> +++ b/arch/arm/dts/stm32mp15-ddr3-2x4Gb-1066-binG.dtsi
> @@ -100,20 +100,8 @@
> #define DDR_ODTCR 0x00010000
> #define DDR_ZQ0CR1 0x00000038
> #define DDR_DX0GCR 0x0000CE81
> -#define DDR_DX0DLLCR 0x40000000
> -#define DDR_DX0DQTR 0xFFFFFFFF
> -#define DDR_DX0DQSTR 0x3DB02000
> #define DDR_DX1GCR 0x0000CE81
> -#define DDR_DX1DLLCR 0x40000000
> -#define DDR_DX1DQTR 0xFFFFFFFF
> -#define DDR_DX1DQSTR 0x3DB02000
> #define DDR_DX2GCR 0x0000CE81
> -#define DDR_DX2DLLCR 0x40000000
> -#define DDR_DX2DQTR 0xFFFFFFFF
> -#define DDR_DX2DQSTR 0x3DB02000
> #define DDR_DX3GCR 0x0000CE81
> -#define DDR_DX3DLLCR 0x40000000
> -#define DDR_DX3DQTR 0xFFFFFFFF
> -#define DDR_DX3DQSTR 0x3DB02000
>
> #include "stm32mp15-ddr.dtsi"
> diff --git a/arch/arm/dts/stm32mp15-ddr3-dhsom-2x1Gb-1066-binG.dtsi b/arch/arm/dts/stm32mp15-ddr3-dhsom-2x1Gb-1066-binG.dtsi
> index b3eb280f96..0a277cd675 100644
> --- a/arch/arm/dts/stm32mp15-ddr3-dhsom-2x1Gb-1066-binG.dtsi
> +++ b/arch/arm/dts/stm32mp15-ddr3-dhsom-2x1Gb-1066-binG.dtsi
> @@ -101,20 +101,8 @@
> #define DDR_ODTCR 0x00010000
> #define DDR_ZQ0CR1 0x00000038
> #define DDR_DX0GCR 0x0000CE81
> -#define DDR_DX0DLLCR 0x40000000
> -#define DDR_DX0DQTR 0xFFFFFFFF
> -#define DDR_DX0DQSTR 0x3DB02000
> #define DDR_DX1GCR 0x0000CE81
> -#define DDR_DX1DLLCR 0x40000000
> -#define DDR_DX1DQTR 0xFFFFFFFF
> -#define DDR_DX1DQSTR 0x3DB02000
> #define DDR_DX2GCR 0x0000CE81
> -#define DDR_DX2DLLCR 0x40000000
> -#define DDR_DX2DQTR 0xFFFFFFFF
> -#define DDR_DX2DQSTR 0x3DB02000
> #define DDR_DX3GCR 0x0000CE81
> -#define DDR_DX3DLLCR 0x40000000
> -#define DDR_DX3DQTR 0xFFFFFFFF
> -#define DDR_DX3DQSTR 0x3DB02000
>
> #include "stm32mp15-ddr.dtsi"
> diff --git a/arch/arm/dts/stm32mp15-ddr3-dhsom-2x2Gb-1066-binG.dtsi b/arch/arm/dts/stm32mp15-ddr3-dhsom-2x2Gb-1066-binG.dtsi
> index ed3a5248f8..92774fffb9 100644
> --- a/arch/arm/dts/stm32mp15-ddr3-dhsom-2x2Gb-1066-binG.dtsi
> +++ b/arch/arm/dts/stm32mp15-ddr3-dhsom-2x2Gb-1066-binG.dtsi
> @@ -101,20 +101,8 @@
> #define DDR_ODTCR 0x00010000
> #define DDR_ZQ0CR1 0x00000038
> #define DDR_DX0GCR 0x0000CE81
> -#define DDR_DX0DLLCR 0x40000000
> -#define DDR_DX0DQTR 0xFFFFFFFF
> -#define DDR_DX0DQSTR 0x3DB02000
> #define DDR_DX1GCR 0x0000CE81
> -#define DDR_DX1DLLCR 0x40000000
> -#define DDR_DX1DQTR 0xFFFFFFFF
> -#define DDR_DX1DQSTR 0x3DB02000
> #define DDR_DX2GCR 0x0000CE81
> -#define DDR_DX2DLLCR 0x40000000
> -#define DDR_DX2DQTR 0xFFFFFFFF
> -#define DDR_DX2DQSTR 0x3DB02000
> #define DDR_DX3GCR 0x0000CE81
> -#define DDR_DX3DLLCR 0x40000000
> -#define DDR_DX3DQTR 0xFFFFFFFF
> -#define DDR_DX3DQSTR 0x3DB02000
>
> #include "stm32mp15-ddr.dtsi"
> diff --git a/arch/arm/dts/stm32mp15-ddr3-dhsom-2x4Gb-1066-binG.dtsi b/arch/arm/dts/stm32mp15-ddr3-dhsom-2x4Gb-1066-binG.dtsi
> index d5813d64b0..e53ab18a69 100644
> --- a/arch/arm/dts/stm32mp15-ddr3-dhsom-2x4Gb-1066-binG.dtsi
> +++ b/arch/arm/dts/stm32mp15-ddr3-dhsom-2x4Gb-1066-binG.dtsi
> @@ -101,20 +101,8 @@
> #define DDR_ODTCR 0x00010000
> #define DDR_ZQ0CR1 0x00000038
> #define DDR_DX0GCR 0x0000CE81
> -#define DDR_DX0DLLCR 0x40000000
> -#define DDR_DX0DQTR 0xFFFFFFFF
> -#define DDR_DX0DQSTR 0x3DB02000
> #define DDR_DX1GCR 0x0000CE81
> -#define DDR_DX1DLLCR 0x40000000
> -#define DDR_DX1DQTR 0xFFFFFFFF
> -#define DDR_DX1DQSTR 0x3DB02000
> #define DDR_DX2GCR 0x0000CE81
> -#define DDR_DX2DLLCR 0x40000000
> -#define DDR_DX2DQTR 0xFFFFFFFF
> -#define DDR_DX2DQSTR 0x3DB02000
> #define DDR_DX3GCR 0x0000CE81
> -#define DDR_DX3DLLCR 0x40000000
> -#define DDR_DX3DQTR 0xFFFFFFFF
> -#define DDR_DX3DQSTR 0x3DB02000
>
> #include "stm32mp15-ddr.dtsi"
> diff --git a/arch/arm/dts/stm32mp15-ddr3-icore-1x4Gb-1066-binG.dtsi b/arch/arm/dts/stm32mp15-ddr3-icore-1x4Gb-1066-binG.dtsi
> index 24c81269b0..ff582ac6af 100644
> --- a/arch/arm/dts/stm32mp15-ddr3-icore-1x4Gb-1066-binG.dtsi
> +++ b/arch/arm/dts/stm32mp15-ddr3-icore-1x4Gb-1066-binG.dtsi
> @@ -100,20 +100,8 @@
> #define DDR_ODTCR 0x00010000
> #define DDR_ZQ0CR1 0x00000038
> #define DDR_DX0GCR 0x0000CE81
> -#define DDR_DX0DLLCR 0x40000000
> -#define DDR_DX0DQTR 0xFFFFFFFF
> -#define DDR_DX0DQSTR 0x3DB02000
> #define DDR_DX1GCR 0x0000CE81
> -#define DDR_DX1DLLCR 0x40000000
> -#define DDR_DX1DQTR 0xFFFFFFFF
> -#define DDR_DX1DQSTR 0x3DB02000
> #define DDR_DX2GCR 0x0000CE81
> -#define DDR_DX2DLLCR 0x40000000
> -#define DDR_DX2DQTR 0xFFFFFFFF
> -#define DDR_DX2DQSTR 0x3DB02000
> #define DDR_DX3GCR 0x0000CE81
> -#define DDR_DX3DLLCR 0x40000000
> -#define DDR_DX3DQTR 0xFFFFFFFF
> -#define DDR_DX3DQSTR 0x3DB02000
>
> #include "stm32mp15-ddr.dtsi"
> diff --git a/doc/device-tree-bindings/memory-controllers/st,stm32mp1-ddr.txt b/doc/device-tree-bindings/memory-controllers/st,stm32mp1-ddr.txt
> index ac6a7df432..926e3e83b3 100644
> --- a/doc/device-tree-bindings/memory-controllers/st,stm32mp1-ddr.txt
> +++ b/doc/device-tree-bindings/memory-controllers/st,stm32mp1-ddr.txt
> @@ -128,23 +128,6 @@ phyc attributes:
> MR2
> MR3
>
> -- st,phy-cal : phy cal depending of calibration or tuning of DDR
> - This parameter is optional; when it is absent the built-in PHY
> - calibration is done.
> - for STM32MP15x: 12 values are requested in this order
> - DX0DLLCR
> - DX0DQTR
> - DX0DQSTR
> - DX1DLLCR
> - DX1DQTR
> - DX1DQSTR
> - DX2DLLCR
> - DX2DQTR
> - DX2DQSTR
> - DX3DLLCR
> - DX3DQTR
> - DX3DQSTR
> -
> Example:
>
> / {
> @@ -280,21 +263,6 @@ Example:
> 0x00000000 /*MR3*/
> >;
>
> - st,phy-cal = <
> - 0x40000000 /*DX0DLLCR*/
> - 0xFFFFFFFF /*DX0DQTR*/
> - 0x3DB02000 /*DX0DQSTR*/
> - 0x40000000 /*DX1DLLCR*/
> - 0xFFFFFFFF /*DX1DQTR*/
> - 0x3DB02000 /*DX1DQSTR*/
> - 0x40000000 /*DX2DLLCR*/
> - 0xFFFFFFFF /*DX2DQTR*/
> - 0x3DB02000 /*DX2DQSTR*/
> - 0x40000000 /*DX3DLLCR*/
> - 0xFFFFFFFF /*DX3DQTR*/
> - 0x3DB02000 /*DX3DQSTR*/
> - >;
> -
> status = "okay";
> };
> };
> diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr.c b/drivers/ram/stm32mp1/stm32mp1_ddr.c
> index 1f8422518b..9d086601a4 100644
> --- a/drivers/ram/stm32mp1/stm32mp1_ddr.c
> +++ b/drivers/ram/stm32mp1/stm32mp1_ddr.c
> @@ -68,7 +68,6 @@ struct reg_desc {
>
> #define DDRPHY_REG_REG_SIZE 11 /* st,phy-reg */
> #define DDRPHY_REG_TIMING_SIZE 10 /* st,phy-timing */
> -#define DDRPHY_REG_CAL_SIZE 12 /* st,phy-cal */
>
> #define DDRCTL_REG_REG(x) DDRCTL_REG(x, stm32mp1_ddrctrl_reg)
> static const struct reg_desc ddr_reg[DDRCTL_REG_REG_SIZE] = {
> @@ -178,22 +177,6 @@ static const struct reg_desc ddrphy_timing[DDRPHY_REG_TIMING_SIZE] = {
> DDRPHY_REG_TIMING(mr3),
> };
>
> -#define DDRPHY_REG_CAL(x) DDRPHY_REG(x, stm32mp1_ddrphy_cal)
> -static const struct reg_desc ddrphy_cal[DDRPHY_REG_CAL_SIZE] = {
> - DDRPHY_REG_CAL(dx0dllcr),
> - DDRPHY_REG_CAL(dx0dqtr),
> - DDRPHY_REG_CAL(dx0dqstr),
> - DDRPHY_REG_CAL(dx1dllcr),
> - DDRPHY_REG_CAL(dx1dqtr),
> - DDRPHY_REG_CAL(dx1dqstr),
> - DDRPHY_REG_CAL(dx2dllcr),
> - DDRPHY_REG_CAL(dx2dqtr),
> - DDRPHY_REG_CAL(dx2dqstr),
> - DDRPHY_REG_CAL(dx3dllcr),
> - DDRPHY_REG_CAL(dx3dqtr),
> - DDRPHY_REG_CAL(dx3dqstr),
> -};
> -
> /**************************************************************
> * DYNAMIC REGISTERS: only used for debug purpose (read/modify)
> **************************************************************/
> @@ -218,12 +201,24 @@ static const struct reg_desc ddrphy_dyn[] = {
> DDRPHY_REG_DYN(zq0sr1),
> DDRPHY_REG_DYN(dx0gsr0),
> DDRPHY_REG_DYN(dx0gsr1),
> + DDRPHY_REG_DYN(dx0dllcr),
> + DDRPHY_REG_DYN(dx0dqtr),
> + DDRPHY_REG_DYN(dx0dqstr),
> DDRPHY_REG_DYN(dx1gsr0),
> DDRPHY_REG_DYN(dx1gsr1),
> + DDRPHY_REG_DYN(dx1dllcr),
> + DDRPHY_REG_DYN(dx1dqtr),
> + DDRPHY_REG_DYN(dx1dqstr),
> DDRPHY_REG_DYN(dx2gsr0),
> DDRPHY_REG_DYN(dx2gsr1),
> + DDRPHY_REG_DYN(dx2dllcr),
> + DDRPHY_REG_DYN(dx2dqtr),
> + DDRPHY_REG_DYN(dx2dqstr),
> DDRPHY_REG_DYN(dx3gsr0),
> DDRPHY_REG_DYN(dx3gsr1),
> + DDRPHY_REG_DYN(dx3dllcr),
> + DDRPHY_REG_DYN(dx3dqtr),
> + DDRPHY_REG_DYN(dx3dqstr),
> };
>
> #define DDRPHY_REG_DYN_SIZE ARRAY_SIZE(ddrphy_dyn)
> @@ -240,7 +235,6 @@ enum reg_type {
> REG_MAP,
> REGPHY_REG,
> REGPHY_TIMING,
> - REGPHY_CAL,
> #ifdef CONFIG_STM32MP1_DDR_INTERACTIVE
> /* dynamic registers => managed in driver or not changed,
> * can be dumped in interactive mode
> @@ -264,8 +258,6 @@ struct ddr_reg_info {
> enum base_type base;
> };
>
> -#define DDRPHY_REG_CAL(x) DDRPHY_REG(x, stm32mp1_ddrphy_cal)
> -
> const struct ddr_reg_info ddr_registers[REG_TYPE_NB] = {
> [REG_REG] = {
> "static", ddr_reg, DDRCTL_REG_REG_SIZE, DDR_BASE},
> @@ -279,8 +271,6 @@ const struct ddr_reg_info ddr_registers[REG_TYPE_NB] = {
> "static", ddrphy_reg, DDRPHY_REG_REG_SIZE, DDRPHY_BASE},
> [REGPHY_TIMING] = {
> "timing", ddrphy_timing, DDRPHY_REG_TIMING_SIZE, DDRPHY_BASE},
> -[REGPHY_CAL] = {
> - "cal", ddrphy_cal, DDRPHY_REG_CAL_SIZE, DDRPHY_BASE},
> #ifdef CONFIG_STM32MP1_DDR_INTERACTIVE
> [REG_DYN] = {
> "dyn", ddr_dyn, DDR_REG_DYN_SIZE, DDR_BASE},
> @@ -456,9 +446,6 @@ static u32 get_par_addr(const struct stm32mp1_ddr_config *config,
> case REGPHY_TIMING:
> par_addr = (u32)&config->p_timing;
> break;
> - case REGPHY_CAL:
> - par_addr = (u32)&config->p_cal;
> - break;
> case REG_DYN:
> case REGPHY_DYN:
> case REG_TYPE_NB:
> @@ -774,8 +761,6 @@ start:
> */
> set_reg(priv, REGPHY_REG, &config->p_reg);
> set_reg(priv, REGPHY_TIMING, &config->p_timing);
> - if (config->p_cal_present)
> - set_reg(priv, REGPHY_CAL, &config->p_cal);
>
> if (INTERACTIVE(STEP_PHY_INIT))
> goto start;
> @@ -810,36 +795,32 @@ start:
>
> wait_operating_mode(priv, DDRCTRL_STAT_OPERATING_MODE_NORMAL);
>
> - if (config->p_cal_present) {
> - log_debug("DDR DQS training skipped.\n");
> - } else {
> - log_debug("DDR DQS training : ");
> + log_debug("DDR DQS training : ");
> /* 8. Disable Auto refresh and power down by setting
> * - RFSHCTL3.dis_au_refresh = 1
> * - PWRCTL.powerdown_en = 0
> * - DFIMISC.dfiinit_complete_en = 0
> */
> - stm32mp1_refresh_disable(priv->ctl);
> + stm32mp1_refresh_disable(priv->ctl);
>
> /* 9. Program PUBL PGCR to enable refresh during training and rank to train
> * not done => keep the programed value in PGCR
> */
>
> /* 10. configure PUBL PIR register to specify which training step to run */
> - /* RVTRN is excuted only on LPDDR2/LPDDR3 */
> - if (config->c_reg.mstr & DDRCTRL_MSTR_DDR3)
> - pir = DDRPHYC_PIR_QSTRN;
> - else
> - pir = DDRPHYC_PIR_QSTRN | DDRPHYC_PIR_RVTRN;
> - stm32mp1_ddrphy_init(priv->phy, pir);
> + /* RVTRN is excuted only on LPDDR2/LPDDR3 */
> + if (config->c_reg.mstr & DDRCTRL_MSTR_DDR3)
> + pir = DDRPHYC_PIR_QSTRN;
> + else
> + pir = DDRPHYC_PIR_QSTRN | DDRPHYC_PIR_RVTRN;
> + stm32mp1_ddrphy_init(priv->phy, pir);
>
> /* 11. monitor PUB PGSR.IDONE to poll cpmpletion of training sequence */
> - ddrphy_idone_wait(priv->phy);
> + ddrphy_idone_wait(priv->phy);
>
> /* 12. set back registers in step 8 to the orginal values if desidered */
> - stm32mp1_refresh_restore(priv->ctl, config->c_reg.rfshctl3,
> - config->c_reg.pwrctl);
> - } /* if (config->p_cal_present) */
> + stm32mp1_refresh_restore(priv->ctl, config->c_reg.rfshctl3,
> + config->c_reg.pwrctl);
>
> /* enable uMCTL2 AXI port 0 and 1 */
> setbits_le32(&priv->ctl->pctrl_0, DDRCTRL_PCTRL_N_PORT_EN);
> diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr.h b/drivers/ram/stm32mp1/stm32mp1_ddr.h
> index 4998f04439..3bfcb85a8f 100644
> --- a/drivers/ram/stm32mp1/stm32mp1_ddr.h
> +++ b/drivers/ram/stm32mp1/stm32mp1_ddr.h
> @@ -140,21 +140,6 @@ struct stm32mp1_ddrphy_timing {
> u32 mr3;
> };
>
> -struct stm32mp1_ddrphy_cal {
> - u32 dx0dllcr;
> - u32 dx0dqtr;
> - u32 dx0dqstr;
> - u32 dx1dllcr;
> - u32 dx1dqtr;
> - u32 dx1dqstr;
> - u32 dx2dllcr;
> - u32 dx2dqtr;
> - u32 dx2dqstr;
> - u32 dx3dllcr;
> - u32 dx3dqtr;
> - u32 dx3dqstr;
> -};
> -
> struct stm32mp1_ddr_info {
> const char *name;
> u32 speed; /* in kHZ */
> @@ -169,8 +154,6 @@ struct stm32mp1_ddr_config {
> struct stm32mp1_ddrctrl_perf c_perf;
> struct stm32mp1_ddrphy_reg p_reg;
> struct stm32mp1_ddrphy_timing p_timing;
> - struct stm32mp1_ddrphy_cal p_cal;
> - bool p_cal_present;
> };
>
> int stm32mp1_ddr_clk_enable(struct ddr_info *priv, u32 mem_speed);
> diff --git a/drivers/ram/stm32mp1/stm32mp1_interactive.c b/drivers/ram/stm32mp1/stm32mp1_interactive.c
> index 8c2310ac90..a667d49cff 100644
> --- a/drivers/ram/stm32mp1/stm32mp1_interactive.c
> +++ b/drivers/ram/stm32mp1/stm32mp1_interactive.c
> @@ -111,7 +111,7 @@ static void stm32mp1_do_usage(void)
> "help displays help\n"
> "info displays DDR information\n"
> "info <param> <val> changes DDR information\n"
> - " with <param> = step, name, size, speed or cal\n"
> + " with <param> = step, name, size or speed\n"
> "freq displays the DDR PHY frequency in kHz\n"
> "freq <freq> changes the DDR PHY frequency\n"
> "param [type|reg] prints input parameters\n"
> @@ -132,7 +132,7 @@ static void stm32mp1_do_usage(void)
> "\nwith for [type|reg]:\n"
> " all registers if absent\n"
> " <type> = ctl, phy\n"
> - " or one category (static, timing, map, perf, cal, dyn)\n"
> + " or one category (static, timing, map, perf, dyn)\n"
> " <reg> = name of the register\n"
> };
>
> @@ -165,7 +165,6 @@ static void stm32mp1_do_info(struct ddr_info *priv,
> printf("name = %s\n", config->info.name);
> printf("size = 0x%x\n", config->info.size);
> printf("speed = %d kHz\n", config->info.speed);
> - printf("cal = %d\n", config->p_cal_present);
> return;
> }
>
> @@ -214,16 +213,6 @@ static void stm32mp1_do_info(struct ddr_info *priv,
> }
> return;
> }
> - if (!strcmp(argv[1], "cal")) {
> - if (strict_strtoul(argv[2], 10, &value) < 0 ||
> - (value != 0 && value != 1)) {
> - printf("invalid value %s\n", argv[2]);
> - } else {
> - config->p_cal_present = value;
> - printf("cal = %d\n", config->p_cal_present);
> - }
> - return;
> - }
> printf("argument %s invalid\n", argv[1]);
> }
>
> diff --git a/drivers/ram/stm32mp1/stm32mp1_ram.c b/drivers/ram/stm32mp1/stm32mp1_ram.c
> index 98fa1f4f11..3b65269b98 100644
> --- a/drivers/ram/stm32mp1/stm32mp1_ram.c
> +++ b/drivers/ram/stm32mp1/stm32mp1_ram.c
> @@ -95,26 +95,22 @@ static __maybe_unused int stm32mp1_ddr_setup(struct udevice *dev)
> { .name = x, \
> .offset = offsetof(struct stm32mp1_ddr_config, y), \
> .size = sizeof(config.y) / sizeof(u32), \
> - .present = z, \
> }
>
> #define CTL_PARAM(x) PARAM("st,ctl-"#x, c_##x, NULL)
> #define PHY_PARAM(x) PARAM("st,phy-"#x, p_##x, NULL)
> -#define PHY_PARAM_OPT(x) PARAM("st,phy-"#x, p_##x, &config.p_##x##_present)
>
> const struct {
> const char *name; /* name in DT */
> const u32 offset; /* offset in config struct */
> const u32 size; /* size of parameters */
> - bool * const present; /* presence indication for opt */
> } param[] = {
> CTL_PARAM(reg),
> CTL_PARAM(timing),
> CTL_PARAM(map),
> CTL_PARAM(perf),
> PHY_PARAM(reg),
> - PHY_PARAM(timing),
> - PHY_PARAM_OPT(cal)
> + PHY_PARAM(timing)
> };
>
> config.info.speed = ofnode_read_u32_default(node, "st,mem-speed", 0);
> @@ -133,25 +129,11 @@ static __maybe_unused int stm32mp1_ddr_setup(struct udevice *dev)
> param[idx].size);
> dev_dbg(dev, "%s: %s[0x%x] = %d\n", __func__,
> param[idx].name, param[idx].size, ret);
> - if (ret &&
> - (ret != -FDT_ERR_NOTFOUND || !param[idx].present)) {
> + if (ret) {
> dev_err(dev, "Cannot read %s, error=%d\n",
> param[idx].name, ret);
> return -EINVAL;
> }
> - if (param[idx].present) {
> - /* save presence of optional parameters */
> - *param[idx].present = true;
> - if (ret == -FDT_ERR_NOTFOUND) {
> - *param[idx].present = false;
> -#ifdef CONFIG_STM32MP1_DDR_INTERACTIVE
> - /* reset values if used later */
> - memset((void *)((u32)&config +
> - param[idx].offset),
> - 0, param[idx].size * sizeof(u32));
> -#endif
> - }
> - }
> }
>
> ret = clk_get_by_name(dev, "axidcg", &axidcg);
>
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
Thanks
Patrice
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 3/3] stm32mp1: ram: remove tuning support
2021-11-15 14:32 ` [PATCH 3/3] stm32mp1: ram: remove tuning support Patrick Delaunay
@ 2021-11-22 7:51 ` Patrice CHOTARD
0 siblings, 0 replies; 7+ messages in thread
From: Patrice CHOTARD @ 2021-11-22 7:51 UTC (permalink / raw)
To: Patrick Delaunay, u-boot; +Cc: U-Boot STM32
Hi Patrick
On 11/15/21 3:32 PM, Patrick Delaunay wrote:
> Remove the DDR interactive command tuning, as the support of a predefined
> DDR PHY tuning is removed for STM32MP1 driver in SPL and in TF-A
> and the result of this tuning will be never used.
>
> Moreover this SW tuning procedure can failed on some hardware
> configuration (to many BIST errors and no convergence); it will be no
> more supported in the next delivery of the DDR utilities included in
> the CubeMX tool of STMicroelectronics.
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
> ---
>
> drivers/ram/stm32mp1/Makefile | 1 -
> drivers/ram/stm32mp1/stm32mp1_ddr.c | 8 +-
> drivers/ram/stm32mp1/stm32mp1_ddr.h | 5 -
> drivers/ram/stm32mp1/stm32mp1_ddr_regs.h | 64 +-
> drivers/ram/stm32mp1/stm32mp1_interactive.c | 22 +-
> drivers/ram/stm32mp1/stm32mp1_tests.h | 3 -
> drivers/ram/stm32mp1/stm32mp1_tuning.c | 1540 -------------------
> 7 files changed, 7 insertions(+), 1636 deletions(-)
> delete mode 100644 drivers/ram/stm32mp1/stm32mp1_tuning.c
>
> diff --git a/drivers/ram/stm32mp1/Makefile b/drivers/ram/stm32mp1/Makefile
> index e1e9135603..71ded6bed4 100644
> --- a/drivers/ram/stm32mp1/Makefile
> +++ b/drivers/ram/stm32mp1/Makefile
> @@ -8,7 +8,6 @@ obj-y += stm32mp1_ddr.o
>
> obj-$(CONFIG_STM32MP1_DDR_INTERACTIVE) += stm32mp1_interactive.o
> obj-$(CONFIG_STM32MP1_DDR_TESTS) += stm32mp1_tests.o
> -obj-$(CONFIG_STM32MP1_DDR_TUNING) += stm32mp1_tuning.o
>
> ifneq ($(DDR_INTERACTIVE),)
> CFLAGS_stm32mp1_interactive.o += -DCONFIG_STM32MP1_DDR_INTERACTIVE_FORCE=y
> diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr.c b/drivers/ram/stm32mp1/stm32mp1_ddr.c
> index 9d086601a4..4d78aa5cb1 100644
> --- a/drivers/ram/stm32mp1/stm32mp1_ddr.c
> +++ b/drivers/ram/stm32mp1/stm32mp1_ddr.c
> @@ -557,7 +557,7 @@ static void ddrphy_idone_wait(struct stm32mp1_ddrphy *phy)
> (u32)&phy->pgsr, pgsr, ret);
> }
>
> -void stm32mp1_ddrphy_init(struct stm32mp1_ddrphy *phy, u32 pir)
> +static void stm32mp1_ddrphy_init(struct stm32mp1_ddrphy *phy, u32 pir)
> {
> pir |= DDRPHYC_PIR_INIT;
> writel(pir, &phy->pir);
> @@ -626,7 +626,7 @@ static void wait_operating_mode(struct ddr_info *priv, int mode)
> log_debug("[0x%08x] stat = 0x%08x\n", (u32)&priv->ctl->stat, stat);
> }
>
> -void stm32mp1_refresh_disable(struct stm32mp1_ddrctl *ctl)
> +static void stm32mp1_refresh_disable(struct stm32mp1_ddrctl *ctl)
> {
> start_sw_done(ctl);
> /* quasi-dynamic register update*/
> @@ -637,8 +637,8 @@ void stm32mp1_refresh_disable(struct stm32mp1_ddrctl *ctl)
> wait_sw_done_ack(ctl);
> }
>
> -void stm32mp1_refresh_restore(struct stm32mp1_ddrctl *ctl,
> - u32 rfshctl3, u32 pwrctl)
> +static void stm32mp1_refresh_restore(struct stm32mp1_ddrctl *ctl,
> + u32 rfshctl3, u32 pwrctl)
> {
> start_sw_done(ctl);
> if (!(rfshctl3 & DDRCTRL_RFSHCTL3_DIS_AUTO_REFRESH))
> diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr.h b/drivers/ram/stm32mp1/stm32mp1_ddr.h
> index 3bfcb85a8f..861efff92b 100644
> --- a/drivers/ram/stm32mp1/stm32mp1_ddr.h
> +++ b/drivers/ram/stm32mp1/stm32mp1_ddr.h
> @@ -157,11 +157,6 @@ struct stm32mp1_ddr_config {
> };
>
> int stm32mp1_ddr_clk_enable(struct ddr_info *priv, u32 mem_speed);
> -void stm32mp1_ddrphy_init(struct stm32mp1_ddrphy *phy, u32 pir);
> -void stm32mp1_refresh_disable(struct stm32mp1_ddrctl *ctl);
> -void stm32mp1_refresh_restore(struct stm32mp1_ddrctl *ctl,
> - u32 rfshctl3,
> - u32 pwrctl);
>
> void stm32mp1_ddr_init(
> struct ddr_info *priv,
> diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h b/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h
> index ada3087328..be89d81018 100644
> --- a/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h
> +++ b/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h
> @@ -6,8 +6,9 @@
> #ifndef _RAM_STM32MP1_DDR_REGS_H
> #define _RAM_STM32MP1_DDR_REGS_H
>
> -/* DDR3/LPDDR2/LPDDR3 Controller (DDRCTRL) registers */
> #include <linux/bitops.h>
> +
> +/* DDR3/LPDDR2/LPDDR3 Controller (DDRCTRL) registers */
> struct stm32mp1_ddrctl {
> u32 mstr ; /* 0x0 Master*/
> u32 stat; /* 0x4 Operating Mode Status*/
> @@ -275,25 +276,6 @@ struct stm32mp1_ddrphy {
>
> #define DDRCTRL_DFIMISC_DFI_INIT_COMPLETE_EN BIT(0)
>
> -#define DDRCTRL_DBG1_DIS_HIF BIT(1)
> -
> -#define DDRCTRL_DBGCAM_WR_DATA_PIPELINE_EMPTY BIT(29)
> -#define DDRCTRL_DBGCAM_RD_DATA_PIPELINE_EMPTY BIT(28)
> -#define DDRCTRL_DBGCAM_DBG_WR_Q_EMPTY BIT(26)
> -#define DDRCTRL_DBGCAM_DBG_LPR_Q_DEPTH GENMASK(12, 8)
> -#define DDRCTRL_DBGCAM_DBG_HPR_Q_DEPTH GENMASK(4, 0)
> -#define DDRCTRL_DBGCAM_DATA_PIPELINE_EMPTY \
> - (DDRCTRL_DBGCAM_WR_DATA_PIPELINE_EMPTY | \
> - DDRCTRL_DBGCAM_RD_DATA_PIPELINE_EMPTY)
> -#define DDRCTRL_DBGCAM_DBG_Q_DEPTH \
> - (DDRCTRL_DBGCAM_DBG_WR_Q_EMPTY | \
> - DDRCTRL_DBGCAM_DBG_LPR_Q_DEPTH | \
> - DDRCTRL_DBGCAM_DBG_HPR_Q_DEPTH)
> -
> -#define DDRCTRL_DBGCMD_RANK0_REFRESH BIT(0)
> -
> -#define DDRCTRL_DBGSTAT_RANK0_REFRESH_BUSY BIT(0)
> -
> #define DDRCTRL_SWCTL_SW_DONE BIT(0)
>
> #define DDRCTRL_SWSTAT_SW_DONE_ACK BIT(0)
> @@ -312,11 +294,6 @@ struct stm32mp1_ddrphy {
> #define DDRPHYC_PIR_RVTRN BIT(8)
> #define DDRPHYC_PIR_ICPC BIT(16)
> #define DDRPHYC_PIR_ZCALBYP BIT(30)
> -#define DDRPHYC_PIR_INITSTEPS_MASK GENMASK(31, 7)
> -
> -#define DDRPHYC_PGCR_DFTCMP BIT(2)
> -#define DDRPHYC_PGCR_PDDISDX BIT(24)
> -#define DDRPHYC_PGCR_RFSHDT_MASK GENMASK(28, 25)
>
> #define DDRPHYC_PGSR_IDONE BIT(0)
> #define DDRPHYC_PGSR_DTERR BIT(5)
> @@ -325,43 +302,6 @@ struct stm32mp1_ddrphy {
> #define DDRPHYC_PGSR_RVERR BIT(8)
> #define DDRPHYC_PGSR_RVEIRR BIT(9)
>
> -#define DDRPHYC_DLLGCR_BPS200 BIT(23)
> -
> -#define DDRPHYC_ACDLLCR_DLLDIS BIT(31)
> -
> -#define DDRPHYC_ZQ0CRN_ZDATA_MASK GENMASK(27, 0)
> -#define DDRPHYC_ZQ0CRN_ZDATA_SHIFT 0
> -#define DDRPHYC_ZQ0CRN_ZDEN BIT(28)
> -
> -#define DDRPHYC_DXNGCR_DXEN BIT(0)
> -
> -#define DDRPHYC_DXNDLLCR_DLLSRST BIT(30)
> -#define DDRPHYC_DXNDLLCR_DLLDIS BIT(31)
> -#define DDRPHYC_DXNDLLCR_SDPHASE_MASK GENMASK(17, 14)
> -#define DDRPHYC_DXNDLLCR_SDPHASE_SHIFT 14
> -
> -#define DDRPHYC_DXNDQTR_DQDLY_SHIFT(bit) (4 * (bit))
> -#define DDRPHYC_DXNDQTR_DQDLY_MASK GENMASK(3, 0)
> -#define DDRPHYC_DXNDQTR_DQDLY_LOW_MASK GENMASK(1, 0)
> -#define DDRPHYC_DXNDQTR_DQDLY_HIGH_MASK GENMASK(3, 2)
> -
> -#define DDRPHYC_DXNDQSTR_DQSDLY_MASK GENMASK(22, 20)
> -#define DDRPHYC_DXNDQSTR_DQSDLY_SHIFT 20
> -#define DDRPHYC_DXNDQSTR_DQSNDLY_MASK GENMASK(25, 23)
> -#define DDRPHYC_DXNDQSTR_DQSNDLY_SHIFT 23
> -#define DDRPHYC_DXNDQSTR_R0DGSL_MASK GENMASK(2, 0)
> -#define DDRPHYC_DXNDQSTR_R0DGSL_SHIFT 0
> -#define DDRPHYC_DXNDQSTR_R0DGPS_MASK GENMASK(13, 12)
> -#define DDRPHYC_DXNDQSTR_R0DGPS_SHIFT 12
> -
> -#define DDRPHYC_BISTRR_BDXSEL_MASK GENMASK(22, 19)
> -#define DDRPHYC_BISTRR_BDXSEL_SHIFT 19
> -
> -#define DDRPHYC_BISTGSR_BDDONE BIT(0)
> -#define DDRPHYC_BISTGSR_BDXERR BIT(2)
> -
> -#define DDRPHYC_BISTWCSR_DXWCNT_SHIFT 16
> -
> /* PWR registers */
> #define PWR_CR3 0x00C
> #define PWR_CR3_DDRSRDIS BIT(11)
> diff --git a/drivers/ram/stm32mp1/stm32mp1_interactive.c b/drivers/ram/stm32mp1/stm32mp1_interactive.c
> index a667d49cff..f0fe7e61e3 100644
> --- a/drivers/ram/stm32mp1/stm32mp1_interactive.c
> +++ b/drivers/ram/stm32mp1/stm32mp1_interactive.c
> @@ -32,7 +32,6 @@ enum ddr_command {
> DDR_CMD_NEXT,
> DDR_CMD_GO,
> DDR_CMD_TEST,
> - DDR_CMD_TUNING,
> DDR_CMD_UNKNOWN,
> };
>
> @@ -59,9 +58,6 @@ enum ddr_command stm32mp1_get_command(char *cmd, int argc)
> [DDR_CMD_GO] = "go",
> #ifdef CONFIG_STM32MP1_DDR_TESTS
> [DDR_CMD_TEST] = "test",
> -#endif
> -#ifdef CONFIG_STM32MP1_DDR_TUNING
> - [DDR_CMD_TUNING] = "tuning",
> #endif
> };
> /* min and max number of argument */
> @@ -78,9 +74,6 @@ enum ddr_command stm32mp1_get_command(char *cmd, int argc)
> [DDR_CMD_GO] = { 0, 0 },
> #ifdef CONFIG_STM32MP1_DDR_TESTS
> [DDR_CMD_TEST] = { 0, 255 },
> -#endif
> -#ifdef CONFIG_STM32MP1_DDR_TUNING
> - [DDR_CMD_TUNING] = { 0, 255 },
> #endif
> };
> int i;
> @@ -125,9 +118,6 @@ static void stm32mp1_do_usage(void)
> "reset reboots machine\n"
> #ifdef CONFIG_STM32MP1_DDR_TESTS
> "test [help] | <n> [...] lists (with help) or executes test <n>\n"
> -#endif
> -#ifdef CONFIG_STM32MP1_DDR_TUNING
> - "tuning [help] | <n> [...] lists (with help) or execute tuning <n>\n"
> #endif
> "\nwith for [type|reg]:\n"
> " all registers if absent\n"
> @@ -311,7 +301,7 @@ end:
> return step;
> }
>
> -#if defined(CONFIG_STM32MP1_DDR_TESTS) || defined(CONFIG_STM32MP1_DDR_TUNING)
> +#if defined(CONFIG_STM32MP1_DDR_TESTS)
> static const char * const s_result[] = {
> [TEST_PASSED] = "Pass",
> [TEST_FAILED] = "Failed",
> @@ -468,16 +458,6 @@ bool stm32mp1_ddr_interactive(void *priv,
> stm32mp1_ddr_subcmd(priv, argc, argv, test, test_nb);
> break;
> #endif
> -
> -#ifdef CONFIG_STM32MP1_DDR_TUNING
> - case DDR_CMD_TUNING:
> - if (!stm32mp1_check_step(step, STEP_DDR_READY))
> - continue;
> - stm32mp1_ddr_subcmd(priv, argc, argv,
> - tuning, tuning_nb);
> - break;
> -#endif
> -
> default:
> break;
> }
> diff --git a/drivers/ram/stm32mp1/stm32mp1_tests.h b/drivers/ram/stm32mp1/stm32mp1_tests.h
> index 55f5d6d93b..8436780790 100644
> --- a/drivers/ram/stm32mp1/stm32mp1_tests.h
> +++ b/drivers/ram/stm32mp1/stm32mp1_tests.h
> @@ -28,7 +28,4 @@ struct test_desc {
> extern const struct test_desc test[];
> extern const int test_nb;
>
> -extern const struct test_desc tuning[];
> -extern const int tuning_nb;
> -
> #endif
> diff --git a/drivers/ram/stm32mp1/stm32mp1_tuning.c b/drivers/ram/stm32mp1/stm32mp1_tuning.c
> deleted file mode 100644
> index c8cd7c3cea..0000000000
> --- a/drivers/ram/stm32mp1/stm32mp1_tuning.c
> +++ /dev/null
> @@ -1,1540 +0,0 @@
> -// SPDX-License-Identifier: GPL-2.0+ OR BSD-3-Clause
> -/*
> - * Copyright (C) 2019, STMicroelectronics - All Rights Reserved
> - */
> -
> -#define LOG_CATEGORY UCLASS_RAM
> -
> -#include <common.h>
> -#include <console.h>
> -#include <clk.h>
> -#include <log.h>
> -#include <ram.h>
> -#include <rand.h>
> -#include <reset.h>
> -#include <asm/io.h>
> -#include <linux/bitops.h>
> -#include <linux/delay.h>
> -#include <linux/iopoll.h>
> -
> -#include "stm32mp1_ddr_regs.h"
> -#include "stm32mp1_ddr.h"
> -#include "stm32mp1_tests.h"
> -
> -#define MAX_DQS_PHASE_IDX _144deg
> -#define MAX_DQS_UNIT_IDX 7
> -#define MAX_GSL_IDX 5
> -#define MAX_GPS_IDX 3
> -
> -/* Number of bytes used in this SW. ( min 1--> max 4). */
> -#define NUM_BYTES 4
> -
> -enum dqs_phase_enum {
> - _36deg = 0,
> - _54deg = 1,
> - _72deg = 2,
> - _90deg = 3,
> - _108deg = 4,
> - _126deg = 5,
> - _144deg = 6
> -};
> -
> -/* BIST Result struct */
> -struct BIST_result {
> - /* Overall test result:
> - * 0 Fail (any bit failed) ,
> - * 1 Success (All bits success)
> - */
> - bool test_result;
> - /* 1: true, all fail / 0: False, not all bits fail */
> - bool all_bits_fail;
> - bool bit_i_test_result[8]; /* 0 fail / 1 success */
> -};
> -
> -/* a struct that defines tuning parameters of a byte. */
> -struct tuning_position {
> - u8 phase; /* DQS phase */
> - u8 unit; /* DQS unit delay */
> - u32 bits_delay; /* Bits deskew in this byte */
> -};
> -
> -/* 36deg, 54deg, 72deg, 90deg, 108deg, 126deg, 144deg */
> -const u8 dx_dll_phase[7] = {3, 2, 1, 0, 14, 13, 12};
> -
> -static u8 BIST_error_max = 1;
> -static u32 BIST_seed = 0x1234ABCD;
> -
> -static u8 get_nb_bytes(struct stm32mp1_ddrctl *ctl)
> -{
> - u32 data_bus = readl(&ctl->mstr) & DDRCTRL_MSTR_DATA_BUS_WIDTH_MASK;
> - u8 nb_bytes = NUM_BYTES;
> -
> - switch (data_bus) {
> - case DDRCTRL_MSTR_DATA_BUS_WIDTH_HALF:
> - nb_bytes /= 2;
> - break;
> - case DDRCTRL_MSTR_DATA_BUS_WIDTH_QUARTER:
> - nb_bytes /= 4;
> - break;
> - default:
> - break;
> - }
> -
> - return nb_bytes;
> -}
> -
> -static u8 get_nb_bank(struct stm32mp1_ddrctl *ctl)
> -{
> - /* Count bank address bits */
> - u8 bits = 0;
> - u32 reg, val;
> -
> - reg = readl(&ctl->addrmap1);
> - /* addrmap1.addrmap_bank_b1 */
> - val = (reg & GENMASK(5, 0)) >> 0;
> - if (val <= 31)
> - bits++;
> - /* addrmap1.addrmap_bank_b2 */
> - val = (reg & GENMASK(13, 8)) >> 8;
> - if (val <= 31)
> - bits++;
> - /* addrmap1.addrmap_bank_b3 */
> - val = (reg & GENMASK(21, 16)) >> 16;
> - if (val <= 31)
> - bits++;
> -
> - return bits;
> -}
> -
> -static u8 get_nb_col(struct stm32mp1_ddrctl *ctl)
> -{
> - u8 bits;
> - u32 reg, val;
> -
> - /* Count column address bits, start at 2 for b0 and b1 (fixed) */
> - bits = 2;
> -
> - reg = readl(&ctl->addrmap2);
> - /* addrmap2.addrmap_col_b2 */
> - val = (reg & GENMASK(3, 0)) >> 0;
> - if (val <= 7)
> - bits++;
> - /* addrmap2.addrmap_col_b3 */
> - val = (reg & GENMASK(11, 8)) >> 8;
> - if (val <= 7)
> - bits++;
> - /* addrmap2.addrmap_col_b4 */
> - val = (reg & GENMASK(19, 16)) >> 16;
> - if (val <= 7)
> - bits++;
> - /* addrmap2.addrmap_col_b5 */
> - val = (reg & GENMASK(27, 24)) >> 24;
> - if (val <= 7)
> - bits++;
> -
> - reg = readl(&ctl->addrmap3);
> - /* addrmap3.addrmap_col_b6 */
> - val = (reg & GENMASK(3, 0)) >> 0;
> - if (val <= 7)
> - bits++;
> - /* addrmap3.addrmap_col_b7 */
> - val = (reg & GENMASK(11, 8)) >> 8;
> - if (val <= 7)
> - bits++;
> - /* addrmap3.addrmap_col_b8 */
> - val = (reg & GENMASK(19, 16)) >> 16;
> - if (val <= 7)
> - bits++;
> - /* addrmap3.addrmap_col_b9 */
> - val = (reg & GENMASK(27, 24)) >> 24;
> - if (val <= 7)
> - bits++;
> -
> - reg = readl(&ctl->addrmap4);
> - /* addrmap4.addrmap_col_b10 */
> - val = (reg & GENMASK(3, 0)) >> 0;
> - if (val <= 7)
> - bits++;
> - /* addrmap4.addrmap_col_b11 */
> - val = (reg & GENMASK(11, 8)) >> 8;
> - if (val <= 7)
> - bits++;
> -
> - return bits;
> -}
> -
> -static u8 get_nb_row(struct stm32mp1_ddrctl *ctl)
> -{
> - /* Count row address bits */
> - u8 bits = 0;
> - u32 reg, val;
> -
> - reg = readl(&ctl->addrmap5);
> - /* addrmap5.addrmap_row_b0 */
> - val = (reg & GENMASK(3, 0)) >> 0;
> - if (val <= 11)
> - bits++;
> - /* addrmap5.addrmap_row_b1 */
> - val = (reg & GENMASK(11, 8)) >> 8;
> - if (val <= 11)
> - bits++;
> - /* addrmap5.addrmap_row_b2_10 */
> - val = (reg & GENMASK(19, 16)) >> 16;
> - if (val <= 11)
> - bits += 9;
> - else
> - printf("warning: addrmap5.addrmap_row_b2_10 not supported\n");
> - /* addrmap5.addrmap_row_b11 */
> - val = (reg & GENMASK(27, 24)) >> 24;
> - if (val <= 11)
> - bits++;
> -
> - reg = readl(&ctl->addrmap6);
> - /* addrmap6.addrmap_row_b12 */
> - val = (reg & GENMASK(3, 0)) >> 0;
> - if (val <= 7)
> - bits++;
> - /* addrmap6.addrmap_row_b13 */
> - val = (reg & GENMASK(11, 8)) >> 8;
> - if (val <= 7)
> - bits++;
> - /* addrmap6.addrmap_row_b14 */
> - val = (reg & GENMASK(19, 16)) >> 16;
> - if (val <= 7)
> - bits++;
> - /* addrmap6.addrmap_row_b15 */
> - val = (reg & GENMASK(27, 24)) >> 24;
> - if (val <= 7)
> - bits++;
> -
> - return bits;
> -}
> -
> -static void itm_soft_reset(struct stm32mp1_ddrphy *phy)
> -{
> - stm32mp1_ddrphy_init(phy, DDRPHYC_PIR_ITMSRST);
> -}
> -
> -/* Read DQ unit delay register and provides the retrieved value for DQS
> - * We are assuming that we have the same delay when clocking
> - * by DQS and when clocking by DQSN
> - */
> -static u8 DQ_unit_index(struct stm32mp1_ddrphy *phy, u8 byte, u8 bit)
> -{
> - u32 index;
> - u32 addr = DXNDQTR(phy, byte);
> -
> - /* We are assuming that we have the same delay when clocking by DQS
> - * and when clocking by DQSN : use only the low bits
> - */
> - index = (readl(addr) >> DDRPHYC_DXNDQTR_DQDLY_SHIFT(bit))
> - & DDRPHYC_DXNDQTR_DQDLY_LOW_MASK;
> -
> - log_debug("[%x]: %x => DQ unit index = %x\n", addr, readl(addr), index);
> -
> - return index;
> -}
> -
> -/* Sets the DQS phase delay for a byte lane.
> - *phase delay is specified by giving the index of the desired delay
> - * in the dx_dll_phase array.
> - */
> -static void DQS_phase_delay(struct stm32mp1_ddrphy *phy, u8 byte, u8 phase_idx)
> -{
> - u8 sdphase_val = 0;
> -
> - /* Write DXNDLLCR.SDPHASE = dx_dll_phase(phase_index); */
> - sdphase_val = dx_dll_phase[phase_idx];
> - clrsetbits_le32(DXNDLLCR(phy, byte),
> - DDRPHYC_DXNDLLCR_SDPHASE_MASK,
> - sdphase_val << DDRPHYC_DXNDLLCR_SDPHASE_SHIFT);
> -}
> -
> -/* Sets the DQS unit delay for a byte lane.
> - * unit delay is specified by giving the index of the desired delay
> - * for dgsdly and dqsndly (same value).
> - */
> -static void DQS_unit_delay(struct stm32mp1_ddrphy *phy,
> - u8 byte, u8 unit_dly_idx)
> -{
> - /* Write the same value in DXNDQSTR.DQSDLY and DXNDQSTR.DQSNDLY */
> - clrsetbits_le32(DXNDQSTR(phy, byte),
> - DDRPHYC_DXNDQSTR_DQSDLY_MASK |
> - DDRPHYC_DXNDQSTR_DQSNDLY_MASK,
> - (unit_dly_idx << DDRPHYC_DXNDQSTR_DQSDLY_SHIFT) |
> - (unit_dly_idx << DDRPHYC_DXNDQSTR_DQSNDLY_SHIFT));
> -
> - /* After changing this value, an ITM soft reset (PIR.ITMSRST=1,
> - * plus PIR.INIT=1) must be issued.
> - */
> - stm32mp1_ddrphy_init(phy, DDRPHYC_PIR_ITMSRST);
> -}
> -
> -/* Sets the DQ unit delay for a bit line in particular byte lane.
> - * unit delay is specified by giving the desired delay
> - */
> -static void set_DQ_unit_delay(struct stm32mp1_ddrphy *phy,
> - u8 byte, u8 bit,
> - u8 dq_delay_index)
> -{
> - u8 dq_bit_delay_val = dq_delay_index | (dq_delay_index << 2);
> -
> - /* same value on delay for clock DQ an DQS_b */
> - clrsetbits_le32(DXNDQTR(phy, byte),
> - DDRPHYC_DXNDQTR_DQDLY_MASK
> - << DDRPHYC_DXNDQTR_DQDLY_SHIFT(bit),
> - dq_bit_delay_val << DDRPHYC_DXNDQTR_DQDLY_SHIFT(bit));
> -}
> -
> -static void set_r0dgsl_delay(struct stm32mp1_ddrphy *phy,
> - u8 byte, u8 r0dgsl_idx)
> -{
> - clrsetbits_le32(DXNDQSTR(phy, byte),
> - DDRPHYC_DXNDQSTR_R0DGSL_MASK,
> - r0dgsl_idx << DDRPHYC_DXNDQSTR_R0DGSL_SHIFT);
> -}
> -
> -static void set_r0dgps_delay(struct stm32mp1_ddrphy *phy,
> - u8 byte, u8 r0dgps_idx)
> -{
> - clrsetbits_le32(DXNDQSTR(phy, byte),
> - DDRPHYC_DXNDQSTR_R0DGPS_MASK,
> - r0dgps_idx << DDRPHYC_DXNDQSTR_R0DGPS_SHIFT);
> -}
> -
> -/* Basic BIST configuration for data lane tests. */
> -static void config_BIST(struct stm32mp1_ddrctl *ctl,
> - struct stm32mp1_ddrphy *phy)
> -{
> - u8 nb_bank = get_nb_bank(ctl);
> - u8 nb_row = get_nb_row(ctl);
> - u8 nb_col = get_nb_col(ctl);
> -
> - /* Selects the SDRAM bank address to be used during BIST. */
> - u32 bbank = 0;
> - /* Selects the SDRAM row address to be used during BIST. */
> - u32 brow = 0;
> - /* Selects the SDRAM column address to be used during BIST. */
> - u32 bcol = 0;
> - /* Selects the value by which the SDRAM address is incremented
> - * for each write/read access.
> - */
> - u32 bainc = 0x00000008;
> - /* Specifies the maximum SDRAM rank to be used during BIST.
> - * The default value is set to maximum ranks minus 1.
> - * must be 0 with single rank
> - */
> - u32 bmrank = 0;
> - /* Selects the SDRAM rank to be used during BIST.
> - * must be 0 with single rank
> - */
> - u32 brank = 0;
> -
> - /* Specifies the maximum SDRAM bank address to be used during
> - * BIST before the address & increments to the next rank.
> - */
> - u32 bmbank = (1 << nb_bank) - 1;
> - /* Specifies the maximum SDRAM row address to be used during
> - * BIST before the address & increments to the next bank.
> - */
> - u32 bmrow = (1 << nb_row) - 1;
> - /* Specifies the maximum SDRAM column address to be used during
> - * BIST before the address & increments to the next row.
> - */
> - u32 bmcol = (1 << nb_col) - 1;
> -
> - u32 bmode_conf = 0x00000001; /* DRam mode */
> - u32 bdxen_conf = 0x00000001; /* BIST on Data byte */
> - u32 bdpat_conf = 0x00000002; /* Select LFSR pattern */
> -
> - /*Setup BIST for DRAM mode, and LFSR-random data pattern.*/
> - /*Write BISTRR.BMODE = 1?b1;*/
> - /*Write BISTRR.BDXEN = 1?b1;*/
> - /*Write BISTRR.BDPAT = 2?b10;*/
> -
> - /* reset BIST */
> - writel(0x3, &phy->bistrr);
> -
> - writel((bmode_conf << 3) | (bdxen_conf << 14) | (bdpat_conf << 17),
> - &phy->bistrr);
> -
> - /*Setup BIST Word Count*/
> - /*Write BISTWCR.BWCNT = 16?b0008;*/
> - writel(0x00000200, &phy->bistwcr); /* A multiple of BL/2 */
> -
> - writel(bcol | (brow << 12) | (bbank << 28), &phy->bistar0);
> - writel(brank | (bmrank << 2) | (bainc << 4), &phy->bistar1);
> - writel(bmcol | (bmrow << 12) | (bmbank << 28), &phy->bistar2);
> -}
> -
> -/* Select the Byte lane to be tested by BIST. */
> -static void BIST_datx8_sel(struct stm32mp1_ddrphy *phy, u8 datx8)
> -{
> - clrsetbits_le32(&phy->bistrr,
> - DDRPHYC_BISTRR_BDXSEL_MASK,
> - datx8 << DDRPHYC_BISTRR_BDXSEL_SHIFT);
> -
> - /*(For example, selecting Byte Lane 3, BISTRR.BDXSEL = 4?b0011)*/
> - /* Write BISTRR.BDXSEL = datx8; */
> -}
> -
> -/* Perform BIST Write_Read test on a byte lane and return test result. */
> -static void BIST_test(struct stm32mp1_ddrphy *phy, u8 byte,
> - struct BIST_result *bist)
> -{
> - bool result = true; /* BIST_SUCCESS */
> - u32 cnt = 0;
> - u32 error = 0;
> - u32 val;
> - int ret;
> -
> - bist->test_result = true;
> -
> -run:
> - itm_soft_reset(phy);
> -
> - /*Perform BIST Reset*/
> - /* Write BISTRR.BINST = 3?b011; */
> - clrsetbits_le32(&phy->bistrr,
> - 0x00000007,
> - 0x00000003);
> -
> - /*Re-seed LFSR*/
> - /* Write BISTLSR.SEED = 32'h1234ABCD; */
> - if (BIST_seed)
> - writel(BIST_seed, &phy->bistlsr);
> - else
> - writel(rand(), &phy->bistlsr);
> -
> - /* some delay to reset BIST */
> - udelay(10);
> -
> - /*Perform BIST Run*/
> - clrsetbits_le32(&phy->bistrr,
> - 0x00000007,
> - 0x00000001);
> - /* Write BISTRR.BINST = 3?b001; */
> -
> - /* poll on BISTGSR.BDONE and wait max 1000 us */
> - ret = readl_poll_timeout(&phy->bistgsr, val,
> - val & DDRPHYC_BISTGSR_BDDONE, 1000);
> -
> - if (ret < 0) {
> - printf("warning: BIST timeout\n");
> - result = false; /* BIST_FAIL; */
> - /*Perform BIST Stop */
> - clrsetbits_le32(&phy->bistrr, 0x00000007, 0x00000002);
> - } else {
> - /*Check if received correct number of words*/
> - /* if (Read BISTWCSR.DXWCNT = Read BISTWCR.BWCNT) */
> - if (((readl(&phy->bistwcsr)) >> DDRPHYC_BISTWCSR_DXWCNT_SHIFT)
> - == readl(&phy->bistwcr)) {
> - /*Determine if there is a data comparison error*/
> - /* if (Read BISTGSR.BDXERR = 1?b0) */
> - if (readl(&phy->bistgsr) & DDRPHYC_BISTGSR_BDXERR)
> - result = false; /* BIST_FAIL; */
> - else
> - result = true; /* BIST_SUCCESS; */
> - } else {
> - result = false; /* BIST_FAIL; */
> - }
> - }
> -
> - /* loop while success */
> - cnt++;
> - if (result && cnt != 1000)
> - goto run;
> -
> - if (!result)
> - error++;
> -
> - if (error < BIST_error_max) {
> - if (cnt != 1000)
> - goto run;
> - bist->test_result = true;
> - } else {
> - bist->test_result = false;
> - }
> -}
> -
> -/* After running the deskew algo, this function applies the new DQ delays
> - * by reading them from the array "deskew_delay"and writing in PHY registers.
> - * The bits that are not deskewed parfectly (too much skew on them,
> - * or data eye very wide) are marked in the array deskew_non_converge.
> - */
> -static void apply_deskew_results(struct stm32mp1_ddrphy *phy, u8 byte,
> - u8 deskew_delay[NUM_BYTES][8],
> - u8 deskew_non_converge[NUM_BYTES][8])
> -{
> - u8 bit_i;
> - u8 index;
> -
> - for (bit_i = 0; bit_i < 8; bit_i++) {
> - set_DQ_unit_delay(phy, byte, bit_i, deskew_delay[byte][bit_i]);
> - index = DQ_unit_index(phy, byte, bit_i);
> - log_debug("Byte %d ; bit %d : The new DQ delay (%d) index=%d [delta=%d, 3 is the default]",
> - byte, bit_i, deskew_delay[byte][bit_i],
> - index, index - 3);
> - printf("Byte %d, bit %d, DQ delay = %d",
> - byte, bit_i, deskew_delay[byte][bit_i]);
> - if (deskew_non_converge[byte][bit_i] == 1)
> - log_debug(" - not converged : still more skew");
> - printf("\n");
> - }
> -}
> -
> -/* DQ Bit de-skew algorithm.
> - * Deskews data lines as much as possible.
> - * 1. Add delay to DQS line until finding the failure
> - * (normally a hold time violation)
> - * 2. Reduce DQS line by small steps until finding the very first time
> - * we go back to "Pass" condition.
> - * 3. For each DQ line, Reduce DQ delay until finding the very first failure
> - * (normally a hold time fail)
> - * 4. When all bits are at their first failure delay, we can consider them
> - * aligned.
> - * Handle conrer situation (Can't find Pass-fail, or fail-pass transitions
> - * at any step)
> - * TODO Provide a return Status. Improve doc
> - */
> -static enum test_result bit_deskew(struct stm32mp1_ddrctl *ctl,
> - struct stm32mp1_ddrphy *phy, char *string)
> -{
> - /* New DQ delay value (index), set during Deskew algo */
> - u8 deskew_delay[NUM_BYTES][8];
> - /*If there is still skew on a bit, mark this bit. */
> - u8 deskew_non_converge[NUM_BYTES][8];
> - struct BIST_result result;
> - s8 dqs_unit_delay_index = 0;
> - u8 datx8 = 0;
> - u8 bit_i = 0;
> - s8 phase_idx = 0;
> - s8 bit_i_delay_index = 0;
> - u8 success = 0;
> - struct tuning_position last_right_ok;
> - u8 force_stop = 0;
> - u8 fail_found;
> - u8 error = 0;
> - u8 nb_bytes = get_nb_bytes(ctl);
> - /* u8 last_pass_dqs_unit = 0; */
> -
> - memset(deskew_delay, 0, sizeof(deskew_delay));
> - memset(deskew_non_converge, 0, sizeof(deskew_non_converge));
> -
> - /*Disable DQS Drift Compensation*/
> - clrbits_le32(&phy->pgcr, DDRPHYC_PGCR_DFTCMP);
> - /*Disable all bytes*/
> - /* Disable automatic power down of DLL and IOs when disabling
> - * a byte (To avoid having to add programming and delay
> - * for a DLL re-lock when later re-enabling a disabled Byte Lane)
> - */
> - clrbits_le32(&phy->pgcr, DDRPHYC_PGCR_PDDISDX);
> -
> - /* Disable all data bytes */
> - clrbits_le32(&phy->dx0gcr, DDRPHYC_DXNGCR_DXEN);
> - clrbits_le32(&phy->dx1gcr, DDRPHYC_DXNGCR_DXEN);
> - clrbits_le32(&phy->dx2gcr, DDRPHYC_DXNGCR_DXEN);
> - clrbits_le32(&phy->dx3gcr, DDRPHYC_DXNGCR_DXEN);
> -
> - /* Config the BIST block */
> - config_BIST(ctl, phy);
> - log_debug("BIST Config done.\n");
> -
> - /* Train each byte */
> - for (datx8 = 0; datx8 < nb_bytes; datx8++) {
> - if (ctrlc()) {
> - sprintf(string, "interrupted at byte %d/%d, error=%d",
> - datx8 + 1, nb_bytes, error);
> - return TEST_FAILED;
> - }
> - log_debug("\n======================\n");
> - log_debug("Start deskew byte %d .\n", datx8);
> - log_debug("======================\n");
> - /* Enable Byte (DXNGCR, bit DXEN) */
> - setbits_le32(DXNGCR(phy, datx8), DDRPHYC_DXNGCR_DXEN);
> -
> - /* Select the byte lane for comparison of read data */
> - BIST_datx8_sel(phy, datx8);
> -
> - /* Set all DQDLYn to maximum value. All bits within the byte
> - * will be delayed with DQSTR = 2 instead of max = 3
> - * to avoid inter bits fail influence
> - */
> - writel(0xAAAAAAAA, DXNDQTR(phy, datx8));
> -
> - /* Set the DQS phase delay to 90 DEG (default).
> - * What is defined here is the index of the desired config
> - * in the PHASE array.
> - */
> - phase_idx = _90deg;
> -
> - /* Set DQS unit delay to the max value. */
> - dqs_unit_delay_index = MAX_DQS_UNIT_IDX;
> - DQS_unit_delay(phy, datx8, dqs_unit_delay_index);
> - DQS_phase_delay(phy, datx8, phase_idx);
> -
> - /* Issue a DLL soft reset */
> - clrbits_le32(DXNDLLCR(phy, datx8), DDRPHYC_DXNDLLCR_DLLSRST);
> - setbits_le32(DXNDLLCR(phy, datx8), DDRPHYC_DXNDLLCR_DLLSRST);
> -
> - /* Test this typical init condition */
> - BIST_test(phy, datx8, &result);
> - success = result.test_result;
> -
> - /* If the test pass in this typical condition,
> - * start the algo with it.
> - * Else, look for Pass init condition
> - */
> - if (!success) {
> - log_debug("Fail at init condtion. Let's look for a good init condition.\n");
> - success = 0; /* init */
> - /* Make sure we start with a PASS condition before
> - * looking for a fail condition.
> - * Find the first PASS PHASE condition
> - */
> -
> - /* escape if we find a PASS */
> - log_debug("increase Phase idx\n");
> - while (!success && (phase_idx <= MAX_DQS_PHASE_IDX)) {
> - DQS_phase_delay(phy, datx8, phase_idx);
> - BIST_test(phy, datx8, &result);
> - success = result.test_result;
> - phase_idx++;
> - }
> - /* if ended with success
> - * ==>> Restore the fist success condition
> - */
> - if (success)
> - phase_idx--; /* because it ended with ++ */
> - }
> - if (ctrlc()) {
> - sprintf(string, "interrupted at byte %d/%d, error=%d",
> - datx8 + 1, nb_bytes, error);
> - return TEST_FAILED;
> - }
> - /* We couldn't find a successful condition, its seems
> - * we have hold violation, lets try reduce DQS_unit Delay
> - */
> - if (!success) {
> - /* We couldn't find a successful condition, its seems
> - * we have hold violation, lets try reduce DQS_unit
> - * Delay
> - */
> - log_debug("Still fail. Try decrease DQS Unit delay\n");
> -
> - phase_idx = 0;
> - dqs_unit_delay_index = 0;
> - DQS_phase_delay(phy, datx8, phase_idx);
> -
> - /* escape if we find a PASS */
> - while (!success &&
> - (dqs_unit_delay_index <=
> - MAX_DQS_UNIT_IDX)) {
> - DQS_unit_delay(phy, datx8,
> - dqs_unit_delay_index);
> - BIST_test(phy, datx8, &result);
> - success = result.test_result;
> - dqs_unit_delay_index++;
> - }
> - if (success) {
> - /* Restore the first success condition*/
> - dqs_unit_delay_index--;
> - /* last_pass_dqs_unit = dqs_unit_delay_index;*/
> - DQS_unit_delay(phy, datx8,
> - dqs_unit_delay_index);
> - } else {
> - /* No need to continue,
> - * there is no pass region.
> - */
> - force_stop = 1;
> - }
> - }
> -
> - /* There is an initial PASS condition
> - * Look for the first failing condition by PHASE stepping.
> - * This part of the algo can finish without converging.
> - */
> - if (force_stop) {
> - printf("Result: Failed ");
> - printf("[Cannot Deskew lines, ");
> - printf("there is no PASS region]\n");
> - error++;
> - continue;
> - }
> - if (ctrlc()) {
> - sprintf(string, "interrupted at byte %d/%d, error=%d",
> - datx8 + 1, nb_bytes, error);
> - return TEST_FAILED;
> - }
> -
> - log_debug("there is a pass region for phase idx %d\n",
> - phase_idx);
> - log_debug("Step1: Find the first failing condition\n");
> - /* Look for the first failing condition by PHASE stepping.
> - * This part of the algo can finish without converging.
> - */
> -
> - /* escape if we find a fail (hold time violation)
> - * condition at any bit or if out of delay range.
> - */
> - while (success && (phase_idx <= MAX_DQS_PHASE_IDX)) {
> - DQS_phase_delay(phy, datx8, phase_idx);
> - BIST_test(phy, datx8, &result);
> - success = result.test_result;
> - phase_idx++;
> - }
> - if (ctrlc()) {
> - sprintf(string, "interrupted at byte %d/%d, error=%d",
> - datx8 + 1, nb_bytes, error);
> - return TEST_FAILED;
> - }
> -
> - /* if the loop ended with a failing condition at any bit,
> - * lets look for the first previous success condition by unit
> - * stepping (minimal delay)
> - */
> - if (!success) {
> - log_debug("Fail region (PHASE) found phase idx %d\n",
> - phase_idx);
> - log_debug("Let's look for first success by DQS Unit steps\n");
> - /* This part, the algo always converge */
> - phase_idx--;
> -
> - /* escape if we find a success condition
> - * or if out of delay range.
> - */
> - while (!success && dqs_unit_delay_index >= 0) {
> - DQS_unit_delay(phy, datx8,
> - dqs_unit_delay_index);
> - BIST_test(phy, datx8, &result);
> - success = result.test_result;
> - dqs_unit_delay_index--;
> - }
> - /* if the loop ended with a success condition,
> - * the last delay Right OK (before hold violation)
> - * condition is then defined as following:
> - */
> - if (success) {
> - /* Hold the dely parameters of the the last
> - * delay Right OK condition.
> - * -1 to get back to current condition
> - */
> - last_right_ok.phase = phase_idx;
> - /*+1 to get back to current condition */
> - last_right_ok.unit = dqs_unit_delay_index + 1;
> - last_right_ok.bits_delay = 0xFFFFFFFF;
> - log_debug("Found %d\n", dqs_unit_delay_index);
> - } else {
> - /* the last OK condition is then with the
> - * previous phase_idx.
> - * -2 instead of -1 because at the last
> - * iteration of the while(),
> - * we incremented phase_idx
> - */
> - last_right_ok.phase = phase_idx - 1;
> - /* Nominal+1. Because we want the previous
> - * delay after reducing the phase delay.
> - */
> - last_right_ok.unit = 1;
> - last_right_ok.bits_delay = 0xFFFFFFFF;
> - log_debug("Not Found : try previous phase %d\n",
> - phase_idx - 1);
> -
> - DQS_phase_delay(phy, datx8, phase_idx - 1);
> - dqs_unit_delay_index = 0;
> - success = true;
> - while (success &&
> - (dqs_unit_delay_index <
> - MAX_DQS_UNIT_IDX)) {
> - DQS_unit_delay(phy, datx8,
> - dqs_unit_delay_index);
> - BIST_test(phy, datx8, &result);
> - success = result.test_result;
> - dqs_unit_delay_index++;
> - log_debug("dqs_unit_delay_index = %d, result = %d\n",
> - dqs_unit_delay_index, success);
> - }
> -
> - if (!success) {
> - last_right_ok.unit =
> - dqs_unit_delay_index - 1;
> - } else {
> - last_right_ok.unit = 0;
> - log_debug("ERROR: failed region not FOUND");
> - }
> - }
> - } else {
> - /* we can't find a failing condition at all bits
> - * ==> Just hold the last test condition
> - * (the max DQS delay)
> - * which is the most likely,
> - * the closest to a hold violation
> - * If we can't find a Fail condition after
> - * the Pass region, stick at this position
> - * In order to have max chances to find a fail
> - * when reducing DQ delays.
> - */
> - last_right_ok.phase = MAX_DQS_PHASE_IDX;
> - last_right_ok.unit = MAX_DQS_UNIT_IDX;
> - last_right_ok.bits_delay = 0xFFFFFFFF;
> - log_debug("Can't find the a fail condition\n");
> - }
> -
> - /* step 2:
> - * if we arrive at this stage, it means that we found the last
> - * Right OK condition (by tweeking the DQS delay). Or we simply
> - * pushed DQS delay to the max
> - * This means that by reducing the delay on some DQ bits,
> - * we should find a failing condition.
> - */
> - printf("Byte %d, DQS unit = %d, phase = %d\n",
> - datx8, last_right_ok.unit, last_right_ok.phase);
> - log_debug("Step2, unit = %d, phase = %d, bits delay=%x\n",
> - last_right_ok.unit, last_right_ok.phase,
> - last_right_ok.bits_delay);
> -
> - /* Restore the last_right_ok condtion. */
> - DQS_unit_delay(phy, datx8, last_right_ok.unit);
> - DQS_phase_delay(phy, datx8, last_right_ok.phase);
> - writel(last_right_ok.bits_delay, DXNDQTR(phy, datx8));
> -
> - /* train each bit
> - * reduce delay on each bit, and perform a write/read test
> - * and stop at the very first time it fails.
> - * the goal is the find the first failing condition
> - * for each bit.
> - * When we achieve this condition< for all the bits,
> - * we are sure they are aligned (+/- step resolution)
> - */
> - fail_found = 0;
> - for (bit_i = 0; bit_i < 8; bit_i++) {
> - if (ctrlc()) {
> - sprintf(string,
> - "interrupted at byte %d/%d, error=%d",
> - datx8 + 1, nb_bytes, error);
> - return error;
> - }
> - log_debug("deskewing bit %d:\n", bit_i);
> - success = 1; /* init */
> - /* Set all DQDLYn to maximum value.
> - * Only bit_i will be down-delayed
> - * ==> if we have a fail, it will be definitely
> - * from bit_i
> - */
> - writel(0xFFFFFFFF, DXNDQTR(phy, datx8));
> - /* Arriving at this stage,
> - * we have a success condition with delay = 3;
> - */
> - bit_i_delay_index = 3;
> -
> - /* escape if bit delay is out of range or
> - * if a fatil occurs
> - */
> - while ((bit_i_delay_index >= 0) && success) {
> - set_DQ_unit_delay(phy, datx8,
> - bit_i,
> - bit_i_delay_index);
> - BIST_test(phy, datx8, &result);
> - success = result.test_result;
> - bit_i_delay_index--;
> - }
> -
> - /* if escape with a fail condition
> - * ==> save this position for bit_i
> - */
> - if (!success) {
> - /* save the delay position.
> - * Add 1 because the while loop ended with a --,
> - * and that we need to hold the last success
> - * delay
> - */
> - deskew_delay[datx8][bit_i] =
> - bit_i_delay_index + 2;
> - if (deskew_delay[datx8][bit_i] > 3)
> - deskew_delay[datx8][bit_i] = 3;
> -
> - /* A flag that states we found at least a fail
> - * at one bit.
> - */
> - fail_found = 1;
> - log_debug("Fail found on bit %d, for delay = %d => deskew[%d][%d] = %d\n",
> - bit_i, bit_i_delay_index + 1,
> - datx8, bit_i,
> - deskew_delay[datx8][bit_i]);
> - } else {
> - /* if we can find a success condition by
> - * back-delaying this bit, just set the delay
> - * to 0 (the best deskew
> - * possible) and mark the bit.
> - */
> - deskew_delay[datx8][bit_i] = 0;
> - /* set a flag that will be used later
> - * in the report.
> - */
> - deskew_non_converge[datx8][bit_i] = 1;
> - log_debug("Fail not found on bit %d => deskew[%d][%d] = %d\n",
> - bit_i, datx8, bit_i,
> - deskew_delay[datx8][bit_i]);
> - }
> - }
> - log_debug("**********byte %d tuning complete************\n",
> - datx8);
> - /* If we can't find any failure by back delaying DQ lines,
> - * hold the default values
> - */
> - if (!fail_found) {
> - for (bit_i = 0; bit_i < 8; bit_i++)
> - deskew_delay[datx8][bit_i] = 0;
> - log_debug("The Deskew algorithm can't converge, there is too much margin in your design. Good job!\n");
> - }
> -
> - apply_deskew_results(phy, datx8, deskew_delay,
> - deskew_non_converge);
> - /* Restore nominal value for DQS delay */
> - DQS_phase_delay(phy, datx8, 3);
> - DQS_unit_delay(phy, datx8, 3);
> - /* disable byte after byte bits deskew */
> - clrbits_le32(DXNGCR(phy, datx8), DDRPHYC_DXNGCR_DXEN);
> - } /* end of byte deskew */
> -
> - /* re-enable all data bytes */
> - setbits_le32(&phy->dx0gcr, DDRPHYC_DXNGCR_DXEN);
> - setbits_le32(&phy->dx1gcr, DDRPHYC_DXNGCR_DXEN);
> - setbits_le32(&phy->dx2gcr, DDRPHYC_DXNGCR_DXEN);
> - setbits_le32(&phy->dx3gcr, DDRPHYC_DXNGCR_DXEN);
> -
> - if (error) {
> - sprintf(string, "error = %d", error);
> - return TEST_FAILED;
> - }
> -
> - return TEST_PASSED;
> -} /* end function */
> -
> -/* Trim DQS timings and set it in the centre of data eye.
> - * Look for a PPPPF region, then look for a FPPP region and finally select
> - * the mid of the FPPPPPF region
> - */
> -static enum test_result eye_training(struct stm32mp1_ddrctl *ctl,
> - struct stm32mp1_ddrphy *phy, char *string)
> -{
> - /*Stores the DQS trim values (PHASE index, unit index) */
> - u8 eye_training_val[NUM_BYTES][2];
> - u8 byte = 0;
> - struct BIST_result result;
> - s8 dqs_unit_delay_index = 0;
> - s8 phase_idx = 0;
> - s8 dqs_unit_delay_index_pass = 0;
> - s8 phase_idx_pass = 0;
> - u8 success = 0;
> - u8 left_phase_bound_found, right_phase_bound_found;
> - u8 left_unit_bound_found, right_unit_bound_found;
> - u8 left_bound_found, right_bound_found;
> - struct tuning_position left_bound, right_bound;
> - u8 error = 0;
> - u8 nb_bytes = get_nb_bytes(ctl);
> -
> - /*Disable DQS Drift Compensation*/
> - clrbits_le32(&phy->pgcr, DDRPHYC_PGCR_DFTCMP);
> - /*Disable all bytes*/
> - /* Disable automatic power down of DLL and IOs when disabling a byte
> - * (To avoid having to add programming and delay
> - * for a DLL re-lock when later re-enabling a disabled Byte Lane)
> - */
> - clrbits_le32(&phy->pgcr, DDRPHYC_PGCR_PDDISDX);
> -
> - /*Disable all data bytes */
> - clrbits_le32(&phy->dx0gcr, DDRPHYC_DXNGCR_DXEN);
> - clrbits_le32(&phy->dx1gcr, DDRPHYC_DXNGCR_DXEN);
> - clrbits_le32(&phy->dx2gcr, DDRPHYC_DXNGCR_DXEN);
> - clrbits_le32(&phy->dx3gcr, DDRPHYC_DXNGCR_DXEN);
> -
> - /* Config the BIST block */
> - config_BIST(ctl, phy);
> -
> - for (byte = 0; byte < nb_bytes; byte++) {
> - if (ctrlc()) {
> - sprintf(string, "interrupted at byte %d/%d, error=%d",
> - byte + 1, nb_bytes, error);
> - return TEST_FAILED;
> - }
> - right_bound.phase = 0;
> - right_bound.unit = 0;
> -
> - left_bound.phase = 0;
> - left_bound.unit = 0;
> -
> - left_phase_bound_found = 0;
> - right_phase_bound_found = 0;
> -
> - left_unit_bound_found = 0;
> - right_unit_bound_found = 0;
> -
> - left_bound_found = 0;
> - right_bound_found = 0;
> -
> - /* Enable Byte (DXNGCR, bit DXEN) */
> - setbits_le32(DXNGCR(phy, byte), DDRPHYC_DXNGCR_DXEN);
> -
> - /* Select the byte lane for comparison of read data */
> - BIST_datx8_sel(phy, byte);
> -
> - /* Set DQS phase delay to the nominal value. */
> - phase_idx = _90deg;
> - phase_idx_pass = phase_idx;
> -
> - /* Set DQS unit delay to the nominal value. */
> - dqs_unit_delay_index = 3;
> - dqs_unit_delay_index_pass = dqs_unit_delay_index;
> - success = 0;
> -
> - log_debug("STEP0: Find Init delay\n");
> - /* STEP0: Find Init delay: a delay that put the system
> - * in a "Pass" condition then (TODO) update
> - * dqs_unit_delay_index_pass & phase_idx_pass
> - */
> - DQS_unit_delay(phy, byte, dqs_unit_delay_index);
> - DQS_phase_delay(phy, byte, phase_idx);
> - BIST_test(phy, byte, &result);
> - success = result.test_result;
> - /* If we have a fail in the nominal condition */
> - if (!success) {
> - /* Look at the left */
> - while (phase_idx >= 0 && !success) {
> - phase_idx--;
> - DQS_phase_delay(phy, byte, phase_idx);
> - BIST_test(phy, byte, &result);
> - success = result.test_result;
> - }
> - }
> - if (!success) {
> - /* if we can't find pass condition,
> - * then look at the right
> - */
> - phase_idx = _90deg;
> - while (phase_idx <= MAX_DQS_PHASE_IDX &&
> - !success) {
> - phase_idx++;
> - DQS_phase_delay(phy, byte,
> - phase_idx);
> - BIST_test(phy, byte, &result);
> - success = result.test_result;
> - }
> - }
> - /* save the pass condition */
> - if (success) {
> - phase_idx_pass = phase_idx;
> - } else {
> - printf("Result: Failed ");
> - printf("[Cannot DQS timings, ");
> - printf("there is no PASS region]\n");
> - error++;
> - continue;
> - }
> -
> - if (ctrlc()) {
> - sprintf(string, "interrupted at byte %d/%d, error=%d",
> - byte + 1, nb_bytes, error);
> - return TEST_FAILED;
> - }
> - log_debug("STEP1: Find LEFT PHASE DQS Bound\n");
> - /* STEP1: Find LEFT PHASE DQS Bound */
> - while ((phase_idx >= 0) &&
> - (phase_idx <= MAX_DQS_PHASE_IDX) &&
> - !left_phase_bound_found) {
> - DQS_unit_delay(phy, byte,
> - dqs_unit_delay_index);
> - DQS_phase_delay(phy, byte,
> - phase_idx);
> - BIST_test(phy, byte, &result);
> - success = result.test_result;
> -
> - /*TODO: Manage the case were at the beginning
> - * there is already a fail
> - */
> - if (!success) {
> - /* the last pass condition */
> - left_bound.phase = ++phase_idx;
> - left_phase_bound_found = 1;
> - } else if (success) {
> - phase_idx--;
> - }
> - }
> - if (!left_phase_bound_found) {
> - left_bound.phase = 0;
> - phase_idx = 0;
> - }
> - /* If not found, lets take 0 */
> -
> - if (ctrlc()) {
> - sprintf(string, "interrupted at byte %d/%d, error=%d",
> - byte + 1, nb_bytes, error);
> - return TEST_FAILED;
> - }
> - log_debug("STEP2: Find UNIT left bound\n");
> - /* STEP2: Find UNIT left bound */
> - while ((dqs_unit_delay_index >= 0) &&
> - !left_unit_bound_found) {
> - DQS_unit_delay(phy, byte,
> - dqs_unit_delay_index);
> - DQS_phase_delay(phy, byte, phase_idx);
> - BIST_test(phy, byte, &result);
> - success = result.test_result;
> - if (!success) {
> - left_bound.unit =
> - ++dqs_unit_delay_index;
> - left_unit_bound_found = 1;
> - left_bound_found = 1;
> - } else if (success) {
> - dqs_unit_delay_index--;
> - }
> - }
> -
> - /* If not found, lets take 0 */
> - if (!left_unit_bound_found)
> - left_bound.unit = 0;
> -
> - if (ctrlc()) {
> - sprintf(string, "interrupted at byte %d/%d, error=%d",
> - byte + 1, nb_bytes, error);
> - return TEST_FAILED;
> - }
> - log_debug("STEP3: Find PHase right bound\n");
> - /* STEP3: Find PHase right bound, start with "pass"
> - * condition
> - */
> -
> - /* Set DQS phase delay to the pass value. */
> - phase_idx = phase_idx_pass;
> -
> - /* Set DQS unit delay to the pass value. */
> - dqs_unit_delay_index = dqs_unit_delay_index_pass;
> -
> - while ((phase_idx <= MAX_DQS_PHASE_IDX) &&
> - !right_phase_bound_found) {
> - DQS_unit_delay(phy, byte,
> - dqs_unit_delay_index);
> - DQS_phase_delay(phy, byte, phase_idx);
> - BIST_test(phy, byte, &result);
> - success = result.test_result;
> - if (!success) {
> - /* the last pass condition */
> - right_bound.phase = --phase_idx;
> - right_phase_bound_found = 1;
> - } else if (success) {
> - phase_idx++;
> - }
> - }
> -
> - /* If not found, lets take the max value */
> - if (!right_phase_bound_found) {
> - right_bound.phase = MAX_DQS_PHASE_IDX;
> - phase_idx = MAX_DQS_PHASE_IDX;
> - }
> -
> - if (ctrlc()) {
> - sprintf(string, "interrupted at byte %d/%d, error=%d",
> - byte + 1, nb_bytes, error);
> - return TEST_FAILED;
> - }
> - log_debug("STEP4: Find UNIT right bound\n");
> - /* STEP4: Find UNIT right bound */
> - while ((dqs_unit_delay_index <= MAX_DQS_UNIT_IDX) &&
> - !right_unit_bound_found) {
> - DQS_unit_delay(phy, byte,
> - dqs_unit_delay_index);
> - DQS_phase_delay(phy, byte, phase_idx);
> - BIST_test(phy, byte, &result);
> - success = result.test_result;
> - if (!success) {
> - right_bound.unit =
> - --dqs_unit_delay_index;
> - right_unit_bound_found = 1;
> - right_bound_found = 1;
> - } else if (success) {
> - dqs_unit_delay_index++;
> - }
> - }
> - /* If not found, lets take the max value */
> - if (!right_unit_bound_found)
> - right_bound.unit = MAX_DQS_UNIT_IDX;
> -
> - /* If we found a regular FAil Pass FAil pattern
> - * FFPPPPPPFF
> - * OR PPPPPFF Or FFPPPPP
> - */
> -
> - if (left_bound_found || right_bound_found) {
> - eye_training_val[byte][0] = (right_bound.phase +
> - left_bound.phase) / 2;
> - eye_training_val[byte][1] = (right_bound.unit +
> - left_bound.unit) / 2;
> -
> - /* If we already lost 1/2PHASE Tuning,
> - * let's try to recover by ++ on unit
> - */
> - if (((right_bound.phase + left_bound.phase) % 2 == 1) &&
> - eye_training_val[byte][1] != MAX_DQS_UNIT_IDX)
> - eye_training_val[byte][1]++;
> - log_debug("** found phase : %d - %d & unit %d - %d\n",
> - right_bound.phase, left_bound.phase,
> - right_bound.unit, left_bound.unit);
> - log_debug("** calculating mid region: phase: %d unit: %d (nominal is 3)\n",
> - eye_training_val[byte][0],
> - eye_training_val[byte][1]);
> - } else {
> - /* PPPPPPPPPP, we're already good.
> - * Set nominal values.
> - */
> - eye_training_val[byte][0] = 3;
> - eye_training_val[byte][1] = 3;
> - }
> - DQS_phase_delay(phy, byte, eye_training_val[byte][0]);
> - DQS_unit_delay(phy, byte, eye_training_val[byte][1]);
> -
> - printf("Byte %d, DQS unit = %d, phase = %d\n",
> - byte,
> - eye_training_val[byte][1],
> - eye_training_val[byte][0]);
> - }
> -
> - if (error) {
> - sprintf(string, "error = %d", error);
> - return TEST_FAILED;
> - }
> -
> - return TEST_PASSED;
> -}
> -
> -static void display_reg_results(struct stm32mp1_ddrphy *phy, u8 byte)
> -{
> - u8 i = 0;
> -
> - printf("Byte %d Dekew result, bit0 delay, bit1 delay...bit8 delay\n ",
> - byte);
> -
> - for (i = 0; i < 8; i++)
> - printf("%d ", DQ_unit_index(phy, byte, i));
> - printf("\n");
> -
> - printf("dxndllcr: [%08x] val:%08x\n",
> - DXNDLLCR(phy, byte),
> - readl(DXNDLLCR(phy, byte)));
> - printf("dxnqdstr: [%08x] val:%08x\n",
> - DXNDQSTR(phy, byte),
> - readl(DXNDQSTR(phy, byte)));
> - printf("dxndqtr: [%08x] val:%08x\n",
> - DXNDQTR(phy, byte),
> - readl(DXNDQTR(phy, byte)));
> -}
> -
> -/* analyse the dgs gating log table, and determine the midpoint.*/
> -static u8 set_midpoint_read_dqs_gating(struct stm32mp1_ddrphy *phy, u8 byte,
> - u8 dqs_gating[NUM_BYTES]
> - [MAX_GSL_IDX + 1]
> - [MAX_GPS_IDX + 1])
> -{
> - /* stores the dqs gate values (gsl index, gps index) */
> - u8 dqs_gate_values[NUM_BYTES][2];
> - u8 gsl_idx, gps_idx = 0;
> - u8 left_bound_idx[2] = {0, 0};
> - u8 right_bound_idx[2] = {0, 0};
> - u8 left_bound_found = 0;
> - u8 right_bound_found = 0;
> - u8 intermittent = 0;
> - u8 value;
> -
> - for (gsl_idx = 0; gsl_idx <= MAX_GSL_IDX; gsl_idx++) {
> - for (gps_idx = 0; gps_idx <= MAX_GPS_IDX; gps_idx++) {
> - value = dqs_gating[byte][gsl_idx][gps_idx];
> - if (value == 1 && left_bound_found == 0) {
> - left_bound_idx[0] = gsl_idx;
> - left_bound_idx[1] = gps_idx;
> - left_bound_found = 1;
> - } else if (value == 0 &&
> - left_bound_found == 1 &&
> - !right_bound_found) {
> - if (gps_idx == 0) {
> - right_bound_idx[0] = gsl_idx - 1;
> - right_bound_idx[1] = MAX_GPS_IDX;
> - } else {
> - right_bound_idx[0] = gsl_idx;
> - right_bound_idx[1] = gps_idx - 1;
> - }
> - right_bound_found = 1;
> - } else if (value == 1 &&
> - right_bound_found == 1) {
> - intermittent = 1;
> - }
> - }
> - }
> -
> - /* if only ppppppp is found, there is no mid region. */
> - if (left_bound_idx[0] == 0 && left_bound_idx[1] == 0 &&
> - right_bound_idx[0] == 0 && right_bound_idx[1] == 0)
> - intermittent = 1;
> -
> - /*if we found a regular fail pass fail pattern ffppppppff
> - * or pppppff or ffppppp
> - */
> - if (!intermittent) {
> - /*if we found a regular fail pass fail pattern ffppppppff
> - * or pppppff or ffppppp
> - */
> - if (left_bound_found || right_bound_found) {
> - log_debug("idx0(%d): %d %d idx1(%d) : %d %d\n",
> - left_bound_found,
> - right_bound_idx[0], left_bound_idx[0],
> - right_bound_found,
> - right_bound_idx[1], left_bound_idx[1]);
> - dqs_gate_values[byte][0] =
> - (right_bound_idx[0] + left_bound_idx[0]) / 2;
> - dqs_gate_values[byte][1] =
> - (right_bound_idx[1] + left_bound_idx[1]) / 2;
> - /* if we already lost 1/2gsl tuning,
> - * let's try to recover by ++ on gps
> - */
> - if (((right_bound_idx[0] +
> - left_bound_idx[0]) % 2 == 1) &&
> - dqs_gate_values[byte][1] != MAX_GPS_IDX)
> - dqs_gate_values[byte][1]++;
> - /* if we already lost 1/2gsl tuning and gps is on max*/
> - else if (((right_bound_idx[0] +
> - left_bound_idx[0]) % 2 == 1) &&
> - dqs_gate_values[byte][1] == MAX_GPS_IDX) {
> - dqs_gate_values[byte][1] = 0;
> - dqs_gate_values[byte][0]++;
> - }
> - /* if we have gsl left and write limit too close
> - * (difference=1)
> - */
> - if (((right_bound_idx[0] - left_bound_idx[0]) == 1)) {
> - dqs_gate_values[byte][1] = (left_bound_idx[1] +
> - right_bound_idx[1] +
> - 4) / 2;
> - if (dqs_gate_values[byte][1] >= 4) {
> - dqs_gate_values[byte][0] =
> - right_bound_idx[0];
> - dqs_gate_values[byte][1] -= 4;
> - } else {
> - dqs_gate_values[byte][0] =
> - left_bound_idx[0];
> - }
> - }
> - log_debug("*******calculating mid region: system latency: %d phase: %d********\n",
> - dqs_gate_values[byte][0],
> - dqs_gate_values[byte][1]);
> - log_debug("*******the nominal values were system latency: 0 phase: 2*******\n");
> - }
> - } else {
> - /* if intermitant, restore defaut values */
> - log_debug("dqs gating:no regular fail/pass/fail found. defaults values restored.\n");
> - dqs_gate_values[byte][0] = 0;
> - dqs_gate_values[byte][1] = 2;
> - }
> - set_r0dgsl_delay(phy, byte, dqs_gate_values[byte][0]);
> - set_r0dgps_delay(phy, byte, dqs_gate_values[byte][1]);
> - printf("Byte %d, R0DGSL = %d, R0DGPS = %d\n",
> - byte, dqs_gate_values[byte][0], dqs_gate_values[byte][1]);
> -
> - /* return 0 if intermittent or if both left_bound
> - * and right_bound are not found
> - */
> - return !(intermittent || (left_bound_found && right_bound_found));
> -}
> -
> -static enum test_result read_dqs_gating(struct stm32mp1_ddrctl *ctl,
> - struct stm32mp1_ddrphy *phy,
> - char *string)
> -{
> - /* stores the log of pass/fail */
> - u8 dqs_gating[NUM_BYTES][MAX_GSL_IDX + 1][MAX_GPS_IDX + 1];
> - u8 byte, gsl_idx, gps_idx = 0;
> - struct BIST_result result;
> - u8 success = 0;
> - u8 nb_bytes = get_nb_bytes(ctl);
> -
> - memset(dqs_gating, 0x0, sizeof(dqs_gating));
> -
> - /*disable dqs drift compensation*/
> - clrbits_le32(&phy->pgcr, DDRPHYC_PGCR_DFTCMP);
> - /*disable all bytes*/
> - /* disable automatic power down of dll and ios when disabling a byte
> - * (to avoid having to add programming and delay
> - * for a dll re-lock when later re-enabling a disabled byte lane)
> - */
> - clrbits_le32(&phy->pgcr, DDRPHYC_PGCR_PDDISDX);
> -
> - /* disable all data bytes */
> - clrbits_le32(&phy->dx0gcr, DDRPHYC_DXNGCR_DXEN);
> - clrbits_le32(&phy->dx1gcr, DDRPHYC_DXNGCR_DXEN);
> - clrbits_le32(&phy->dx2gcr, DDRPHYC_DXNGCR_DXEN);
> - clrbits_le32(&phy->dx3gcr, DDRPHYC_DXNGCR_DXEN);
> -
> - /* config the bist block */
> - config_BIST(ctl, phy);
> -
> - for (byte = 0; byte < nb_bytes; byte++) {
> - if (ctrlc()) {
> - sprintf(string, "interrupted at byte %d/%d",
> - byte + 1, nb_bytes);
> - return TEST_FAILED;
> - }
> - /* enable byte x (dxngcr, bit dxen) */
> - setbits_le32(DXNGCR(phy, byte), DDRPHYC_DXNGCR_DXEN);
> -
> - /* select the byte lane for comparison of read data */
> - BIST_datx8_sel(phy, byte);
> - for (gsl_idx = 0; gsl_idx <= MAX_GSL_IDX; gsl_idx++) {
> - for (gps_idx = 0; gps_idx <= MAX_GPS_IDX; gps_idx++) {
> - if (ctrlc()) {
> - sprintf(string,
> - "interrupted at byte %d/%d",
> - byte + 1, nb_bytes);
> - return TEST_FAILED;
> - }
> - /* write cfg to dxndqstr */
> - set_r0dgsl_delay(phy, byte, gsl_idx);
> - set_r0dgps_delay(phy, byte, gps_idx);
> -
> - BIST_test(phy, byte, &result);
> - success = result.test_result;
> - if (success)
> - dqs_gating[byte][gsl_idx][gps_idx] = 1;
> - itm_soft_reset(phy);
> - }
> - }
> - set_midpoint_read_dqs_gating(phy, byte, dqs_gating);
> - /* dummy reads */
> - readl(0xc0000000);
> - readl(0xc0000000);
> - }
> -
> - /* re-enable drift compensation */
> - /* setbits_le32(&phy->pgcr, DDRPHYC_PGCR_DFTCMP); */
> - return TEST_PASSED;
> -}
> -
> -/****************************************************************
> - * TEST
> - ****************************************************************
> - */
> -static enum test_result do_read_dqs_gating(struct stm32mp1_ddrctl *ctl,
> - struct stm32mp1_ddrphy *phy,
> - char *string, int argc,
> - char *argv[])
> -{
> - u32 rfshctl3 = readl(&ctl->rfshctl3);
> - u32 pwrctl = readl(&ctl->pwrctl);
> - u32 derateen = readl(&ctl->derateen);
> - enum test_result res;
> -
> - writel(0x0, &ctl->derateen);
> - stm32mp1_refresh_disable(ctl);
> -
> - res = read_dqs_gating(ctl, phy, string);
> -
> - stm32mp1_refresh_restore(ctl, rfshctl3, pwrctl);
> - writel(derateen, &ctl->derateen);
> -
> - return res;
> -}
> -
> -static enum test_result do_bit_deskew(struct stm32mp1_ddrctl *ctl,
> - struct stm32mp1_ddrphy *phy,
> - char *string, int argc, char *argv[])
> -{
> - u32 rfshctl3 = readl(&ctl->rfshctl3);
> - u32 pwrctl = readl(&ctl->pwrctl);
> - u32 derateen = readl(&ctl->derateen);
> - enum test_result res;
> -
> - writel(0x0, &ctl->derateen);
> - stm32mp1_refresh_disable(ctl);
> -
> - res = bit_deskew(ctl, phy, string);
> -
> - stm32mp1_refresh_restore(ctl, rfshctl3, pwrctl);
> - writel(derateen, &ctl->derateen);
> -
> - return res;
> -}
> -
> -static enum test_result do_eye_training(struct stm32mp1_ddrctl *ctl,
> - struct stm32mp1_ddrphy *phy,
> - char *string, int argc, char *argv[])
> -{
> - u32 rfshctl3 = readl(&ctl->rfshctl3);
> - u32 pwrctl = readl(&ctl->pwrctl);
> - u32 derateen = readl(&ctl->derateen);
> - enum test_result res;
> -
> - writel(0x0, &ctl->derateen);
> - stm32mp1_refresh_disable(ctl);
> -
> - res = eye_training(ctl, phy, string);
> -
> - stm32mp1_refresh_restore(ctl, rfshctl3, pwrctl);
> - writel(derateen, &ctl->derateen);
> -
> - return res;
> -}
> -
> -static enum test_result do_display(struct stm32mp1_ddrctl *ctl,
> - struct stm32mp1_ddrphy *phy,
> - char *string, int argc, char *argv[])
> -{
> - int byte;
> - u8 nb_bytes = get_nb_bytes(ctl);
> -
> - for (byte = 0; byte < nb_bytes; byte++)
> - display_reg_results(phy, byte);
> -
> - return TEST_PASSED;
> -}
> -
> -static enum test_result do_bist_config(struct stm32mp1_ddrctl *ctl,
> - struct stm32mp1_ddrphy *phy,
> - char *string, int argc, char *argv[])
> -{
> - unsigned long value;
> -
> - if (argc > 0) {
> - if (strict_strtoul(argv[0], 0, &value) < 0) {
> - sprintf(string, "invalid nbErr %s", argv[0]);
> - return TEST_FAILED;
> - }
> - BIST_error_max = value;
> - }
> - if (argc > 1) {
> - if (strict_strtoul(argv[1], 0, &value) < 0) {
> - sprintf(string, "invalid Seed %s", argv[1]);
> - return TEST_FAILED;
> - }
> - BIST_seed = value;
> - }
> - printf("Bist.nbErr = %d\n", BIST_error_max);
> - if (BIST_seed)
> - printf("Bist.Seed = 0x%x\n", BIST_seed);
> - else
> - printf("Bist.Seed = random\n");
> -
> - return TEST_PASSED;
> -}
> -
> -/****************************************************************
> - * TEST Description
> - ****************************************************************
> - */
> -
> -const struct test_desc tuning[] = {
> - {do_read_dqs_gating, "Read DQS gating",
> - "software read DQS Gating", "", 0 },
> - {do_bit_deskew, "Bit de-skew", "", "", 0 },
> - {do_eye_training, "Eye Training", "or DQS training", "", 0 },
> - {do_display, "Display registers", "", "", 0 },
> - {do_bist_config, "Bist config", "[nbErr] [seed]",
> - "configure Bist test", 2},
> -};
> -
> -const int tuning_nb = ARRAY_SIZE(tuning);
>
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
Thanks
Patrice
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] stm32mp1: ram: add read valid training support
2021-11-15 14:32 [PATCH 1/3] stm32mp1: ram: add read valid training support Patrick Delaunay
` (2 preceding siblings ...)
2021-11-22 7:48 ` [PATCH 1/3] stm32mp1: ram: add read valid training support Patrice CHOTARD
@ 2021-11-30 10:36 ` Patrice CHOTARD
3 siblings, 0 replies; 7+ messages in thread
From: Patrice CHOTARD @ 2021-11-30 10:36 UTC (permalink / raw)
To: Patrick Delaunay, u-boot; +Cc: U-Boot STM32
Hi Patrick
On 11/15/21 3:32 PM, Patrick Delaunay wrote:
> Add the read data eye training = training for optimal read valid placement
> (RVTRN) when the built-in calibration is executed for LPDDR2 and LPDDR3.
>
> This training is supported on the PUBL integrated in the STM32MP15x
> DDR subsystem and it is not required for DDR3.
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
> ---
>
> drivers/ram/stm32mp1/stm32mp1_ddr.c | 8 ++++++--
> drivers/ram/stm32mp1/stm32mp1_ddr_regs.h | 1 +
> 2 files changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr.c b/drivers/ram/stm32mp1/stm32mp1_ddr.c
> index 0457166b12..1f8422518b 100644
> --- a/drivers/ram/stm32mp1/stm32mp1_ddr.c
> +++ b/drivers/ram/stm32mp1/stm32mp1_ddr.c
> @@ -826,8 +826,12 @@ start:
> */
>
> /* 10. configure PUBL PIR register to specify which training step to run */
> - /* warning : RVTRN is not supported by this PUBL */
> - stm32mp1_ddrphy_init(priv->phy, DDRPHYC_PIR_QSTRN);
> + /* RVTRN is excuted only on LPDDR2/LPDDR3 */
> + if (config->c_reg.mstr & DDRCTRL_MSTR_DDR3)
> + pir = DDRPHYC_PIR_QSTRN;
> + else
> + pir = DDRPHYC_PIR_QSTRN | DDRPHYC_PIR_RVTRN;
> + stm32mp1_ddrphy_init(priv->phy, pir);
>
> /* 11. monitor PUB PGSR.IDONE to poll cpmpletion of training sequence */
> ddrphy_idone_wait(priv->phy);
> diff --git a/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h b/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h
> index 3c8885a965..ada3087328 100644
> --- a/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h
> +++ b/drivers/ram/stm32mp1/stm32mp1_ddr_regs.h
> @@ -309,6 +309,7 @@ struct stm32mp1_ddrphy {
> #define DDRPHYC_PIR_DRAMRST BIT(5)
> #define DDRPHYC_PIR_DRAMINIT BIT(6)
> #define DDRPHYC_PIR_QSTRN BIT(7)
> +#define DDRPHYC_PIR_RVTRN BIT(8)
> #define DDRPHYC_PIR_ICPC BIT(16)
> #define DDRPHYC_PIR_ZCALBYP BIT(30)
> #define DDRPHYC_PIR_INITSTEPS_MASK GENMASK(31, 7)
>
For the series, applied to u-boot-stm32/next
Thanks
Patrice
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-11-30 10:36 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-15 14:32 [PATCH 1/3] stm32mp1: ram: add read valid training support Patrick Delaunay
2021-11-15 14:32 ` [PATCH 2/3] stm32mp1: ram: remove the support of calibration result Patrick Delaunay
2021-11-22 7:49 ` Patrice CHOTARD
2021-11-15 14:32 ` [PATCH 3/3] stm32mp1: ram: remove tuning support Patrick Delaunay
2021-11-22 7:51 ` Patrice CHOTARD
2021-11-22 7:48 ` [PATCH 1/3] stm32mp1: ram: add read valid training support Patrice CHOTARD
2021-11-30 10:36 ` Patrice CHOTARD
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.