* [PATCH] drivers: ntb: Kconfig: pedantic cleanups
@ 2019-03-06 18:51 Enrico Weigelt, metux IT consult
2019-03-06 22:02 ` [PATCH v2] " Enrico Weigelt, metux IT consult
2019-03-20 21:13 ` [PATCH] " Bjorn Helgaas
0 siblings, 2 replies; 5+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2019-03-06 18:51 UTC (permalink / raw)
To: linux-kernel; +Cc: kurt.schwemmer, logang, linux-ntb, linux-pci
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
---
drivers/ntb/Kconfig | 20 ++++++++++----------
drivers/ntb/hw/amd/Kconfig | 4 ++--
drivers/ntb/hw/idt/Kconfig | 41 ++++++++++++++++++++---------------------
drivers/ntb/hw/intel/Kconfig | 4 ++--
drivers/ntb/hw/mscc/Kconfig | 8 ++++----
drivers/ntb/test/Kconfig | 26 +++++++++++++-------------
6 files changed, 51 insertions(+), 52 deletions(-)
diff --git a/drivers/ntb/Kconfig b/drivers/ntb/Kconfig
index 95944e5..5ce3fdd 100644
--- a/drivers/ntb/Kconfig
+++ b/drivers/ntb/Kconfig
@@ -2,13 +2,13 @@ menuconfig NTB
tristate "Non-Transparent Bridge support"
depends on PCI
help
- The PCI-E Non-transparent bridge hardware is a point-to-point PCI-E bus
- connecting 2 systems. When configured, writes to the device's PCI
- mapped memory will be mirrored to a buffer on the remote system. The
- ntb Linux driver uses this point-to-point communication as a method to
- transfer data from one system to the other.
+ The PCI-E Non-transparent bridge hardware is a point-to-point PCI-E bus
+ connecting 2 systems. When configured, writes to the device's PCI
+ mapped memory will be mirrored to a buffer on the remote system. The
+ ntb Linux driver uses this point-to-point communication as a method to
+ transfer data from one system to the other.
- If unsure, say N.
+ If unsure, say N.
if NTB
@@ -19,10 +19,10 @@ source "drivers/ntb/test/Kconfig"
config NTB_TRANSPORT
tristate "NTB Transport Client"
help
- This is a transport driver that enables connected systems to exchange
- messages over the ntb hardware. The transport exposes a queue pair api
- to client drivers.
+ This is a transport driver that enables connected systems to exchange
+ messages over the ntb hardware. The transport exposes a queue pair api
+ to client drivers.
- If unsure, say N.
+ If unsure, say N.
endif # NTB
diff --git a/drivers/ntb/hw/amd/Kconfig b/drivers/ntb/hw/amd/Kconfig
index cfe903c..9a90f17 100644
--- a/drivers/ntb/hw/amd/Kconfig
+++ b/drivers/ntb/hw/amd/Kconfig
@@ -2,6 +2,6 @@ config NTB_AMD
tristate "AMD Non-Transparent Bridge support"
depends on X86_64
help
- This driver supports AMD NTB on capable Zeppelin hardware.
+ This driver supports AMD NTB on capable Zeppelin hardware.
- If unsure, say N.
+ If unsure, say N.
diff --git a/drivers/ntb/hw/idt/Kconfig b/drivers/ntb/hw/idt/Kconfig
index f8948cf..5d106ac 100644
--- a/drivers/ntb/hw/idt/Kconfig
+++ b/drivers/ntb/hw/idt/Kconfig
@@ -3,28 +3,27 @@ config NTB_IDT
depends on PCI
select HWMON
help
- This driver supports NTB of cappable IDT PCIe-switches.
+ This driver supports NTB of cappable IDT PCIe-switches.
- Some of the pre-initializations must be made before IDT PCIe-switch
- exposes it NT-functions correctly. It should be done by either proper
- initialisation of EEPROM connected to master smbus of the switch or
- by BIOS using slave-SMBus interface changing corresponding registers
- value. Evidently it must be done before PCI bus enumeration is
- finished in Linux kernel.
+ Some of the pre-initializations must be made before IDT PCIe-switch
+ exposes it NT-functions correctly. It should be done by either proper
+ initialisation of EEPROM connected to master smbus of the switch or
+ by BIOS using slave-SMBus interface changing corresponding registers
+ value. Evidently it must be done before PCI bus enumeration is
+ finished in Linux kernel.
- First of all partitions must be activated and properly assigned to all
- the ports with NT-functions intended to be activated (see SWPARTxCTL
- and SWPORTxCTL registers). Then all NT-function BARs must be enabled
- with chosen valid aperture. For memory windows related BARs the
- aperture settings shall determine the maximum size of memory windows
- accepted by a BAR. Note that BAR0 must map PCI configuration space
- registers.
+ First of all partitions must be activated and properly assigned to all
+ the ports with NT-functions intended to be activated (see SWPARTxCTL
+ and SWPORTxCTL registers). Then all NT-function BARs must be enabled
+ with chosen valid aperture. For memory windows related BARs the
+ aperture settings shall determine the maximum size of memory windows
+ accepted by a BAR. Note that BAR0 must map PCI configuration space
+ registers.
- It's worth to note, that since a part of this driver relies on the
- BAR settings of peer NT-functions, the BAR setups can't be done over
- kernel PCI fixups. That's why the alternative pre-initialization
- techniques like BIOS using SMBus interface or EEPROM should be
- utilized.
-
- If unsure, say N.
+ It's worth to note, that since a part of this driver relies on the
+ BAR settings of peer NT-functions, the BAR setups can't be done over
+ kernel PCI fixups. That's why the alternative pre-initialization
+ techniques like BIOS using SMBus interface or EEPROM should be
+ utilized.
+ If unsure, say N.
diff --git a/drivers/ntb/hw/intel/Kconfig b/drivers/ntb/hw/intel/Kconfig
index 91f995e..c166d50 100644
--- a/drivers/ntb/hw/intel/Kconfig
+++ b/drivers/ntb/hw/intel/Kconfig
@@ -2,6 +2,6 @@ config NTB_INTEL
tristate "Intel Non-Transparent Bridge support"
depends on X86_64
help
- This driver supports Intel NTB on capable Xeon and Atom hardware.
+ This driver supports Intel NTB on capable Xeon and Atom hardware.
- If unsure, say N.
+ If unsure, say N.
diff --git a/drivers/ntb/hw/mscc/Kconfig b/drivers/ntb/hw/mscc/Kconfig
index 013ed67..74ec015 100644
--- a/drivers/ntb/hw/mscc/Kconfig
+++ b/drivers/ntb/hw/mscc/Kconfig
@@ -2,8 +2,8 @@ config NTB_SWITCHTEC
tristate "MicroSemi Switchtec Non-Transparent Bridge Support"
select PCI_SW_SWITCHTEC
help
- Enables NTB support for Switchtec PCI switches. This also
- selects the Switchtec management driver as they share the same
- hardware interface.
+ Enables NTB support for Switchtec PCI switches. This also
+ selects the Switchtec management driver as they share the same
+ hardware interface.
- If unsure, say N.
+ If unsure, say N.
diff --git a/drivers/ntb/test/Kconfig b/drivers/ntb/test/Kconfig
index a5d0eda..416ab56 100644
--- a/drivers/ntb/test/Kconfig
+++ b/drivers/ntb/test/Kconfig
@@ -1,27 +1,27 @@
config NTB_PINGPONG
tristate "NTB Ping Pong Test Client"
help
- This is a simple ping pong driver that exercises the scratchpads and
- doorbells of the ntb hardware. This driver may be used to test that
- your ntb hardware and drivers are functioning at a basic level.
+ This is a simple ping pong driver that exercises the scratchpads and
+ doorbells of the ntb hardware. This driver may be used to test that
+ your ntb hardware and drivers are functioning at a basic level.
- If unsure, say N.
+ If unsure, say N.
config NTB_TOOL
tristate "NTB Debugging Tool Test Client"
help
- This is a simple debugging driver that enables the doorbell and
- scratchpad registers to be read and written from the debugfs. This
- enables more complicated debugging to be scripted from user space.
- This driver may be used to test that your ntb hardware and drivers are
- functioning at a basic level.
+ This is a simple debugging driver that enables the doorbell and
+ scratchpad registers to be read and written from the debugfs. This
+ enables more complicated debugging to be scripted from user space.
+ This driver may be used to test that your ntb hardware and drivers are
+ functioning at a basic level.
- If unsure, say N.
+ If unsure, say N.
config NTB_PERF
tristate "NTB RAW Perf Measuring Tool"
help
- This is a tool to measure raw NTB performance by transferring data
- to and from the window without additional software interaction.
+ This is a tool to measure raw NTB performance by transferring data
+ to and from the window without additional software interaction.
- If unsure, say N.
+ If unsure, say N.
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH v2] drivers: ntb: Kconfig: pedantic cleanups
2019-03-06 18:51 [PATCH] drivers: ntb: Kconfig: pedantic cleanups Enrico Weigelt, metux IT consult
@ 2019-03-06 22:02 ` Enrico Weigelt, metux IT consult
2019-03-20 21:15 ` Bjorn Helgaas
2019-03-20 21:13 ` [PATCH] " Bjorn Helgaas
1 sibling, 1 reply; 5+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2019-03-06 22:02 UTC (permalink / raw)
To: linux-kernel; +Cc: kurt.schwemmer, logang, linux-ntb, linux-pci
Formatting of Kconfig files doesn't look so pretty, so just
take damp cloth and clean it up.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
---
drivers/ntb/Kconfig | 20 ++++++++++----------
drivers/ntb/hw/amd/Kconfig | 4 ++--
drivers/ntb/hw/idt/Kconfig | 41 ++++++++++++++++++++---------------------
drivers/ntb/hw/intel/Kconfig | 4 ++--
drivers/ntb/hw/mscc/Kconfig | 8 ++++----
drivers/ntb/test/Kconfig | 26 +++++++++++++-------------
6 files changed, 51 insertions(+), 52 deletions(-)
diff --git a/drivers/ntb/Kconfig b/drivers/ntb/Kconfig
index 95944e5..5ce3fdd 100644
--- a/drivers/ntb/Kconfig
+++ b/drivers/ntb/Kconfig
@@ -2,13 +2,13 @@ menuconfig NTB
tristate "Non-Transparent Bridge support"
depends on PCI
help
- The PCI-E Non-transparent bridge hardware is a point-to-point PCI-E bus
- connecting 2 systems. When configured, writes to the device's PCI
- mapped memory will be mirrored to a buffer on the remote system. The
- ntb Linux driver uses this point-to-point communication as a method to
- transfer data from one system to the other.
+ The PCI-E Non-transparent bridge hardware is a point-to-point PCI-E bus
+ connecting 2 systems. When configured, writes to the device's PCI
+ mapped memory will be mirrored to a buffer on the remote system. The
+ ntb Linux driver uses this point-to-point communication as a method to
+ transfer data from one system to the other.
- If unsure, say N.
+ If unsure, say N.
if NTB
@@ -19,10 +19,10 @@ source "drivers/ntb/test/Kconfig"
config NTB_TRANSPORT
tristate "NTB Transport Client"
help
- This is a transport driver that enables connected systems to exchange
- messages over the ntb hardware. The transport exposes a queue pair api
- to client drivers.
+ This is a transport driver that enables connected systems to exchange
+ messages over the ntb hardware. The transport exposes a queue pair api
+ to client drivers.
- If unsure, say N.
+ If unsure, say N.
endif # NTB
diff --git a/drivers/ntb/hw/amd/Kconfig b/drivers/ntb/hw/amd/Kconfig
index cfe903c..9a90f17 100644
--- a/drivers/ntb/hw/amd/Kconfig
+++ b/drivers/ntb/hw/amd/Kconfig
@@ -2,6 +2,6 @@ config NTB_AMD
tristate "AMD Non-Transparent Bridge support"
depends on X86_64
help
- This driver supports AMD NTB on capable Zeppelin hardware.
+ This driver supports AMD NTB on capable Zeppelin hardware.
- If unsure, say N.
+ If unsure, say N.
diff --git a/drivers/ntb/hw/idt/Kconfig b/drivers/ntb/hw/idt/Kconfig
index f8948cf..5d106ac 100644
--- a/drivers/ntb/hw/idt/Kconfig
+++ b/drivers/ntb/hw/idt/Kconfig
@@ -3,28 +3,27 @@ config NTB_IDT
depends on PCI
select HWMON
help
- This driver supports NTB of cappable IDT PCIe-switches.
+ This driver supports NTB of cappable IDT PCIe-switches.
- Some of the pre-initializations must be made before IDT PCIe-switch
- exposes it NT-functions correctly. It should be done by either proper
- initialisation of EEPROM connected to master smbus of the switch or
- by BIOS using slave-SMBus interface changing corresponding registers
- value. Evidently it must be done before PCI bus enumeration is
- finished in Linux kernel.
+ Some of the pre-initializations must be made before IDT PCIe-switch
+ exposes it NT-functions correctly. It should be done by either proper
+ initialisation of EEPROM connected to master smbus of the switch or
+ by BIOS using slave-SMBus interface changing corresponding registers
+ value. Evidently it must be done before PCI bus enumeration is
+ finished in Linux kernel.
- First of all partitions must be activated and properly assigned to all
- the ports with NT-functions intended to be activated (see SWPARTxCTL
- and SWPORTxCTL registers). Then all NT-function BARs must be enabled
- with chosen valid aperture. For memory windows related BARs the
- aperture settings shall determine the maximum size of memory windows
- accepted by a BAR. Note that BAR0 must map PCI configuration space
- registers.
+ First of all partitions must be activated and properly assigned to all
+ the ports with NT-functions intended to be activated (see SWPARTxCTL
+ and SWPORTxCTL registers). Then all NT-function BARs must be enabled
+ with chosen valid aperture. For memory windows related BARs the
+ aperture settings shall determine the maximum size of memory windows
+ accepted by a BAR. Note that BAR0 must map PCI configuration space
+ registers.
- It's worth to note, that since a part of this driver relies on the
- BAR settings of peer NT-functions, the BAR setups can't be done over
- kernel PCI fixups. That's why the alternative pre-initialization
- techniques like BIOS using SMBus interface or EEPROM should be
- utilized.
-
- If unsure, say N.
+ It's worth to note, that since a part of this driver relies on the
+ BAR settings of peer NT-functions, the BAR setups can't be done over
+ kernel PCI fixups. That's why the alternative pre-initialization
+ techniques like BIOS using SMBus interface or EEPROM should be
+ utilized.
+ If unsure, say N.
diff --git a/drivers/ntb/hw/intel/Kconfig b/drivers/ntb/hw/intel/Kconfig
index 91f995e..c166d50 100644
--- a/drivers/ntb/hw/intel/Kconfig
+++ b/drivers/ntb/hw/intel/Kconfig
@@ -2,6 +2,6 @@ config NTB_INTEL
tristate "Intel Non-Transparent Bridge support"
depends on X86_64
help
- This driver supports Intel NTB on capable Xeon and Atom hardware.
+ This driver supports Intel NTB on capable Xeon and Atom hardware.
- If unsure, say N.
+ If unsure, say N.
diff --git a/drivers/ntb/hw/mscc/Kconfig b/drivers/ntb/hw/mscc/Kconfig
index 013ed67..74ec015 100644
--- a/drivers/ntb/hw/mscc/Kconfig
+++ b/drivers/ntb/hw/mscc/Kconfig
@@ -2,8 +2,8 @@ config NTB_SWITCHTEC
tristate "MicroSemi Switchtec Non-Transparent Bridge Support"
select PCI_SW_SWITCHTEC
help
- Enables NTB support for Switchtec PCI switches. This also
- selects the Switchtec management driver as they share the same
- hardware interface.
+ Enables NTB support for Switchtec PCI switches. This also
+ selects the Switchtec management driver as they share the same
+ hardware interface.
- If unsure, say N.
+ If unsure, say N.
diff --git a/drivers/ntb/test/Kconfig b/drivers/ntb/test/Kconfig
index a5d0eda..416ab56 100644
--- a/drivers/ntb/test/Kconfig
+++ b/drivers/ntb/test/Kconfig
@@ -1,27 +1,27 @@
config NTB_PINGPONG
tristate "NTB Ping Pong Test Client"
help
- This is a simple ping pong driver that exercises the scratchpads and
- doorbells of the ntb hardware. This driver may be used to test that
- your ntb hardware and drivers are functioning at a basic level.
+ This is a simple ping pong driver that exercises the scratchpads and
+ doorbells of the ntb hardware. This driver may be used to test that
+ your ntb hardware and drivers are functioning at a basic level.
- If unsure, say N.
+ If unsure, say N.
config NTB_TOOL
tristate "NTB Debugging Tool Test Client"
help
- This is a simple debugging driver that enables the doorbell and
- scratchpad registers to be read and written from the debugfs. This
- enables more complicated debugging to be scripted from user space.
- This driver may be used to test that your ntb hardware and drivers are
- functioning at a basic level.
+ This is a simple debugging driver that enables the doorbell and
+ scratchpad registers to be read and written from the debugfs. This
+ enables more complicated debugging to be scripted from user space.
+ This driver may be used to test that your ntb hardware and drivers are
+ functioning at a basic level.
- If unsure, say N.
+ If unsure, say N.
config NTB_PERF
tristate "NTB RAW Perf Measuring Tool"
help
- This is a tool to measure raw NTB performance by transferring data
- to and from the window without additional software interaction.
+ This is a tool to measure raw NTB performance by transferring data
+ to and from the window without additional software interaction.
- If unsure, say N.
+ If unsure, say N.
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] drivers: ntb: Kconfig: pedantic cleanups
2019-03-06 18:51 [PATCH] drivers: ntb: Kconfig: pedantic cleanups Enrico Weigelt, metux IT consult
2019-03-06 22:02 ` [PATCH v2] " Enrico Weigelt, metux IT consult
@ 2019-03-20 21:13 ` Bjorn Helgaas
1 sibling, 0 replies; 5+ messages in thread
From: Bjorn Helgaas @ 2019-03-20 21:13 UTC (permalink / raw)
To: Enrico Weigelt, metux IT consult
Cc: linux-kernel, kurt.schwemmer, logang, linux-ntb, linux-pci,
Jon Mason, Dave Jiang, Allen Hubbe
[+cc Jon, Dave, Allen (NTB core maintainers)]
Hi Enrico,
I added the NTB maintainers, who will deal with these, but since
you're fixing pedantic issues, I'll give you some pedantic comments :)
The first is that you might put something here in the commit log, eg,
"fix Kconfig help text indentation" or whatever you're doing.
On Wed, Mar 06, 2019 at 07:51:05PM +0100, Enrico Weigelt, metux IT consult wrote:
> Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
> ---
> drivers/ntb/Kconfig | 20 ++++++++++----------
> drivers/ntb/hw/amd/Kconfig | 4 ++--
> drivers/ntb/hw/idt/Kconfig | 41 ++++++++++++++++++++---------------------
> drivers/ntb/hw/intel/Kconfig | 4 ++--
> drivers/ntb/hw/mscc/Kconfig | 8 ++++----
> drivers/ntb/test/Kconfig | 26 +++++++++++++-------------
> 6 files changed, 51 insertions(+), 52 deletions(-)
>
> diff --git a/drivers/ntb/Kconfig b/drivers/ntb/Kconfig
> index 95944e5..5ce3fdd 100644
> --- a/drivers/ntb/Kconfig
> +++ b/drivers/ntb/Kconfig
> @@ -2,13 +2,13 @@ menuconfig NTB
> tristate "Non-Transparent Bridge support"
> depends on PCI
> help
> - The PCI-E Non-transparent bridge hardware is a point-to-point PCI-E bus
> - connecting 2 systems. When configured, writes to the device's PCI
> - mapped memory will be mirrored to a buffer on the remote system. The
> - ntb Linux driver uses this point-to-point communication as a method to
> - transfer data from one system to the other.
> + The PCI-E Non-transparent bridge hardware is a point-to-point PCI-E bus
The usual spelling is "PCIe" instead of "PCI-E".
s/Non-transparent bridge/non-transparent bridge/
A "point-to-point PCIe bus" is usually called a "PCIe Link".
> + connecting 2 systems. When configured, writes to the device's PCI
> + mapped memory will be mirrored to a buffer on the remote system. The
> + ntb Linux driver uses this point-to-point communication as a method to
s/ntb Linux driver/Linux ntb driver/
> + transfer data from one system to the other.
>
> - If unsure, say N.
> + If unsure, say N.
>
> if NTB
>
> @@ -19,10 +19,10 @@ source "drivers/ntb/test/Kconfig"
> config NTB_TRANSPORT
> tristate "NTB Transport Client"
> help
> - This is a transport driver that enables connected systems to exchange
> - messages over the ntb hardware. The transport exposes a queue pair api
> - to client drivers.
> + This is a transport driver that enables connected systems to exchange
> + messages over the ntb hardware. The transport exposes a queue pair api
s/ntb hardware/NTB hardware/
(I think it makes sense to use "ntb" when referring to the driver,
since that's its name, but when referring to the hardware, "NTB" is
being used as an acronym and probably should be capitalized.)
> + to client drivers.
>
> - If unsure, say N.
> + If unsure, say N.
>
> endif # NTB
> diff --git a/drivers/ntb/hw/amd/Kconfig b/drivers/ntb/hw/amd/Kconfig
> index cfe903c..9a90f17 100644
> --- a/drivers/ntb/hw/amd/Kconfig
> +++ b/drivers/ntb/hw/amd/Kconfig
> @@ -2,6 +2,6 @@ config NTB_AMD
> tristate "AMD Non-Transparent Bridge support"
> depends on X86_64
> help
> - This driver supports AMD NTB on capable Zeppelin hardware.
> + This driver supports AMD NTB on capable Zeppelin hardware.
>
> - If unsure, say N.
> + If unsure, say N.
> diff --git a/drivers/ntb/hw/idt/Kconfig b/drivers/ntb/hw/idt/Kconfig
> index f8948cf..5d106ac 100644
> --- a/drivers/ntb/hw/idt/Kconfig
> +++ b/drivers/ntb/hw/idt/Kconfig
> @@ -3,28 +3,27 @@ config NTB_IDT
> depends on PCI
> select HWMON
> help
> - This driver supports NTB of cappable IDT PCIe-switches.
> + This driver supports NTB of cappable IDT PCIe-switches.
>
> - Some of the pre-initializations must be made before IDT PCIe-switch
> - exposes it NT-functions correctly. It should be done by either proper
> - initialisation of EEPROM connected to master smbus of the switch or
> - by BIOS using slave-SMBus interface changing corresponding registers
> - value. Evidently it must be done before PCI bus enumeration is
> - finished in Linux kernel.
> + Some of the pre-initializations must be made before IDT PCIe-switch
> + exposes it NT-functions correctly. It should be done by either proper
s/it/its/
s/correctly//
s/proper//
I always think "correct" and "proper" are sort of useless in comments.
Who wants to write incorrect or improper code? Besides, they don't
convey any specific information.
> + initialisation of EEPROM connected to master smbus of the switch or
> + by BIOS using slave-SMBus interface changing corresponding registers
Capitalize "smbus/SMBus" consistently. I think "SMBus" is typical.
> + value. Evidently it must be done before PCI bus enumeration is
> + finished in Linux kernel.
>
> - First of all partitions must be activated and properly assigned to all
> - the ports with NT-functions intended to be activated (see SWPARTxCTL
> - and SWPORTxCTL registers). Then all NT-function BARs must be enabled
> - with chosen valid aperture. For memory windows related BARs the
> - aperture settings shall determine the maximum size of memory windows
> - accepted by a BAR. Note that BAR0 must map PCI configuration space
> - registers.
> + First of all partitions must be activated and properly assigned to all
s/properly// again :)
> + the ports with NT-functions intended to be activated (see SWPARTxCTL
> + and SWPORTxCTL registers). Then all NT-function BARs must be enabled
> + with chosen valid aperture. For memory windows related BARs the
> + aperture settings shall determine the maximum size of memory windows
> + accepted by a BAR. Note that BAR0 must map PCI configuration space
> + registers.
>
> - It's worth to note, that since a part of this driver relies on the
> - BAR settings of peer NT-functions, the BAR setups can't be done over
> - kernel PCI fixups. That's why the alternative pre-initialization
> - techniques like BIOS using SMBus interface or EEPROM should be
> - utilized.
> -
> - If unsure, say N.
> + It's worth to note, that since a part of this driver relies on the
s/note,/note/
> + BAR settings of peer NT-functions, the BAR setups can't be done over
> + kernel PCI fixups. That's why the alternative pre-initialization
> + techniques like BIOS using SMBus interface or EEPROM should be
> + utilized.
>
> + If unsure, say N.
> diff --git a/drivers/ntb/hw/intel/Kconfig b/drivers/ntb/hw/intel/Kconfig
> index 91f995e..c166d50 100644
> --- a/drivers/ntb/hw/intel/Kconfig
> +++ b/drivers/ntb/hw/intel/Kconfig
> @@ -2,6 +2,6 @@ config NTB_INTEL
> tristate "Intel Non-Transparent Bridge support"
> depends on X86_64
> help
> - This driver supports Intel NTB on capable Xeon and Atom hardware.
> + This driver supports Intel NTB on capable Xeon and Atom hardware.
>
> - If unsure, say N.
> + If unsure, say N.
> diff --git a/drivers/ntb/hw/mscc/Kconfig b/drivers/ntb/hw/mscc/Kconfig
> index 013ed67..74ec015 100644
> --- a/drivers/ntb/hw/mscc/Kconfig
> +++ b/drivers/ntb/hw/mscc/Kconfig
> @@ -2,8 +2,8 @@ config NTB_SWITCHTEC
> tristate "MicroSemi Switchtec Non-Transparent Bridge Support"
> select PCI_SW_SWITCHTEC
> help
> - Enables NTB support for Switchtec PCI switches. This also
> - selects the Switchtec management driver as they share the same
> - hardware interface.
> + Enables NTB support for Switchtec PCI switches. This also
> + selects the Switchtec management driver as they share the same
> + hardware interface.
>
> - If unsure, say N.
> + If unsure, say N.
> diff --git a/drivers/ntb/test/Kconfig b/drivers/ntb/test/Kconfig
> index a5d0eda..416ab56 100644
> --- a/drivers/ntb/test/Kconfig
> +++ b/drivers/ntb/test/Kconfig
> @@ -1,27 +1,27 @@
> config NTB_PINGPONG
> tristate "NTB Ping Pong Test Client"
> help
> - This is a simple ping pong driver that exercises the scratchpads and
> - doorbells of the ntb hardware. This driver may be used to test that
> - your ntb hardware and drivers are functioning at a basic level.
> + This is a simple ping pong driver that exercises the scratchpads and
> + doorbells of the ntb hardware. This driver may be used to test that
> + your ntb hardware and drivers are functioning at a basic level.
s/ntb/NTB/ (twice)
> - If unsure, say N.
> + If unsure, say N.
>
> config NTB_TOOL
> tristate "NTB Debugging Tool Test Client"
> help
> - This is a simple debugging driver that enables the doorbell and
> - scratchpad registers to be read and written from the debugfs. This
> - enables more complicated debugging to be scripted from user space.
> - This driver may be used to test that your ntb hardware and drivers are
> - functioning at a basic level.
> + This is a simple debugging driver that enables the doorbell and
> + scratchpad registers to be read and written from the debugfs. This
> + enables more complicated debugging to be scripted from user space.
> + This driver may be used to test that your ntb hardware and drivers are
> + functioning at a basic level.
s/ntb/NTB/
> - If unsure, say N.
> + If unsure, say N.
>
> config NTB_PERF
> tristate "NTB RAW Perf Measuring Tool"
> help
> - This is a tool to measure raw NTB performance by transferring data
> - to and from the window without additional software interaction.
> + This is a tool to measure raw NTB performance by transferring data
> + to and from the window without additional software interaction.
>
> - If unsure, say N.
> + If unsure, say N.
> --
> 1.9.1
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] drivers: ntb: Kconfig: pedantic cleanups
2019-03-06 22:02 ` [PATCH v2] " Enrico Weigelt, metux IT consult
@ 2019-03-20 21:15 ` Bjorn Helgaas
2019-03-28 11:45 ` Jon Mason
0 siblings, 1 reply; 5+ messages in thread
From: Bjorn Helgaas @ 2019-03-20 21:15 UTC (permalink / raw)
To: Enrico Weigelt, metux IT consult
Cc: linux-kernel, kurt.schwemmer, logang, linux-ntb, linux-pci,
Jon Mason, Dave Jiang, Allen Hubbe
[+cc Jon, Dave, Allen]
On Wed, Mar 06, 2019 at 11:02:54PM +0100, Enrico Weigelt, metux IT consult wrote:
> Formatting of Kconfig files doesn't look so pretty, so just
> take damp cloth and clean it up.
Oops, I didn't notice that this was a v2. I first thought this was a
2/2 patch. Sorry for the noise. Most of my comments still apply
here, I think.
> Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
> ---
> drivers/ntb/Kconfig | 20 ++++++++++----------
> drivers/ntb/hw/amd/Kconfig | 4 ++--
> drivers/ntb/hw/idt/Kconfig | 41 ++++++++++++++++++++---------------------
> drivers/ntb/hw/intel/Kconfig | 4 ++--
> drivers/ntb/hw/mscc/Kconfig | 8 ++++----
> drivers/ntb/test/Kconfig | 26 +++++++++++++-------------
> 6 files changed, 51 insertions(+), 52 deletions(-)
>
> diff --git a/drivers/ntb/Kconfig b/drivers/ntb/Kconfig
> index 95944e5..5ce3fdd 100644
> --- a/drivers/ntb/Kconfig
> +++ b/drivers/ntb/Kconfig
> @@ -2,13 +2,13 @@ menuconfig NTB
> tristate "Non-Transparent Bridge support"
> depends on PCI
> help
> - The PCI-E Non-transparent bridge hardware is a point-to-point PCI-E bus
> - connecting 2 systems. When configured, writes to the device's PCI
> - mapped memory will be mirrored to a buffer on the remote system. The
> - ntb Linux driver uses this point-to-point communication as a method to
> - transfer data from one system to the other.
> + The PCI-E Non-transparent bridge hardware is a point-to-point PCI-E bus
> + connecting 2 systems. When configured, writes to the device's PCI
> + mapped memory will be mirrored to a buffer on the remote system. The
> + ntb Linux driver uses this point-to-point communication as a method to
> + transfer data from one system to the other.
>
> - If unsure, say N.
> + If unsure, say N.
>
> if NTB
>
> @@ -19,10 +19,10 @@ source "drivers/ntb/test/Kconfig"
> config NTB_TRANSPORT
> tristate "NTB Transport Client"
> help
> - This is a transport driver that enables connected systems to exchange
> - messages over the ntb hardware. The transport exposes a queue pair api
> - to client drivers.
> + This is a transport driver that enables connected systems to exchange
> + messages over the ntb hardware. The transport exposes a queue pair api
> + to client drivers.
>
> - If unsure, say N.
> + If unsure, say N.
>
> endif # NTB
> diff --git a/drivers/ntb/hw/amd/Kconfig b/drivers/ntb/hw/amd/Kconfig
> index cfe903c..9a90f17 100644
> --- a/drivers/ntb/hw/amd/Kconfig
> +++ b/drivers/ntb/hw/amd/Kconfig
> @@ -2,6 +2,6 @@ config NTB_AMD
> tristate "AMD Non-Transparent Bridge support"
> depends on X86_64
> help
> - This driver supports AMD NTB on capable Zeppelin hardware.
> + This driver supports AMD NTB on capable Zeppelin hardware.
>
> - If unsure, say N.
> + If unsure, say N.
> diff --git a/drivers/ntb/hw/idt/Kconfig b/drivers/ntb/hw/idt/Kconfig
> index f8948cf..5d106ac 100644
> --- a/drivers/ntb/hw/idt/Kconfig
> +++ b/drivers/ntb/hw/idt/Kconfig
> @@ -3,28 +3,27 @@ config NTB_IDT
> depends on PCI
> select HWMON
> help
> - This driver supports NTB of cappable IDT PCIe-switches.
> + This driver supports NTB of cappable IDT PCIe-switches.
>
> - Some of the pre-initializations must be made before IDT PCIe-switch
> - exposes it NT-functions correctly. It should be done by either proper
> - initialisation of EEPROM connected to master smbus of the switch or
> - by BIOS using slave-SMBus interface changing corresponding registers
> - value. Evidently it must be done before PCI bus enumeration is
> - finished in Linux kernel.
> + Some of the pre-initializations must be made before IDT PCIe-switch
> + exposes it NT-functions correctly. It should be done by either proper
> + initialisation of EEPROM connected to master smbus of the switch or
> + by BIOS using slave-SMBus interface changing corresponding registers
> + value. Evidently it must be done before PCI bus enumeration is
> + finished in Linux kernel.
>
> - First of all partitions must be activated and properly assigned to all
> - the ports with NT-functions intended to be activated (see SWPARTxCTL
> - and SWPORTxCTL registers). Then all NT-function BARs must be enabled
> - with chosen valid aperture. For memory windows related BARs the
> - aperture settings shall determine the maximum size of memory windows
> - accepted by a BAR. Note that BAR0 must map PCI configuration space
> - registers.
> + First of all partitions must be activated and properly assigned to all
> + the ports with NT-functions intended to be activated (see SWPARTxCTL
> + and SWPORTxCTL registers). Then all NT-function BARs must be enabled
> + with chosen valid aperture. For memory windows related BARs the
> + aperture settings shall determine the maximum size of memory windows
> + accepted by a BAR. Note that BAR0 must map PCI configuration space
> + registers.
>
> - It's worth to note, that since a part of this driver relies on the
> - BAR settings of peer NT-functions, the BAR setups can't be done over
> - kernel PCI fixups. That's why the alternative pre-initialization
> - techniques like BIOS using SMBus interface or EEPROM should be
> - utilized.
> -
> - If unsure, say N.
> + It's worth to note, that since a part of this driver relies on the
> + BAR settings of peer NT-functions, the BAR setups can't be done over
> + kernel PCI fixups. That's why the alternative pre-initialization
> + techniques like BIOS using SMBus interface or EEPROM should be
> + utilized.
>
> + If unsure, say N.
> diff --git a/drivers/ntb/hw/intel/Kconfig b/drivers/ntb/hw/intel/Kconfig
> index 91f995e..c166d50 100644
> --- a/drivers/ntb/hw/intel/Kconfig
> +++ b/drivers/ntb/hw/intel/Kconfig
> @@ -2,6 +2,6 @@ config NTB_INTEL
> tristate "Intel Non-Transparent Bridge support"
> depends on X86_64
> help
> - This driver supports Intel NTB on capable Xeon and Atom hardware.
> + This driver supports Intel NTB on capable Xeon and Atom hardware.
>
> - If unsure, say N.
> + If unsure, say N.
> diff --git a/drivers/ntb/hw/mscc/Kconfig b/drivers/ntb/hw/mscc/Kconfig
> index 013ed67..74ec015 100644
> --- a/drivers/ntb/hw/mscc/Kconfig
> +++ b/drivers/ntb/hw/mscc/Kconfig
> @@ -2,8 +2,8 @@ config NTB_SWITCHTEC
> tristate "MicroSemi Switchtec Non-Transparent Bridge Support"
> select PCI_SW_SWITCHTEC
> help
> - Enables NTB support for Switchtec PCI switches. This also
> - selects the Switchtec management driver as they share the same
> - hardware interface.
> + Enables NTB support for Switchtec PCI switches. This also
> + selects the Switchtec management driver as they share the same
> + hardware interface.
>
> - If unsure, say N.
> + If unsure, say N.
> diff --git a/drivers/ntb/test/Kconfig b/drivers/ntb/test/Kconfig
> index a5d0eda..416ab56 100644
> --- a/drivers/ntb/test/Kconfig
> +++ b/drivers/ntb/test/Kconfig
> @@ -1,27 +1,27 @@
> config NTB_PINGPONG
> tristate "NTB Ping Pong Test Client"
> help
> - This is a simple ping pong driver that exercises the scratchpads and
> - doorbells of the ntb hardware. This driver may be used to test that
> - your ntb hardware and drivers are functioning at a basic level.
> + This is a simple ping pong driver that exercises the scratchpads and
> + doorbells of the ntb hardware. This driver may be used to test that
> + your ntb hardware and drivers are functioning at a basic level.
>
> - If unsure, say N.
> + If unsure, say N.
>
> config NTB_TOOL
> tristate "NTB Debugging Tool Test Client"
> help
> - This is a simple debugging driver that enables the doorbell and
> - scratchpad registers to be read and written from the debugfs. This
> - enables more complicated debugging to be scripted from user space.
> - This driver may be used to test that your ntb hardware and drivers are
> - functioning at a basic level.
> + This is a simple debugging driver that enables the doorbell and
> + scratchpad registers to be read and written from the debugfs. This
> + enables more complicated debugging to be scripted from user space.
> + This driver may be used to test that your ntb hardware and drivers are
> + functioning at a basic level.
>
> - If unsure, say N.
> + If unsure, say N.
>
> config NTB_PERF
> tristate "NTB RAW Perf Measuring Tool"
> help
> - This is a tool to measure raw NTB performance by transferring data
> - to and from the window without additional software interaction.
> + This is a tool to measure raw NTB performance by transferring data
> + to and from the window without additional software interaction.
>
> - If unsure, say N.
> + If unsure, say N.
> --
> 1.9.1
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] drivers: ntb: Kconfig: pedantic cleanups
2019-03-20 21:15 ` Bjorn Helgaas
@ 2019-03-28 11:45 ` Jon Mason
0 siblings, 0 replies; 5+ messages in thread
From: Jon Mason @ 2019-03-28 11:45 UTC (permalink / raw)
To: Bjorn Helgaas
Cc: Enrico Weigelt, metux IT consult, linux-kernel, Kurt Schwemmer,
Logan Gunthorpe, linux-ntb, linux-pci, Dave Jiang, Allen Hubbe
On Wed, Mar 20, 2019 at 9:15 PM Bjorn Helgaas <helgaas@kernel.org> wrote:
>
> [+cc Jon, Dave, Allen]
>
> On Wed, Mar 06, 2019 at 11:02:54PM +0100, Enrico Weigelt, metux IT consult wrote:
> > Formatting of Kconfig files doesn't look so pretty, so just
> > take damp cloth and clean it up.
>
> Oops, I didn't notice that this was a v2. I first thought this was a
> 2/2 patch. Sorry for the noise. Most of my comments still apply
> here, I think.
I 100% agree with the comments form Bjorn. Though most of the
comments were from previous versions, but if you are going to change
it then might as well make it right. :)
Assuming you make the changes, I'll pull it in.
Thanks,
Jon
>
> > Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
> > ---
> > drivers/ntb/Kconfig | 20 ++++++++++----------
> > drivers/ntb/hw/amd/Kconfig | 4 ++--
> > drivers/ntb/hw/idt/Kconfig | 41 ++++++++++++++++++++---------------------
> > drivers/ntb/hw/intel/Kconfig | 4 ++--
> > drivers/ntb/hw/mscc/Kconfig | 8 ++++----
> > drivers/ntb/test/Kconfig | 26 +++++++++++++-------------
> > 6 files changed, 51 insertions(+), 52 deletions(-)
> >
> > diff --git a/drivers/ntb/Kconfig b/drivers/ntb/Kconfig
> > index 95944e5..5ce3fdd 100644
> > --- a/drivers/ntb/Kconfig
> > +++ b/drivers/ntb/Kconfig
> > @@ -2,13 +2,13 @@ menuconfig NTB
> > tristate "Non-Transparent Bridge support"
> > depends on PCI
> > help
> > - The PCI-E Non-transparent bridge hardware is a point-to-point PCI-E bus
> > - connecting 2 systems. When configured, writes to the device's PCI
> > - mapped memory will be mirrored to a buffer on the remote system. The
> > - ntb Linux driver uses this point-to-point communication as a method to
> > - transfer data from one system to the other.
> > + The PCI-E Non-transparent bridge hardware is a point-to-point PCI-E bus
> > + connecting 2 systems. When configured, writes to the device's PCI
> > + mapped memory will be mirrored to a buffer on the remote system. The
> > + ntb Linux driver uses this point-to-point communication as a method to
> > + transfer data from one system to the other.
> >
> > - If unsure, say N.
> > + If unsure, say N.
> >
> > if NTB
> >
> > @@ -19,10 +19,10 @@ source "drivers/ntb/test/Kconfig"
> > config NTB_TRANSPORT
> > tristate "NTB Transport Client"
> > help
> > - This is a transport driver that enables connected systems to exchange
> > - messages over the ntb hardware. The transport exposes a queue pair api
> > - to client drivers.
> > + This is a transport driver that enables connected systems to exchange
> > + messages over the ntb hardware. The transport exposes a queue pair api
> > + to client drivers.
> >
> > - If unsure, say N.
> > + If unsure, say N.
> >
> > endif # NTB
> > diff --git a/drivers/ntb/hw/amd/Kconfig b/drivers/ntb/hw/amd/Kconfig
> > index cfe903c..9a90f17 100644
> > --- a/drivers/ntb/hw/amd/Kconfig
> > +++ b/drivers/ntb/hw/amd/Kconfig
> > @@ -2,6 +2,6 @@ config NTB_AMD
> > tristate "AMD Non-Transparent Bridge support"
> > depends on X86_64
> > help
> > - This driver supports AMD NTB on capable Zeppelin hardware.
> > + This driver supports AMD NTB on capable Zeppelin hardware.
> >
> > - If unsure, say N.
> > + If unsure, say N.
> > diff --git a/drivers/ntb/hw/idt/Kconfig b/drivers/ntb/hw/idt/Kconfig
> > index f8948cf..5d106ac 100644
> > --- a/drivers/ntb/hw/idt/Kconfig
> > +++ b/drivers/ntb/hw/idt/Kconfig
> > @@ -3,28 +3,27 @@ config NTB_IDT
> > depends on PCI
> > select HWMON
> > help
> > - This driver supports NTB of cappable IDT PCIe-switches.
> > + This driver supports NTB of cappable IDT PCIe-switches.
> >
> > - Some of the pre-initializations must be made before IDT PCIe-switch
> > - exposes it NT-functions correctly. It should be done by either proper
> > - initialisation of EEPROM connected to master smbus of the switch or
> > - by BIOS using slave-SMBus interface changing corresponding registers
> > - value. Evidently it must be done before PCI bus enumeration is
> > - finished in Linux kernel.
> > + Some of the pre-initializations must be made before IDT PCIe-switch
> > + exposes it NT-functions correctly. It should be done by either proper
> > + initialisation of EEPROM connected to master smbus of the switch or
> > + by BIOS using slave-SMBus interface changing corresponding registers
> > + value. Evidently it must be done before PCI bus enumeration is
> > + finished in Linux kernel.
> >
> > - First of all partitions must be activated and properly assigned to all
> > - the ports with NT-functions intended to be activated (see SWPARTxCTL
> > - and SWPORTxCTL registers). Then all NT-function BARs must be enabled
> > - with chosen valid aperture. For memory windows related BARs the
> > - aperture settings shall determine the maximum size of memory windows
> > - accepted by a BAR. Note that BAR0 must map PCI configuration space
> > - registers.
> > + First of all partitions must be activated and properly assigned to all
> > + the ports with NT-functions intended to be activated (see SWPARTxCTL
> > + and SWPORTxCTL registers). Then all NT-function BARs must be enabled
> > + with chosen valid aperture. For memory windows related BARs the
> > + aperture settings shall determine the maximum size of memory windows
> > + accepted by a BAR. Note that BAR0 must map PCI configuration space
> > + registers.
> >
> > - It's worth to note, that since a part of this driver relies on the
> > - BAR settings of peer NT-functions, the BAR setups can't be done over
> > - kernel PCI fixups. That's why the alternative pre-initialization
> > - techniques like BIOS using SMBus interface or EEPROM should be
> > - utilized.
> > -
> > - If unsure, say N.
> > + It's worth to note, that since a part of this driver relies on the
> > + BAR settings of peer NT-functions, the BAR setups can't be done over
> > + kernel PCI fixups. That's why the alternative pre-initialization
> > + techniques like BIOS using SMBus interface or EEPROM should be
> > + utilized.
> >
> > + If unsure, say N.
> > diff --git a/drivers/ntb/hw/intel/Kconfig b/drivers/ntb/hw/intel/Kconfig
> > index 91f995e..c166d50 100644
> > --- a/drivers/ntb/hw/intel/Kconfig
> > +++ b/drivers/ntb/hw/intel/Kconfig
> > @@ -2,6 +2,6 @@ config NTB_INTEL
> > tristate "Intel Non-Transparent Bridge support"
> > depends on X86_64
> > help
> > - This driver supports Intel NTB on capable Xeon and Atom hardware.
> > + This driver supports Intel NTB on capable Xeon and Atom hardware.
> >
> > - If unsure, say N.
> > + If unsure, say N.
> > diff --git a/drivers/ntb/hw/mscc/Kconfig b/drivers/ntb/hw/mscc/Kconfig
> > index 013ed67..74ec015 100644
> > --- a/drivers/ntb/hw/mscc/Kconfig
> > +++ b/drivers/ntb/hw/mscc/Kconfig
> > @@ -2,8 +2,8 @@ config NTB_SWITCHTEC
> > tristate "MicroSemi Switchtec Non-Transparent Bridge Support"
> > select PCI_SW_SWITCHTEC
> > help
> > - Enables NTB support for Switchtec PCI switches. This also
> > - selects the Switchtec management driver as they share the same
> > - hardware interface.
> > + Enables NTB support for Switchtec PCI switches. This also
> > + selects the Switchtec management driver as they share the same
> > + hardware interface.
> >
> > - If unsure, say N.
> > + If unsure, say N.
> > diff --git a/drivers/ntb/test/Kconfig b/drivers/ntb/test/Kconfig
> > index a5d0eda..416ab56 100644
> > --- a/drivers/ntb/test/Kconfig
> > +++ b/drivers/ntb/test/Kconfig
> > @@ -1,27 +1,27 @@
> > config NTB_PINGPONG
> > tristate "NTB Ping Pong Test Client"
> > help
> > - This is a simple ping pong driver that exercises the scratchpads and
> > - doorbells of the ntb hardware. This driver may be used to test that
> > - your ntb hardware and drivers are functioning at a basic level.
> > + This is a simple ping pong driver that exercises the scratchpads and
> > + doorbells of the ntb hardware. This driver may be used to test that
> > + your ntb hardware and drivers are functioning at a basic level.
> >
> > - If unsure, say N.
> > + If unsure, say N.
> >
> > config NTB_TOOL
> > tristate "NTB Debugging Tool Test Client"
> > help
> > - This is a simple debugging driver that enables the doorbell and
> > - scratchpad registers to be read and written from the debugfs. This
> > - enables more complicated debugging to be scripted from user space.
> > - This driver may be used to test that your ntb hardware and drivers are
> > - functioning at a basic level.
> > + This is a simple debugging driver that enables the doorbell and
> > + scratchpad registers to be read and written from the debugfs. This
> > + enables more complicated debugging to be scripted from user space.
> > + This driver may be used to test that your ntb hardware and drivers are
> > + functioning at a basic level.
> >
> > - If unsure, say N.
> > + If unsure, say N.
> >
> > config NTB_PERF
> > tristate "NTB RAW Perf Measuring Tool"
> > help
> > - This is a tool to measure raw NTB performance by transferring data
> > - to and from the window without additional software interaction.
> > + This is a tool to measure raw NTB performance by transferring data
> > + to and from the window without additional software interaction.
> >
> > - If unsure, say N.
> > + If unsure, say N.
> > --
> > 1.9.1
> >
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-03-28 11:45 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-06 18:51 [PATCH] drivers: ntb: Kconfig: pedantic cleanups Enrico Weigelt, metux IT consult
2019-03-06 22:02 ` [PATCH v2] " Enrico Weigelt, metux IT consult
2019-03-20 21:15 ` Bjorn Helgaas
2019-03-28 11:45 ` Jon Mason
2019-03-20 21:13 ` [PATCH] " Bjorn Helgaas
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).