On Thu Aug 14, 2025 at 12:59 PM CEST, Himal Prasad Ghimiray wrote: > From: Boris Brezillon <boris.brezil...@collabora.com> > > 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. :) > Cc: Danilo Krummrich <d...@kernel.org> > Cc: Brendan King <brendan.k...@imgtec.com> > Cc: Boris Brezillon <bbrezil...@kernel.org> > Cc: Caterina Shablia <caterina.shab...@collabora.com> > Cc: Rob Clark <robin.cl...@oss.qualcomm.com> > Cc: Matthew Brost <matthew.br...@intel.com> > Cc: <dri-devel@lists.freedesktop.org> > Co-developed-by: Himal Prasad Ghimiray <himal.prasad.ghimi...@intel.com> > Signed-off-by: Himal Prasad Ghimiray <himal.prasad.ghimi...@intel.com> > Signed-off-by: Boris Brezillon <boris.brezil...@collabora.com> > Acked-by: Danilo Krummrich <d...@kernel.org> #v4 > Reviewed-by: Matthew Brost <matthew.br...@intel.com> #v6
