diff options
| author | Felix Held <felix-coreboot@felixheld.de> | 2022-12-13 19:38:01 +0100 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2022-12-14 17:58:08 +0000 |
| commit | c4f5241e66ef245cd8061972dd83716b351fd972 (patch) | |
| tree | 09d82a1d9877bf6fd57347a520b82b56c439a988 /src/soc/amd/common/block | |
| parent | 993092039be26a4c822a05e345eada72654adf65 (diff) | |
soc/amd/common/block/espi_util: drop unneeded check in espi_get_config
Since soc_get_common_config will either return a valid pointer or cause
a linking error, this function will also return a valid pointer or cause
a linking error, so no need for additional runtime checks.
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I99661247b9f8f47a708e3a6ff3f9e5359b505509
Reviewed-on: https://review.coreboot.org/c/coreboot/+/70739
Reviewed-by: Martin Roth <martin.roth@amd.corp-partner.google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
Reviewed-by: Varshit Pandya <pandyavarshit@gmail.com>
Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>
Diffstat (limited to 'src/soc/amd/common/block')
| -rw-r--r-- | src/soc/amd/common/block/lpc/espi_util.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/soc/amd/common/block/lpc/espi_util.c b/src/soc/amd/common/block/lpc/espi_util.c index 0cd35e5ed8..c0f2354f4f 100644 --- a/src/soc/amd/common/block/lpc/espi_util.c +++ b/src/soc/amd/common/block/lpc/espi_util.c @@ -374,10 +374,6 @@ enum cb_err espi_open_mmio_window(uint32_t base, size_t size) static const struct espi_config *espi_get_config(void) { const struct soc_amd_common_config *soc_cfg = soc_get_common_config(); - - if (!soc_cfg) - die("Common config structure is NULL!\n"); - return &soc_cfg->espi_config; } |