| Message ID | 20190401042547.14067-1-andrew.smirnov@gmail.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | i.MX6, DesignWare PCI improvements | expand |
On Sun, Mar 31, 2019 at 09:25:36PM -0700, Andrey Smirnov wrote: > Everyone: > > This is the series containing various small improvements that I made > while reading the code and researching commit history of pci-imx6.c > and pcie-designware*.c files. All changes are optional, so commits > that don't seem like an improvement can be easily dropped. Hopefully > each patch is self-explanatory. > > I tested this series on i.MX6Q, i.MX7D and i.MX8MQ. > > Feedback is welcome! > > Thanks, > Andrey Smirnov > > Changes since [v2]: > > - All non i.MX6 patches dropped, since they were accepted as a > seprarte series > > - Series rebased on lates 'dwc-pci' branch of PCI tree > > - Patches "PCI: imx6: Use flags to indicate support for suspend" > and "PCI: imx6: Replace calls to udelay() with usleep_range()" > added to the series > > Changes since [v1]: > > - Dropped "PCI: imx6: Drop imx6_pcie_link_up()" due to the matter > already having been addressed by "PCI: imx6: Fix link training > status detection in link up check" from Trent Piepho > > - Changed "designware" -> "dwc" for all subject lines > > - Collected Acked-by's from Gustavo Pimentel > > [v2] lkml.kernel.org/r/20190104174925.17153-1-andrew.smirnov@gmail.com > [v1] lkml.kernel.org/r/20181221072716.29017-1-andrew.smirnov@gmail.com > > Andrey Smirnov (11): > PCI: imx6: Simplify imx7d_pcie_wait_for_phy_pll_lock() > PCI: imx6: Remove redundant debug tracing > PCI: imx6: Return -ETIMEOUT from imx6_pcie_wait_for_speed_change() > PCI: imx6: Remove PCIE_PL_PFLR_* constants > PCI: dwc: imx6: Share PHY debug register definitions > PCI: imx6: Make use of BIT() in constant definitions > PCI: imx6: Simplify bit operations in PHY functions > PCI: imx6: Simplify pcie_phy_poll_ack() > PCI: imx6: Restrict PHY register data to 16-bit > PCI: imx6: Use flags to indicate support for suspend > PCI: imx6: Replace calls to udelay() with usleep_range() > > drivers/pci/controller/dwc/pci-imx6.c | 119 ++++++++----------- > drivers/pci/controller/dwc/pcie-designware.c | 12 +- > drivers/pci/controller/dwc/pcie-designware.h | 3 + > 3 files changed, 54 insertions(+), 80 deletions(-) I am waiting for Lucas's review/ACKs to proceed with this series. Thanks, Lorenzo
Everyone: This is the series containing various small improvements that I made while reading the code and researching commit history of pci-imx6.c and pcie-designware*.c files. All changes are optional, so commits that don't seem like an improvement can be easily dropped. Hopefully each patch is self-explanatory. I tested this series on i.MX6Q, i.MX7D and i.MX8MQ. Feedback is welcome! Thanks, Andrey Smirnov Changes since [v2]: - All non i.MX6 patches dropped, since they were accepted as a seprarte series - Series rebased on lates 'dwc-pci' branch of PCI tree - Patches "PCI: imx6: Use flags to indicate support for suspend" and "PCI: imx6: Replace calls to udelay() with usleep_range()" added to the series Changes since [v1]: - Dropped "PCI: imx6: Drop imx6_pcie_link_up()" due to the matter already having been addressed by "PCI: imx6: Fix link training status detection in link up check" from Trent Piepho - Changed "designware" -> "dwc" for all subject lines - Collected Acked-by's from Gustavo Pimentel [v2] lkml.kernel.org/r/20190104174925.17153-1-andrew.smirnov@gmail.com [v1] lkml.kernel.org/r/20181221072716.29017-1-andrew.smirnov@gmail.com Andrey Smirnov (11): PCI: imx6: Simplify imx7d_pcie_wait_for_phy_pll_lock() PCI: imx6: Remove redundant debug tracing PCI: imx6: Return -ETIMEOUT from imx6_pcie_wait_for_speed_change() PCI: imx6: Remove PCIE_PL_PFLR_* constants PCI: dwc: imx6: Share PHY debug register definitions PCI: imx6: Make use of BIT() in constant definitions PCI: imx6: Simplify bit operations in PHY functions PCI: imx6: Simplify pcie_phy_poll_ack() PCI: imx6: Restrict PHY register data to 16-bit PCI: imx6: Use flags to indicate support for suspend PCI: imx6: Replace calls to udelay() with usleep_range() drivers/pci/controller/dwc/pci-imx6.c | 119 ++++++++----------- drivers/pci/controller/dwc/pcie-designware.c | 12 +- drivers/pci/controller/dwc/pcie-designware.h | 3 + 3 files changed, 54 insertions(+), 80 deletions(-)