Further mvebu PCIe patches
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest2 of 9 patches 0 repliesNew
2 of 9 patches are in this archive. The cover letter announces 9.
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 2/9 | pci: mvebu: fix memory leaks and refcount leaks | +0 | 2 replies | |
| 2/9 | pci: mvebu: fix memory leaks and refcount leaks | +0 | 4 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCH 0/9] Further mvebu PCIe patches
Hello Russell,
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.
Thanks for these patches! They look good. However, I'm away for the Embedded Linux Conference Europe in Dublin right now, and therefore don't have access to my boards to test that it works OK on platforms other than Armada 38x (which you tested). I will try to do this testing next week, hopefully on Friday.
Thanks!
Thomas
Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com