diff options
| author | Jamie Chen <jamie.chen@intel.com> | 2021-06-03 11:10:57 +0800 |
|---|---|---|
| committer | Nick Vaccaro <nvaccaro@google.com> | 2021-08-12 18:00:54 +0000 |
| commit | adda399234603c38b983331a8deeebc070fb82aa (patch) | |
| tree | 86d8071b83b373f27c1de4d96ff065bfa77581de /src/soc/intel/apollolake/include | |
| parent | 9f7e9cbc2686c478af3d9cd17ea3103cf9fb1d20 (diff) | |
soc/intel/apollolake: change LPDDR4 density enum definition
Originally we use rank_density=0 to mean disable the channel, but actually
rank_density=0 means 4Gb density in the FSP.
This patch changes the LPDDR4 enum values to the real density number and
adds a switch statement to mapping the density define in the FSP.
BUG=b:178665760
BRANCH=NONE
TEST=build fw and flash to the dut, the dut can boot up successfully.
Change-Id: I36dba2cef130211e7aea9e2a4f82c5db78f82a83
Signed-off-by: Jamie Chen <jamie.chen@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/56805
Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-by: Karthik Ramasubramanian <kramasub@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc/intel/apollolake/include')
| -rw-r--r-- | src/soc/intel/apollolake/include/soc/meminit.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/soc/intel/apollolake/include/soc/meminit.h b/src/soc/intel/apollolake/include/soc/meminit.h index 83cf809453..b08427026f 100644 --- a/src/soc/intel/apollolake/include/soc/meminit.h +++ b/src/soc/intel/apollolake/include/soc/meminit.h @@ -51,9 +51,9 @@ enum { /* LPDDR4 module density in bits. */ enum { - LP4_8Gb_DENSITY = 2, - LP4_12Gb_DENSITY, - LP4_16Gb_DENSITY, + LP4_8Gb_DENSITY = 8, + LP4_12Gb_DENSITY = 12, + LP4_16Gb_DENSITY = 16, }; /* @@ -93,7 +93,7 @@ void meminit_lpddr4(FSP_M_CONFIG *cfg, int speed); * to the memory reference code. */ void meminit_lpddr4_enable_channel(FSP_M_CONFIG *cfg, int logical_chan, - int rank_density, int dual_rank, + int rank_density_gb, int dual_rank, const struct lpddr4_swizzle_cfg *scfg); struct lpddr4_sku { |