diff mbox series

[v2,11/29] docs: devicetree: bindings: submitting-patches.rst: avoid using ReST :doc:`foo` markup

Message ID 0048c23d47b582dd1a1959628fd2b895209ac826.1623824363.git.mchehab+huawei@kernel.org
State Accepted
Commit 6dce82b28a93492af7a817b2b3166aaf775e4aba
Headers show
Series None | expand

Commit Message

Mauro Carvalho Chehab June 16, 2021, 6:27 a.m. UTC
The :doc:`foo` tag is auto-generated via automarkup.py.
So, use the filename at the sources, instead of :doc:`foo`.

Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
---
 .../devicetree/bindings/submitting-patches.rst        | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

Comments

Rob Herring June 16, 2021, 10:08 p.m. UTC | #1
On Wed, Jun 16, 2021 at 12:27 AM Mauro Carvalho Chehab
<mchehab+huawei@kernel.org> wrote:
>
> The :doc:`foo` tag is auto-generated via automarkup.py.
> So, use the filename at the sources, instead of :doc:`foo`.
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
> ---
>  .../devicetree/bindings/submitting-patches.rst        | 11 ++++++-----
>  1 file changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/submitting-patches.rst b/Documentation/devicetree/bindings/submitting-patches.rst
> index 104fa8fb2c17..8087780f1685 100644
> --- a/Documentation/devicetree/bindings/submitting-patches.rst
> +++ b/Documentation/devicetree/bindings/submitting-patches.rst
> @@ -7,8 +7,8 @@ Submitting Devicetree (DT) binding patches
>  I. For patch submitters
>  =======================
>
> -  0) Normal patch submission rules from Documentation/process/submitting-patches.rst
> -     applies.
> +  0) Normal patch submission rules from
> +     Documentation/process/submitting-patches.rst applies.
>
>    1) The Documentation/ and include/dt-bindings/ portion of the patch should
>       be a separate patch. The preferred subject prefix for binding patches is::
> @@ -25,8 +25,8 @@ I. For patch submitters
>
>         make dt_binding_check
>
> -     See Documentation/devicetree/bindings/writing-schema.rst for more details about
> -     schema and tools setup.
> +     See Documentation/devicetree/bindings/writing-schema.rst for more details
> +     about schema and tools setup.

These don't match $subject.

>
>    3) DT binding files should be dual licensed. The preferred license tag is
>       (GPL-2.0-only OR BSD-2-Clause).
> @@ -84,7 +84,8 @@ II. For kernel maintainers
>  III. Notes
>  ==========
>
> -  0) Please see :doc:`ABI` for details regarding devicetree ABI.
> +  0) Please see Documentation/devicetree/bindings/ABI.rst for details
> +     regarding devicetree ABI.

Is this new? You did review my oneliner adding ':doc:' not too long ago.

>
>    1) This document is intended as a general familiarization with the process as
>       decided at the 2013 Kernel Summit.  When in doubt, the current word of the
> --
> 2.31.1
>
Mauro Carvalho Chehab June 17, 2021, 6:38 a.m. UTC | #2
Em Wed, 16 Jun 2021 16:08:05 -0600
Rob Herring <robh+dt@kernel.org> escreveu:

> On Wed, Jun 16, 2021 at 12:27 AM Mauro Carvalho Chehab

> <mchehab+huawei@kernel.org> wrote:

> >

> > The :doc:`foo` tag is auto-generated via automarkup.py.

> > So, use the filename at the sources, instead of :doc:`foo`.

> >

> > Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>

> > ---

> >  .../devicetree/bindings/submitting-patches.rst        | 11 ++++++-----

> >  1 file changed, 6 insertions(+), 5 deletions(-)

> >

> > diff --git a/Documentation/devicetree/bindings/submitting-patches.rst b/Documentation/devicetree/bindings/submitting-patches.rst

> > index 104fa8fb2c17..8087780f1685 100644

