On Wed, Feb 01, 2023 at 11:02:59AM +0800, Mad Horse wrote: > 在 2023/1/18 03:46, Bjorn Helgaas 写道: > > Suggest structure like this (flesh out with the relevant DT property > > names, etc): > > > > For PCI ath9k devices, load calibration data only if there is a DT > > node corresponding to the device with XXX ... > > > > More details: https://chris.beams.io/posts/git-commit/ > > > Thanks for pointing it out. The commit log of the last two patch has > been reworded with required content, and sent to the current thread > earlier. Thanks. Sending updated version as replies to the original series posting makes it hard to keep track of what's happening. Better if you send a "v2" series of *all* the patches, even if some of them haven't changed. There are a few hints here, although it's not very explicit: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?id=v6.1#n633 Also, you use multiple email addresses, and if you can figure out how to at least make the display names the same, it will make the conversation easier to follow. Here's what it looks like in mutt: - Jan 7 9:14 Mad Horse [PATCH] pci: Add functions ... - Jan 10 9:24 To Mad Horse ├─> - Jan 17 4:01 Mad Horse ├─>[PATCH 1/3] PCI: of: Match ... - Jan 17 4:02 Mad Horse ├─>[PATCH 2/3] wifi: ath9k: stop ... - Jan 17 4:02 Mad Horse ├─>[PATCH 3/3] wifi: ath10k: only ... - Jan 17 4:29 Mad Horse ├─>[PATCH 1/3] PCI: of: Match pci ... - Jan 17 3:27 Edward Chow ├─>[PATCH 1/3] PCI: of: Match pci ... - Jan 17 3:28 Edward Chow ├─>[PATCH 3/3] wifi: ath10k: only ... - Jan 21 4:00 persmule@gmail. ├─>[PATCH 2/3] wifi: ath9k: stop ... - Jan 21 4:06 persmule@gmail. ├─>[PATCH 3/3] wifi: ath10k: only ... - Jan 17 3:27 Edward Chow └─>[PATCH 2/3] wifi: ath9k: stop ... - Jan 17 13:46 To Edward Chow └─> - Jan 31 21:02 Mad Horse └─> This also shows the problem with updated versions in the same thread. I can work out that "Jan 17 1/3", "Jan 21 2/3", and "Jan 21 3/3" are supposed to be the newest versions, but tools like patchwork and b4 are not smart enough to do that, so it's really hard to apply this. If you just post an updated series, e.g., [PATCH v2 0/3] PCI: ... |-> [PATCH v2 1/3] PCI: of: Match ... |-> [PATCH v2 1/3] PCI: of: Match ... |-> [PATCH v2 1/3] PCI: of: Match ... then everything will be nice and clean and easy to review and apply. Bjorn
