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:   Mon, 20 May 2019 23:05:39 +0800
From:   Changbin Du <changbin.du@gmail.com>
To:     Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Cc:     Changbin Du <changbin.du@gmail.com>, bhelgaas@google.com,
        corbet@lwn.net, linux-pci@vger.kernel.org,
        linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v6 00/12] Include linux PCI docs into Sphinx TOC tree
Message-ID: <20190520150538.hq4jnqeugpucjtwo@mail.google.com>
References: <20190514144734.19760-1-changbin.du@gmail.com>
 <20190520061014.qtq6tc366pnnqcio@mail.google.com>
 <20190520112350.4679df1c@coco.lan>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20190520112350.4679df1c@coco.lan>
User-Agent: NeoMutt/20180716
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Mon, May 20, 2019 at 11:23:50AM -0300, Mauro Carvalho Chehab wrote:
> Em Mon, 20 May 2019 06:10:15 +0000
> Changbin Du <changbin.du@gmail.com> escreveu:
> 
> > Bjorn and Jonathan,
> > Could we consider to merge this serias now? Thanks.
> 
> Before merging it, did you check if the renames won't cause broken
> reference links? There were such breakages with your x86 and acpi
> patch series. I'm sending the fixes right now, but it would be
> great if you could run the ./scripts/documentation-file-ref-check
> script and address any file name change this series would be
> introducing. There's even a --fix option there that allows
> to automatically fix them (you need to double-check the results).
>
I just ran documentation-file-ref-check, and *no* broken reference found in PCI
documentation. This tool is great!

> Regards,
> Mauro
> 
> > 
> > On Tue, May 14, 2019 at 10:47:22PM +0800, Changbin Du wrote:
> > > Hi all,
> > > 
> > > 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://www.bytemem.com:8080/kernel-doc/PCI/index.html
> > > 
> > > Thank you!
> > > 
> > > v2: trivial style update.
> > > v3: update titles. (Bjorn Helgaas)
> > > v4: fix comments from Mauro Carvalho Chehab
> > > v5: update MAINTAINERS (Joe Perches)
> > > v6: fix comments.
> > > 
> > > Changbin Du (12):
> > >   Documentation: add Linux PCI to Sphinx TOC tree
> > >   Documentation: PCI: convert pci.txt to reST
> > >   Documentation: PCI: convert PCIEBUS-HOWTO.txt to reST
> > >   Documentation: PCI: convert pci-iov-howto.txt to reST
> > >   Documentation: PCI: convert MSI-HOWTO.txt to reST
> > >   Documentation: PCI: convert acpi-info.txt to reST
> > >   Documentation: PCI: convert pci-error-recovery.txt to reST
> > >   Documentation: PCI: convert pcieaer-howto.txt to reST
> > >   Documentation: PCI: convert endpoint/pci-endpoint.txt to reST
> > >   Documentation: PCI: convert endpoint/pci-endpoint-cfs.txt to reST
> > >   Documentation: PCI: convert endpoint/pci-test-function.txt to reST
> > >   Documentation: PCI: convert endpoint/pci-test-howto.txt to reST
> > > 
> > >  .../PCI/{acpi-info.txt => acpi-info.rst}      |  15 +-
> > >  Documentation/PCI/endpoint/index.rst          |  13 +
> > >  ...-endpoint-cfs.txt => pci-endpoint-cfs.rst} |  99 ++---
> > >  .../{pci-endpoint.txt => pci-endpoint.rst}    |  92 +++--
> > >  ...est-function.txt => pci-test-function.rst} |  84 +++--
> > >  ...{pci-test-howto.txt => pci-test-howto.rst} |  81 ++--
> > >  Documentation/PCI/index.rst                   |  18 +
> > >  .../PCI/{MSI-HOWTO.txt => msi-howto.rst}      |  85 +++--
> > >  ...or-recovery.txt => pci-error-recovery.rst} | 287 +++++++-------
> > >  .../{pci-iov-howto.txt => pci-iov-howto.rst}  | 161 ++++----
> > >  Documentation/PCI/{pci.txt => pci.rst}        | 356 ++++++++----------
> > >  .../{pcieaer-howto.txt => pcieaer-howto.rst}  | 156 +++++---
> > >  .../{PCIEBUS-HOWTO.txt => picebus-howto.rst}  | 140 ++++---
> > >  Documentation/index.rst                       |   1 +
> > >  MAINTAINERS                                   |   4 +-
> > >  include/linux/mod_devicetable.h               |  19 +
> > >  include/linux/pci.h                           |  37 ++
> > >  17 files changed, 938 insertions(+), 710 deletions(-)
> > >  rename Documentation/PCI/{acpi-info.txt => acpi-info.rst} (96%)
> > >  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} (83%)
> > >  rename Documentation/PCI/endpoint/{pci-test-function.txt => pci-test-function.rst} (55%)
> > >  rename Documentation/PCI/endpoint/{pci-test-howto.txt => pci-test-howto.rst} (78%)
> > >  create mode 100644 Documentation/PCI/index.rst
> > >  rename Documentation/PCI/{MSI-HOWTO.txt => msi-howto.rst} (88%)
> > >  rename Documentation/PCI/{pci-error-recovery.txt => pci-error-recovery.rst} (67%)
> > >  rename Documentation/PCI/{pci-iov-howto.txt => pci-iov-howto.rst} (63%)
> > >  rename Documentation/PCI/{pci.txt => pci.rst} (68%)
> > >  rename Documentation/PCI/{pcieaer-howto.txt => pcieaer-howto.rst} (72%)
> > >  rename Documentation/PCI/{PCIEBUS-HOWTO.txt => picebus-howto.rst} (70%)
> > > 
> > > -- 
> > > 2.20.1
> > >   
> > 
> 
> 
> 
> Thanks,
> Mauro

-- 
Cheers,
Changbin Du