[RFC PATCH] Replace dev_err with drm_err
Sukrut Heroorkar
hsukrut3 at gmail.com
Mon Jul 28 14:58:28 UTC 2025
Hello,
This patch converts dev_err logging to drm_err, as recommended
in gpu/todo.rst for drivers supporting multiple DRM instances.
Before proceeding further, I'd like to confirm if this is the
right direction for amdgpu.
The below change builds successfully. I am happy to send this
as a formal patch if it's acceptable.
Thank you.
Signed-off-by: Sukrut Heroorkar <hsukrut3 at gmail.com>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
index 13f0cdeb59c4..effffba39b5a 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
@@ -460,7 +460,7 @@ static int amdgpu_virt_read_pf2vf_data(struct amdgpu_device *adev)
return -EINVAL;
if (pf2vf_info->size > 1024) {
- dev_err(adev->dev, "invalid pf2vf message size: 0x%x\n", pf2vf_info->size);
+ drm_err(adev_to_drm(adev), "invalid pf2vf message size: 0x%x\n", pf2vf_info->size);
return -EINVAL;
}
@@ -471,7 +471,7 @@ static int amdgpu_virt_read_pf2vf_data(struct amdgpu_device *adev)
adev->virt.fw_reserve.p_pf2vf, pf2vf_info->size,
adev->virt.fw_reserve.checksum_key, checksum);
if (checksum != checkval) {
- dev_err(adev->dev,
+ drm_err(adev_to_drm(adev),
"invalid pf2vf message: header checksum=0x%x calculated checksum=0x%x\n",
checksum, checkval);
return -EINVAL;
--
2.43.0
More information about the dri-devel
mailing list