On Thu, 15 May 2025, Lukas Wunner wrote: > On Thu, May 15, 2025 at 05:43:46PM +0900, Krzysztof Wilczy??ski wrote: > > Done. Squashed with the first commit from Ilpo, see: Thanks Krzysztof for handling this, I should have put the note about squashing it to the resubmission but I forgot (this time I didn't do the diff against the previous version before sending it which I normally do). > > https://git.kernel.org/pub/scm/linux/kernel/git/pci/pci.git/commit/?h=bwctrl&id=2389d8dc38fee18176c49e9c4804f5ecc55807fa > > Awesome, thank you! > > > Let me know if there is anything else needed. > > Actually, two small things: > > - That patch on the pci/bwctrl topic branch is still marked "New" > in patchwork, even though it's been applied: > https://patchwork.kernel.org/project/linux-pci/patch/20250422115548.1483-1-ilpo.jarvinen@xxxxxxxxxxxxxxx/ > > - Version 1 of the same patch is likewise marked "New", even though > it's been superseded: > https://patchwork.kernel.org/project/linux-pci/patch/20250417124633.11470-1-ilpo.jarvinen@xxxxxxxxxxxxxxx/ > > Unfortunately I can't update it myself because I'm not the submitter. > (Ilpo could do it if he has a patchwork.kernel.org account.) I'm a pdx86 maintainer so I do have an account, yes. I actually had the patchwork page listing my PCI patches already open, but I just hadn't hit "update" button yet. I've done those two changes now. I'm a bit hesitant to mark "Accepted" state though, I did it this time but in general I feel I might be overstepping my authority even if I know some patches have been accepted. > Something unrelated (only if you feel like doing it): > > On the pci/enumeration branch, Bjorn queued up a revert which was > waiting to be ack'ed by AMD IOMMU maintainers: > https://lore.kernel.org/r/20250425163259.GA546441@bhelgaas/ > > In the meantime the ack has arrived: > https://lore.kernel.org/r/aCLv7cN_s1Z4abEl@xxxxxxxxxx/ > > So the remaining housekeeping items are: > > - Add Jörg's Acked-by to commit e86c7278eba8 on pci/enumeration > - Remove the "XXX" marker from the subject line > - Remove "Needs AMD IOMMU ack" from the commit message > > Again, only if you feel like doing it. No urgency. -- i.
