From: Jon Mason <jdmason@kudzu.us>
To: Bjorn Helgaas <helgaas@kernel.org>
Cc: "Enrico Weigelt, metux IT consult" <info@metux.net>,
linux-kernel <linux-kernel@vger.kernel.org>,
Kurt Schwemmer <kurt.schwemmer@microsemi.com>,
Logan Gunthorpe <logang@deltatee.com>,
linux-ntb <linux-ntb@googlegroups.com>,
linux-pci@vger.kernel.org, Dave Jiang <dave.jiang@intel.com>,
Allen Hubbe <allenbh@gmail.com>
Subject: Re: [PATCH v2] drivers: ntb: Kconfig: pedantic cleanups
Date: Thu, 28 Mar 2019 11:45:28 +0000 [thread overview]
Message-ID: <CAPoiz9x6bSdFpE-cuOBinE7WdNvzaD5R+tsweUqWe+KsL87GQw@mail.gmail.com> (raw)
In-Reply-To: <20190320211517.GG251185@google.com>
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
> >
next prev parent reply other threads:[~2019-03-28 11:45 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
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 [this message]
2019-03-20 21:13 ` [PATCH] " Bjorn Helgaas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAPoiz9x6bSdFpE-cuOBinE7WdNvzaD5R+tsweUqWe+KsL87GQw@mail.gmail.com \
--to=jdmason@kudzu.us \
--cc=allenbh@gmail.com \
--cc=dave.jiang@intel.com \
--cc=helgaas@kernel.org \
--cc=info@metux.net \
--cc=kurt.schwemmer@microsemi.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-ntb@googlegroups.com \
--cc=linux-pci@vger.kernel.org \
--cc=logang@deltatee.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).