Hi Markus, Thanks for the review and reminding! On Wed, Feb 05, 2025 at 06:48:35PM +0100, Markus Elfring wrote: > … > > Add the necessary wait to comply with PCIe spec. The waiting logic refers > > existing pcie_poll_cmd(). > > * How do you think about to add any tags (like “Fixes” and “Cc”) accordingly? The code went through some refactor, and the related commit should be: commit 2bd50dd800b52245294cfceb56be62020cdc7515 Author: Rafael J. Wysocki <rjw@xxxxxxxxxxxxx> Date: Sat Aug 21 01:57:39 2010 +0200 PCI: PCIe: Disable PCIe port services during port initialization In principle PCIe port services may be enabled by the BIOS, so it's better to disable them during port initialization to avoid spurious events from being generated. Signed-off-by: Rafael J. Wysocki <rjw@xxxxxxx> Reviewed-by: Hidetoshi Seto <seto.hidetoshi@xxxxxxxxxxxxxx> Signed-off-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> Add Rafael to cc. > * Will cover letters be usually helpful for such patch series? Initially I planned to send the 2 patches seprately as they handle different issues, but as 2/2 will reuse the helper function added by 1/2, I put them together. > * You would probably like to avoid a typo in an email address. Could you be more specific? I got the mail addresses from get_maintainers.pl. thanks! - Feng
