Further mvebu PCIe patches
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest0 of 9 patches 0 repliesNew
Patches in v1 No applies recorded yet
None of the 9 patches the cover letter announces are in this archive.
Reviews trailers per patch
| Patch | Thomas Petazzoni |
|---|---|
| Series | R-b T-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH 0/9] Further mvebu PCIe patches
Rusell, all,
Quoted 17 lines · Russell King - ARM Linux
Here are further PCIe patches which follow on from the previous set of six I sent earlier in September. This set:
* Separates the DT parsing from the use of this parsed data.
* Fixes memory leaks fromkasprintf()and refcount leaks of the DT node where we break out from the loop early.
* Usesgpio_set_value_cansleep()so that GPIOs on I2C expanders can be used for the PCIe reset functionality.
* Switch to usingdevm_kcalloc()instead ofdevm_kzalloc(), which eliminates the multiplication, moving it into core code.
* Switch to using a gpio descriptor for the reset gpio, which can contain the active level information from DT. (It would be nice if gpiolib automated some of the resource claiming there.)
* Make PERST# vs clock timing to match PCIe specifications. PCIe specs require the clock to be running for 100us prior to releasing reset.
* Add the standard PCIe capability block in root port form to the
Quoted 8 lines
emulated PCIe configuration block. This allows the PCI layer to identify the "host bridge" as a PCIe device, and allows us to take advantage of the code in
drivers/pci/pcie, particularly aspm for link power management.
* As a result of identifying ourselves as a PCIe root port, this eliminates the need to special case accesses to non-slot 0 in the driver; the lack of other "slots" is something which the generic PCI code knows about for PCIe root ports.
Entire series tested on Armada XP GP, with 3 PCIe cards plugged in. They seem to work, and the lspci -v output seems sane. I've also reviewed the patch series, and it looks good to me.
Thanks!
Thomas
Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com