On Tue, Jul 02, 2024 at 11:12:21AM +0200, Lorenzo Bianconi wrote: > > On Sat, Jun 29, 2024 at 03:51:54PM +0200, Lorenzo Bianconi wrote: > > > Introduce support for Airoha EN7581 PCIe controller to mediatek-gen3 > > > PCIe controller driver. > > > > > +/* PCIe reset line delay in ms */ > > > +#define PCIE_RESET_TIME_MS 100 > > > > Is this something required by the PCIe base spec, or is it specific to > > EN7581? Either way it would be nice to have a citation to the spec > > (revision and section number). If it's generic to PCIe, it should be > > in drivers/pci/pci.h so other drivers can use the same thing. > > It is just the time needed by the EN7581 reset controller to > complete the operation, it is not something PCIe generic (it is > something just related to EN7581 SoC). Do you think we should move > it in EN7581 reset controller codebase? I have no opinion about moving it. But it sounds like maybe it should have a less generic name so it doesn't look like a generic PCIe thing. And also a spec citation would be helpful for future maintenance.
