[PATCH 1/1] MAINTAINERS: Add maintainers for Media Controller

[PATCH 1/1] MAINTAINERS: Add maintainers for Media Controller

[Sakari Ailus]

When Media Controller was merged to mainline long, long time ago, no-one
bothered to think what its MAINTAINERS entry should be. Now that Media
Controller is moved into its own directory, address this at the same time.

So tell people to mail patches to myself and Laurent Pinchart.

Note that the patches are still merged through the Media tree, just like
any other driver or framework bits that have separate "mail patches to"
entries different from the main drivers/media one.

Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
---
This patch adds a MAINTAINERS entry for the MC framework and API, as
discussed in this week's weekly meeting.

 MAINTAINERS | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index ad2bf808b02c..0a064d10a5df 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -9643,6 +9643,17 @@ L:	linux-iio@vger.kernel.org
 S:	Maintained
 F:	drivers/iio/dac/cio-dac.c
 
+MEDIA CONTROLLER FRAMEWORK
+M:	Sakari Ailus <sakari.ailus@linux.intel.com>
+M:	Laurent Pinchart <laurent.pinchart@ideasonboard.com>
+L:	linux-media@vger.kernel.org
+W:	https://www.linuxtv.org
+T:	git git://linuxtv.org/media_tree.git
+S:	Supported
+F:	drivers/media/mc/
+F:	include/media/media-*.h
+F:	include/uapi/linux/media.h
+
 MEDIA DRIVERS FOR ASCOT2E
 M:	Sergey Kozlov <serjk@netup.ru>
 M:	Abylay Ospan <aospan@netup.ru>
-- 
2.11.0

Re: [PATCH 1/1] MAINTAINERS: Add maintainers for Media Controller

[Laurent Pinchart]

Hi Sakari,

Thank you for the patch.

On Thu, Jun 20, 2019 at 05:17:53PM +0300, Sakari Ailus wrote:
> When Media Controller was merged to mainline long, long time ago, no-one
> bothered to think what its MAINTAINERS entry should be. Now that Media
> Controller is moved into its own directory, address this at the same time.
> 
> So tell people to mail patches to myself and Laurent Pinchart.
> 
> Note that the patches are still merged through the Media tree, just like
> any other driver or framework bits that have separate "mail patches to"
> entries different from the main drivers/media one.
> 
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>

This will help me catching changes to the media controller, and being
more active on reviews, so I welcome that change.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

(pick the tag you find the most appropriate, or both if you desire)

> ---
> This patch adds a MAINTAINERS entry for the MC framework and API, as
> discussed in this week's weekly meeting.
> 
>  MAINTAINERS | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index ad2bf808b02c..0a064d10a5df 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -9643,6 +9643,17 @@ L:	linux-iio@vger.kernel.org
>  S:	Maintained
>  F:	drivers/iio/dac/cio-dac.c
>  
> +MEDIA CONTROLLER FRAMEWORK
> +M:	Sakari Ailus <sakari.ailus@linux.intel.com>
> +M:	Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> +L:	linux-media@vger.kernel.org
> +W:	https://www.linuxtv.org
> +T:	git git://linuxtv.org/media_tree.git
> +S:	Supported
> +F:	drivers/media/mc/
> +F:	include/media/media-*.h
> +F:	include/uapi/linux/media.h
> +
>  MEDIA DRIVERS FOR ASCOT2E
>  M:	Sergey Kozlov <serjk@netup.ru>
>  M:	Abylay Ospan <aospan@netup.ru>

-- 
Regards,

Laurent Pinchart

Re: [PATCH 1/1] MAINTAINERS: Add maintainers for Media Controller

[Mauro Carvalho Chehab]

Em Thu, 20 Jun 2019 17:19:55 +0300
Laurent Pinchart <laurent.pinchart@ideasonboard.com> escreveu:

> Hi Sakari,
> 
> Thank you for the patch.
> 
> On Thu, Jun 20, 2019 at 05:17:53PM +0300, Sakari Ailus wrote:
> > When Media Controller was merged to mainline long, long time ago, no-one
> > bothered to think what its MAINTAINERS entry should be. Now that Media
> > Controller is moved into its own directory, address this at the same time.
> > 
> > So tell people to mail patches to myself and Laurent Pinchart.
> > 
> > Note that the patches are still merged through the Media tree, just like
> > any other driver or framework bits that have separate "mail patches to"
> > entries different from the main drivers/media one.
> > 
> > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>  
> 
> This will help me catching changes to the media controller, and being
> more active on reviews, so I welcome that change.

Ok, but please notice that this is a core part of the subsystem, and
not a driver one.

So, you should be sure that you'll have enough bandwidth to not
get patches accumulated here for no more than a reasonable time
(a couple of weeks).

If both you and Sakari are ok with that, I'll pick it.

Regards,
Mauro

Re: [PATCH 1/1] MAINTAINERS: Add maintainers for Media Controller

[Sakari Ailus]

