|
Okay, Thanks for the clarifying. Yes, we will need the interrupt handler. So let’s just keep the set function as is. Regards, From: Pan, Xinhui <Xinhui.Pan@xxxxxxx> #1 can’t drop it, Need install irq handler. irq_get will fail it ->set is empty. #4, move all codes in the {}… I did not notice it too…. From: Zhang, Hawking <Hawking.Zhang@xxxxxxx>
Patch #1 Can you just drop the whole function (sdma_v4_0_ecc_irq_funcs) ? Patch #4 What the patch actually do seems doesn’t match with the description. I only see you adjust some code alignment in patch #4, anything I missed? From: Pan, Xinhui <Xinhui.Pan@xxxxxxx>
Hi, This is to fix some issues reported recently. 1) no need to do sdma irq setting. Ras ta do it. 2) gpu reset is not stable on some boards, so make amdgpu_ras_reset_gpu a empty function. 3) fix build warnings. 4) ta might be missing, in such case, hit null pointer. xinhui pan (4): drm/amdgpu: do not set sdma ecc interrupt drm/amdgpu: skip gpu reset when ras error occured drm/amdgpu: fix compile warning drm/amdgpu: fix NULL pointer when ta is missing drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 1 - drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 5 +++-- drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h | 3 +++ drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 2 +- drivers/gpu/drm/amd/amdgpu/psp_v11_0.c | 12 ++++++------ drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c | 11 ----------- 7 files changed, 14 insertions(+), 22 deletions(-) |
_______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx
