diff options
| author | Subrata Banik <subrata.banik@intel.com> | 2021-03-22 15:50:08 +0530 |
|---|---|---|
| committer | Subrata Banik <subrata.banik@intel.com> | 2021-03-26 04:52:57 +0000 |
| commit | c8ac8f5ce969196f9e38e24629120ab8c5a4c873 (patch) | |
| tree | b05c98e9ea1c3bfac49e92de86faf44544598cd9 /src/mainboard/intel/adlrvp | |
| parent | 74406e10539336e6eb201e56c4ee559c54973f03 (diff) | |
soc/intel/alderlake: Align RcompResistor definition as per MRC
List of changes:
1. Alder Lake MRC is expecting a RcompResistor value of word width.
Reference RCOMP resistors on motherboard are ~ 100 Ohms but coreboot
is passing an array of RcompResistor which is not completely in use.
Note: Rcomp resistor value represents rcomp resistor attached to
the DDR_COMP pins on the SoC.
2. Also, remove usage of '&' with memcpy the required value into
RcompTarget array.
3. Also, update RcompResistor value for ADLRVP.
BUG=b:183341229
TEST=Enable FSP debug log to verify the override value for
RcompResistor is reflecting correctly.
Change-Id: I69c7cec55b65036fc039c33374a3fd363ef7004e
Signed-off-by: Subrata Banik <subrata.banik@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/51704
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'src/mainboard/intel/adlrvp')
| -rw-r--r-- | src/mainboard/intel/adlrvp/memory.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/mainboard/intel/adlrvp/memory.c b/src/mainboard/intel/adlrvp/memory.c index 6f158a057f..68f7fca483 100644 --- a/src/mainboard/intel/adlrvp/memory.c +++ b/src/mainboard/intel/adlrvp/memory.c @@ -14,8 +14,8 @@ static const struct mb_cfg ddr4_mem_config = { .ddr_config = { - /* Baseboard uses only 100ohm Rcomp resistors */ - .rcomp_resistor = {100, 100, 100}, + /* Baseboard uses only 100ohm Rcomp resistor */ + .rcomp_resistor = 100, /* Baseboard Rcomp target values */ .rcomp_targets = {40, 30, 33, 33, 30}, @@ -148,8 +148,8 @@ static const struct mb_cfg ddr5_mem_config = { .UserBd = BOARD_TYPE_MOBILE, .ddr_config = { - /* Baseboard uses only 100ohm Rcomp resistors */ - .rcomp_resistor = {100, 100, 100}, + /* Baseboard uses only 100ohm Rcomp resistor */ + .rcomp_resistor = 100, /* Baseboard Rcomp target values */ .rcomp_targets = {50, 30, 30, 30, 27}, |