Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Tue, 2 Apr 2019 23:08:54 +0800
From:   Changbin Du <changbin.du@gmail.com>
To:     Bjorn Helgaas <helgaas@kernel.org>
Cc:     Changbin Du <changbin.du@gmail.com>,
        Jonathan Corbet <corbet@lwn.net>, linux-pci@vger.kernel.org,
        linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 00/12] Include linux PCI docs into Sphinx TOC tree
Message-ID: <20190402150852.juugbq7gripqbrmz@mail.google.com>
References: <20190329160413.4293-1-changbin.du@gmail.com>
 <20190401230321.GB141706@google.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20190401230321.GB141706@google.com>
User-Agent: NeoMutt/20180716
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Mon, Apr 01, 2019 at 06:03:21PM -0500, Bjorn Helgaas wrote:
> On Sat, Mar 30, 2019 at 12:04:01AM +0800, Changbin Du wrote:
> > Hi Corbet,
> 
> Conventional address would be "Hi Jonathan", just FYI.
>
I know this convention. I said 'Hi Corbet' because I saw the name of email
address. Sorry if this is not the case. :)

> > I also did the conversion for PCI documentation. Please check, Thanks!
> > 
> > The kernel now uses Sphinx to generate intelligent and beautiful documentation
> > from reStructuredText files. I converted most of the Linux PCI docs to rst
> > format in this serias.
> > 
> > For you to preview, please visit below url:
> > http://104.238.181.70:8080/kernel-doc/PCI/index.html
> > 
> > Thank you!
> > 
> > Changbin Du (12):
> >   Documentation: add Linux PCI to Sphinx TOC tree
> >   pci doc: convert PCI/pci.txt to rst format
> >   pci doc: convert PCI/PCIEBUS-HOWTO.txt to rst format
> >   pci doc: convert PCI/pci-iov-howto.txt to rst format
> >   pci doc: convert PCI/MSI-HOWTO.txt to rst format
> >   pci doc: convert PCI/acpi-info.txt to rst format
> >   pci doc: convert PCI/pci-error-recovery.txt to rst format
> >   pci doc: convert PCI/pcieaer-howto.txt to rst format
> >   pci doc: convert PCI/endpoint/pci-endpoint.txt to rst format
> >   pci doc: convert PCI/endpoint/pci-endpoint-cfs.txt to rst format
> >   pci doc: convert PCI/endpoint/pci-test-function.txt to rst format
> >   pci doc: convert PCI/endpoint/pci-test-howto.txt to rst format
> 
> Thanks for doing all this work!
> 
> Acked-by: Bjorn Helgaas <bhelgaas@google.com>
> 
> There's not a real convention yet for subject lines in Documentation/,
> but I would propose:
> 
>   Documentation: PCI: Convert pci.txt to reST
>   Documentation: PCI: Convert endpoint/pci-endpoint.txt to reST
> 
> because that at least goes with some recent Documentation/PCI changes
> and it identifies "reST" as something other than a plain English word
> (RST or ReST would also work for me).
> 
I agree. I am just a little lazy to update all of them this time :)
Thanks for your ACK!

> >  .../PCI/{MSI-HOWTO.txt => MSI-HOWTO.rst}      |  56 +++--
> >  .../{PCIEBUS-HOWTO.txt => PCIEBUS-HOWTO.rst}  | 112 +++++----
> >  .../PCI/{acpi-info.txt => acpi-info.rst}      |  11 +-
> >  Documentation/PCI/endpoint/index.rst          |  13 ++
> >  ...-endpoint-cfs.txt => pci-endpoint-cfs.rst} |  99 ++++----
> >  .../{pci-endpoint.txt => pci-endpoint.rst}    |  74 +++---
> >  ...est-function.txt => pci-test-function.rst} |  32 +--
> >  ...{pci-test-howto.txt => pci-test-howto.rst} |  51 ++--
> >  Documentation/PCI/index.rst                   |  17 ++
> >  ...or-recovery.txt => pci-error-recovery.rst} | 180 ++++++++-------
> >  .../{pci-iov-howto.txt => pci-iov-howto.rst}  | 144 +++++++-----
> >  Documentation/PCI/{pci.txt => pci.rst}        | 218 +++++++++---------
> >  .../{pcieaer-howto.txt => pcieaer-howto.rst}  |  67 ++++--
> >  Documentation/index.rst                       |   1 +
> >  MAINTAINERS                                   |   2 +-
> >  15 files changed, 641 insertions(+), 436 deletions(-)
> >  rename Documentation/PCI/{MSI-HOWTO.txt => MSI-HOWTO.rst} (89%)
> >  rename Documentation/PCI/{PCIEBUS-HOWTO.txt => PCIEBUS-HOWTO.rst} (75%)
> >  rename Documentation/PCI/{acpi-info.txt => acpi-info.rst} (97%)
> >  create mode 100644 Documentation/PCI/endpoint/index.rst
> >  rename Documentation/PCI/endpoint/{pci-endpoint-cfs.txt => pci-endpoint-cfs.rst} (64%)
> >  rename Documentation/PCI/endpoint/{pci-endpoint.txt => pci-endpoint.rst} (86%)
> >  rename Documentation/PCI/endpoint/{pci-test-function.txt => pci-test-function.rst} (84%)
> >  rename Documentation/PCI/endpoint/{pci-test-howto.txt => pci-test-howto.rst} (83%)
> >  create mode 100644 Documentation/PCI/index.rst
> >  rename Documentation/PCI/{pci-error-recovery.txt => pci-error-recovery.rst} (80%)
> >  rename Documentation/PCI/{pci-iov-howto.txt => pci-iov-howto.rst} (65%)
> >  rename Documentation/PCI/{pci.txt => pci.rst} (80%)
> >  rename Documentation/PCI/{pcieaer-howto.txt => pcieaer-howto.rst} (83%)
> > 
> > -- 
> > 2.20.1
> > 

-- 
Cheers,
Changbin Du