* [PATCH -next] fpga: dfl-pci: rectify ReST formatting @ 2021-01-11 11:21 ` Lukas Bulwahn 0 siblings, 0 replies; 24+ 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] 24+ messages in thread
* [PATCH -next] fpga: dfl-pci: rectify ReST formatting @ 2021-01-11 11:21 ` Lukas Bulwahn 0 siblings, 0 replies; 24+ 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] 24+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting 2021-01-11 11:21 ` Lukas Bulwahn @ 2021-01-11 15:11 ` Tom Rix -1 siblings, 0 replies; 24+ 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] 24+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting @ 2021-01-11 15:11 ` Tom Rix 0 siblings, 0 replies; 24+ 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] 24+ 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 -1 siblings, 0 replies; 24+ 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] 24+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting @ 2021-01-11 15:53 ` Greg Kroah-Hartman 0 siblings, 0 replies; 24+ 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] 24+ 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 -1 siblings, 0 replies; 24+ 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] 24+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting @ 2021-01-11 16:07 ` Tom Rix 0 siblings, 0 replies; 24+ 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] 24+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting 2021-01-11 11:21 ` Lukas Bulwahn @ 2021-01-11 15:53 ` Greg Kroah-Hartman -1 siblings, 0 replies; 24+ 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] 24+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting @ 2021-01-11 15:53 ` Greg Kroah-Hartman 0 siblings, 0 replies; 24+ 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] 24+ 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 -1 siblings, 0 replies; 24+ 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] 24+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting @ 2021-01-11 16:34 ` Lukas Bulwahn 0 siblings, 0 replies; 24+ 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] 24+ 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 -1 siblings, 0 replies; 24+ 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] 24+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting @ 2021-01-11 18:14 ` Greg Kroah-Hartman 0 siblings, 0 replies; 24+ 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] 24+ 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 -1 siblings, 0 replies; 24+ 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] 24+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting @ 2021-01-14 16:45 ` Moritz Fischer 0 siblings, 0 replies; 24+ 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] 24+ 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 -1 siblings, 0 replies; 24+ 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] 24+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting @ 2021-01-14 17:05 ` Greg Kroah-Hartman 0 siblings, 0 replies; 24+ 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] 24+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting 2021-01-11 11:21 ` Lukas Bulwahn @ 2021-01-11 15:55 ` Moritz Fischer -1 siblings, 0 replies; 24+ 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] 24+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting @ 2021-01-11 15:55 ` Moritz Fischer 0 siblings, 0 replies; 24+ 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] 24+ 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 -1 siblings, 0 replies; 24+ 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] 24+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting @ 2021-01-11 17:10 ` Lukas Bulwahn 0 siblings, 0 replies; 24+ 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] 24+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting 2021-01-11 15:55 ` Moritz Fischer @ 2021-01-11 18:16 ` matthew.gerlach -1 siblings, 0 replies; 24+ 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] 24+ messages in thread
* Re: [PATCH -next] fpga: dfl-pci: rectify ReST formatting @ 2021-01-11 18:16 ` matthew.gerlach 0 siblings, 0 replies; 24+ 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] 24+ messages in thread
end of thread, other threads:[~2021-01-14 17:06 UTC | newest] Thread overview: 24+ 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 11:21 ` Lukas Bulwahn 2021-01-11 15:11 ` Tom Rix 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 16:07 ` Tom Rix 2021-01-11 16:07 ` Tom Rix 2021-01-11 15:53 ` Greg Kroah-Hartman 2021-01-11 15:53 ` Greg Kroah-Hartman 2021-01-11 16:34 ` Lukas Bulwahn 2021-01-11 16:34 ` Lukas Bulwahn 2021-01-11 18:14 ` Greg Kroah-Hartman 2021-01-11 18:14 ` Greg Kroah-Hartman 2021-01-14 16:45 ` Moritz Fischer 2021-01-14 16:45 ` Moritz Fischer 2021-01-14 17:05 ` Greg Kroah-Hartman 2021-01-14 17:05 ` Greg Kroah-Hartman 2021-01-11 15:55 ` Moritz Fischer 2021-01-11 15:55 ` Moritz Fischer 2021-01-11 17:10 ` Lukas Bulwahn 2021-01-11 17:10 ` Lukas Bulwahn 2021-01-11 18:16 ` matthew.gerlach 2021-01-11 18:16 ` matthew.gerlach
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.