diff mbox

MAINTAINERS: add separate entry for DRM TTM

Message ID 20171214133708.2641-1-christian.koenig@amd.com (mailing list archive)
State New, archived
Headers show

Commit Message

Christian König Dec. 14, 2017, 1:37 p.m. UTC
AMD is the major user of TTM, so it also makes sense that we maintain
it.

Signed-off-by: Christian König <christian.koenig@amd.com>
---
 MAINTAINERS | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Alex Deucher Dec. 14, 2017, 1:45 p.m. UTC | #1
On Thu, Dec 14, 2017 at 8:37 AM, Christian König
<ckoenig.leichtzumerken@gmail.com> wrote:
> AMD is the major user of TTM, so it also makes sense that we maintain
> it.
>
> Signed-off-by: Christian König <christian.koenig@amd.com>

Acked-by: Alex Deucher <alexander.deucher@amd.com>

> ---
>  MAINTAINERS | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 069ba63190b2..72e8f8f750ec 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -4805,6 +4805,14 @@ S:       Maintained
>  F:     drivers/gpu/drm/tinydrm/
>  F:     include/drm/tinydrm/
>
> +DRM TTM SUBSYSTEM
> +M:     Christian Koenig <christian.koenig@amd.com>
> +M:     Roger He <Hongbo.He@amd.com>
> +S:     Maintained
> +L:     dri-devel@lists.freedesktop.org
> +F:     include/drm/ttm/
> +F:     drivers/gpu/drm/ttm/
> +
>  DSBR100 USB FM RADIO DRIVER
>  M:     Alexey Klimov <klimov.linux@gmail.com>
>  L:     linux-media@vger.kernel.org
> --
> 2.11.0
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
Daniel Vetter Dec. 14, 2017, 2:41 p.m. UTC | #2
On Thu, Dec 14, 2017 at 08:45:10AM -0500, Alex Deucher wrote:
> On Thu, Dec 14, 2017 at 8:37 AM, Christian König
> <ckoenig.leichtzumerken@gmail.com> wrote:
> > AMD is the major user of TTM, so it also makes sense that we maintain
> > it.
> >
> > Signed-off-by: Christian König <christian.koenig@amd.com>
> 
> Acked-by: Alex Deucher <alexander.deucher@amd.com>
> 
> > ---
> >  MAINTAINERS | 8 ++++++++
> >  1 file changed, 8 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 069ba63190b2..72e8f8f750ec 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -4805,6 +4805,14 @@ S:       Maintained
> >  F:     drivers/gpu/drm/tinydrm/
> >  F:     include/drm/tinydrm/
> >
> > +DRM TTM SUBSYSTEM
> > +M:     Christian Koenig <christian.koenig@amd.com>
> > +M:     Roger He <Hongbo.He@amd.com>
> > +S:     Maintained
> > +L:     dri-devel@lists.freedesktop.org
> > +F:     include/drm/ttm/
> > +F:     drivers/gpu/drm/ttm/

A git tree would be good here I think. Defacto that's right now the amd
tree.
-Daniel

> > +
> >  DSBR100 USB FM RADIO DRIVER
> >  M:     Alexey Klimov <klimov.linux@gmail.com>
> >  L:     linux-media@vger.kernel.org
> > --
> > 2.11.0
> >
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
Christian König Dec. 14, 2017, 3:15 p.m. UTC | #3
Am 14.12.2017 um 15:41 schrieb Daniel Vetter:
> On Thu, Dec 14, 2017 at 08:45:10AM -0500, Alex Deucher wrote:
>> On Thu, Dec 14, 2017 at 8:37 AM, Christian König
>> <ckoenig.leichtzumerken@gmail.com> wrote:
>>> AMD is the major user of TTM, so it also makes sense that we maintain
>>> it.
>>>
>>> Signed-off-by: Christian König <christian.koenig@amd.com>
>> Acked-by: Alex Deucher <alexander.deucher@amd.com>
>>
>>> ---
>>>   MAINTAINERS | 8 ++++++++
>>>   1 file changed, 8 insertions(+)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index 069ba63190b2..72e8f8f750ec 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -4805,6 +4805,14 @@ S:       Maintained
>>>   F:     drivers/gpu/drm/tinydrm/
>>>   F:     include/drm/tinydrm/
>>>
>>> +DRM TTM SUBSYSTEM
>>> +M:     Christian Koenig <christian.koenig@amd.com>
>>> +M:     Roger He <Hongbo.He@amd.com>
>>> +S:     Maintained
>>> +L:     dri-devel@lists.freedesktop.org
>>> +F:     include/drm/ttm/
>>> +F:     drivers/gpu/drm/ttm/
> A git tree would be good here I think. Defacto that's right now the amd
> tree.

Yeah, I was wondering what to do with that as well.

I see three possible options of hand:
A) Stick with the amd trees maintained by Alex.
B) Come up with separate ones.
C) Use drm-misc.

For now I think I will stick with A if there are no objections.

Christian.

> -Daniel
>
>>> +
>>>   DSBR100 USB FM RADIO DRIVER
>>>   M:     Alexey Klimov <klimov.linux@gmail.com>
>>>   L:     linux-media@vger.kernel.org
>>> --
>>> 2.11.0
>>>
>>> _______________________________________________
>>> dri-devel mailing list
>>> dri-devel@lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
Emil Velikov Dec. 14, 2017, 5:48 p.m. UTC | #4
On 14 December 2017 at 13:37, Christian König
<ckoenig.leichtzumerken@gmail.com> wrote:
> AMD is the major user of TTM, so it also makes sense that we maintain
> it.
>
> Signed-off-by: Christian König <christian.koenig@amd.com>
> ---
>  MAINTAINERS | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 069ba63190b2..72e8f8f750ec 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -4805,6 +4805,14 @@ S:       Maintained
>  F:     drivers/gpu/drm/tinydrm/
>  F:     include/drm/tinydrm/
>
> +DRM TTM SUBSYSTEM
> +M:     Christian Koenig <christian.koenig@amd.com>
> +M:     Roger He <Hongbo.He@amd.com>
> +S:     Maintained
> +L:     dri-devel@lists.freedesktop.org
> +F:     include/drm/ttm/
> +F:     drivers/gpu/drm/ttm/
> +

