2021-08-26 09:15:44

by Daniel Vetter

[permalink] [raw]
Subject: [PATCH] MAINTAINERS: Add dri-devel for component.[hc]

dri-devel is the main user, and somehow there's been the assumption
that component stuff is unmaintained.

References: https://lore.kernel.org/dri-devel/CAAEAJfDWOzCJxZFNtxeT7Cvr2pWbYrfz-YnA81sVNs-rM=8n4Q@mail.gmail.com/
Cc: Ezequiel Garcia <[email protected]>
Cc: Greg Kroah-Hartman <[email protected]>
Cc: "Rafael J. Wysocki" <[email protected]>
Signed-off-by: Daniel Vetter <[email protected]>
---
MAINTAINERS | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index ac58d0032abd..7cdc19815ec4 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -5707,6 +5707,11 @@ F: Documentation/admin-guide/blockdev/
F: drivers/block/drbd/
F: lib/lru_cache.c

+DRIVER COMPONENT FRAMEWORK
+L: [email protected]
+F: drivers/base/component.c
+F: include/linux/component.h
+
DRIVER CORE, KOBJECTS, DEBUGFS AND SYSFS
M: Greg Kroah-Hartman <[email protected]>
R: "Rafael J. Wysocki" <[email protected]>
--
2.32.0


2021-08-26 10:58:53

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add dri-devel for component.[hc]

On Thu, Aug 26, 2021 at 11:13:43AM +0200, Daniel Vetter wrote:
> dri-devel is the main user, and somehow there's been the assumption
> that component stuff is unmaintained.
>
> References: https://lore.kernel.org/dri-devel/CAAEAJfDWOzCJxZFNtxeT7Cvr2pWbYrfz-YnA81sVNs-rM=8n4Q@mail.gmail.com/
> Cc: Ezequiel Garcia <[email protected]>
> Cc: Greg Kroah-Hartman <[email protected]>
> Cc: "Rafael J. Wysocki" <[email protected]>
> Signed-off-by: Daniel Vetter <[email protected]>
> ---
> MAINTAINERS | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index ac58d0032abd..7cdc19815ec4 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -5707,6 +5707,11 @@ F: Documentation/admin-guide/blockdev/
> F: drivers/block/drbd/
> F: lib/lru_cache.c
>
> +DRIVER COMPONENT FRAMEWORK
> +L: [email protected]
> +F: drivers/base/component.c
> +F: include/linux/component.h

Acked-by: Greg Kroah-Hartman <[email protected]>

2021-08-26 19:41:37

by Daniel Vetter

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add dri-devel for component.[hc]

On Thu, Aug 26, 2021 at 12:56:26PM +0200, Greg Kroah-Hartman wrote:
> On Thu, Aug 26, 2021 at 11:13:43AM +0200, Daniel Vetter wrote:
> > dri-devel is the main user, and somehow there's been the assumption
> > that component stuff is unmaintained.
> >
> > References: https://lore.kernel.org/dri-devel/CAAEAJfDWOzCJxZFNtxeT7Cvr2pWbYrfz-YnA81sVNs-rM=8n4Q@mail.gmail.com/
> > Cc: Ezequiel Garcia <[email protected]>
> > Cc: Greg Kroah-Hartman <[email protected]>
> > Cc: "Rafael J. Wysocki" <[email protected]>
> > Signed-off-by: Daniel Vetter <[email protected]>
> > ---
> > MAINTAINERS | 5 +++++
> > 1 file changed, 5 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index ac58d0032abd..7cdc19815ec4 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -5707,6 +5707,11 @@ F: Documentation/admin-guide/blockdev/
> > F: drivers/block/drbd/
> > F: lib/lru_cache.c
> >
> > +DRIVER COMPONENT FRAMEWORK
> > +L: [email protected]
> > +F: drivers/base/component.c
> > +F: include/linux/component.h
>
> Acked-by: Greg Kroah-Hartman <[email protected]>

Will you merge this through your tree? I think merging component patches
through driver core makes sense (or topic branch or whatever) still. This
is just so that there's a wider reviewer pool than just you & Rafael.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch

2021-08-27 08:41:34

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add dri-devel for component.[hc]

On Thu, Aug 26, 2021 at 09:39:44PM +0200, Daniel Vetter wrote:
> On Thu, Aug 26, 2021 at 12:56:26PM +0200, Greg Kroah-Hartman wrote:
> > On Thu, Aug 26, 2021 at 11:13:43AM +0200, Daniel Vetter wrote:
> > > dri-devel is the main user, and somehow there's been the assumption
> > > that component stuff is unmaintained.
> > >
> > > References: https://lore.kernel.org/dri-devel/CAAEAJfDWOzCJxZFNtxeT7Cvr2pWbYrfz-YnA81sVNs-rM=8n4Q@mail.gmail.com/
> > > Cc: Ezequiel Garcia <[email protected]>
> > > Cc: Greg Kroah-Hartman <[email protected]>
> > > Cc: "Rafael J. Wysocki" <[email protected]>
> > > Signed-off-by: Daniel Vetter <[email protected]>
> > > ---
> > > MAINTAINERS | 5 +++++
> > > 1 file changed, 5 insertions(+)
> > >
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index ac58d0032abd..7cdc19815ec4 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -5707,6 +5707,11 @@ F: Documentation/admin-guide/blockdev/
> > > F: drivers/block/drbd/
> > > F: lib/lru_cache.c
> > >
> > > +DRIVER COMPONENT FRAMEWORK
> > > +L: [email protected]
> > > +F: drivers/base/component.c
> > > +F: include/linux/component.h
> >
> > Acked-by: Greg Kroah-Hartman <[email protected]>
>
> Will you merge this through your tree? I think merging component patches
> through driver core makes sense (or topic branch or whatever) still. This
> is just so that there's a wider reviewer pool than just you & Rafael.

Sure, now merged in my tree.

thanks,

greg k-h

2021-08-27 12:31:15

by Ezequiel Garcia

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add dri-devel for component.[hc]

Hi Daniel,

On Thu, 26 Aug 2021 at 06:13, Daniel Vetter <[email protected]> wrote:
>
> dri-devel is the main user, and somehow there's been the assumption
> that component stuff is unmaintained.
>
> References: https://lore.kernel.org/dri-devel/CAAEAJfDWOzCJxZFNtxeT7Cvr2pWbYrfz-YnA81sVNs-rM=8n4Q@mail.gmail.com/
> Cc: Ezequiel Garcia <[email protected]>
> Cc: Greg Kroah-Hartman <[email protected]>
> Cc: "Rafael J. Wysocki" <[email protected]>
> Signed-off-by: Daniel Vetter <[email protected]>
> ---
> MAINTAINERS | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index ac58d0032abd..7cdc19815ec4 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -5707,6 +5707,11 @@ F: Documentation/admin-guide/blockdev/
> F: drivers/block/drbd/
> F: lib/lru_cache.c
>
> +DRIVER COMPONENT FRAMEWORK
> +L: [email protected]
> +F: drivers/base/component.c
> +F: include/linux/component.h
> +

Thanks for taking care of this, much appreciated.

Ezequiel