[PATCH v4 0/4] PCI: Add delay macros for better code readability and maintainability

Hans Zhang posted 4 patches 3 months, 1 week ago
drivers/pci/hotplug/pciehp_hpc.c |  7 +++++--
drivers/pci/pci.c                | 11 +++++------
drivers/pci/pci.h                |  3 +++
drivers/pci/pcie/dpc.c           |  4 +++-
4 files changed, 16 insertions(+), 9 deletions(-)
[PATCH v4 0/4] PCI: Add delay macros for better code readability and maintainability
Posted by Hans Zhang 3 months, 1 week ago
This series improves code readability and maintainability in the PCI
subsystem by replacing hard-coded delay values with descriptive macros.

---
Hi Bjorn,

I wonder if there is still a need to advance this series? If it's not necessary,
please drop it. Thank you very much.

Best regards
Hans


Changes for v4:
https://patchwork.kernel.org/project/linux-pci/patch/20250826170315.721551-1-18255117159@163.com/

- According to Bjorn's feedback, the benefits of using fsleep are not significant.
  drop the 0002 patch in v3. (Bjorn)
- For the controller drivers, the added macros do no good and provide no value.
  So if you ever respin this series, you can drop them. (Mani)

Changes for v3:
https://patchwork.kernel.org/project/linux-pci/cover/20250822155908.625553-1-18255117159@163.com/

- According to Bjorn's suggestion, split the first patch of v2 and add
  macro definitions to the remaining patches.

Changes for v2:
https://patchwork.kernel.org/project/linux-pci/patch/20250820160944.489061-1-18255117159@163.com/

- According to the Maintainer's suggestion, it was modified to fsleep,
  usleep_range, and macro definitions were used instead of hard code. (Bjorn)
---

Hans Zhang (4):
  PCI: Add macro for secondary bus reset delay
  PCI: Add macro for link status check delay
  PCI: pciehp: Add macros for hotplug operation delays
  PCI/DPC: Add macro for RP busy check delay

 drivers/pci/hotplug/pciehp_hpc.c |  7 +++++--
 drivers/pci/pci.c                | 11 +++++------
 drivers/pci/pci.h                |  3 +++
 drivers/pci/pcie/dpc.c           |  4 +++-
 4 files changed, 16 insertions(+), 9 deletions(-)


base-commit: ba36dd5ee6fd4643ebbf6ee6eefcecf0b07e35c7
-- 
2.34.1