* [PATCH v2] NTB: switchtec_ntb: Update switchtec documentation with pre-requisites for NTB
@ 2018-09-27 6:36 ` Wesley.Sheng
0 siblings, 0 replies; 3+ messages in thread
From: Wesley.Sheng @ 2018-09-27 6:36 UTC (permalink / raw)
To: kurt.schwemmer, logang, bhelgaas, corbet
Cc: linux-pci, linux-doc, linux-kernel, Wesley.Sheng, linux-ntb, linux-ntb
Update switchtec documentation
1.add pre-requisites for NTB driver
2.misc updates
change NTB hardware driver name to ntb_hw_switchtec
update the description of 2 partitions limit
add reference to Linux Documentation/ntb.txt
add use of term 'Command ID'
add use of term 'Command Return Value'
add a missing ')'
Signed-off-by: Kelvin Cao <kelvin.cao@microchip.com>
Signed-off-by: Wesley Yung <Wesley.Yung@microchip.com>
Signed-off-by: Wesley Sheng <Wesley.sheng@microchip.com>
---
Documentation/switchtec.txt | 30 ++++++++++++++--------
1 file changed, 20 insertions(+), 10 deletions(-)
diff --git a/Documentation/switchtec.txt b/Documentation/switchtec.txt index f788264..40e839d 100644
--- a/Documentation/switchtec.txt
+++ b/Documentation/switchtec.txt
@@ -23,7 +23,7 @@ The primary means of communicating with the Switchtec management firmware is
through the Memory-mapped Remote Procedure Call (MRPC) interface.
Commands are submitted to the interface with a 4-byte command
identifier and up to 1KB of command specific data. The firmware will
-respond with a 4 bytes return code and up to 1KB of command specific
+respond with a 4-byte return code and up to 1KB of command specific
data. The interface only processes a single command at a time.
@@ -36,8 +36,8 @@ device: /dev/switchtec#, one for each management endpoint in the system.
The char device has the following semantics:
* A write must consist of at least 4 bytes and no more than 1028 bytes.
- The first four bytes will be interpreted as the command to run and
- the remainder will be used as the input data. A write will send the
+ The first 4 bytes will be interpreted as the Command ID and the
+ remainder will be used as the input data. A write will send the
command to the firmware to begin processing.
* Each write must be followed by exactly one read. Any double write will
@@ -45,9 +45,9 @@ The char device has the following semantics:
produce an error.
* A read will block until the firmware completes the command and return
- the four bytes of status plus up to 1024 bytes of output data. (The
- length will be specified by the size parameter of the read call --
- reading less than 4 bytes will produce an error.
+ the 4-byte Command Return Value plus up to 1024 bytes of output
+ data. (The length will be specified by the size parameter of the read
+ call -- reading less than 4 bytes will produce an error.)
* The poll call will also be supported for userspace applications that
need to do other things while waiting for the command to complete.
@@ -83,10 +83,20 @@ The following IOCTLs are also supported by the device:
Non-Transparent Bridge (NTB) Driver
===================================
-An NTB driver is provided for the switchtec hardware in switchtec_ntb.
-Currently, it only supports switches configured with exactly 2
-partitions. It also requires the following configuration settings:
+An NTB hardware driver is provided for the Switchtec hardware in
+ntb_hw_switchtec. Currently, it only supports switches configured with
+exactly 2 NT partitions and zero or more non-NT partitions. It also
+requires the following configuration settings:
-* Both partitions must be able to access each other's GAS spaces.
+* Both NT partitions must be able to access each other's GAS spaces.
Thus, the bits in the GAS Access Vector under Management Settings
must be set to support this.
+* Kernel configuration MUST include support for NTB (CONFIG_NTB needs
+ to be set)
+
+NT EP BAR 2 will be dynamically configured as a Direct Window, and the
+configuration file does not need to configure it explicitly.
+
+Please refer to Documentation/ntb.txt in Linux source tree for an
+overall understanding of the Linux NTB stack. ntb_hw_switchtec works as
+an NTB Hardware Driver in this stack.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH v2] NTB: switchtec_ntb: Update switchtec documentation with pre-requisites for NTB
@ 2018-09-27 6:36 ` Wesley.Sheng
0 siblings, 0 replies; 3+ messages in thread
From: Wesley.Sheng @ 2018-09-27 6:36 UTC (permalink / raw)
To: kurt.schwemmer, logang, bhelgaas, corbet
Cc: linux-pci, linux-doc, linux-kernel, Wesley.Sheng, linux-ntb, linux-ntb
Update switchtec documentation
1.add pre-requisites for NTB driver
2.misc updates
change NTB hardware driver name to ntb_hw_switchtec
update the description of 2 partitions limit
add reference to Linux Documentation/ntb.txt
add use of term 'Command ID'
add use of term 'Command Return Value'
add a missing ')'
Signed-off-by: Kelvin Cao <kelvin.cao@microchip.com>
Signed-off-by: Wesley Yung <Wesley.Yung@microchip.com>
Signed-off-by: Wesley Sheng <Wesley.sheng@microchip.com>
---
Documentation/switchtec.txt | 30 ++++++++++++++--------
1 file changed, 20 insertions(+), 10 deletions(-)
diff --git a/Documentation/switchtec.txt b/Documentation/switchtec.txt index f788264..40e839d 100644
--- a/Documentation/switchtec.txt
+++ b/Documentation/switchtec.txt
@@ -23,7 +23,7 @@ The primary means of communicating with the Switchtec management firmware is
through the Memory-mapped Remote Procedure Call (MRPC) interface.
Commands are submitted to the interface with a 4-byte command
identifier and up to 1KB of command specific data. The firmware will
-respond with a 4 bytes return code and up to 1KB of command specific
+respond with a 4-byte return code and up to 1KB of command specific
data. The interface only processes a single command at a time.
@@ -36,8 +36,8 @@ device: /dev/switchtec#, one for each management endpoint in the system.
The char device has the following semantics:
* A write must consist of at least 4 bytes and no more than 1028 bytes.
- The first four bytes will be interpreted as the command to run and
- the remainder will be used as the input data. A write will send the
+ The first 4 bytes will be interpreted as the Command ID and the
+ remainder will be used as the input data. A write will send the
command to the firmware to begin processing.
* Each write must be followed by exactly one read. Any double write will
@@ -45,9 +45,9 @@ The char device has the following semantics:
produce an error.
* A read will block until the firmware completes the command and return
- the four bytes of status plus up to 1024 bytes of output data. (The
- length will be specified by the size parameter of the read call --
- reading less than 4 bytes will produce an error.
+ the 4-byte Command Return Value plus up to 1024 bytes of output
+ data. (The length will be specified by the size parameter of the read
+ call -- reading less than 4 bytes will produce an error.)
* The poll call will also be supported for userspace applications that
need to do other things while waiting for the command to complete.
@@ -83,10 +83,20 @@ The following IOCTLs are also supported by the device:
Non-Transparent Bridge (NTB) Driver
===================================
-An NTB driver is provided for the switchtec hardware in switchtec_ntb.
-Currently, it only supports switches configured with exactly 2
-partitions. It also requires the following configuration settings:
+An NTB hardware driver is provided for the Switchtec hardware in
+ntb_hw_switchtec. Currently, it only supports switches configured with
+exactly 2 NT partitions and zero or more non-NT partitions. It also
+requires the following configuration settings:
-* Both partitions must be able to access each other's GAS spaces.
+* Both NT partitions must be able to access each other's GAS spaces.
Thus, the bits in the GAS Access Vector under Management Settings
must be set to support this.
+* Kernel configuration MUST include support for NTB (CONFIG_NTB needs
+ to be set)
+
+NT EP BAR 2 will be dynamically configured as a Direct Window, and the
+configuration file does not need to configure it explicitly.
+
+Please refer to Documentation/ntb.txt in Linux source tree for an
+overall understanding of the Linux NTB stack. ntb_hw_switchtec works as
+an NTB Hardware Driver in this stack.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] NTB: switchtec_ntb: Update switchtec documentation with pre-requisites for NTB
2018-09-27 6:36 ` Wesley.Sheng
(?)
@ 2018-09-27 16:38 ` Logan Gunthorpe
-1 siblings, 0 replies; 3+ messages in thread
From: Logan Gunthorpe @ 2018-09-27 16:38 UTC (permalink / raw)
To: Wesley.Sheng, kurt.schwemmer, bhelgaas, corbet
Cc: linux-pci, linux-doc, linux-kernel, linux-ntb
On 2018-09-27 12:36 a.m., Wesley.Sheng@microchip.com wrote:
> Update switchtec documentation
> 1.add pre-requisites for NTB driver
> 2.misc updates
> change NTB hardware driver name to ntb_hw_switchtec
> update the description of 2 partitions limit
> add reference to Linux Documentation/ntb.txt
> add use of term 'Command ID'
> add use of term 'Command Return Value'
> add a missing ')'
You need a better commit message. It needs to describe, in plain
English, why the maintainers should merge this, not itemize the
individual changes (that's what the patch body tells us).
> * A write must consist of at least 4 bytes and no more than 1028 bytes.
> - The first four bytes will be interpreted as the command to run and
> - the remainder will be used as the input data. A write will send the
> + The first 4 bytes will be interpreted as the Command ID and the
> + remainder will be used as the input data. A write will send the
> command to the firmware to begin processing.
You still have some broken white space in this patch. Notice the spacing
before "remainder" compared to the previous line.
> * Each write must be followed by exactly one read. Any double write will
> @@ -45,9 +45,9 @@ The char device has the following semantics:
> produce an error.
>
> * A read will block until the firmware completes the command and return
> - the four bytes of status plus up to 1024 bytes of output data. (The
> - length will be specified by the size parameter of the read call --
> - reading less than 4 bytes will produce an error.
> + the 4-byte Command Return Value plus up to 1024 bytes of output
> + data. (The length will be specified by the size parameter of the read
> + call -- reading less than 4 bytes will produce an error.)
Here too.
> * The poll call will also be supported for userspace applications that
> need to do other things while waiting for the command to complete.
> @@ -83,10 +83,20 @@ The following IOCTLs are also supported by the device:
> Non-Transparent Bridge (NTB) Driver
> ===================================
>
> -An NTB driver is provided for the switchtec hardware in switchtec_ntb.
> -Currently, it only supports switches configured with exactly 2
> -partitions. It also requires the following configuration settings:
> +An NTB hardware driver is provided for the Switchtec hardware in
> +ntb_hw_switchtec. Currently, it only supports switches configured with
> +exactly 2 NT partitions and zero or more non-NT partitions. It also
> +requires the following configuration settings:
>
> -* Both partitions must be able to access each other's GAS spaces.
> +* Both NT partitions must be able to access each other's GAS spaces.
> Thus, the bits in the GAS Access Vector under Management Settings
> must be set to support this.
> +* Kernel configuration MUST include support for NTB (CONFIG_NTB needs
> + to be set)
> +
> +NT EP BAR 2 will be dynamically configured as a Direct Window, and the
> +configuration file does not need to configure it explicitly.
> +
> +Please refer to Documentation/ntb.txt in Linux source tree for an
> +overall understanding of the Linux NTB stack. ntb_hw_switchtec works as
> +an NTB Hardware Driver in this stack.
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-09-27 16:38 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-27 6:36 [PATCH v2] NTB: switchtec_ntb: Update switchtec documentation with pre-requisites for NTB Wesley.Sheng
2018-09-27 6:36 ` Wesley.Sheng
2018-09-27 16:38 ` Logan Gunthorpe
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.