diff options
| author | Felix Held <felix-coreboot@felixheld.de> | 2023-12-20 22:47:35 +0100 |
|---|---|---|
| committer | Shelley Chen <shchen@google.com> | 2023-12-26 17:21:42 +0000 |
| commit | aa8ae1a9b832a2a3f7ee6c013dbac6c00ddfaacf (patch) | |
| tree | 9db3361527e295c6a4eb68caa3960c4fa90d4150 /src/soc/amd | |
| parent | 727ee667561909ecb05a2671bd7d885aa111cd60 (diff) | |
| download | coreboot-aa8ae1a9b832a2a3f7ee6c013dbac6c00ddfaacf.tar.gz coreboot-aa8ae1a9b832a2a3f7ee6c013dbac6c00ddfaacf.tar.bz2 coreboot-aa8ae1a9b832a2a3f7ee6c013dbac6c00ddfaacf.zip | |
soc/amd/common/pi/agesawrapper: use is_dev_enabled(DEV_PTR())
Since we have chipset devicetrees for all SoCs that include this code in
the build, we can use the DEV_PTR macro instead of using
pcidev_path_on_root to get the device struct pointer. We can also use
the is_dev_enabled function instead of checking the value of the enabled
element of the device struct directly.
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I5dcd92399e2d3f304352f2170dd3ef8761e86541
Reviewed-on: https://review.coreboot.org/c/coreboot/+/79672
Reviewed-by: Felix Singer <service+coreboot-gerrit@felixsinger.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc/amd')
| -rw-r--r-- | src/soc/amd/common/pi/agesawrapper.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/soc/amd/common/pi/agesawrapper.c b/src/soc/amd/common/pi/agesawrapper.c index 2539a1d68af3..beae4baa9c72 100644 --- a/src/soc/amd/common/pi/agesawrapper.c +++ b/src/soc/amd/common/pi/agesawrapper.c @@ -260,8 +260,7 @@ static AGESA_STATUS amd_init_late(AMD_LATE_PARAMS *LateParams) { AGESA_STATUS Status; - const struct device *dev = pcidev_path_on_root(IOMMU_DEVFN); - if (dev && dev->enabled) { + if (is_dev_enabled(DEV_PTR(iommu))) { LateParams->GnbLateConfiguration.GnbIoapicId = GNB_IOAPIC_ID; LateParams->GnbLateConfiguration.FchIoapicId = FCH_IOAPIC_ID; } |