IIRC Thomas was one of the people heavily involved in TTM, at least in
the early days.
An Ack from him might be a good idea, for posterity?

-Emil
Thomas Hellstrom Dec. 14, 2017, 5:55 p.m. UTC | #5
On 12/14/2017 06:48 PM, Emil Velikov wrote:
> On 14 December 2017 at 13:37, Christian König
> <ckoenig.leichtzumerken@gmail.com> wrote:
>> AMD is the major user of TTM, so it also makes sense that we maintain
>> it.
>>
>> Signed-off-by: Christian König <christian.koenig@amd.com>
>> ---
>>   MAINTAINERS | 8 ++++++++
>>   1 file changed, 8 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 069ba63190b2..72e8f8f750ec 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -4805,6 +4805,14 @@ S:       Maintained
>>   F:     drivers/gpu/drm/tinydrm/
>>   F:     include/drm/tinydrm/
>>
>> +DRM TTM SUBSYSTEM
>> +M:     Christian Koenig <christian.koenig@amd.com>
>> +M:     Roger He <Hongbo.He@amd.com>
>> +S:     Maintained
>> +L:     dri-devel@lists.freedesktop.org
>> +F:     include/drm/ttm/
>> +F:     drivers/gpu/drm/ttm/
>> +
> IIRC Thomas was one of the people heavily involved in TTM, at least in
> the early days.
> An Ack from him might be a good idea, for posterity?
>
> -Emil

Acked-by: Thomas Hellstrom <thellstrom@vmware.com>

Although of course I think it's important that we keep sending patches 
out before merging for reviewing and commenting, since even if AMD is 
the major user, it's still used by a number of other drivers.

Thanks,

/Thomas
Daniel Vetter Dec. 14, 2017, 8:05 p.m. UTC | #6
On Thu, Dec 14, 2017 at 04:15:27PM +0100, Christian König wrote:
> Am 14.12.2017 um 15:41 schrieb Daniel Vetter:
> > On Thu, Dec 14, 2017 at 08:45:10AM -0500, Alex Deucher wrote:
> > > On Thu, Dec 14, 2017 at 8:37 AM, Christian König
> > > <ckoenig.leichtzumerken@gmail.com> wrote:
> > > > AMD is the major user of TTM, so it also makes sense that we maintain
> > > > it.
> > > > 
> > > > Signed-off-by: Christian König <christian.koenig@amd.com>
> > > Acked-by: Alex Deucher <alexander.deucher@amd.com>
> > > 
> > > > ---
> > > >   MAINTAINERS | 8 ++++++++
> > > >   1 file changed, 8 insertions(+)
> > > > 
> > > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > > index 069ba63190b2..72e8f8f750ec 100644
> > > > --- a/MAINTAINERS
> > > > +++ b/MAINTAINERS
> > > > @@ -4805,6 +4805,14 @@ S:       Maintained
> > > >   F:     drivers/gpu/drm/tinydrm/
> > > >   F:     include/drm/tinydrm/
> > > > 
> > > > +DRM TTM SUBSYSTEM
> > > > +M:     Christian Koenig <christian.koenig@amd.com>
> > > > +M:     Roger He <Hongbo.He@amd.com>
> > > > +S:     Maintained
> > > > +L:     dri-devel@lists.freedesktop.org
> > > > +F:     include/drm/ttm/
> > > > +F:     drivers/gpu/drm/ttm/
> > A git tree would be good here I think. Defacto that's right now the amd
> > tree.
> 
> Yeah, I was wondering what to do with that as well.
> 
> I see three possible options of hand:
> A) Stick with the amd trees maintained by Alex.
> B) Come up with separate ones.
> C) Use drm-misc.
> 
> For now I think I will stick with A if there are no objections.

For C, would be happy to add Roger He as committer if there's a need.
Assuming you make sure he writes good commit message ofc :-)
-Daniel

> 
> Christian.
> 
> > -Daniel
> > 
> > > > +
> > > >   DSBR100 USB FM RADIO DRIVER
> > > >   M:     Alexey Klimov <klimov.linux@gmail.com>
> > > >   L:     linux-media@vger.kernel.org
> > > > --
> > > > 2.11.0
> > > > 
> > > > _______________________________________________
> > > > dri-devel mailing list
> > > > dri-devel@lists.freedesktop.org
> > > > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> > > _______________________________________________
> > > dri-devel mailing list
> > > dri-devel@lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
diff mbox

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 069ba63190b2..72e8f8f750ec 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -4805,6 +4805,14 @@  S:	Maintained
 F:	drivers/gpu/drm/tinydrm/
 F:	include/drm/tinydrm/
 
+DRM TTM SUBSYSTEM
+M:	Christian Koenig <christian.koenig@amd.com>
+M:	Roger He <Hongbo.He@amd.com>
+S:	Maintained
+L:	dri-devel@lists.freedesktop.org
+F:	include/drm/ttm/
+F:	drivers/gpu/drm/ttm/
+
 DSBR100 USB FM RADIO DRIVER
 M:	Alexey Klimov <klimov.linux@gmail.com>
 L:	linux-media@vger.kernel.org