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

device_util: Handle domain device in dev_get_domain

Browse Source 
When the input device pointer pointing to a domain device,
dev_get_domain returns the input device itself.

TEST=Build and boot on intel/archercity CRB

Change-Id: I3a278a8f573de95406ee256fba17767def4ad75d
Signed-off-by: Shuo Liu <shuo.liu@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/81957
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
This commit is contained in:
Shuo Liu
2024-04-18 00:44:42 +08:00
committed by Felix Held
parent 48097a17f1
commit 101685de0c
2 changed files with 3 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/device/device_util.c
View File

@@ -252,9 +252,9 @@ const struct device *dev_get_domain(const struct device *dev)
{ {
/* Walk up the tree up to the domain */ /* Walk up the tree up to the domain */
while (dev && dev->upstream && !is_root_device(dev)) { while (dev && dev->upstream && !is_root_device(dev)) {
dev = dev->upstream->dev;
if (dev->path.type == DEVICE_PATH_DOMAIN) if (dev->path.type == DEVICE_PATH_DOMAIN)
return dev; return dev;
dev = dev->upstream->dev;
} }
return NULL; return NULL;

8
src/soc/intel/xeon_sp/chip_common.c
View File

@@ -136,11 +136,7 @@ int iio_pci_domain_socket_from_dev(const struct device *dev)
const struct device *domain; const struct device *domain;
union xeon_domain_path dn; union xeon_domain_path dn;
if (dev->path.type == DEVICE_PATH_DOMAIN)
domain = dev;
else
domain = dev_get_domain(dev); domain = dev_get_domain(dev);
if (!domain) if (!domain)
return -1; return -1;
@@ -162,11 +158,7 @@ int iio_pci_domain_stack_from_dev(const struct device *dev)
const struct device *domain; const struct device *domain;
union xeon_domain_path dn; union xeon_domain_path dn;
if (dev->path.type == DEVICE_PATH_DOMAIN)
domain = dev;
else
domain = dev_get_domain(dev); domain = dev_get_domain(dev);
if (!domain) if (!domain)
return -1; return -1;