Skip to contentSkip to conversation

MAINTAINERS: Add maintainers for Media Controller

PATCH In review Sakari Ailus 4 participants 6 messages Jun 20, 2019 – Jun 24, 2019 Media (V4L/DVB)

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 MAINTAINERS: Add maintainers for Media Controller

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.

---
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(+)
MAINTAINERS +11 -0
index ad2bf808b02c..0a064d10a5df 100644
@@ -9643,6 +9643,17 @@ L:	[email protected]
 S:	Maintained
 F:	drivers/iio/dac/cio-dac.c
 
+MEDIA CONTROLLER FRAMEWORK
+M:	Sakari Ailus <[email protected]>
+M:	Laurent Pinchart <[email protected]>
+L:	[email protected]
+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 <[email protected]>
 M:	Abylay Ospan <[email protected]>
Signature · 1 line
2.11.0
Reviewed-by on v1 1/1
#1 lore raw

Hi Sakari,

Thank you for the patch.

Quoted 11 lines · 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.

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

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

Quoted 29 lines

---
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(+)
MAINTAINERS +11 -0
index ad2bf808b02c..0a064d10a5df 100644
@@ -9643,6 +9643,17 @@ L:	[email protected]
 S:	Maintained
 F:	drivers/iio/dac/cio-dac.c
 
+MEDIA CONTROLLER FRAMEWORK
+M:	Sakari Ailus <[email protected]>
+M:	Laurent Pinchart <[email protected]>
+L:	[email protected]
+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 <[email protected]>
 M:	Abylay Ospan <[email protected]>
Signature · 3 lines
Regards,

Laurent Pinchart
#2 lore raw

Em Thu, 20 Jun 2019 17:19:55 +0300
Laurent Pinchart <[email protected]> escreveu:

Quoted 11 lines

Hi Sakari,

Thank you for the patch.

Quoted 6 lines · 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.

Quoted 8 lines
Quoted 5 lines

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.

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

#3 lore raw

Hi Mauro,

Quoted 23 lines · Mauro Carvalho Chehab

Em Thu, 20 Jun 2019 17:19:55 +0300
Laurent Pinchart <[email protected]> escreveu:

Quoted 11 lines

Hi Sakari,

Thank you for the patch.

Quoted 6 lines · 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.

Quoted 8 lines
Quoted 5 lines

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.

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

Quoted 8 lines

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.

Signature · 4 lines
Kind regards,

Sakari Ailus
[email protected]
#4 lore raw
In reply to Mauro Carvalho Chehab

Hi Mauro,

Quoted 17 lines · Mauro Carvalho Chehab

Em Thu, 20 Jun 2019 17:19:55 +0300 Laurent Pinchart escreveu:

Quoted 7 lines · Sakari Ailus
Quoted 6 lines

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.

Quoted 8 lines
Quoted 5 lines

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.

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

Quoted 8 lines

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.

Signature · 3 lines
Regards,

Laurent Pinchart
Acked-by on v1 1/1
#5 lore raw
In reply to Sakari Ailus
Quoted 11 lines · 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.

Regards,

Hans

Quoted 30 lines

---
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(+)
MAINTAINERS +11 -0
index ad2bf808b02c..0a064d10a5df 100644
@@ -9643,6 +9643,17 @@ L:	[email protected]
 S:	Maintained
 F:	drivers/iio/dac/cio-dac.c
 
+MEDIA CONTROLLER FRAMEWORK
+M:	Sakari Ailus <[email protected]>
+M:	Laurent Pinchart <[email protected]>
+L:	[email protected]
+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 <[email protected]>
 M:	Abylay Ospan <[email protected]>