From: Andi Shyti <andi.sh...@linux.intel.com>
Coverity warns that 'cap' may be used uninitialised. If pcode
is disabled there is no need to go through the hassle of a
pcode read or taking a PM reference.
Check skip_pcode early in the function and return if it is set.
No change for platforms where pcode is enabled.
Signed-off-by: Andi Shyti <andi.sh...@linux.intel.com>
---
drivers/gpu/drm/xe/xe_device_sysfs.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/xe/xe_device_sysfs.c
b/drivers/gpu/drm/xe/xe_device_sysfs.c
index bd9015761aa0..3a083c215891 100644
--- a/drivers/gpu/drm/xe/xe_device_sysfs.c
+++ b/drivers/gpu/drm/xe/xe_device_sysfs.c
@@ -156,6 +156,9 @@ static int late_bind_create_files(struct device *dev)
u32 cap;
int ret;
+ if (xe->info.skip_pcode)
+ return 0;
+
xe_pm_runtime_get(xe);
ret = xe_pcode_read(root, PCODE_MBOX(PCODE_LATE_BINDING,
GET_CAPABILITY_STATUS, 0),
--
2.50.0