Hi Mauro,

On Mon, Jun 24, 2019 at 09:30:12AM -0300, Mauro Carvalho Chehab wrote:
> Em Thu, 20 Jun 2019 17:19:55 +0300
> Laurent Pinchart <laurent.pinchart@ideasonboard.com> escreveu:
> 
> > Hi Sakari,
> > 
> > Thank you for the patch.
> > 
> > On Thu, Jun 20, 2019 at 05:17:53PM +0300, Sakari Ailus wrote:
> > > When Media Controller was merged to mainline long, long time ago, no-one
> > > bothered to think what its MAINTAINERS entry should be. Now that Media
> > > Controller is moved into its own directory, address this at the same time.
> > > 
> > > So tell people to mail patches to myself and Laurent Pinchart.
> > > 
> > > Note that the patches are still merged through the Media tree, just like
> > > any other driver or framework bits that have separate "mail patches to"
> > > entries different from the main drivers/media one.
> > > 
> > > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>  
> > 
> > This will help me catching changes to the media controller, and being
> > more active on reviews, so I welcome that change.
> 
> Ok, but please notice that this is a core part of the subsystem, and
> not a driver one.
> 
> So, you should be sure that you'll have enough bandwidth to not
> get patches accumulated here for no more than a reasonable time
> (a couple of weeks).
> 
> If both you and Sakari are ok with that, I'll pick it.

I already do review MC patches (at least as long as I've noticed the
patches myself, I believed a few have slipped past my eyes over the years),
so this isn't really adding to my workload.

So to answer the question, yes, I've prepared to do this.

-- 
Kind regards,

Sakari Ailus
sakari.ailus@linux.intel.com

Re: [PATCH 1/1] MAINTAINERS: Add maintainers for Media Controller

[Laurent Pinchart]

Hi Mauro,

On Mon, Jun 24, 2019 at 09:30:12AM -0300, Mauro Carvalho Chehab wrote:
> Em Thu, 20 Jun 2019 17:19:55 +0300 Laurent Pinchart escreveu:
> > On Thu, Jun 20, 2019 at 05:17:53PM +0300, Sakari Ailus wrote:
> >> When Media Controller was merged to mainline long, long time ago, no-one
> >> bothered to think what its MAINTAINERS entry should be. Now that Media
> >> Controller is moved into its own directory, address this at the same time.
> >> 
> >> So tell people to mail patches to myself and Laurent Pinchart.
> >> 
> >> Note that the patches are still merged through the Media tree, just like
> >> any other driver or framework bits that have separate "mail patches to"
> >> entries different from the main drivers/media one.
> >> 
> >> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>  
> > 
> > This will help me catching changes to the media controller, and being
> > more active on reviews, so I welcome that change.
> 
> Ok, but please notice that this is a core part of the subsystem, and
> not a driver one.
> 
> So, you should be sure that you'll have enough bandwidth to not
> get patches accumulated here for no more than a reasonable time
> (a couple of weeks).
> 
> If both you and Sakari are ok with that, I'll pick it.

I think that will be fine, based on the existing development rate for
that code. Of course, if tomorrow 10 developers decide to work full time
on it, I may not be able to follow, but that's hardly specific to MC,
the media subsystem or me.

-- 
Regards,

Laurent Pinchart

Re: [PATCH 1/1] MAINTAINERS: Add maintainers for Media Controller

[Hans Verkuil]

On 6/20/19 4:17 PM, Sakari Ailus wrote:
> When Media Controller was merged to mainline long, long time ago, no-one
> bothered to think what its MAINTAINERS entry should be. Now that Media
> Controller is moved into its own directory, address this at the same time.
> 
> So tell people to mail patches to myself and Laurent Pinchart.
> 
> Note that the patches are still merged through the Media tree, just like
> any other driver or framework bits that have separate "mail patches to"
> entries different from the main drivers/media one.
> 
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>

Acked-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>

Regards,

	Hans

> ---
> This patch adds a MAINTAINERS entry for the MC framework and API, as
> discussed in this week's weekly meeting.
> 
>  MAINTAINERS | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index ad2bf808b02c..0a064d10a5df 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -9643,6 +9643,17 @@ L:	linux-iio@vger.kernel.org
>  S:	Maintained
>  F:	drivers/iio/dac/cio-dac.c
>  
> +MEDIA CONTROLLER FRAMEWORK
> +M:	Sakari Ailus <sakari.ailus@linux.intel.com>
> +M:	Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> +L:	linux-media@vger.kernel.org
> +W:	https://www.linuxtv.org
> +T:	git git://linuxtv.org/media_tree.git
> +S:	Supported
> +F:	drivers/media/mc/
> +F:	include/media/media-*.h
> +F:	include/uapi/linux/media.h
> +
>  MEDIA DRIVERS FOR ASCOT2E
>  M:	Sergey Kozlov <serjk@netup.ru>
>  M:	Abylay Ospan <aospan@netup.ru>
>