sravan/system76-coreboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/amd/common/pi/agesawrapper: use is_dev_enabled(DEV_PTR())

Browse Source 
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>
This commit is contained in:
Felix Held 2023-12-20 22:47:35 +01:00 committed by Shelley Chen
parent 727ee66756
commit aa8ae1a9b8
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/soc/amd/common/pi/agesawrapper.c
View File

@ -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;
}