tree 312dc8b299106c662ee94555a7e9d0c2e981852f
parent 0d0a893f089f8a8c30ddd9b17a1348538a6406e8
author Marshall Dawson <marshalldawson3rd@gmail.com> 1568232154 -0600
committer Commit Bot <commit-bot@chromium.org> 1568383115 +0000

UPSTREAM: drivers/intel/fsp2_0: Allocate cfg_region_size for UPD

In FSP-S, the driver constructs its pointer to UPD using the offset
in the header.  Similarly, use the header's cfg_region_size for
allocating memory and copying the default configuration.

Add sanity checks for unexpedted configuration and UPD header
conditions.

TEST=Verify OK on Mandolin, verify a mock error condition
BUG=b:140648081

Change-Id: I1cee7da35e3b705e03caa9d519db3a5ad57fa945
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Original-Commit-Id: 71dbcf127ad26b6caa68e242f1404aae27bb49ae
Original-Change-Id: I20fad0e27a2ad537898b6d01e5241e1508da690c
Original-Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Original-Reviewed-on: https://review.coreboot.org/c/coreboot/+/35349
Original-Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Original-Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Original-Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/coreboot/+/1801644
Reviewed-by: Patrick Georgi <pgeorgi@chromium.org>
Commit-Queue: Patrick Georgi <pgeorgi@chromium.org>
Tested-by: Patrick Georgi <pgeorgi@chromium.org>