> > --- a/Documentation/devicetree/bindings/submitting-patches.rst

> > +++ b/Documentation/devicetree/bindings/submitting-patches.rst

> > @@ -7,8 +7,8 @@ Submitting Devicetree (DT) binding patches

> >  I. For patch submitters

> >  =======================

> >

> > -  0) Normal patch submission rules from Documentation/process/submitting-patches.rst

> > -     applies.

> > +  0) Normal patch submission rules from

> > +     Documentation/process/submitting-patches.rst applies.

> >

> >    1) The Documentation/ and include/dt-bindings/ portion of the patch should

> >       be a separate patch. The preferred subject prefix for binding patches is::

> > @@ -25,8 +25,8 @@ I. For patch submitters

> >

> >         make dt_binding_check

> >

> > -     See Documentation/devicetree/bindings/writing-schema.rst for more details about

> > -     schema and tools setup.

> > +     See Documentation/devicetree/bindings/writing-schema.rst for more details

> > +     about schema and tools setup.  

> 

> These don't match $subject.


Well, I just seek for all places where Documentation/* are, and adjusted
them to 80-columns. I'll add a comment. 

> 

> >

> >    3) DT binding files should be dual licensed. The preferred license tag is

> >       (GPL-2.0-only OR BSD-2-Clause).

> > @@ -84,7 +84,8 @@ II. For kernel maintainers

> >  III. Notes

> >  ==========

> >

> > -  0) Please see :doc:`ABI` for details regarding devicetree ABI.

> > +  0) Please see Documentation/devicetree/bindings/ABI.rst for details

> > +     regarding devicetree ABI.  

> 

> Is this new? You did review my oneliner adding ':doc:' not too long ago.


Yes. It is due to some discussion we had upstream:

	https://lore.kernel.org/linux-doc/871r9k6rmy.fsf@meer.lwn.net/

While the html/latex/pdf output is identical (as automarkup.py will
convert Documentation/foo.rst into a cross-reference), using full path
names allow developers to jump directly to other documents when they
have the path name.

> 

> >

> >    1) This document is intended as a general familiarization with the process as

> >       decided at the 2013 Kernel Summit.  When in doubt, the current word of the

> > --

> > 2.31.1

> >  




Thanks,
Mauro
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/submitting-patches.rst b/Documentation/devicetree/bindings/submitting-patches.rst
index 104fa8fb2c17..8087780f1685 100644
--- a/Documentation/devicetree/bindings/submitting-patches.rst
+++ b/Documentation/devicetree/bindings/submitting-patches.rst
@@ -7,8 +7,8 @@  Submitting Devicetree (DT) binding patches
 I. For patch submitters
 =======================
 
-  0) Normal patch submission rules from Documentation/process/submitting-patches.rst
-     applies.
+  0) Normal patch submission rules from
+     Documentation/process/submitting-patches.rst applies.
 
   1) The Documentation/ and include/dt-bindings/ portion of the patch should
      be a separate patch. The preferred subject prefix for binding patches is::
@@ -25,8 +25,8 @@  I. For patch submitters
 
        make dt_binding_check
 
-     See Documentation/devicetree/bindings/writing-schema.rst for more details about
-     schema and tools setup.
+     See Documentation/devicetree/bindings/writing-schema.rst for more details
+     about schema and tools setup.
 
   3) DT binding files should be dual licensed. The preferred license tag is
      (GPL-2.0-only OR BSD-2-Clause).
@@ -84,7 +84,8 @@  II. For kernel maintainers
 III. Notes
 ==========
 
-  0) Please see :doc:`ABI` for details regarding devicetree ABI.
+  0) Please see Documentation/devicetree/bindings/ABI.rst for details
+     regarding devicetree ABI.
 
   1) This document is intended as a general familiarization with the process as
      decided at the 2013 Kernel Summit.  When in doubt, the current word of the