summaryrefslogtreecommitdiff
path: root/src/superio/nsc/pc87384/superio.c
diff options
context:
space:
mode:
authorFurquan Shaikh <furquan@google.com>2019-03-19 20:34:11 -0700
committerFurquan Shaikh <furquan@google.com>2019-03-20 13:49:54 +0000
commit4dfd8d690d17a484eef83006a843d561dd7ca13d (patch)
tree41ec1290d9c22adff76323b8a2087211fa99e8ab /src/superio/nsc/pc87384/superio.c
parent52d79fbc6e1243a7ecaf34835893476c5a303271 (diff)
soc/intel/cannonlake: Fix return values for get_param_value
Commit 41483c9 (soc/intel/cannonlake: Add required FSP UPD changes for CML) changed the enum values for PCH_SERIAL_IO_MODE so that 0 is invalid and valid values start from 1. However, get_param_value was not updated to correctly subtract 1 before returning any value. This change adds a macro PCH_SERIAL_IO_INDEX to apply the subtract 1 operation on any value that get_param_value needs to return. BUG=b:128946016 TEST=Verified that hatch boots successfully. Change-Id: I4e32fcd1efe4a535251f0ec58662a2dc5f70e8b0 Signed-off-by: Furquan Shaikh <furquan@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/31974 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Duncan Laurie <dlaurie@chromium.org> Reviewed-by: Rizwan Qureshi <rizwan.qureshi@intel.com> Reviewed-by: Maulik V Vaghela <maulik.v.vaghela@intel.com> Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/superio/nsc/pc87384/superio.c')
0 files changed, 0 insertions, 0 deletions