ddr: marvell: a38x: add support for twin-die combined memory device
commit 6285efb8a118940877522c4c07bd7c64569b4f5f upstream. the twin-die combined memory device should be treatened as X8 device and not as X16 one Signed-off-by: Moti Buskila <motib@marvell.com> Reviewed-by: Kostya Porotchkin <kostap@marvell.com> [ - the default value for twin_die_combined is set to NOT_COMBINED for all boards, as this was default behaviour prior this change ] Signed-off-by: Marek Behún <marek.behun@nic.cz> Tested-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
This commit is contained in:
parent
52dd9b96fc
commit
32e7a6baef
@ -286,6 +286,7 @@ static struct mv_ddr_topology_map board_topology_map_1g = {
|
||||
MV_DDR_TIM_2T} }, /* timing */
|
||||
BUS_MASK_32BIT, /* Busses mask */
|
||||
MV_DDR_CFG_DEFAULT, /* ddr configuration data source */
|
||||
NOT_COMBINED, /* ddr twin-die combined */
|
||||
{ {0} }, /* raw spd data */
|
||||
{0} /* timing parameters */
|
||||
};
|
||||
@ -308,6 +309,7 @@ static struct mv_ddr_topology_map board_topology_map_2g = {
|
||||
MV_DDR_TIM_2T} }, /* timing */
|
||||
BUS_MASK_32BIT, /* Busses mask */
|
||||
MV_DDR_CFG_DEFAULT, /* ddr configuration data source */
|
||||
NOT_COMBINED, /* ddr twin-die combined */
|
||||
{ {0} }, /* raw spd data */
|
||||
{0} /* timing parameters */
|
||||
};
|
||||
|
@ -73,6 +73,7 @@ static struct mv_ddr_topology_map board_topology_map = {
|
||||
MV_DDR_TIM_DEFAULT} }, /* timing */
|
||||
BUS_MASK_32BIT, /* Busses mask */
|
||||
MV_DDR_CFG_DEFAULT, /* ddr configuration data source */
|
||||
NOT_COMBINED, /* ddr twin-die combined */
|
||||
{ {0} }, /* raw spd data */
|
||||
{0} /* timing parameters */
|
||||
};
|
||||
|
@ -94,6 +94,7 @@ static struct mv_ddr_topology_map board_topology_map = {
|
||||
MV_DDR_TIM_DEFAULT} }, /* timing */
|
||||
BUS_MASK_32BIT, /* Busses mask */
|
||||
MV_DDR_CFG_DEFAULT, /* ddr configuration data source */
|
||||
NOT_COMBINED, /* ddr twin-die combined */
|
||||
{ {0} }, /* raw spd data */
|
||||
{0} /* timing parameters */
|
||||
};
|
||||
|
@ -68,6 +68,7 @@ static struct mv_ddr_topology_map board_topology_map = {
|
||||
MV_DDR_TIM_2T} }, /* timing */
|
||||
BUS_MASK_32BIT_ECC, /* subphys mask */
|
||||
MV_DDR_CFG_DEFAULT, /* ddr configuration data source */
|
||||
NOT_COMBINED, /* ddr twin-die combined */
|
||||
{ {0} }, /* raw spd data */
|
||||
{0}, /* timing parameters */
|
||||
{ {0} }, /* electrical configuration */
|
||||
|
@ -71,6 +71,7 @@ static struct mv_ddr_topology_map ddr_topology_map = {
|
||||
MV_DDR_TIM_DEFAULT} }, /* timing */
|
||||
BUS_MASK_32BIT, /* Busses mask */
|
||||
MV_DDR_CFG_DEFAULT, /* ddr configuration data source */
|
||||
NOT_COMBINED, /* ddr twin-die combined */
|
||||
{ {0} }, /* raw spd data */
|
||||
{0} /* timing parameters */
|
||||
|
||||
|
@ -71,6 +71,7 @@ static struct mv_ddr_topology_map board_topology_map = {
|
||||
MV_DDR_TIM_DEFAULT} }, /* timing */
|
||||
BUS_MASK_32BIT_ECC, /* Busses mask */
|
||||
MV_DDR_CFG_DEFAULT, /* ddr configuration data source */
|
||||
NOT_COMBINED, /* ddr twin-die combined */
|
||||
{ {0} }, /* raw spd data */
|
||||
{0} /* timing parameters */
|
||||
};
|
||||
|
@ -142,6 +142,7 @@ static struct mv_ddr_topology_map board_topology_map = {
|
||||
MV_DDR_TIM_DEFAULT} }, /* timing */
|
||||
BUS_MASK_32BIT, /* Busses mask */
|
||||
MV_DDR_CFG_DEFAULT, /* ddr configuration data source */
|
||||
NOT_COMBINED, /* ddr twin-die combined */
|
||||
{ {0} }, /* raw spd data */
|
||||
{0}, /* timing parameters */
|
||||
{ {0} }, /* electrical configuration */
|
||||
|
@ -14,6 +14,11 @@
|
||||
#define MV_DDR_MAX_BUS_NUM 9
|
||||
#define MV_DDR_MAX_IFACE_NUM 1
|
||||
|
||||
enum mv_ddr_twin_die {
|
||||
COMBINED,
|
||||
NOT_COMBINED,
|
||||
};
|
||||
|
||||
struct bus_params {
|
||||
/* Chip Select (CS) bitmask (bits 0-CS0, bit 1- CS1 ...) */
|
||||
u8 cs_bitmask;
|
||||
@ -47,6 +52,9 @@ struct if_params {
|
||||
/* The DDR frequency for each interfaces */
|
||||
enum mv_ddr_freq memory_freq;
|
||||
|
||||
/* ddr twin-die */
|
||||
enum mv_ddr_twin_die twin_die_combined;
|
||||
|
||||
/*
|
||||
* delay CAS Write Latency
|
||||
* - 0 for using default value (jedec suggested)
|
||||
@ -113,6 +121,9 @@ struct mv_ddr_topology_map {
|
||||
/* source of ddr configuration data */
|
||||
enum mv_ddr_cfg_src cfg_src;
|
||||
|
||||
/* ddr twin-die */
|
||||
enum mv_ddr_twin_die twin_die_combined;
|
||||
|
||||
/* raw spd data */
|
||||
union mv_ddr_spd_data spd_data;
|
||||
|
||||
@ -193,6 +204,7 @@ struct mv_ddr_iface {
|
||||
|
||||
/* ddr interface topology map */
|
||||
struct mv_ddr_topology_map tm;
|
||||
|
||||
};
|
||||
|
||||
struct mv_ddr_iface *mv_ddr_iface_get(void);
|
||||
|
@ -127,6 +127,11 @@ int mv_ddr_topology_map_update(void)
|
||||
speed_bin_index = iface_params->speed_bin_index;
|
||||
freq = iface_params->memory_freq;
|
||||
|
||||
if (tm->twin_die_combined == COMBINED) {
|
||||
iface_params->bus_width = MV_DDR_DEV_WIDTH_8BIT;
|
||||
iface_params->memory_size -= 1;
|
||||
}
|
||||
|
||||
if (iface_params->cas_l == 0)
|
||||
iface_params->cas_l = mv_ddr_cl_val_get(speed_bin_index, freq);
|
||||
|
||||
@ -281,7 +286,6 @@ unsigned long long mv_ddr_mem_sz_per_cs_get(void)
|
||||
mem_sz_per_cs = (unsigned long long)mem_size[iface_params->memory_size] *
|
||||
(unsigned long long)sphys /
|
||||
(unsigned long long)sphys_per_dunit;
|
||||
|
||||
return mem_sz_per_cs;
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user