Re: [U-Boot] [PATCH v4 1/7] driver/ddr: Add support for setting timing in hws_topology_map

[Chris Packham]

Hi Marek,

On Sat, Jun 10, 2017 at 5:28 AM, Marek Behún  wrote:
> The DDR3 training code for Marvell A38X currently computes 1t timing
> when given board topology map of the Turris Omnia, but Omnia needs 2t.
>
> This patch adds support for enforcing the 2t timing in struct
> hws_topology_map, through a new enum hws_timing, which can assume
> following values:
>   HWS_TIM_DEFAULT - default behaviour, compute whether to enable 2t
> from the number of CSs
>   HWS_TIM_1T  - enforce 1t
>   HWS_TIM_2T  - enforce 2t
>
> This patch also sets all the board topology maps (db-88f6820-amc,
> db-88f6820-gp, controlcenterdc and clearfog) to have timing set to
> HWS_TIM_DEFAULT.
>
> Signed-off-by: Marek Behun 
> Reviewed-by: Stefan Roese 

Sorry for only noticing this now but I'd like to add to this
discussion that Marvell _unconditionally_ enable 2T mode in their
bootloader (personally I hope they adopt your change). This may be an
indication that we should set HWS_TIM_2T for the clearfog and
db-88f6820 boards.
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH v4 1/7] driver/ddr: Add support for setting timing in hws_topology_map

[Marek Behún]

The DDR3 training code for Marvell A38X currently computes 1t timing
when given board topology map of the Turris Omnia, but Omnia needs 2t.

This patch adds support for enforcing the 2t timing in struct
hws_topology_map, through a new enum hws_timing, which can assume
following values:
  HWS_TIM_DEFAULT - default behaviour, compute whether to enable 2t
from the number of CSs
  HWS_TIM_1T  - enforce 1t
  HWS_TIM_2T  - enforce 2t

This patch also sets all the board topology maps (db-88f6820-amc,
db-88f6820-gp, controlcenterdc and clearfog) to have timing set to
HWS_TIM_DEFAULT.

Signed-off-by: Marek Behun 
Reviewed-by: Stefan Roese 

diff --git a/board/Marvell/db-88f6820-amc/db-88f6820-amc.c 
b/board/Marvell/db-88f6820-amc/db-88f6820-amc.c
index cade99c8d7..40fa599865 100644
--- a/board/Marvell/db-88f6820-amc/db-88f6820-amc.c
+++ b/board/Marvell/db-88f6820-amc/db-88f6820-amc.c
@@ -69,7 +69,8 @@ static struct hws_topology_map board_topology_map = {
MEM_4G, /* mem_size */
DDR_FREQ_800,   /* frequency */
0, 0,   /* cas_l cas_wl */
-   HWS_TEMP_LOW} },/* temperature */
+   HWS_TEMP_LOW,   /* temperature */
+   HWS_TIM_DEFAULT} }, /* timing */
5,  /* Num Of Bus Per Interface*/
BUS_MASK_32BIT  /* Busses mask */
 };
diff --git a/board/Marvell/db-88f6820-gp/db-88f6820-gp.c 
b/board/Marvell/db-88f6820-gp/db-88f6820-gp.c
index e700781103..a1974cb4bd 100644
--- a/board/Marvell/db-88f6820-gp/db-88f6820-gp.c
+++ b/board/Marvell/db-88f6820-gp/db-88f6820-gp.c
@@ -90,7 +90,8 @@ static struct hws_topology_map board_topology_map = {
MEM_4G, /* mem_size */
DDR_FREQ_800,   /* frequency */
0, 0,   /* cas_l cas_wl */
-   HWS_TEMP_LOW} },/* temperature */
+   HWS_TEMP_LOW,   /* temperature */
+   HWS_TIM_DEFAULT} }, /* timing */
5,  /* Num Of Bus Per Interface*/
BUS_MASK_32BIT  /* Busses mask */
 };
diff --git a/board/gdsys/a38x/controlcenterdc.c 
b/board/gdsys/a38x/controlcenterdc.c
index f0efb53447..32168d3576 100644
--- a/board/gdsys/a38x/controlcenterdc.c
+++ b/board/gdsys/a38x/controlcenterdc.c
@@ -53,7 +53,8 @@ static struct hws_topology_map ddr_topology_map = {
MEM_4G, /* mem_size */
DDR_FREQ_533,   /* frequency */
0, 0,   /* cas_l cas_wl */
-   HWS_TEMP_LOW} },/* temperature */
+   HWS_TEMP_LOW,   /* temperature */
+   HWS_TIM_DEFAULT} }, /* timing */
5,  /* Num Of Bus Per Interface*/
BUS_MASK_32BIT  /* Busses mask */
 };
diff --git a/board/solidrun/clearfog/clearfog.c 
b/board/solidrun/clearfog/clearfog.c
index 3a8257cac3..8906636f76 100644
--- a/board/solidrun/clearfog/clearfog.c
+++ b/board/solidrun/clearfog/clearfog.c
@@ -83,7 +83,8 @@ static struct hws_topology_map board_topology_map = {
MEM_4G, /* mem_size */
DDR_FREQ_800,   /* frequency */
0, 0,   /* cas_l cas_wl */
-   HWS_TEMP_LOW} },/* temperature */
+   HWS_TEMP_LOW,   /* temperature */
+   HWS_TIM_DEFAULT} }, /* timing */
5,  /* Num Of Bus Per Interface*/
BUS_MASK_32BIT  /* Busses mask */
 };
diff --git a/drivers/ddr/marvell/a38x/ddr3_training.c 
b/drivers/ddr/marvell/a38x/ddr3_training.c
index 7e0749fde3..e70ca4b425 100644
--- a/drivers/ddr/marvell/a38x/ddr3_training.c
+++ b/drivers/ddr/marvell/a38x/ddr3_training.c
@@ -308,6 +308,7 @@ int hws_ddr3_tip_init_controller(u32 dev_num, struct 
init_cntr_param *init_cntr_
enum hws_speed_bin speed_bin_index = SPEED_BIN_DDR_2133N;
enum hws_mem_size memory_size = MEM_2G;
enum hws_ddr_freq freq = init_freq;
+   enum hws_timing timing;
u32 cs_mask = 0;
u32 cl_value = 0, cwl_val = 0;
u32 refresh_interval_cnt = 0, bus_cnt = 0, adll_tap = 0;
@@ -569,8 +570,13 @@ int hws_ddr3_tip_init_controller(u32 dev_num, struct 
init_cntr_param *init_cntr_
  DUNIT_CONTROL_HIGH_REG,
  (init_cntr_prm->msys_init << 7), (1 << 
7)));
 
+   timing = tm->interface_params[if_id].timing;
+
if (mode2_t != 0xff) {
t2t = mode2_t;
+   } else if (timing != HWS_TIM_DEFAULT) {
+   /* Board topology map is forcing timing */
+