[SeaBIOS] [PATCH v2 0/3] pci: resource reserve capability found

[Liu, Jing2]

On 8/14/2018 10:01 PM, Laszlo Ersek wrote:
> 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.
Let me re-send this serial with the same version.
Sorry for disturbing.

Jing

> 
> Thanks
> Laszlo
>