* [PATCH -next] fpga: dfl-pci: rectify ReST formatting
@ 2021-01-11 11:21 Lukas Bulwahn
2021-01-11 15:11 ` Tom Rix
` (2 more replies)
0 siblings, 3 replies; 12+ messages in thread
From: Lukas Bulwahn @ 2021-01-11 11:21 UTC (permalink / raw)
To: Greg Kroah-Hartman, Wu Hao, Moritz Fischer, Matthew Gerlach, linux-fpga
Cc: Tom Rix, linux-doc, kernel-janitors, linux-kernel, Lukas Bulwahn
Commit fa41d10589be ("fpga: dfl-pci: locate DFLs by PCIe vendor specific
capability") provides documentation to the FPGA Device Feature List (DFL)
Framework Overview, but introduced new documentation warnings:
./Documentation/fpga/dfl.rst:
505: WARNING: Title underline too short.
523: WARNING: Unexpected indentation.
523: WARNING: Blank line required after table.
524: WARNING: Block quote ends without a blank line; unexpected unindent.
Rectify ReST formatting in ./Documentation/fpga/dfl.rst.
Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
---
applies cleanly on next-20210111
Moritz, Matthew, please ack.
Greg, please pick this doc fixup to your fpga -next tree on top of
the commit above.
Documentation/fpga/dfl.rst | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/Documentation/fpga/dfl.rst b/Documentation/fpga/dfl.rst
index ea8cefc18bdb..c41ac76ffaae 100644
--- a/Documentation/fpga/dfl.rst
+++ b/Documentation/fpga/dfl.rst
@@ -502,7 +502,7 @@ FME Partial Reconfiguration Sub Feature driver (see drivers/fpga/dfl-fme-pr.c)
could be a reference.
Location of DFLs on a PCI Device
-===========================
+================================
The original method for finding a DFL on a PCI device assumed the start of the
first DFL to offset 0 of bar 0. If the first node of the DFL is an FME,
then further DFLs in the port(s) are specified in FME header registers.
@@ -514,6 +514,7 @@ data begins with a 4 byte vendor specific register for the number of DFLs follow
Offset/BIR vendor specific registers for each DFL. Bits 2:0 of Offset/BIR register
indicates the BAR, and bits 31:3 form the 8 byte aligned offset where bits 2:0 are
zero.
+::
+----------------------------+
|31 Number of DFLS 0|
--
2.17.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting
2021-01-11 11:21 [PATCH -next] fpga: dfl-pci: rectify ReST formatting Lukas Bulwahn
@ 2021-01-11 15:11 ` Tom Rix
2021-01-11 15:53 ` Greg Kroah-Hartman
2021-01-11 15:53 ` Greg Kroah-Hartman
2021-01-11 15:55 ` Moritz Fischer
2 siblings, 1 reply; 12+ messages in thread
From: Tom Rix @ 2021-01-11 15:11 UTC (permalink / raw)
To: Lukas Bulwahn, Greg Kroah-Hartman, Wu Hao, Moritz Fischer,
Matthew Gerlach, linux-fpga
Cc: linux-doc, kernel-janitors, linux-kernel
On 1/11/21 3:21 AM, Lukas Bulwahn wrote:
> Commit fa41d10589be ("fpga: dfl-pci: locate DFLs by PCIe vendor specific
> capability") provides documentation to the FPGA Device Feature List (DFL)
> Framework Overview, but introduced new documentation warnings:
>
> ./Documentation/fpga/dfl.rst:
> 505: WARNING: Title underline too short.
> 523: WARNING: Unexpected indentation.
> 523: WARNING: Blank line required after table.
> 524: WARNING: Block quote ends without a blank line; unexpected unindent.
>
> Rectify ReST formatting in ./Documentation/fpga/dfl.rst.
Can you explain how to reproduce this problem ?
Tom
>
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> ---
> applies cleanly on next-20210111
>
> Moritz, Matthew, please ack.
>
> Greg, please pick this doc fixup to your fpga -next tree on top of
> the commit above.
>
> Documentation/fpga/dfl.rst | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/fpga/dfl.rst b/Documentation/fpga/dfl.rst
> index ea8cefc18bdb..c41ac76ffaae 100644
> --- a/Documentation/fpga/dfl.rst
> +++ b/Documentation/fpga/dfl.rst
> @@ -502,7 +502,7 @@ FME Partial Reconfiguration Sub Feature driver (see drivers/fpga/dfl-fme-pr.c)
> could be a reference.
>
> Location of DFLs on a PCI Device
> -===========================
> +================================
> The original method for finding a DFL on a PCI device assumed the start of the
> first DFL to offset 0 of bar 0. If the first node of the DFL is an FME,
> then further DFLs in the port(s) are specified in FME header registers.
> @@ -514,6 +514,7 @@ data begins with a 4 byte vendor specific register for the number of DFLs follow
> Offset/BIR vendor specific registers for each DFL. Bits 2:0 of Offset/BIR register
> indicates the BAR, and bits 31:3 form the 8 byte aligned offset where bits 2:0 are
> zero.
> +::
>
> +----------------------------+
> |31 Number of DFLS 0|
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting
2021-01-11 15:11 ` Tom Rix
@ 2021-01-11 15:53 ` Greg Kroah-Hartman
2021-01-11 16:07 ` Tom Rix
0 siblings, 1 reply; 12+ messages in thread
From: Greg Kroah-Hartman @ 2021-01-11 15:53 UTC (permalink / raw)
To: Tom Rix
Cc: Lukas Bulwahn, Wu Hao, Moritz Fischer, Matthew Gerlach,
linux-fpga, linux-doc, kernel-janitors, linux-kernel
On Mon, Jan 11, 2021 at 07:11:59AM -0800, Tom Rix wrote:
>
> On 1/11/21 3:21 AM, Lukas Bulwahn wrote:
> > Commit fa41d10589be ("fpga: dfl-pci: locate DFLs by PCIe vendor specific
> > capability") provides documentation to the FPGA Device Feature List (DFL)
> > Framework Overview, but introduced new documentation warnings:
> >
> > ./Documentation/fpga/dfl.rst:
> > 505: WARNING: Title underline too short.
> > 523: WARNING: Unexpected indentation.
> > 523: WARNING: Blank line required after table.
> > 524: WARNING: Block quote ends without a blank line; unexpected unindent.
> >
> > Rectify ReST formatting in ./Documentation/fpga/dfl.rst.
>
> Can you explain how to reproduce this problem ?
'make htmldocs'
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting
2021-01-11 15:53 ` Greg Kroah-Hartman
@ 2021-01-11 16:07 ` Tom Rix
0 siblings, 0 replies; 12+ messages in thread
From: Tom Rix @ 2021-01-11 16:07 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Lukas Bulwahn, Wu Hao, Moritz Fischer, Matthew Gerlach,
linux-fpga, linux-doc, kernel-janitors, linux-kernel
On 1/11/21 7:53 AM, Greg Kroah-Hartman wrote:
> On Mon, Jan 11, 2021 at 07:11:59AM -0800, Tom Rix wrote:
>> On 1/11/21 3:21 AM, Lukas Bulwahn wrote:
>>> Commit fa41d10589be ("fpga: dfl-pci: locate DFLs by PCIe vendor specific
>>> capability") provides documentation to the FPGA Device Feature List (DFL)
>>> Framework Overview, but introduced new documentation warnings:
>>>
>>> ./Documentation/fpga/dfl.rst:
>>> 505: WARNING: Title underline too short.
>>> 523: WARNING: Unexpected indentation.
>>> 523: WARNING: Blank line required after table.
>>> 524: WARNING: Block quote ends without a blank line; unexpected unindent.
>>>
>>> Rectify ReST formatting in ./Documentation/fpga/dfl.rst.
>> Can you explain how to reproduce this problem ?
> 'make htmldocs'
Thanks, I have verified the problem and its resolution.
Tested-by: Tom Rix <trix@redhat.com>
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting
2021-01-11 11:21 [PATCH -next] fpga: dfl-pci: rectify ReST formatting Lukas Bulwahn
2021-01-11 15:11 ` Tom Rix
@ 2021-01-11 15:53 ` Greg Kroah-Hartman
2021-01-11 16:34 ` Lukas Bulwahn
2021-01-11 15:55 ` Moritz Fischer
2 siblings, 1 reply; 12+ messages in thread
From: Greg Kroah-Hartman @ 2021-01-11 15:53 UTC (permalink / raw)
To: Lukas Bulwahn
Cc: Wu Hao, Moritz Fischer, Matthew Gerlach, linux-fpga, Tom Rix,
linux-doc, kernel-janitors, linux-kernel
On Mon, Jan 11, 2021 at 12:21:13PM +0100, Lukas Bulwahn wrote:
> Commit fa41d10589be ("fpga: dfl-pci: locate DFLs by PCIe vendor specific
> capability") provides documentation to the FPGA Device Feature List (DFL)
> Framework Overview, but introduced new documentation warnings:
>
> ./Documentation/fpga/dfl.rst:
> 505: WARNING: Title underline too short.
> 523: WARNING: Unexpected indentation.
> 523: WARNING: Blank line required after table.
> 524: WARNING: Block quote ends without a blank line; unexpected unindent.
>
> Rectify ReST formatting in ./Documentation/fpga/dfl.rst.
>
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
You forgot a Reported-by: tag for the person who notified you of this :(
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting
2021-01-11 15:53 ` Greg Kroah-Hartman
@ 2021-01-11 16:34 ` Lukas Bulwahn
2021-01-11 18:14 ` Greg Kroah-Hartman
0 siblings, 1 reply; 12+ messages in thread
From: Lukas Bulwahn @ 2021-01-11 16:34 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Wu Hao, Moritz Fischer, Matthew Gerlach, linux-fpga, Tom Rix,
open list:DOCUMENTATION, kernel-janitors,
Linux Kernel Mailing List
On Mon, Jan 11, 2021 at 4:52 PM Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
>
> On Mon, Jan 11, 2021 at 12:21:13PM +0100, Lukas Bulwahn wrote:
> > Commit fa41d10589be ("fpga: dfl-pci: locate DFLs by PCIe vendor specific
> > capability") provides documentation to the FPGA Device Feature List (DFL)
> > Framework Overview, but introduced new documentation warnings:
> >
> > ./Documentation/fpga/dfl.rst:
> > 505: WARNING: Title underline too short.
> > 523: WARNING: Unexpected indentation.
> > 523: WARNING: Blank line required after table.
> > 524: WARNING: Block quote ends without a blank line; unexpected unindent.
> >
> > Rectify ReST formatting in ./Documentation/fpga/dfl.rst.
> >
> > Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
>
> You forgot a Reported-by: tag for the person who notified you of this :(
Greg, would you believe that I run 'make htmldocs' on linux-next myself?
Of course, Stephen Rothwell reports such documentation warnings as
well, but I take care independent of Stephen's reporting.
Lukas
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting
2021-01-11 16:34 ` Lukas Bulwahn
@ 2021-01-11 18:14 ` Greg Kroah-Hartman
2021-01-14 16:45 ` Moritz Fischer
0 siblings, 1 reply; 12+ messages in thread
From: Greg Kroah-Hartman @ 2021-01-11 18:14 UTC (permalink / raw)
To: Lukas Bulwahn
Cc: Wu Hao, Moritz Fischer, Matthew Gerlach, linux-fpga, Tom Rix,
open list:DOCUMENTATION, kernel-janitors,
Linux Kernel Mailing List
On Mon, Jan 11, 2021 at 05:34:57PM +0100, Lukas Bulwahn wrote:
> On Mon, Jan 11, 2021 at 4:52 PM Greg Kroah-Hartman
> <gregkh@linuxfoundation.org> wrote:
> >
> > On Mon, Jan 11, 2021 at 12:21:13PM +0100, Lukas Bulwahn wrote:
> > > Commit fa41d10589be ("fpga: dfl-pci: locate DFLs by PCIe vendor specific
> > > capability") provides documentation to the FPGA Device Feature List (DFL)
> > > Framework Overview, but introduced new documentation warnings:
> > >
> > > ./Documentation/fpga/dfl.rst:
> > > 505: WARNING: Title underline too short.
> > > 523: WARNING: Unexpected indentation.
> > > 523: WARNING: Blank line required after table.
> > > 524: WARNING: Block quote ends without a blank line; unexpected unindent.
> > >
> > > Rectify ReST formatting in ./Documentation/fpga/dfl.rst.
> > >
> > > Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> >
>
> > You forgot a Reported-by: tag for the person who notified you of this :(
>
> Greg, would you believe that I run 'make htmldocs' on linux-next myself?
Sure, just seemed to match up with when Stephen reported this a few
hours earlier...
thanks,
greg k-h
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting
2021-01-11 18:14 ` Greg Kroah-Hartman
@ 2021-01-14 16:45 ` Moritz Fischer
2021-01-14 17:05 ` Greg Kroah-Hartman
0 siblings, 1 reply; 12+ messages in thread
From: Moritz Fischer @ 2021-01-14 16:45 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Lukas Bulwahn, Wu Hao, Moritz Fischer, Matthew Gerlach,
linux-fpga, Tom Rix, open list:DOCUMENTATION, kernel-janitors,
Linux Kernel Mailing List
Hi Greg,
On Mon, Jan 11, 2021 at 07:14:22PM +0100, Greg Kroah-Hartman wrote:
> On Mon, Jan 11, 2021 at 05:34:57PM +0100, Lukas Bulwahn wrote:
> > On Mon, Jan 11, 2021 at 4:52 PM Greg Kroah-Hartman
> > <gregkh@linuxfoundation.org> wrote:
> > >
> > > On Mon, Jan 11, 2021 at 12:21:13PM +0100, Lukas Bulwahn wrote:
> > > > Commit fa41d10589be ("fpga: dfl-pci: locate DFLs by PCIe vendor specific
> > > > capability") provides documentation to the FPGA Device Feature List (DFL)
> > > > Framework Overview, but introduced new documentation warnings:
> > > >
> > > > ./Documentation/fpga/dfl.rst:
> > > > 505: WARNING: Title underline too short.
> > > > 523: WARNING: Unexpected indentation.
> > > > 523: WARNING: Blank line required after table.
> > > > 524: WARNING: Block quote ends without a blank line; unexpected unindent.
> > > >
> > > > Rectify ReST formatting in ./Documentation/fpga/dfl.rst.
> > > >
> > > > Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> > >
> >
> > > You forgot a Reported-by: tag for the person who notified you of this :(
> >
> > Greg, would you believe that I run 'make htmldocs' on linux-next myself?
>
> Sure, just seemed to match up with when Stephen reported this a few
> hours earlier...
>
> thanks,
>
> greg k-h
Are you waiting for a resend on this? Just asking because we got three
versions of this patch by now :)
I haven't re-sent this since you were already on the thread.
- Moritz
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting
2021-01-14 16:45 ` Moritz Fischer
@ 2021-01-14 17:05 ` Greg Kroah-Hartman
0 siblings, 0 replies; 12+ messages in thread
From: Greg Kroah-Hartman @ 2021-01-14 17:05 UTC (permalink / raw)
To: Moritz Fischer
Cc: Lukas Bulwahn, Wu Hao, Matthew Gerlach, linux-fpga, Tom Rix,
open list:DOCUMENTATION, kernel-janitors,
Linux Kernel Mailing List
On Thu, Jan 14, 2021 at 08:45:15AM -0800, Moritz Fischer wrote:
> Hi Greg,
>
> On Mon, Jan 11, 2021 at 07:14:22PM +0100, Greg Kroah-Hartman wrote:
> > On Mon, Jan 11, 2021 at 05:34:57PM +0100, Lukas Bulwahn wrote:
> > > On Mon, Jan 11, 2021 at 4:52 PM Greg Kroah-Hartman
> > > <gregkh@linuxfoundation.org> wrote:
> > > >
> > > > On Mon, Jan 11, 2021 at 12:21:13PM +0100, Lukas Bulwahn wrote:
> > > > > Commit fa41d10589be ("fpga: dfl-pci: locate DFLs by PCIe vendor specific
> > > > > capability") provides documentation to the FPGA Device Feature List (DFL)
> > > > > Framework Overview, but introduced new documentation warnings:
> > > > >
> > > > > ./Documentation/fpga/dfl.rst:
> > > > > 505: WARNING: Title underline too short.
> > > > > 523: WARNING: Unexpected indentation.
> > > > > 523: WARNING: Blank line required after table.
> > > > > 524: WARNING: Block quote ends without a blank line; unexpected unindent.
> > > > >
> > > > > Rectify ReST formatting in ./Documentation/fpga/dfl.rst.
> > > > >
> > > > > Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> > > >
> > >
> > > > You forgot a Reported-by: tag for the person who notified you of this :(
> > >
> > > Greg, would you believe that I run 'make htmldocs' on linux-next myself?
> >
> > Sure, just seemed to match up with when Stephen reported this a few
> > hours earlier...
> >
> > thanks,
> >
> > greg k-h
>
> Are you waiting for a resend on this? Just asking because we got three
> versions of this patch by now :)
>
> I haven't re-sent this since you were already on the thread.
I'll just take this one, for some reason I thought Lukas was going to
resend, my fault...
greg k-h
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting
2021-01-11 11:21 [PATCH -next] fpga: dfl-pci: rectify ReST formatting Lukas Bulwahn
2021-01-11 15:11 ` Tom Rix
2021-01-11 15:53 ` Greg Kroah-Hartman
@ 2021-01-11 15:55 ` Moritz Fischer
2021-01-11 17:10 ` Lukas Bulwahn
2021-01-11 18:16 ` matthew.gerlach
2 siblings, 2 replies; 12+ messages in thread
From: Moritz Fischer @ 2021-01-11 15:55 UTC (permalink / raw)
To: Lukas Bulwahn
Cc: Greg Kroah-Hartman, Wu Hao, Moritz Fischer, Matthew Gerlach,
linux-fpga, Tom Rix, linux-doc, kernel-janitors, linux-kernel
Hi Lukas,
On Mon, Jan 11, 2021 at 12:21:13PM +0100, Lukas Bulwahn wrote:
> Commit fa41d10589be ("fpga: dfl-pci: locate DFLs by PCIe vendor specific
> capability") provides documentation to the FPGA Device Feature List (DFL)
Nit: Do you want to make this a Fixes: tag instead?
> Framework Overview, but introduced new documentation warnings:
>
> ./Documentation/fpga/dfl.rst:
> 505: WARNING: Title underline too short.
> 523: WARNING: Unexpected indentation.
> 523: WARNING: Blank line required after table.
> 524: WARNING: Block quote ends without a blank line; unexpected unindent.
>
> Rectify ReST formatting in ./Documentation/fpga/dfl.rst.
>
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
Acked-by: Moritz Fischer <mdf@kernel.org>
> ---
> applies cleanly on next-20210111
>
> Moritz, Matthew, please ack.
>
> Greg, please pick this doc fixup to your fpga -next tree on top of
> the commit above.
>
> Documentation/fpga/dfl.rst | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/fpga/dfl.rst b/Documentation/fpga/dfl.rst
> index ea8cefc18bdb..c41ac76ffaae 100644
> --- a/Documentation/fpga/dfl.rst
> +++ b/Documentation/fpga/dfl.rst
> @@ -502,7 +502,7 @@ FME Partial Reconfiguration Sub Feature driver (see drivers/fpga/dfl-fme-pr.c)
> could be a reference.
>
> Location of DFLs on a PCI Device
> -===========================
> +================================
> The original method for finding a DFL on a PCI device assumed the start of the
> first DFL to offset 0 of bar 0. If the first node of the DFL is an FME,
> then further DFLs in the port(s) are specified in FME header registers.
> @@ -514,6 +514,7 @@ data begins with a 4 byte vendor specific register for the number of DFLs follow
> Offset/BIR vendor specific registers for each DFL. Bits 2:0 of Offset/BIR register
> indicates the BAR, and bits 31:3 form the 8 byte aligned offset where bits 2:0 are
> zero.
> +::
>
> +----------------------------+
> |31 Number of DFLS 0|
> --
> 2.17.1
>
Thanks for doing this, I was about to send that same patch myself.
- Moritz
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting
2021-01-11 15:55 ` Moritz Fischer
@ 2021-01-11 17:10 ` Lukas Bulwahn
2021-01-11 18:16 ` matthew.gerlach
1 sibling, 0 replies; 12+ messages in thread
From: Lukas Bulwahn @ 2021-01-11 17:10 UTC (permalink / raw)
To: Moritz Fischer
Cc: Greg Kroah-Hartman, Wu Hao, Matthew Gerlach, linux-fpga, Tom Rix,
open list:DOCUMENTATION, kernel-janitors,
Linux Kernel Mailing List
On Mon, Jan 11, 2021 at 4:55 PM Moritz Fischer <mdf@kernel.org> wrote:
>
> Hi Lukas,
>
> On Mon, Jan 11, 2021 at 12:21:13PM +0100, Lukas Bulwahn wrote:
> > Commit fa41d10589be ("fpga: dfl-pci: locate DFLs by PCIe vendor specific
> > capability") provides documentation to the FPGA Device Feature List (DFL)
> Nit: Do you want to make this a Fixes: tag instead?
IMHO, it is just adjusting the mark-up of the documentation and that
does not deserve a Fixes: tag.
Also, in the past, I got more frequent rejections when adding Fixes:
tags for such patches compared to proposals adding a Fixes: tag. So, I
settled for not adding Fixes: tags for such minor adjustment patches.
Lukas
> > Framework Overview, but introduced new documentation warnings:
> >
> > ./Documentation/fpga/dfl.rst:
> > 505: WARNING: Title underline too short.
> > 523: WARNING: Unexpected indentation.
> > 523: WARNING: Blank line required after table.
> > 524: WARNING: Block quote ends without a blank line; unexpected unindent.
> >
> > Rectify ReST formatting in ./Documentation/fpga/dfl.rst.
> >
> > Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> Acked-by: Moritz Fischer <mdf@kernel.org>
> > ---
> > applies cleanly on next-20210111
> >
> > Moritz, Matthew, please ack.
> >
> > Greg, please pick this doc fixup to your fpga -next tree on top of
> > the commit above.
> >
> > Documentation/fpga/dfl.rst | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/Documentation/fpga/dfl.rst b/Documentation/fpga/dfl.rst
> > index ea8cefc18bdb..c41ac76ffaae 100644
> > --- a/Documentation/fpga/dfl.rst
> > +++ b/Documentation/fpga/dfl.rst
> > @@ -502,7 +502,7 @@ FME Partial Reconfiguration Sub Feature driver (see drivers/fpga/dfl-fme-pr.c)
> > could be a reference.
> >
> > Location of DFLs on a PCI Device
> > -===========================
> > +================================
> > The original method for finding a DFL on a PCI device assumed the start of the
> > first DFL to offset 0 of bar 0. If the first node of the DFL is an FME,
> > then further DFLs in the port(s) are specified in FME header registers.
> > @@ -514,6 +514,7 @@ data begins with a 4 byte vendor specific register for the number of DFLs follow
> > Offset/BIR vendor specific registers for each DFL. Bits 2:0 of Offset/BIR register
> > indicates the BAR, and bits 31:3 form the 8 byte aligned offset where bits 2:0 are
> > zero.
> > +::
> >
> > +----------------------------+
> > |31 Number of DFLS 0|
> > --
> > 2.17.1
> >
>
> Thanks for doing this, I was about to send that same patch myself.
>
> - Moritz
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting
2021-01-11 15:55 ` Moritz Fischer
2021-01-11 17:10 ` Lukas Bulwahn
@ 2021-01-11 18:16 ` matthew.gerlach
1 sibling, 0 replies; 12+ messages in thread
From: matthew.gerlach @ 2021-01-11 18:16 UTC (permalink / raw)
To: Moritz Fischer
Cc: Lukas Bulwahn, Greg Kroah-Hartman, Wu Hao, linux-fpga, Tom Rix,
linux-doc, kernel-janitors, linux-kernel
On Mon, 11 Jan 2021, Moritz Fischer wrote:
> Hi Lukas,
>
> On Mon, Jan 11, 2021 at 12:21:13PM +0100, Lukas Bulwahn wrote:
>> Commit fa41d10589be ("fpga: dfl-pci: locate DFLs by PCIe vendor specific
>> capability") provides documentation to the FPGA Device Feature List (DFL)
> Nit: Do you want to make this a Fixes: tag instead?
>> Framework Overview, but introduced new documentation warnings:
>>
>> ./Documentation/fpga/dfl.rst:
>> 505: WARNING: Title underline too short.
>> 523: WARNING: Unexpected indentation.
>> 523: WARNING: Blank line required after table.
>> 524: WARNING: Block quote ends without a blank line; unexpected unindent.
>>
>> Rectify ReST formatting in ./Documentation/fpga/dfl.rst.
>>
>> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> Acked-by: Moritz Fischer <mdf@kernel.org>
Acked-by: Matthew Gerlach <matthew.gerlach@linux.intel.com>
>> ---
>> applies cleanly on next-20210111
>>
>> Moritz, Matthew, please ack.
>>
>> Greg, please pick this doc fixup to your fpga -next tree on top of
>> the commit above.
>>
>> Documentation/fpga/dfl.rst | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/Documentation/fpga/dfl.rst b/Documentation/fpga/dfl.rst
>> index ea8cefc18bdb..c41ac76ffaae 100644
>> --- a/Documentation/fpga/dfl.rst
>> +++ b/Documentation/fpga/dfl.rst
>> @@ -502,7 +502,7 @@ FME Partial Reconfiguration Sub Feature driver (see drivers/fpga/dfl-fme-pr.c)
>> could be a reference.
>>
>> Location of DFLs on a PCI Device
>> -===========================
>> +================================
>> The original method for finding a DFL on a PCI device assumed the start of the
>> first DFL to offset 0 of bar 0. If the first node of the DFL is an FME,
>> then further DFLs in the port(s) are specified in FME header registers.
>> @@ -514,6 +514,7 @@ data begins with a 4 byte vendor specific register for the number of DFLs follow
>> Offset/BIR vendor specific registers for each DFL. Bits 2:0 of Offset/BIR register
>> indicates the BAR, and bits 31:3 form the 8 byte aligned offset where bits 2:0 are
>> zero.
>> +::
>>
>> +----------------------------+
>> |31 Number of DFLS 0|
>> --
>> 2.17.1
>>
>
> Thanks for doing this, I was about to send that same patch myself.
>
> - Moritz
>
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2021-01-14 17:06 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-11 11:21 [PATCH -next] fpga: dfl-pci: rectify ReST formatting Lukas Bulwahn
2021-01-11 15:11 ` Tom Rix
2021-01-11 15:53 ` Greg Kroah-Hartman
2021-01-11 16:07 ` Tom Rix
2021-01-11 15:53 ` Greg Kroah-Hartman
2021-01-11 16:34 ` Lukas Bulwahn
2021-01-11 18:14 ` Greg Kroah-Hartman
2021-01-14 16:45 ` Moritz Fischer
2021-01-14 17:05 ` Greg Kroah-Hartman
2021-01-11 15:55 ` Moritz Fischer
2021-01-11 17:10 ` Lukas Bulwahn
2021-01-11 18:16 ` matthew.gerlach
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).