mbox series

[0/2] amdgpu's drm_driver becomes const

Message ID 20201104031129.361178-1-luben.tuikov@amd.com (mailing list archive)
Headers show
Series [1/2] drm/amdgpu/virt: fix handling of the atomic flag | expand

Message

Luben Tuikov Nov. 4, 2020, 3:11 a.m. UTC
Hi Daniel,

These two patches follow up your latest
DRM work to make definitions of struct drm_driver
in DRM low-level drivers, constant, in amdgpu.

This set doesn't descend from my previous patch
"drm/amdgpu: Convert to using devm_drm_dev_alloc() (v2)",
since our branch doesn't have it, and I can see
that your const patches aren't in drm-next yet,
but they are based on my conversion patch.
Perhaps you can graft these two patches locally
and dispatch them via drm-next. (There'll be
a one-line conflict, namely the devm_drm_dev_alloc().)

Thanks and Regards,
Luben

Alex Deucher (1):
  drm/amdgpu/virt: fix handling of the atomic flag

Luben Tuikov (1):
  drm/amdgpu: Make struct drm_driver const

 drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c  | 32 +++++++++++++++++++++---
 drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c  | 25 +-----------------
 drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c |  4 ++-
 3 files changed, 32 insertions(+), 29 deletions(-)

Comments

Daniel Vetter Nov. 4, 2020, 9:43 a.m. UTC | #1
On Tue, Nov 03, 2020 at 10:11:27PM -0500, Luben Tuikov wrote:
> Hi Daniel,
> 
> These two patches follow up your latest
> DRM work to make definitions of struct drm_driver
> in DRM low-level drivers, constant, in amdgpu.
> 
> This set doesn't descend from my previous patch
> "drm/amdgpu: Convert to using devm_drm_dev_alloc() (v2)",
> since our branch doesn't have it, and I can see
> that your const patches aren't in drm-next yet,
> but they are based on my conversion patch.
> Perhaps you can graft these two patches locally
> and dispatch them via drm-next. (There'll be
> a one-line conflict, namely the devm_drm_dev_alloc().)

Ah cool, thanks for doing this. I'll take your series here and include it
in mine and resend the entire pile.
-Daniel
> 
> Thanks and Regards,
> Luben
> 
> Alex Deucher (1):
>   drm/amdgpu/virt: fix handling of the atomic flag
> 
> Luben Tuikov (1):
>   drm/amdgpu: Make struct drm_driver const
> 
>  drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c  | 32 +++++++++++++++++++++---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c  | 25 +-----------------
>  drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c |  4 ++-
>  3 files changed, 32 insertions(+), 29 deletions(-)
> 
> -- 
> 2.29.2.154.g7f7ebe054a
>
Luben Tuikov Nov. 4, 2020, 4:24 p.m. UTC | #2
On 2020-11-04 04:43, Daniel Vetter wrote:
> On Tue, Nov 03, 2020 at 10:11:27PM -0500, Luben Tuikov wrote:
>> Hi Daniel,
>>
>> These two patches follow up your latest
>> DRM work to make definitions of struct drm_driver
>> in DRM low-level drivers, constant, in amdgpu.
>>
>> This set doesn't descend from my previous patch
>> "drm/amdgpu: Convert to using devm_drm_dev_alloc() (v2)",
>> since our branch doesn't have it, and I can see
>> that your const patches aren't in drm-next yet,
>> but they are based on my conversion patch.
>> Perhaps you can graft these two patches locally
>> and dispatch them via drm-next. (There'll be
>> a one-line conflict, namely the devm_drm_dev_alloc().)
> 
> Ah cool, thanks for doing this. I'll take your series here and include it
> in mine and resend the entire pile.
> -Daniel

Yeah, no problem, my pleasure.
Thanks for picking this up.

Regards,
Luben

>>
>> Thanks and Regards,
>> Luben
>>
>> Alex Deucher (1):
>>   drm/amdgpu/virt: fix handling of the atomic flag
>>
>> Luben Tuikov (1):
>>   drm/amdgpu: Make struct drm_driver const
>>
>>  drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c  | 32 +++++++++++++++++++++---
>>  drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c  | 25 +-----------------
>>  drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c |  4 ++-
>>  3 files changed, 32 insertions(+), 29 deletions(-)
>>
>> -- 
>> 2.29.2.154.g7f7ebe054a
>>
>