Re: [PATCH 1/5] [media] horus3a: Sony Horus3A DVB-S/S2 tuner driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Em Wed, 18 Mar 2015 02:11:52 +0200
Antti Palosaari <crope@xxxxxx> escreveu:

> On 03/05/2015 10:54 AM, Mauro Carvalho Chehab wrote:
> > Em Mon, 02 Feb 2015 12:22:32 +0300
> > Kozlov Sergey <serjk@xxxxxxxx> escreveu:
> 
> >> diff --git a/MAINTAINERS b/MAINTAINERS
> >> index ddb9ac8..a3a1767 100644
> >> --- a/MAINTAINERS
> >> +++ b/MAINTAINERS
> >> @@ -4365,6 +4365,15 @@ W:	http://linuxtv.org
> >>   S:	Odd Fixes
> >>   F:	drivers/media/usb/hdpvr/
> >>
> >> +HORUS3A MEDIA DRIVER
> >
> > Not a big issue, but could you please rename it to:
> > 	MEDIA DRIVERS FOR HORUS3A
> >
> > We're trying to better organize the media entries at MAINTAINERS, at
> > least for the new drivers.
> 
> What the *ell is that new rule? MAINTAINERS file clearly says entries 
> should be alphabetical order,

Yes, whatever name is given, the MAINTAINERS entries should be in 
alphabetical order.

> but on the other-hand there seems to be 
> PCI and ARM specific stuff already grouped. Is that some new way?

Yeah, there is a tendency to try to group the entries on a similar
way on other subsystems.

So, Laurent proposed to do that for new drivers.

That's said, it makes no sense to rename the existing entries, as such
patch would likely make lots of people unhappy by causing a great number
of merge conflicts, but, as grouping the new entries seem to help
some people that manually looks at MAINTAINERS, it sounds a good
idea to do it for new drivers.

One advantage of grouping the entries for new drivers is that this
reduces the merge conflicts at MAINTAINERS, as it reduces the risk of
two subsystems to touch at the same part of the file.

Regards,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux