driver/ddr: Add support for setting timing in hws_topology_map
authorMarek BehĂșn <marek.behun@nic.cz>
Fri, 9 Jun 2017 17:28:40 +0000 (19:28 +0200)
committerStefan Roese <sr@denx.de>
Wed, 12 Jul 2017 04:56:48 +0000 (06:56 +0200)
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 <marek.behun@nic.cz>
Reviewed-by: Stefan Roese <sr@denx.de>
Signed-off-by: Stefan Roese <sr@denx.de>
board/Marvell/db-88f6820-amc/db-88f6820-amc.c
board/Marvell/db-88f6820-gp/db-88f6820-gp.c
board/gdsys/a38x/controlcenterdc.c
board/solidrun/clearfog/clearfog.c
drivers/ddr/marvell/a38x/ddr3_training.c
drivers/ddr/marvell/a38x/ddr_topology_def.h

index cade99c8d7326faeca1a0cbca62a6f0c723c6ef4..40fa59986573bca384cbfe7a5a0b3666d75e535f 100644 (file)
@@ -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 */
 };
index e700781103fbdf4cf231a062fb8c5eac3adb99b8..a1974cb4bd21138eaad9b9ce9dbe08cc3acfaa13 100644 (file)
@@ -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 */
 };
index f0efb5344781fbeed089eabf45506ad28f9e28bd..32168d35768a1a6375c2497276b5ed7294239c81 100644 (file)
@@ -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 */
 };
index 3a8257cac32e2d7b2875e9060272696d167eab50..8906636f7646d931ee6051b28a58f0a13567d50e 100644 (file)
@@ -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 */
 };
index 7e0749fde3f2ef917e0f61a24ca46a8f0530b8c6..e70ca4b42551f3fff040b840a518134a51df14a4 100644 (file)
@@ -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 */
+                               t2t = (timing == HWS_TIM_2T) ? 1 : 0;
                        } else {
                                /* calculate number of CS (per interface) */
                                CHECK_STATUS(calc_cs_num
index f8894e828a5469d3e3276e7018e775fc691d512f..229c3a127a837434926df47c25ccdd979eefd94c 100644 (file)
@@ -37,6 +37,12 @@ enum hws_mem_size {
        MEM_SIZE_LAST
 };
 
+enum hws_timing {
+       HWS_TIM_DEFAULT,
+       HWS_TIM_1T,
+       HWS_TIM_2T
+};
+
 struct bus_params {
        /* Chip Select (CS) bitmask (bits 0-CS0, bit 1- CS1 ...) */
        u8 cs_bitmask;
@@ -84,6 +90,9 @@ struct if_params {
 
        /* operation temperature */
        enum hws_temperature interface_temp;
+
+       /* 2T vs 1T mode (by default computed from number of CSs) */
+       enum hws_timing timing;
 };
 
 struct hws_topology_map {