On 18-08-2025 22:57, Danilo Krummrich wrote:
On Thu Aug 14, 2025 at 12:59 PM CEST, Himal Prasad Ghimiray wrote:
From: Boris Brezillon <[email protected]>

We are about to pass more arguments to drm_gpuvm_sm_map[_ops_create](),
so, before we do that, let's pass arguments through a struct instead
of changing each call site every time a new optional argument is added.

v5
  - Use drm_gpuva_op_map—same as drm_gpuvm_map_req
  - Rebase changes for drm_gpuvm_sm_map_exec_lock()
  - Fix kernel-docs

v6
  - Use drm_gpuvm_map_req (Danilo/Matt)

v7
  - change member name to map instead of op_map
  - use local variable to minize the code changes in _sm_map

I know it's a thing in DRM, but I'd rather not have the version changes in the
commit message. Please move them below "---" or into the cover letter.

(No need to resend for this. :)

Sure. Thanks


Cc: Danilo Krummrich <[email protected]>
Cc: Brendan King <[email protected]>
Cc: Boris Brezillon <[email protected]>
Cc: Caterina Shablia <[email protected]>
Cc: Rob Clark <[email protected]>
Cc: Matthew Brost <[email protected]>
Cc: <[email protected]>
Co-developed-by: Himal Prasad Ghimiray <[email protected]>
Signed-off-by: Himal Prasad Ghimiray <[email protected]>
Signed-off-by: Boris Brezillon <[email protected]>
Acked-by: Danilo Krummrich <[email protected]> #v4
Reviewed-by: Matthew Brost <[email protected]> #v6

Reply via email to