These are cleanups against 2098142ae87d, the current pci/host-rockchip head in my tree. This is basically the v10 driver posting (it's not identical because I had already tweaked a few things and squashed in Brian's bug fix). These haven't been compiled so I'm sure the 0-day builder will find some errors. After fixing those, I intend to squash everything into one commit, since this driver hasn't been merged yet. I'm just posting these split out patches in case anybody wants to review them. These address most of Guenter's review comments. I didn't change anything related to -EPROBE_DEFER because it wasn't obvious to me what to do. I also made several other changes -- mostly renaming functions and variables so they're more consistent with other PCI host drivers. These are in git as pci/host-rockchip-wip. --- Bjorn Helgaas (9): Remove unused symbols, unnecessary parens, other minor comments from Rename pcie_read() and pcie_write() to rockchip_pcie_read() and Always use "rockchip" as the pointer to per-device struct. Rename struct rockchip_pcie_port to struct rockchip_pcie. Use a local "dev" to avoid repetition of "rockchip->dev". Add comment about why 32-bit read/modify/write isn't safe. Simplify the confusing HIWORD_UPDATE scheme. Simplify testing of link status and speed testing. Move msleeps to address Guenter's comments. drivers/pci/host/pcie-rockchip.c | 680 +++++++++++++++++--------------------- 1 file changed, 307 insertions(+), 373 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html