lkml.org 
[lkml]   [2019]   [Mar]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v2] drivers: ntb: Kconfig: pedantic cleanups
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
> >

\
 
 \ /
  Last update: 2019-03-28 12:47    [W:0.047 / U:0.092 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site