Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1534878 > unrolled thread

[PATCH 0/2] Add maintainers to the admin guide

Back to article view | Back to linux.kernel

Contents

  [PATCH 0/2] Add maintainers to the admin guide Mauro Carvalho Chehab <mchehab@s-opensource.com> - 2016-12-02 13:20 +0100
    [PATCH 1/2] MAINTAINERS: convert first part to ReST markup Mauro Carvalho Chehab <mchehab@s-opensource.com> - 2016-12-02 13:20 +0100
    [PATCH 2/2] MAINTAINERS: add it to the admin-guide Mauro Carvalho Chehab <mchehab@s-opensource.com> - 2016-12-02 13:20 +0100
    Re: [PATCH 0/2] Add maintainers to the admin guide Daniel Vetter <daniel@ffwll.ch> - 2016-12-07 16:50 +0100

#1534878 — [PATCH 0/2] Add maintainers to the admin guide

That's my third attempt to add the MAINTAINERS contents to the admin-guide.

On the past approaches, was planning to keep the documentation
about what's at the MAINTAINERS file inside it, but that would
require running an external script or use some Sphinx extension.

This time, I took a much simpler approach: convert the initial
part of the MAINTAINERS file to ReST and move to a file at the
admin-guide. So, MAINTAINERS file will now contain only the
maintainer's database, and a single line pointing to its documentation.

That should hopefully be a good compromise, as we can add its
contents to a Sphinx file without causing too much hasle.


Mauro Carvalho Chehab (2):
  MAINTAINERS: convert first part to ReST markup
  MAINTAINERS: add it to the admin-guide

 Documentation/admin-guide/index.rst       |   1 +
 Documentation/admin-guide/maintainers.rst | 184 ++++++++++++++++++++++++++++++
 MAINTAINERS                               | 125 +-------------------
 3 files changed, 187 insertions(+), 123 deletions(-)
 create mode 100644 Documentation/admin-guide/maintainers.rst

-- 
2.9.3

[toc] | [next] | [standalone]

#1534879 — [PATCH 1/2] MAINTAINERS: convert first part to ReST markup

- Fix document section markups;
- Use tables;
- Use monotonic font for field names;
- adjust spaces and blank lines.

Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
---
 MAINTAINERS | 213 ++++++++++++++++++++++++++++++++++++++----------------------
 1 file changed, 136 insertions(+), 77 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index e13497c79fe4..1cb31237a2c0 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1,12 +1,14 @@
-
-
-	List of maintainers and how to submit kernel changes
+List of maintainers and how to submit kernel changes
+====================================================
 
 Please try to follow the guidelines below.  This will make things
 easier on the maintainers.  Not all of these guidelines matter for every
 trivial patch so apply some common sense.
 
-1.	Always _test_ your changes, however small, on at least 4 or
+Tips for patch submitters
+-------------------------
+
+1.	Always **test** your changes, however small, on at least 4 or
 	5 people, preferably many more.
 
 2.	Try to release a few ALPHA test versions to the net. Announce
@@ -15,7 +17,7 @@ trivial patch so apply some common sense.
 	you will find things like the fact version 3 firmware needs
 	a magic fix you didn't know about, or some clown changed the
 	chips on a board and not its name.  (Don't laugh!  Look at the
-	SMC etherpower for that.)
+	SMC ``etherpower`` for that.)
 
 3.	Make sure your changes compile correctly in multiple
 	configurations. In particular check that changes work both as a
@@ -25,7 +27,7 @@ trivial patch so apply some common sense.
 	testing and await feedback.
 
 5.	Make a patch available to the relevant maintainer in the list. Use
-	'diff -u' to make the patch easy to merge. Be prepared to get your
+	``diff -u`` to make the patch easy to merge. Be prepared to get your
 	changes sent back with seemingly silly requests about formatting
 	and variable names.  These aren't as silly as they seem. One
 	job the maintainers (and especially Linus) do is to keep things
@@ -33,28 +35,34 @@ trivial patch so apply some common sense.
 	your driver to get around a problem actually needs to become a
 	generalized kernel feature ready for next time.
 
-	PLEASE check your patch with the automated style checker
-	(scripts/checkpatch.pl) to catch trivial style violations.
-	See Documentation/process/coding-style.rst for guidance here.
+	.. attention::
 
-	PLEASE CC: the maintainers and mailing lists that are generated
-	by scripts/get_maintainer.pl.  The results returned by the
-	script will be best if you have git installed and are making
-	your changes in a branch derived from Linus' latest git tree.
-	See Documentation/process/submitting-patches.rst for details.
+	  **PLEASE:**
 
-	PLEASE try to include any credit lines you want added with the
-	patch. It avoids people being missed off by mistake and makes
-	it easier to know who wants adding and who doesn't.
+	  - check your patch with the automated style checker
+	    (``scripts/checkpatch.pl``) to catch trivial style violations.
+	    See :ref:`Documentation/process/coding-style.rst <codingstyle>`
+	    for guidance here.
 
-	PLEASE document known bugs. If it doesn't work for everything
-	or does something very odd once a month document it.
+	  - CC: the maintainers and mailing lists that are generated
+	    by ``scripts/get_maintainer.pl``.  The results returned by the
+	    script will be best if you have git installed and are making
+	    your changes in a branch derived from Linus' latest git tree.
+	    See :ref:`Documentation/process/submitting-patches.rst <submittingpatches>`
+	    for details.
 
-	PLEASE remember that submissions must be made under the terms
-	of the Linux Foundation certificate of contribution and should
-	include a Signed-off-by: line.  The current version of this
-	"Developer's Certificate of Origin" (DCO) is listed in the file
-	Documentation/process/submitting-patches.rst.
+	  - try to include any credit lines you want added with the
+	    patch. It avoids people being missed off by mistake and makes
+	    it easier to know who wants adding and who doesn't.
+
+	  - document known bugs. If it doesn't work for everything
+	    or does something very odd once a month document it.
+
+	  - remember that submissions must be made under the terms
+	    of the Linux Foundation certificate of contribution and should
+	    include a Signed-off-by: line.  The current version of this
+	    "Developer's Certificate of Origin" (DCO) is listed in the file
+	    :ref:`Documentation/process/submitting-patches.rst <submittingpatches>`.
 
 6.	Make sure you have the right to send any changes you make. If you
 	do changes at work you may find your employer owns the patch
@@ -66,64 +74,115 @@ trivial patch so apply some common sense.
 
 8.	Happy hacking.
 
-Descriptions of section entries:
-
-	P: Person (obsolete)
-	M: Mail patches to: FullName <address@domain>
-	R: Designated reviewer: FullName <address@domain>
-	   These reviewers should be CCed on patches.
-	L: Mailing list that is relevant to this area
-	W: Web-page with status/info
-	Q: Patchwork web based patch tracking system site
-	T: SCM tree type and location.
-	   Type is one of: git, hg, quilt, stgit, topgit
-	S: Status, one of the following:
-	   Supported:	Someone is actually paid to look after this.
-	   Maintained:	Someone actually looks after it.
-	   Odd Fixes:	It has a maintainer but they don't have time to do
-			much other than throw the odd patch in. See below..
-	   Orphan:	No current maintainer [but maybe you could take the
-			role as you write your new code].
-	   Obsolete:	Old code. Something tagged obsolete generally means
-			it has been replaced by a better system and you
-			should be using that.
-	F: Files and directories with wildcard patterns.
-	   A trailing slash includes all files and subdirectory files.
-	   F:	drivers/net/	all files in and below drivers/net
-	   F:	drivers/net/*	all files in drivers/net, but not below
-	   F:	*/net/*		all files in "any top level directory"/net
-	   One pattern per line.  Multiple F: lines acceptable.
-	N: Files and directories with regex patterns.
-	   N:	[^a-z]tegra	all files whose path contains the word tegra
-	   One pattern per line.  Multiple N: lines acceptable.
-	   scripts/get_maintainer.pl has different behavior for files that
-	   match F: pattern and matches of N: patterns.  By default,
-	   get_maintainer will not look at git log history when an F: pattern
-	   match occurs.  When an N: match occurs, git log history is used
-	   to also notify the people that have git commit signatures.
-	X: Files and directories that are NOT maintained, same rules as F:
-	   Files exclusions are tested before file matches.
-	   Can be useful for excluding a specific subdirectory, for instance:
-	   F:	net/
-	   X:	net/ipv6/
-	   matches all files in and below net excluding net/ipv6/
-	K: Keyword perl extended regex pattern to match content in a
-	   patch or file.  For instance:
-	   K: of_get_profile
-	      matches patches or files that contain "of_get_profile"
-	   K: \b(printk|pr_(info|err))\b
-	      matches patches or files that contain one or more of the words
-	      printk, pr_info or pr_err
-	   One regex pattern per line.  Multiple K: lines acceptable.
-
-Note: For the hard of thinking, this list is meant to remain in alphabetical
-order. If you could add yourselves to it in alphabetical order that would be
-so much easier [Ed]
+Descriptions of section entries
+-------------------------------
+
+- ``P:`` Person (obsolete)
+
+- ``M:`` Mail patches to: FullName <address@domain>
+
+- ``R:`` Designated reviewer: FullName <address@domain>
+
+	- These reviewers should be CCed on patches.
+
+- ``L:`` Mailing list that is relevant to this area
+
+- ``W:`` Web-page with status/info
+
+- ``Q:`` Patchwork web based patch tracking system site
+
+- ``T:`` SCM tree type and location.
+
+	- Type is one of: **git**, **hg**, **quilt**, **stgit**, **topgit**
+
+- ``S:`` Status, one of the following:
+
+	   - **Supported**:
+	     Someone is actually paid to look after this.
+
+	   - **Maintained**:
+	     Someone actually looks after it.
+
+	   - **Odd Fixes**:
+	     It has a maintainer but they don't have time to do
+	     much other than throw the odd patch in. See below..
+
+	   - **Orphan**:
+	     No current maintainer [but maybe you could take the
+	     role as you write your new code].
+
+	   - **Obsolete**:
+	     Old code. Something tagged obsolete generally means
+	     it has been replaced by a better system and you
+	     should be using that.
+
+- ``F:`` Files and directories with wildcard patterns.
+
+  A trailing slash includes all files and subdirectory files.
+
+  ===============================	================================
+  ``F:``	``drivers/net/``	all files in and below
+					``drivers/net``
+  ``F:``	``drivers/net/*``	all files in ``drivers/net``,
+					but not below
+  ``F:``	``*/net/*``		all files in "any top level
+					directory" ``/net``
+  ===============================	================================
+
+  One pattern per line.  Multiple ``F:`` lines acceptable.
+
+- ``N:`` Files and directories with regex patterns.
+
+  ============================	=============================================
+  ``N:``			``[^a-z]tegra``	all files whose path contains
+				the word ``tegra``
+  ============================	=============================================
+
+  One pattern per line.  Multiple ``N:`` lines acceptable.
+  ``scripts/get_maintainer.pl`` has different behavior for files that
+  match ``F:`` pattern and matches of ``N:`` patterns.  By default,
+  ``scripts/get_maintainer.pl`` will not look at git log history when an ``F:``
+  pattern match occurs.  When an ``N:`` match occurs, git log history
+  is used to also notify the people that have git commit signatures.
+
+- ``X:`` Files and directories that are NOT maintained, same rules as ``F:``.
+
+  Files exclusions are tested before file matches.
+  Can be useful for excluding a specific subdirectory, for instance:
+
+  ============================	========================================
+  ``F:``	``net/``	matches all files in and below
+				``net`` ...
+  ``X:``	``net/ipv6/``	... excluding ``net/ipv6/``
+  ============================	========================================
+
+- ``K:`` Keyword perl extended regex pattern to match content in a
+  patch or file.
+
+  For instance:
+
+  =====================================	=====================================
+  ``K:`` ``of_get_profile``		matches patches or files that contain
+					``of_get_profile``
+  ``K:`` ``\b(printk|pr_(info|err))\b``	matches patches or files that contain
+					one or more of the words ``printk``,
+					``pr_info`` or ``pr_err``
+  =====================================	=====================================
+
+  One regex pattern per line.  Multiple ``K:`` lines acceptable.
+
+.. note::
+
+  For the hard of thinking, this list is meant to remain in alphabetical
+  order. If you could add yourselves to it in alphabetical order that would be
+  so much easier [Ed]
 
 Maintainers List (try to look for most precise areas first)
+-----------------------------------------------------------
 
 		-----------------------------------
 
+
 3C59X NETWORK DRIVER
 M:	Steffen Klassert <klassert@mathematik.tu-chemnitz.de>
 L:	netdev@vger.kernel.org
-- 
2.9.3

[toc] | [prev] | [next] | [standalone]

#1534881 — [PATCH 2/2] MAINTAINERS: add it to the admin-guide

The MAINTAINER's file has different things inside it:
	- Tips for patch submitters;
	- Descriptions for the MAINTAINER file entries;
	- the MAINTAINERS database.

As its contents is useful for someone reporting a bug or
by a newbie submitting a patch, let's add it to the documentation,
keeping just the database at the MAINTAINERS file, and a note
pointing to where the documentation was moved.

Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
---
 Documentation/admin-guide/index.rst       |   1 +
 Documentation/admin-guide/maintainers.rst | 184 ++++++++++++++++++++++++++++++
 MAINTAINERS                               | 182 +----------------------------
 3 files changed, 186 insertions(+), 181 deletions(-)
 create mode 100644 Documentation/admin-guide/maintainers.rst

diff --git a/Documentation/admin-guide/index.rst b/Documentation/admin-guide/index.rst
index 2681cbd24cdd..a2a72b749861 100644
--- a/Documentation/admin-guide/index.rst
+++ b/Documentation/admin-guide/index.rst
@@ -28,6 +28,7 @@ problems and bugs in particular.
    bug-hunting
    bug-bisect
    tainted-kernels
+   maintainers
    ramoops
    dynamic-debug-howto
    init
diff --git a/Documentation/admin-guide/maintainers.rst b/Documentation/admin-guide/maintainers.rst
new file mode 100644
index 000000000000..68daef3df3cf
--- /dev/null
+++ b/Documentation/admin-guide/maintainers.rst
@@ -0,0 +1,184 @@
+List of maintainers and how to submit kernel changes
+====================================================
+
+Please try to follow the guidelines below.  This will make things
+easier on the maintainers.  Not all of these guidelines matter for every
+trivial patch so apply some common sense.
+
+Tips for patch submitters
+-------------------------
+
+1.	Always **test** your changes, however small, on at least 4 or
+	5 people, preferably many more.
+
+2.	Try to release a few ALPHA test versions to the net. Announce
+	them onto the kernel channel and await results. This is especially
+	important for device drivers, because often that's the only way
+	you will find things like the fact version 3 firmware needs
+	a magic fix you didn't know about, or some clown changed the
+	chips on a board and not its name.  (Don't laugh!  Look at the
+	SMC ``etherpower`` for that.)
+
+3.	Make sure your changes compile correctly in multiple
+	configurations. In particular check that changes work both as a
+	module and built into the kernel.
+
+4.	When you are happy with a change make it generally available for
+	testing and await feedback.
+
+5.	Make a patch available to the relevant maintainer in the list. Use
+	``diff -u`` to make the patch easy to merge. Be prepared to get your
+	changes sent back with seemingly silly requests about formatting
+	and variable names.  These aren't as silly as they seem. One
+	job the maintainers (and especially Linus) do is to keep things
+	looking the same. Sometimes this means that the clever hack in
+	your driver to get around a problem actually needs to become a
+	generalized kernel feature ready for next time.
+
+	.. attention::
+
+	  **PLEASE:**
+
+	  - check your patch with the automated style checker
+	    (``scripts/checkpatch.pl``) to catch trivial style violations.
+	    See :ref:`Documentation/process/coding-style.rst <codingstyle>`
+	    for guidance here.
+
+	  - CC: the maintainers and mailing lists that are generated
+	    by ``scripts/get_maintainer.pl``.  The results returned by the
+	    script will be best if you have git installed and are making
+	    your changes in a branch derived from Linus' latest git tree.
+	    See :ref:`Documentation/process/submitting-patches.rst <submittingpatches>`
+	    for details.
+
+	  - try to include any credit lines you want added with the
+	    patch. It avoids people being missed off by mistake and makes
+	    it easier to know who wants adding and who doesn't.
+
+	  - document known bugs. If it doesn't work for everything
+	    or does something very odd once a month document it.
+
+	  - remember that submissions must be made under the terms
+	    of the Linux Foundation certificate of contribution and should
+	    include a Signed-off-by: line.  The current version of this
+	    "Developer's Certificate of Origin" (DCO) is listed in the file
+	    :ref:`Documentation/process/submitting-patches.rst <submittingpatches>`.
+
+6.	Make sure you have the right to send any changes you make. If you
+	do changes at work you may find your employer owns the patch
+	not you.
+
+7.	When sending security related changes or reports to a maintainer
+	please Cc: security@kernel.org, especially if the maintainer
+	does not respond.
+
+8.	Happy hacking.
+
+Descriptions of section entries
+-------------------------------
+
+- ``P:`` Person (obsolete)
+
+- ``M:`` Mail patches to: FullName <address@domain>
+
+- ``R:`` Designated reviewer: FullName <address@domain>
+
+	- These reviewers should be CCed on patches.
+
+- ``L:`` Mailing list that is relevant to this area
+
+- ``W:`` Web-page with status/info
+
+- ``Q:`` Patchwork web based patch tracking system site
+
+- ``T:`` SCM tree type and location.
+
+	- Type is one of: **git**, **hg**, **quilt**, **stgit**, **topgit**
+
+- ``S:`` Status, one of the following:
+
+	   - **Supported**:
+	     Someone is actually paid to look after this.
+
+	   - **Maintained**:
+	     Someone actually looks after it.
+
+	   - **Odd Fixes**:
+	     It has a maintainer but they don't have time to do
+	     much other than throw the odd patch in. See below..
+
+	   - **Orphan**:
+	     No current maintainer [but maybe you could take the
+	     role as you write your new code].
+
+	   - **Obsolete**:
+	     Old code. Something tagged obsolete generally means
+	     it has been replaced by a better system and you
+	     should be using that.
+
+- ``F:`` Files and directories with wildcard patterns.
+
+  A trailing slash includes all files and subdirectory files.
+
+  ===============================	================================
+  ``F:``	``drivers/net/``	all files in and below
+					``drivers/net``
+  ``F:``	``drivers/net/*``	all files in ``drivers/net``,
+					but not below
+  ``F:``	``*/net/*``		all files in "any top level
+					directory" ``/net``
+  ===============================	================================
+
+  One pattern per line.  Multiple ``F:`` lines acceptable.
+
+- ``N:`` Files and directories with regex patterns.
+
+  ============================	=============================================
+  ``N:``			``[^a-z]tegra``	all files whose path contains
+				the word ``tegra``
+  ============================	=============================================
+
+  One pattern per line.  Multiple ``N:`` lines acceptable.
+  ``scripts/get_maintainer.pl`` has different behavior for files that
+  match ``F:`` pattern and matches of ``N:`` patterns.  By default,
+  ``scripts/get_maintainer.pl`` will not look at git log history when an ``F:``
+  pattern match occurs.  When an ``N:`` match occurs, git log history
+  is used to also notify the people that have git commit signatures.
+
+- ``X:`` Files and directories that are NOT maintained, same rules as ``F:``.
+
+  Files exclusions are tested before file matches.
+  Can be useful for excluding a specific subdirectory, for instance:
+
+  ============================	========================================
+  ``F:``	``net/``	matches all files in and below
+				``net`` ...
+  ``X:``	``net/ipv6/``	... excluding ``net/ipv6/``
+  ============================	========================================
+
+- ``K:`` Keyword perl extended regex pattern to match content in a
+  patch or file.
+
+  For instance:
+
+  =====================================	=====================================
+  ``K:`` ``of_get_profile``		matches patches or files that contain
+					``of_get_profile``
+  ``K:`` ``\b(printk|pr_(info|err))\b``	matches patches or files that contain
+					one or more of the words ``printk``,
+					``pr_info`` or ``pr_err``
+  =====================================	=====================================
+
+  One regex pattern per line.  Multiple ``K:`` lines acceptable.
+
+.. note::
+
+  For the hard of thinking, this list is meant to remain in alphabetical
+  order. If you could add yourselves to it in alphabetical order that would be
+  so much easier [Ed]
+
+Maintainer's list
+-----------------
+
+.. include:: ../../MAINTAINERS
+   :literal:
diff --git a/MAINTAINERS b/MAINTAINERS
index 1cb31237a2c0..92bb78b75eb0 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1,187 +1,7 @@
-List of maintainers and how to submit kernel changes
-====================================================
-
-Please try to follow the guidelines below.  This will make things
-easier on the maintainers.  Not all of these guidelines matter for every
-trivial patch so apply some common sense.
-
-Tips for patch submitters
--------------------------
-
-1.	Always **test** your changes, however small, on at least 4 or
-	5 people, preferably many more.
-
-2.	Try to release a few ALPHA test versions to the net. Announce
-	them onto the kernel channel and await results. This is especially
-	important for device drivers, because often that's the only way
-	you will find things like the fact version 3 firmware needs
-	a magic fix you didn't know about, or some clown changed the
-	chips on a board and not its name.  (Don't laugh!  Look at the
-	SMC ``etherpower`` for that.)
-
-3.	Make sure your changes compile correctly in multiple
-	configurations. In particular check that changes work both as a
-	module and built into the kernel.
-
-4.	When you are happy with a change make it generally available for
-	testing and await feedback.
-
-5.	Make a patch available to the relevant maintainer in the list. Use
-	``diff -u`` to make the patch easy to merge. Be prepared to get your
-	changes sent back with seemingly silly requests about formatting
-	and variable names.  These aren't as silly as they seem. One
-	job the maintainers (and especially Linus) do is to keep things
-	looking the same. Sometimes this means that the clever hack in
-	your driver to get around a problem actually needs to become a
-	generalized kernel feature ready for next time.
-
-	.. attention::
-
-	  **PLEASE:**
-
-	  - check your patch with the automated style checker
-	    (``scripts/checkpatch.pl``) to catch trivial style violations.
-	    See :ref:`Documentation/process/coding-style.rst <codingstyle>`
-	    for guidance here.
-
-	  - CC: the maintainers and mailing lists that are generated
-	    by ``scripts/get_maintainer.pl``.  The results returned by the
-	    script will be best if you have git installed and are making
-	    your changes in a branch derived from Linus' latest git tree.
-	    See :ref:`Documentation/process/submitting-patches.rst <submittingpatches>`
-	    for details.
-
-	  - try to include any credit lines you want added with the
-	    patch. It avoids people being missed off by mistake and makes
-	    it easier to know who wants adding and who doesn't.
-
-	  - document known bugs. If it doesn't work for everything
-	    or does something very odd once a month document it.
-
-	  - remember that submissions must be made under the terms
-	    of the Linux Foundation certificate of contribution and should
-	    include a Signed-off-by: line.  The current version of this
-	    "Developer's Certificate of Origin" (DCO) is listed in the file
-	    :ref:`Documentation/process/submitting-patches.rst <submittingpatches>`.
-
-6.	Make sure you have the right to send any changes you make. If you
-	do changes at work you may find your employer owns the patch
-	not you.
-
-7.	When sending security related changes or reports to a maintainer
-	please Cc: security@kernel.org, especially if the maintainer
-	does not respond.
-
-8.	Happy hacking.
-
-Descriptions of section entries
--------------------------------
-
-- ``P:`` Person (obsolete)
-
-- ``M:`` Mail patches to: FullName <address@domain>
-
-- ``R:`` Designated reviewer: FullName <address@domain>
-
-	- These reviewers should be CCed on patches.
-
-- ``L:`` Mailing list that is relevant to this area
-
-- ``W:`` Web-page with status/info
-
-- ``Q:`` Patchwork web based patch tracking system site
-
-- ``T:`` SCM tree type and location.
-
-	- Type is one of: **git**, **hg**, **quilt**, **stgit**, **topgit**
-
-- ``S:`` Status, one of the following:
-
-	   - **Supported**:
-	     Someone is actually paid to look after this.
-
-	   - **Maintained**:
-	     Someone actually looks after it.
-
-	   - **Odd Fixes**:
-	     It has a maintainer but they don't have time to do
-	     much other than throw the odd patch in. See below..
-
-	   - **Orphan**:
-	     No current maintainer [but maybe you could take the
-	     role as you write your new code].
-
-	   - **Obsolete**:
-	     Old code. Something tagged obsolete generally means
-	     it has been replaced by a better system and you
-	     should be using that.
-
-- ``F:`` Files and directories with wildcard patterns.
-
-  A trailing slash includes all files and subdirectory files.
-
-  ===============================	================================
-  ``F:``	``drivers/net/``	all files in and below
-					``drivers/net``
-  ``F:``	``drivers/net/*``	all files in ``drivers/net``,
-					but not below
-  ``F:``	``*/net/*``		all files in "any top level
-					directory" ``/net``
-  ===============================	================================
-
-  One pattern per line.  Multiple ``F:`` lines acceptable.
-
-- ``N:`` Files and directories with regex patterns.
-
-  ============================	=============================================
-  ``N:``			``[^a-z]tegra``	all files whose path contains
-				the word ``tegra``
-  ============================	=============================================
-
-  One pattern per line.  Multiple ``N:`` lines acceptable.
-  ``scripts/get_maintainer.pl`` has different behavior for files that
-  match ``F:`` pattern and matches of ``N:`` patterns.  By default,
-  ``scripts/get_maintainer.pl`` will not look at git log history when an ``F:``
-  pattern match occurs.  When an ``N:`` match occurs, git log history
-  is used to also notify the people that have git commit signatures.
-
-- ``X:`` Files and directories that are NOT maintained, same rules as ``F:``.
-
-  Files exclusions are tested before file matches.
-  Can be useful for excluding a specific subdirectory, for instance:
-
-  ============================	========================================
-  ``F:``	``net/``	matches all files in and below
-				``net`` ...
-  ``X:``	``net/ipv6/``	... excluding ``net/ipv6/``
-  ============================	========================================
-
-- ``K:`` Keyword perl extended regex pattern to match content in a
-  patch or file.
-
-  For instance:
-
-  =====================================	=====================================
-  ``K:`` ``of_get_profile``		matches patches or files that contain
-					``of_get_profile``
-  ``K:`` ``\b(printk|pr_(info|err))\b``	matches patches or files that contain
-					one or more of the words ``printk``,
-					``pr_info`` or ``pr_err``
-  =====================================	=====================================
-
-  One regex pattern per line.  Multiple ``K:`` lines acceptable.
-
-.. note::
-
-  For the hard of thinking, this list is meant to remain in alphabetical
-  order. If you could add yourselves to it in alphabetical order that would be
-  so much easier [Ed]
-
 Maintainers List (try to look for most precise areas first)
 -----------------------------------------------------------
 
-		-----------------------------------
-
+Please read Documentation/admin-guide/maintainers.rst for instructions.
 
 3C59X NETWORK DRIVER
 M:	Steffen Klassert <klassert@mathematik.tu-chemnitz.de>
-- 
2.9.3

[toc] | [prev] | [next] | [standalone]

#1537859

On Fri, Dec 02, 2016 at 10:15:13AM -0200, Mauro Carvalho Chehab wrote:
> That's my third attempt to add the MAINTAINERS contents to the admin-guide.
> 
> On the past approaches, was planning to keep the documentation
> about what's at the MAINTAINERS file inside it, but that would
> require running an external script or use some Sphinx extension.
> 
> This time, I took a much simpler approach: convert the initial
> part of the MAINTAINERS file to ReST and move to a file at the
> admin-guide. So, MAINTAINERS file will now contain only the
> maintainer's database, and a single line pointing to its documentation.
> 
> That should hopefully be a good compromise, as we can add its
> contents to a Sphinx file without causing too much hasle.

Yeah, this seems like a rather neat approach and hopefully keeps everyone
happy. Will probably conflict since there's some in-flight patches to
document the new B: tag a bit better. Anyway:

Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>

> 
> 
> Mauro Carvalho Chehab (2):
>   MAINTAINERS: convert first part to ReST markup
>   MAINTAINERS: add it to the admin-guide
> 
>  Documentation/admin-guide/index.rst       |   1 +
>  Documentation/admin-guide/maintainers.rst | 184 ++++++++++++++++++++++++++++++
>  MAINTAINERS                               | 125 +-------------------
>  3 files changed, 187 insertions(+), 123 deletions(-)
>  create mode 100644 Documentation/admin-guide/maintainers.rst
> 
> -- 
> 2.9.3
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-doc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web