From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Gustavo Pimentel <Gustavo.Pimentel@synopsys.com>
Cc: linux-doc@vger.kernel.org, linux-pci@vger.kernel.org,
linux-kernel@vger.kernel.org,
"Derek Kiernan" <derek.kiernan@xilinx.com>,
"Dragan Cvetic" <dragan.cvetic@xilinx.com>,
"Arnd Bergmann" <arnd@arndb.de>,
"Andrew Morton" <akpm@linux-foundation.org>,
"Jonathan Corbet" <corbet@lwn.net>,
"Bjorn Helgaas" <bhelgaas@google.com>,
"Krzysztof Wilczyński" <kw@linux.com>,
"Stephen Rothwell" <sfr@canb.auug.org.au>
Subject: Re: [PATCH v11 0/4] misc: Add Synopsys DesignWare xData IP driver
Date: Tue, 6 Apr 2021 19:34:02 +0200 [thread overview]
Message-ID: <YGybivASLNlAymk0@kroah.com> (raw)
In-Reply-To: <cover.1617729785.git.gustavo.pimentel@synopsys.com>
On Tue, Apr 06, 2021 at 07:26:45PM +0200, Gustavo Pimentel wrote:
> This patch series adds a new driver called xData-pcie for the Synopsys
> DesignWare PCIe prototype.
>
> The driver configures and enables the Synopsys DesignWare PCIe traffic
> generator IP inside of prototype Endpoint which will generate upstream
> and downstream PCIe traffic. This allows to quickly test the PCIe link
> throughput speed and check is the prototype solution has some limitation
> or not.
>
> Changes:
> V2: Rework driver according to Greg Kroah-Hartman' feedback
> - Replace module parameter by sysfs use.
> - Replace bit fields structure with macros and masks use.
> - Removed SET() and GET() macros by the writel() and readl().
> - Removed some noisy info messages.
> V3: Fixed issues detected while running on 64 bits platforms
> Rebased patches on top of v5.11-rc1 version
> V4: Rework driver according to Greg Kroah-Hartman' feedback
> - Add the ABI doc related to the sysfs implemented on this driver
> V5: Rework driver accordingly to Leon Romanovsky' feedback
> - Removed "default n" on Kconfig
> Rework driver accordingly to Krzysztof Wilczyński' feedback
> - Added some explanatory comments for some steps
> - Added some bit defines instead of magic numbers
> V6: Rework driver according to Greg Kroah-Hartman' feedback
> - Squashed patches #2 and #3
> - Removed units (MB/s) on the sys file
> - Reduced mutex scope on the functions called by sysfs
> Rework driver accordingly to Krzysztof Wilczyński' feedback
> - Fix typo "DesignWare"
> V7: Rework driver according to Greg Kroah-Hartman' feedback
> - Created a sub device (misc device) that will be associated with the PCI driver
> - sysfs group is now associated with the misc drivers instead of the PCI driver
> V8: Rework driver according to Greg Kroah-Hartman' feedback
> - Added more detail to the version changes on the cover letter
> - Squashed patches #1 and #2
> - Removed struct device from the dw_xdata_pcie structure
> - Replaced the pci_*() use by dev_*()
> - Added free call for the misc driver name allocation
> - Added reference counting
> - Removed snps_edda_data structure and their usage
> Rebased patches on top of v5.12-rc4 version
> V9: Squashed temporary development patch #5 into the driver patch #1
> V10: Reworked the write_store() and read_store() to validate the input using kstrtobool()
> Removed stop_store()
> Update ABI documentation accordingly
> V11: Fixed the documentation based on the warnings detected by Stephen Rothwell
As mentioned on the other thread, I can't take these as I took your v10
into my public tree. Please send fixes on top of that.
thanks,
greg k-h
prev parent reply other threads:[~2021-04-06 17:34 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-06 17:26 [PATCH v11 0/4] misc: Add Synopsys DesignWare xData IP driver Gustavo Pimentel
2021-04-06 17:26 ` [PATCH v11 1/4] " Gustavo Pimentel
2021-04-06 17:26 ` [PATCH v11 2/4] Documentation: misc-devices: Add Documentation for dw-xdata-pcie driver Gustavo Pimentel
2021-04-06 17:26 ` [PATCH v11 3/4] MAINTAINERS: Add Synopsys xData IP driver maintainer Gustavo Pimentel
2021-04-06 17:26 ` [PATCH v11 4/4] docs: ABI: Add sysfs documentation interface of dw-xdata-pcie driver Gustavo Pimentel
2021-04-06 17:34 ` Greg Kroah-Hartman [this message]
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=YGybivASLNlAymk0@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=Gustavo.Pimentel@synopsys.com \
--cc=akpm@linux-foundation.org \
--cc=arnd@arndb.de \
--cc=bhelgaas@google.com \
--cc=corbet@lwn.net \
--cc=derek.kiernan@xilinx.com \
--cc=dragan.cvetic@xilinx.com \
--cc=kw@linux.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=sfr@canb.auug.org.au \
/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).