Date: Wed, 26 Oct 2016 22:24:23 -0200
From: Mauro Carvalho Chehab <mchehab@s-opensource.com>
To: Jonathan Corbet <corbet@lwn.net>
Cc: linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
        Jani Nikula <jani.nikula@intel.com>
Subject: Re: [PATCH 07/11] docs: Apply some basic organization to the
 process guide
Message-ID: <20161026222423.40ce921f@vento.lan>
In-Reply-To: <1477523979-5837-8-git-send-email-corbet@lwn.net>
References: <1477523979-5837-1-git-send-email-corbet@lwn.net>
        <1477523979-5837-8-git-send-email-corbet@lwn.net>
Organization: Samsung
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2603
Lines: 96

Em Wed, 26 Oct 2016 17:19:35 -0600
Jonathan Corbet <corbet@lwn.net> escreveu:

> Put like documents together, with the essential ones at the top, and split
> the TOC into sections.
> 
> Signed-off-by: Jonathan Corbet <corbet@lwn.net>

Looks good to me.

Reviewed-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>

> ---
>  Documentation/process/changes.rst |  2 +-
>  Documentation/process/index.rst   | 37 ++++++++++++++++++++++++++++---------
>  2 files changed, 29 insertions(+), 10 deletions(-)
> 
> diff --git a/Documentation/process/changes.rst b/Documentation/process/changes.rst
> index 22797a15dc24..56ce66114665 100644
> --- a/Documentation/process/changes.rst
> +++ b/Documentation/process/changes.rst
> @@ -1,6 +1,6 @@
>  .. _changes:
>  
> -Minimal requerements to compile the Kernel
> +Minimal requirements to compile the Kernel
>  ++++++++++++++++++++++++++++++++++++++++++
>  
>  Intro
> diff --git a/Documentation/process/index.rst b/Documentation/process/index.rst
> index 6ee818752474..0cbc62adbbb1 100644
> --- a/Documentation/process/index.rst
> +++ b/Documentation/process/index.rst
> @@ -4,29 +4,48 @@
>  	\renewcommand\thesubsection*
>  
>  
> -Linux Kernel Development Documentation
> -======================================
> +Working with the kernel development community
> +=============================================
>  
> -Contents:
> +So you want to be a Linux kernel developer?  Welcome!  While there is a lot
> +to be learned about the kernel in a technical sense, it is also important
> +to learn about how our community works.  Reading these documents will make
> +it much easier for you to get your changes merged with a minimum of
> +trouble.
> +
> +Below are the essential guides that every developer should read.
>  
>  .. toctree::
>     :maxdepth: 2
>  
>     howto
> -   changes
> -   coding-style
> +   code-of-conflict
> +   development-process
>     submitting-patches
> +   coding-style
> +   email-clients
> +
> +Other guides to the community that are of interest to most developers are: 
> +
> +.. toctree::
> +   :maxdepth: 2
> +
> +   changes
>     submitting-drivers
>     stable-api-nonsense
>     management-style
>     stable-kernel-rules
> +   submit-checklist
>     kernel-docs
> +
> +These are some overall technical guides that have been put here for now for
> +lack of a better place.
> +
> +.. toctree::
> +   :maxdepth: 2
> +
>     applying-patches
> -   email-clients
> -   submit-checklist
> -   code-of-conflict
>     adding-syscalls
>     magic-number
>     volatile-considered-harmful
>  
> -   development-process


Thanks,
Mauro