Re: [PATCH] drm/amdgpu: fix parameter error of RREG32_PCIE() in amdgpu_regs_pcie

2021-03-02 Thread Wang, Kevin(Yang)
[AMD Public Use]

yes, thanks lijo.

Best Regards,
Kevin


From: Lazar, Lijo 
Sent: Tuesday, March 2, 2021 4:09 PM
To: Wang, Kevin(Yang) ; amd-gfx@lists.freedesktop.org 

Cc: Zhang, Hawking 
Subject: RE: [PATCH] drm/amdgpu: fix parameter error of RREG32_PCIE() in 
amdgpu_regs_pcie

[AMD Public Use]

Same can be done for pcie_write also.

Reviewed-by: Lijo Lazar 

-Original Message-
From: Wang, Kevin(Yang) 
Sent: Tuesday, March 2, 2021 1:34 PM
To: amd-gfx@lists.freedesktop.org
Cc: Zhang, Hawking ; Lazar, Lijo ; 
Wang, Kevin(Yang) 
Subject: [PATCH] drm/amdgpu: fix parameter error of RREG32_PCIE() in 
amdgpu_regs_pcie

the register offset isn't needed division by 4 to pass RREG32_PCIE()

Signed-off-by: Kevin Wang 
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c
index a09469f84251..f3434a6f120f 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c
@@ -317,7 +317,7 @@ static ssize_t amdgpu_debugfs_regs_pcie_read(struct file 
*f, char __user *buf,
 while (size) {
 uint32_t value;

-   value = RREG32_PCIE(*pos >> 2);
+   value = RREG32_PCIE(*pos);
 r = put_user(value, (uint32_t *)buf);
 if (r) {
 pm_runtime_mark_last_busy(adev_to_drm(adev)->dev);
--
2.17.1
___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx


RE: [PATCH] drm/amdgpu: fix parameter error of RREG32_PCIE() in amdgpu_regs_pcie

2021-03-02 Thread Lazar, Lijo
[AMD Public Use]

Same can be done for pcie_write also.

Reviewed-by: Lijo Lazar 

-Original Message-
From: Wang, Kevin(Yang)  
Sent: Tuesday, March 2, 2021 1:34 PM
To: amd-gfx@lists.freedesktop.org
Cc: Zhang, Hawking ; Lazar, Lijo ; 
Wang, Kevin(Yang) 
Subject: [PATCH] drm/amdgpu: fix parameter error of RREG32_PCIE() in 
amdgpu_regs_pcie

the register offset isn't needed division by 4 to pass RREG32_PCIE()

Signed-off-by: Kevin Wang 
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c
index a09469f84251..f3434a6f120f 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c
@@ -317,7 +317,7 @@ static ssize_t amdgpu_debugfs_regs_pcie_read(struct file 
*f, char __user *buf,
while (size) {
uint32_t value;
 
-   value = RREG32_PCIE(*pos >> 2);
+   value = RREG32_PCIE(*pos);
r = put_user(value, (uint32_t *)buf);
if (r) {
pm_runtime_mark_last_busy(adev_to_drm(adev)->dev);
-- 
2.17.1
___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx