On 08/13/18 09:49, Jing Liu wrote: > This patch serial is about QEMU resource reserve capability finding > in firmware. > > Firstly, this fixes a logic bug. When the capability is truncated, > return zero instead of the truncated offset. Secondly, this modified > the debug messages when the capability is not found and when the vendor > ID or device Id doesn't match REDHAT special ones. > > Last, this enables the firmware recongizing the REDHAT PCI BRIDGE device ID, > so that QEMU can also reserve PCI bridge resource capability. > > Jing Liu (3): > pci: fix the return value for truncated capability > pci: clean up the debug message for pci capability found > pci: recognize RH PCI legacy bridge resource reservation capability > > src/fw/pciinit.c | 45 ++++++++++++++++++++++++++++----------------- > src/hw/pci_ids.h | 1 + > 2 files changed, 29 insertions(+), 17 deletions(-)
Something looks wrong with this patch set -- the shortlog above indicates that the 3rd patch is called "pci: recognize RH PCI legacy bridge resource reservation capability", but in the actual series I see: [SeaBIOS][PATCH v2 0/3] pci: resource reserve capability found [SeaBIOS][PATCH v2 1/3] pci: fix the return value for truncated capability [SeaBIOS][PATCH v2 2/3] pci: clean up the debug message for pci capability found [SeaBIOS][PATCH v2] pci: recognize RH PCI legacy bridge resource For the last patch, the "3/3" counter is missing in the subject prefix, plus the subject doesn't match the short log. I also see a separate message [SeaBIOS] [PATCH v2 3/3] pci: recognize RH PCI legacy bridge resource which is apparently not threaded under v2 0/3, and whose subject line also doesn't match the shortlog. Not sure what Kevin and Gerd prefer, but if these patches are meant to form a series, I'd suggest to repost with correct numbering, threading, and subject lines. Thanks Laszlo _______________________________________________ SeaBIOS mailing list SeaBIOS@seabios.org https://mail.coreboot.org/mailman/listinfo/seabios