diff options
| author | Marc Jones <marcj303@gmail.com> | 2017-10-12 15:19:51 -0600 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2017-10-15 22:33:37 +0000 |
| commit | 7e710e047f059025392ed6607909f41114d8e843 (patch) | |
| tree | 919eef301c1aa2cdcc0628b6ac1fb3572c9d8430 | |
| parent | f4dc596a383c2f6ee03d24f37396a8d8b78f155c (diff) | |
| download | coreboot-7e710e047f059025392ed6607909f41114d8e843.tar.xz | |
amd/soc/common: Print an error if an AGESA callout isn't supported
Let the developer or user know that a callout isn't supported.
Change-Id: I73a6c6930a6661627ad76e27bbb78be99e237949
Signed-off-by: Marc Jones <marcj303@gmail.com>
Reviewed-on: https://review.coreboot.org/21998
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
| -rw-r--r-- | src/soc/amd/common/def_callouts.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/soc/amd/common/def_callouts.c b/src/soc/amd/common/def_callouts.c index fe0d5497e1..aa35e345b5 100644 --- a/src/soc/amd/common/def_callouts.c +++ b/src/soc/amd/common/def_callouts.c @@ -33,8 +33,12 @@ AGESA_STATUS GetBiosCallout(UINT32 Func, UINTN Data, VOID *ConfigPtr) if (BiosCallouts[i].CalloutName == Func) break; } - if (i >= BiosCalloutsLen) + + if (i >= BiosCalloutsLen) { + printk(BIOS_ERR, "ERROR: AGESA Callout Not Supported: 0x%x", + (u32)Func); return AGESA_UNSUPPORTED; + } return BiosCallouts[i].CalloutPtr(Func, Data, ConfigPtr); } |