[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20251101160538.10055-1-18255117159@163.com>
Date: Sun, 2 Nov 2025 00:05:34 +0800
From: Hans Zhang <18255117159@....com>
To: bhelgaas@...gle.com,
helgaas@...nel.org,
linux-pci@...r.kernel.org,
linux-kernel@...r.kernel.org
Cc: Hans Zhang <18255117159@....com>
Subject: [PATCH v4 0/4] PCI: Add delay macros for better code readability and maintainability
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
Powered by blists - more mailing lists