[Linaro-mm-sig] Re: [RFC v2 08/13] drm/amdgpu: Use dma-fence driver and timeline name helpers

On 5/9/25 17:33, Tvrtko Ursulin wrote:

Access the dma-fence internals via the previously added helpers.

Signed-off-by: Tvrtko Ursulin tvrtko.ursulin@igalia.com

drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h index 11dd2e0f7979..8e5bf179a6c8 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h @@ -33,7 +33,7 @@ #define TRACE_INCLUDE_FILE amdgpu_trace #define AMDGPU_JOB_GET_TIMELINE_NAME(job) \

• job->base.s_fence->finished.ops->get_timeline_name(&job->base.s_fence->finished)
  

• dma_fence_timeline_name(&job->base.s_fence->finished)
  

I think you can nuke the macro now as well, the new function name is short enough.

Christian.

TRACE_EVENT(amdgpu_device_rreg, TP_PROTO(unsigned did, uint32_t reg, uint32_t value),