linux-kernel - Re: [PATCH v12 0/6] Altera PCIe host controller driver with MSI support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20151027142644.GA8660@localhost>
Date:	Tue, 27 Oct 2015 09:26:44 -0500
From:	Bjorn Helgaas <helgaas@...nel.org>
To:	Ley Foon Tan <lftan@...era.com>
Cc:	Bjorn Helgaas <bhelgaas@...gle.com>,
	Russell King <linux@....linux.org.uk>,
	Marc Zyngier <marc.zyngier@....com>,
	Arnd Bergmann <arnd@...db.de>,
	Dinh Nguyen <dinguyen@...nsource.altera.com>,
	linux-pci@...r.kernel.org, devicetree@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org, linux-doc@...r.kernel.org,
	linux-kernel@...r.kernel.org, lftan.linux@...il.com,
	Rob Herring <robh+dt@...nel.org>,
	Pawel Moll <pawel.moll@....com>,
	Mark Rutland <mark.rutland@....com>,
	Ian Campbell <ijc+devicetree@...lion.org.uk>,
	Kumar Gala <galak@...eaurora.org>,
	Lorenzo Pieralisi <lorenzo.pieralisi@....com>
Subject: Re: [PATCH v12 0/6] Altera PCIe host controller driver with MSI
 support

Hi Ley,

On Fri, Oct 23, 2015 at 06:27:09PM +0800, Ley Foon Tan wrote:
> This is the 12th version of patch set to add support for Altera PCIe host
> controller with MSI feature on Altera FPGA device families. This patchset
> mainly resovle the comments from Bjorn.
> 
> 
> This patchset is based on v4.3-rc6.
> 
> v11->v12 changes:
> - pcie-altera: use DECLARE_PCI_FIXUP_EARLY for pcie retrain fixup
> - pcie-altera: move pcie_bus_configure_settings before pci_bus_add_devices
> - pcie-altera: add %d for irq bit in dev_err()
> - pcie-altera: prevent enumeration of root complex resources in config accessors
> - Documentation: add Acked-by from Rob Herring
> 
> History:
> -------
> [v1]: https://lkml.org/lkml/2015/7/28/395
> [v2]: https://lkml.org/lkml/2015/7/31/267
> [v3]: http://www.kernelhub.org/?msg=811940&p=2
> [v4]: https://lkml.org/lkml/2015/8/17/141
> [v5]: https://lkml.org/lkml/2015/8/25/238
> [v6]: https://lkml.org/lkml/2015/9/1/177
> [v7]: https://lkml.org/lkml/2015/9/20/193
> [v8]: http://www.kernelhub.org/?msg=853553&p=2
> [v9]: https://lkml.org/lkml/2015/10/13/998
> [v10]: https://lkml.org/lkml/2015/10/19/139
> [v11]: https://lkml.org/lkml/2015/10/22/206
> 
> 
> Ley Foon Tan (6):
>   arm: add msi.h to Kbuild
>   pci: add Altera PCI vendor ID
>   pci:host: Add Altera PCIe host controller driver
>   pci: altera: Add Altera PCIe MSI driver
>   Documentation: dt-bindings: pci: altera pcie device tree binding
>   MAINTAINERS: Add Altera PCIe and MSI drivers maintainer
> 
>  .../devicetree/bindings/pci/altera-pcie-msi.txt    |  28 +
>  .../devicetree/bindings/pci/altera-pcie.txt        |  49 ++
>  MAINTAINERS                                        |  16 +
>  arch/arm/include/asm/Kbuild                        |   1 +
>  drivers/pci/host/Kconfig                           |  16 +
>  drivers/pci/host/Makefile                          |   2 +
>  drivers/pci/host/pcie-altera-msi.c                 | 314 +++++++++++
>  drivers/pci/host/pcie-altera.c                     | 580 +++++++++++++++++++++
>  include/linux/pci_ids.h                            |   2 +
>  9 files changed, 1008 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/pci/altera-pcie-msi.txt
>  create mode 100644 Documentation/devicetree/bindings/pci/altera-pcie.txt
>  create mode 100644 drivers/pci/host/pcie-altera-msi.c
>  create mode 100644 drivers/pci/host/pcie-altera.c

I applied these to pci/host-altera for v4.4, thanks!

I squashed these into three patches:

  - add msi.h to Kbuild
  - add Altera host driver (including DT binding and MAINTAINERS update)
  - add Altera MSI driver (including DT binding and MAINTAINERS update)

Since the convention is to add PCI ID #defines only when they are used in
multiple places, I replaced PCI_VENDOR_ID_ALTERA with 0x1172 in the single
place it is used.

Bjorn
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/