From: Keith Busch <keith.busch@intel.com>
To: Bjorn Helgaas <helgaas@kernel.org>
Cc: linux-pci@vger.kernel.org, Borislav Petkov <bp@alien8.de>,
Oza Pawandeep <poza@codeaurora.org>,
linux-kernel@vger.kernel.org
Subject: Re: [RFC PATCH v1 00/13] PCI/AER: Squash into one file
Date: Fri, 8 Jun 2018 14:23:03 -0600 [thread overview]
Message-ID: <20180608202303.GB24554@localhost.localdomain> (raw)
In-Reply-To: <152848785553.11888.12243539903985770441.stgit@bhelgaas-glaptop.roam.corp.google.com>
On Fri, Jun 08, 2018 at 03:04:45PM -0500, Bjorn Helgaas wrote:
> The AER code is scattered across several files and buried a little too deep
> in the hierarchy, which makes it a bit of a hassle to browse. Consolidate
> it all in a single file (plus the optional aer_inject module) and move it
> up one level to drivers/pci/pcie/aer.c.
>
> This is all pure code moves; no functional changes intended.
Thanks for doing this. I got a head start on reviewing when I saw this
merged to your tree earlier today, and the driver is much more pleasant
to read than before!
Reviewed-by: Keith Busch <keith.busch@intel.com>
next prev parent reply other threads:[~2018-06-08 20:23 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-08 20:04 [RFC PATCH v1 00/13] PCI/AER: Squash into one file Bjorn Helgaas
2018-06-08 20:04 ` [PATCH v1 01/13] PCI/AER: Remove forward declarations Bjorn Helgaas
2018-06-08 20:04 ` [PATCH v1 02/13] PCI/AER: Reorder code to group probe/remove stuff together Bjorn Helgaas
2018-06-08 20:05 ` [PATCH v1 03/13] PCI/AER: Squash aerdrv_core.c into aerdrv.c Bjorn Helgaas
2018-06-08 20:05 ` [PATCH v1 04/13] PCI/AER: Squash aerdrv_errprint.c " Bjorn Helgaas
2018-06-08 20:05 ` [PATCH v1 05/13] PCI/AER: Squash aerdrv_acpi.c " Bjorn Helgaas
2018-06-08 20:05 ` [PATCH v1 06/13] PCI/AER: Squash ecrc.c " Bjorn Helgaas
2018-06-08 20:05 ` [PATCH v1 07/13] PCI/AER: Remove duplicate pcie_port_bus_type declaration Bjorn Helgaas
2018-06-08 20:05 ` [PATCH v1 08/13] PCI/AER: Move pcie_aer_get_firmware_first() to portdrv.h Bjorn Helgaas
2018-06-08 20:05 ` [PATCH v1 09/13] PCI/AER: Move aer_irq() declaration " Bjorn Helgaas
2018-06-08 20:17 ` Keith Busch
2018-06-08 20:34 ` Bjorn Helgaas
2018-06-08 20:05 ` [PATCH v1 10/13] PCI/AER: Move private AER things to aerdrv.c Bjorn Helgaas
2018-06-08 20:05 ` [PATCH v1 11/13] PCI/AER: Squash Kconfig.debug into Kconfig Bjorn Helgaas
2018-06-08 20:05 ` [PATCH v1 12/13] PCI/AER: Hoist aerdrv.c, aer_inject.c up to drivers/pci/pcie/ Bjorn Helgaas
2018-06-08 20:06 ` [PATCH v1 13/13] PCI/AER: Use "PCI Express" consistently in Kconfig text Bjorn Helgaas
2018-06-08 20:23 ` Keith Busch [this message]
2018-06-11 13:26 ` [RFC PATCH v1 00/13] PCI/AER: Squash into one file 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=20180608202303.GB24554@localhost.localdomain \
--to=keith.busch@intel.com \
--cc=bp@alien8.de \
--cc=helgaas@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=poza@codeaurora.org \
/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 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.