* [PATCH] Documentation: fpga: cleanup
@ 2018-05-29 17:55 Alan Tull
2018-05-29 17:56 ` Alan Tull
0 siblings, 1 reply; 4+ messages in thread
From: Alan Tull @ 2018-05-29 17:55 UTC (permalink / raw)
To: Greg Kroah-Hartman, Jonathan Corbet, Moritz Fischer
Cc: Randy Dunlap, linux-kernel, linux-fpga, Alan Tull
Hi Greg,
This patch fixes the minor spelling/typos/etc that Randy Dunlap
pointed out in his review.
This patch applies cleanly on char-misc-next; kernel-doc builds
with no warnings for fpga docs.
Thanks,
Alan
Alan Tull (1):
Documentation: fpga: cleanup
Documentation/driver-api/fpga/fpga-mgr.rst | 12 ++++++------
Documentation/driver-api/fpga/fpga-region.rst | 12 ++++++------
Documentation/driver-api/fpga/intro.rst | 14 +++++++-------
3 files changed, 19 insertions(+), 19 deletions(-)
--
2.7.4
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] Documentation: fpga: cleanup
2018-05-29 17:55 [PATCH] Documentation: fpga: cleanup Alan Tull
@ 2018-05-29 17:56 ` Alan Tull
2018-05-29 20:12 ` Randy Dunlap
0 siblings, 1 reply; 4+ messages in thread
From: Alan Tull @ 2018-05-29 17:56 UTC (permalink / raw)
To: Greg Kroah-Hartman, Jonathan Corbet, Moritz Fischer
Cc: Randy Dunlap, linux-kernel, linux-fpga, Alan Tull
Minor fixes including:
* fix some typos
* correct use of a/an
* rephrase explanation of .state ops function
* s/re-use/reuse/ (use only one spelling of 'reuse' in these docs)
* s/cpu/CPU/
Signed-off-by: Alan Tull <atull@kernel.org>
---
Documentation/driver-api/fpga/fpga-mgr.rst | 12 ++++++------
Documentation/driver-api/fpga/fpga-region.rst | 12 ++++++------
Documentation/driver-api/fpga/intro.rst | 14 +++++++-------
3 files changed, 19 insertions(+), 19 deletions(-)
diff --git a/Documentation/driver-api/fpga/fpga-mgr.rst b/Documentation/driver-api/fpga/fpga-mgr.rst
index bcf2dd2..4b3825d 100644
--- a/Documentation/driver-api/fpga/fpga-mgr.rst
+++ b/Documentation/driver-api/fpga/fpga-mgr.rst
@@ -83,7 +83,7 @@ The programming sequence is::
3. .write_complete
The .write_init function will prepare the FPGA to receive the image data. The
-buffer passed into .write_init will be atmost .initial_header_size bytes long,
+buffer passed into .write_init will be at most .initial_header_size bytes long;
if the whole bitstream is not immediately available then the core code will
buffer up at least this much before starting.
@@ -98,9 +98,9 @@ scatter list. This interface is suitable for drivers which use DMA.
The .write_complete function is called after all the image has been written
to put the FPGA into operating mode.
-The ops include a .state function which will read the hardware FPGA manager and
-return a code of type enum fpga_mgr_states. It doesn't result in a change in
-hardware state.
+The ops include a .state function which will determine the state the FPGA is in
+and return a code of type enum fpga_mgr_states. It doesn't result in a change
+in state.
How to write an image buffer to a supported FPGA
------------------------------------------------
@@ -181,8 +181,8 @@ API for implementing a new FPGA Manager driver
.. kernel-doc:: drivers/fpga/fpga-mgr.c
:functions: fpga_mgr_unregister
-API for programming a FPGA
---------------------------
+API for programming an FPGA
+---------------------------
.. kernel-doc:: include/linux/fpga/fpga-mgr.h
:functions: fpga_image_info
diff --git a/Documentation/driver-api/fpga/fpga-region.rst b/Documentation/driver-api/fpga/fpga-region.rst
index f89e4a3..f30333c 100644
--- a/Documentation/driver-api/fpga/fpga-region.rst
+++ b/Documentation/driver-api/fpga/fpga-region.rst
@@ -4,7 +4,7 @@ FPGA Region
Overview
--------
-This document is meant to be an brief overview of the FPGA region API usage. A
+This document is meant to be a brief overview of the FPGA region API usage. A
more conceptual look at regions can be found in the Device Tree binding
document [#f1]_.
@@ -31,11 +31,11 @@ fpga_image_info including:
* pointers to the image as either a scatter-gather buffer, a contiguous
buffer, or the name of firmware file
- * flags indicating specifics such as whether the image if for partial
+ * flags indicating specifics such as whether the image is for partial
reconfiguration.
-How to program a FPGA using a region
-------------------------------------
+How to program an FPGA using a region
+-------------------------------------
First, allocate the info struct::
@@ -77,8 +77,8 @@ An example of usage can be seen in the probe function of [#f2]_.
.. [#f1] ../devicetree/bindings/fpga/fpga-region.txt
.. [#f2] ../../drivers/fpga/of-fpga-region.c
-API to program a FGPA
----------------------
+API to program an FPGA
+----------------------
.. kernel-doc:: drivers/fpga/fpga-region.c
:functions: fpga_region_program_fpga
diff --git a/Documentation/driver-api/fpga/intro.rst b/Documentation/driver-api/fpga/intro.rst
index 51cd81d..50d1cab 100644
--- a/Documentation/driver-api/fpga/intro.rst
+++ b/Documentation/driver-api/fpga/intro.rst
@@ -12,18 +12,18 @@ Linux. Some of the core intentions of the FPGA subsystems are:
* Code should not be shared between upper and lower layers. This
should go without saying. If that seems necessary, there's probably
- framework functionality that that can be added that will benefit
+ framework functionality that can be added that will benefit
other users. Write the linux-fpga mailing list and maintainers and
seek out a solution that expands the framework for broad reuse.
-* Generally, when adding code, think of the future. Plan for re-use.
+* Generally, when adding code, think of the future. Plan for reuse.
The framework in the kernel is divided into:
FPGA Manager
------------
-If you are adding a new FPGA or a new method of programming a FPGA,
+If you are adding a new FPGA or a new method of programming an FPGA,
this is the subsystem for you. Low level FPGA manager drivers contain
the knowledge of how to program a specific device. This subsystem
includes the framework in fpga-mgr.c and the low level drivers that
@@ -32,10 +32,10 @@ are registered with it.
FPGA Bridge
-----------
-FPGA Bridges prevent spurious signals from going out of a FPGA or a
-region of a FPGA during programming. They are disabled before
+FPGA Bridges prevent spurious signals from going out of an FPGA or a
+region of an FPGA during programming. They are disabled before
programming begins and re-enabled afterwards. An FPGA bridge may be
-actual hard hardware that gates a bus to a cpu or a soft ("freeze")
+actual hard hardware that gates a bus to a CPU or a soft ("freeze")
bridge in FPGA fabric that surrounds a partial reconfiguration region
of an FPGA. This subsystem includes fpga-bridge.c and the low level
drivers that are registered with it.
@@ -44,7 +44,7 @@ FPGA Region
-----------
If you are adding a new interface to the FPGA framework, add it on top
-of a FPGA region to allow the most reuse of your interface.
+of an FPGA region to allow the most reuse of your interface.
The FPGA Region framework (fpga-region.c) associates managers and
bridges as reconfigurable regions. A region may refer to the whole
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] Documentation: fpga: cleanup
2018-05-29 17:56 ` Alan Tull
@ 2018-05-29 20:12 ` Randy Dunlap
2018-05-29 20:53 ` Alan Tull
0 siblings, 1 reply; 4+ messages in thread
From: Randy Dunlap @ 2018-05-29 20:12 UTC (permalink / raw)
To: Alan Tull, Greg Kroah-Hartman, Jonathan Corbet, Moritz Fischer
Cc: linux-kernel, linux-fpga
On 05/29/2018 10:56 AM, Alan Tull wrote:
> Minor fixes including:
>
> * fix some typos
> * correct use of a/an
> * rephrase explanation of .state ops function
> * s/re-use/reuse/ (use only one spelling of 'reuse' in these docs)
> * s/cpu/CPU/
>
> Signed-off-by: Alan Tull <atull@kernel.org>
Thanks. Looks good.
Acked-by: Randy Dunlap <rdunlap@infradead.org>
> ---
> Documentation/driver-api/fpga/fpga-mgr.rst | 12 ++++++------
> Documentation/driver-api/fpga/fpga-region.rst | 12 ++++++------
> Documentation/driver-api/fpga/intro.rst | 14 +++++++-------
> 3 files changed, 19 insertions(+), 19 deletions(-)
>
> diff --git a/Documentation/driver-api/fpga/fpga-mgr.rst b/Documentation/driver-api/fpga/fpga-mgr.rst
> index bcf2dd2..4b3825d 100644
> --- a/Documentation/driver-api/fpga/fpga-mgr.rst
> +++ b/Documentation/driver-api/fpga/fpga-mgr.rst
> @@ -83,7 +83,7 @@ The programming sequence is::
> 3. .write_complete
>
> The .write_init function will prepare the FPGA to receive the image data. The
> -buffer passed into .write_init will be atmost .initial_header_size bytes long,
> +buffer passed into .write_init will be at most .initial_header_size bytes long;
> if the whole bitstream is not immediately available then the core code will
> buffer up at least this much before starting.
>
> @@ -98,9 +98,9 @@ scatter list. This interface is suitable for drivers which use DMA.
> The .write_complete function is called after all the image has been written
> to put the FPGA into operating mode.
>
> -The ops include a .state function which will read the hardware FPGA manager and
> -return a code of type enum fpga_mgr_states. It doesn't result in a change in
> -hardware state.
> +The ops include a .state function which will determine the state the FPGA is in
> +and return a code of type enum fpga_mgr_states. It doesn't result in a change
> +in state.
>
> How to write an image buffer to a supported FPGA
> ------------------------------------------------
> @@ -181,8 +181,8 @@ API for implementing a new FPGA Manager driver
> .. kernel-doc:: drivers/fpga/fpga-mgr.c
> :functions: fpga_mgr_unregister
>
> -API for programming a FPGA
> ---------------------------
> +API for programming an FPGA
> +---------------------------
>
> .. kernel-doc:: include/linux/fpga/fpga-mgr.h
> :functions: fpga_image_info
> diff --git a/Documentation/driver-api/fpga/fpga-region.rst b/Documentation/driver-api/fpga/fpga-region.rst
> index f89e4a3..f30333c 100644
> --- a/Documentation/driver-api/fpga/fpga-region.rst
> +++ b/Documentation/driver-api/fpga/fpga-region.rst
> @@ -4,7 +4,7 @@ FPGA Region
> Overview
> --------
>
> -This document is meant to be an brief overview of the FPGA region API usage. A
> +This document is meant to be a brief overview of the FPGA region API usage. A
> more conceptual look at regions can be found in the Device Tree binding
> document [#f1]_.
>
> @@ -31,11 +31,11 @@ fpga_image_info including:
> * pointers to the image as either a scatter-gather buffer, a contiguous
> buffer, or the name of firmware file
>
> - * flags indicating specifics such as whether the image if for partial
> + * flags indicating specifics such as whether the image is for partial
> reconfiguration.
>
> -How to program a FPGA using a region
> -------------------------------------
> +How to program an FPGA using a region
> +-------------------------------------
>
> First, allocate the info struct::
>
> @@ -77,8 +77,8 @@ An example of usage can be seen in the probe function of [#f2]_.
> .. [#f1] ../devicetree/bindings/fpga/fpga-region.txt
> .. [#f2] ../../drivers/fpga/of-fpga-region.c
>
> -API to program a FGPA
> ----------------------
> +API to program an FPGA
> +----------------------
>
> .. kernel-doc:: drivers/fpga/fpga-region.c
> :functions: fpga_region_program_fpga
> diff --git a/Documentation/driver-api/fpga/intro.rst b/Documentation/driver-api/fpga/intro.rst
> index 51cd81d..50d1cab 100644
> --- a/Documentation/driver-api/fpga/intro.rst
> +++ b/Documentation/driver-api/fpga/intro.rst
> @@ -12,18 +12,18 @@ Linux. Some of the core intentions of the FPGA subsystems are:
>
> * Code should not be shared between upper and lower layers. This
> should go without saying. If that seems necessary, there's probably
> - framework functionality that that can be added that will benefit
> + framework functionality that can be added that will benefit
> other users. Write the linux-fpga mailing list and maintainers and
> seek out a solution that expands the framework for broad reuse.
>
> -* Generally, when adding code, think of the future. Plan for re-use.
> +* Generally, when adding code, think of the future. Plan for reuse.
>
> The framework in the kernel is divided into:
>
> FPGA Manager
> ------------
>
> -If you are adding a new FPGA or a new method of programming a FPGA,
> +If you are adding a new FPGA or a new method of programming an FPGA,
> this is the subsystem for you. Low level FPGA manager drivers contain
> the knowledge of how to program a specific device. This subsystem
> includes the framework in fpga-mgr.c and the low level drivers that
> @@ -32,10 +32,10 @@ are registered with it.
> FPGA Bridge
> -----------
>
> -FPGA Bridges prevent spurious signals from going out of a FPGA or a
> -region of a FPGA during programming. They are disabled before
> +FPGA Bridges prevent spurious signals from going out of an FPGA or a
> +region of an FPGA during programming. They are disabled before
> programming begins and re-enabled afterwards. An FPGA bridge may be
> -actual hard hardware that gates a bus to a cpu or a soft ("freeze")
> +actual hard hardware that gates a bus to a CPU or a soft ("freeze")
> bridge in FPGA fabric that surrounds a partial reconfiguration region
> of an FPGA. This subsystem includes fpga-bridge.c and the low level
> drivers that are registered with it.
> @@ -44,7 +44,7 @@ FPGA Region
> -----------
>
> If you are adding a new interface to the FPGA framework, add it on top
> -of a FPGA region to allow the most reuse of your interface.
> +of an FPGA region to allow the most reuse of your interface.
>
> The FPGA Region framework (fpga-region.c) associates managers and
> bridges as reconfigurable regions. A region may refer to the whole
>
--
~Randy
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] Documentation: fpga: cleanup
2018-05-29 20:12 ` Randy Dunlap
@ 2018-05-29 20:53 ` Alan Tull
0 siblings, 0 replies; 4+ messages in thread
From: Alan Tull @ 2018-05-29 20:53 UTC (permalink / raw)
To: Randy Dunlap
Cc: Greg Kroah-Hartman, Jonathan Corbet, Moritz Fischer,
linux-kernel, linux-fpga
On Tue, May 29, 2018 at 3:12 PM, Randy Dunlap <rdunlap@infradead.org> wrote:
> On 05/29/2018 10:56 AM, Alan Tull wrote:
>> Minor fixes including:
>>
>> * fix some typos
>> * correct use of a/an
>> * rephrase explanation of .state ops function
>> * s/re-use/reuse/ (use only one spelling of 'reuse' in these docs)
>> * s/cpu/CPU/
>>
>> Signed-off-by: Alan Tull <atull@kernel.org>
>
> Thanks. Looks good.
> Acked-by: Randy Dunlap <rdunlap@infradead.org>
Hi Randy,
Thanks!
Alan
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-05-29 20:54 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-29 17:55 [PATCH] Documentation: fpga: cleanup Alan Tull
2018-05-29 17:56 ` Alan Tull
2018-05-29 20:12 ` Randy Dunlap
2018-05-29 20:53 ` Alan Tull
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).