Re: [PATCH 2/2] Remove redundant lower_32_bits() calls when settings SDMA doorbell

2022-04-25 Thread Haohui Mai
Sounds good to me. Updated the patch for the CIK / SI hardware. I kept the clamping code to be safe. Signed-off-by: Haohui Mai --- drivers/gpu/drm/amd/amdgpu/cik_sdma.c | 5 ++--- drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c | 4 ++-- drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c | 8 drivers/gpu/

Re: [PATCH 2/2] Remove redundant lower_32_bits() calls when settings SDMA doorbell

2022-04-25 Thread Christian König
Am 25.04.22 um 13:49 schrieb Haohui Mai: I kept the original clamping for CIK / SI in this patch. I don't really care much about them. Keep it as you like, it's only for the older hw generations anyway. Regards, Christian. Please let me know if you want to remove them. Signed-off-by: Hao

[PATCH 2/2] Remove redundant lower_32_bits() calls when settings SDMA doorbell

2022-04-25 Thread Haohui Mai
I kept the original clamping for CIK / SI in this patch. Please let me know if you want to remove them. Signed-off-by: Haohui Mai --- drivers/gpu/drm/amd/amdgpu/cik_sdma.c | 5 ++--- drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c | 4 ++-- drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c | 8 drivers/g