<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2017-07-23 19:30 GMT+03:00 Kevin O'Connor <span dir="ltr"><<a href="mailto:kevin@koconnor.net" target="_blank">kevin@koconnor.net</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">On Sun, Jul 23, 2017 at 01:11:49AM +0300, Aleksandr Bezzubikov wrote:<br>
</span><div><div class="gmail-h5">> On PCI init PCI bridge devices may need some<br>
> extra info about bus number to reserve, IO, memory and<br>
> prefetchable memory limits. QEMU can provide this<br>
> with special vendor-specific PCI capability.<br>
><br>
> This capability is intended to be used only<br>
> for Red Hat PCI bridges, i.e. QEMU cooperation.<br>
><br>
> Sizes of limits match ones from<br>
> PCI Type 1 Configuration Space Header,<br>
> number of buses to reserve occupies only 1 byte<br>
> since it is the size of Subordinate Bus Number register.<br>
><br>
> Signed-off-by: Aleksandr Bezzubikov <<a href="mailto:zuban32s@gmail.com">zuban32s@gmail.com</a>><br>
> ---<br>
>  src/hw/pci_cap.h | 23 +++++++++++++++++++++++<br>
>  1 file changed, 23 insertions(+)<br>
>  create mode 100644 src/hw/pci_cap.h<br>
><br>
> diff --git a/src/hw/pci_cap.h b/src/hw/pci_cap.h<br>
> new file mode 100644<br>
> index 0000000..1382b0b<br>
> --- /dev/null<br>
> +++ b/src/hw/pci_cap.h<br>
> @@ -0,0 +1,23 @@<br>
> +#ifndef _PCI_CAP_H<br>
> +#define _PCI_CAP_H<br>
> +<br>
> +#include "types.h"<br>
> +<br>
> +struct vendor_pci_cap {<br>
> +    u8 id;<br>
> +    u8 next;<br>
> +    u8 len;<br>
> +};<br>
<br>
</div></div>Thanks.  </blockquote><div>Thanks for the review.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">If you respin this series, please add this header to the<br>
src/fw/ directory instead of src/hw/.  Also, I'd prefer to avoid a<br>
"pci_" prefix on the header as it makes it seem similar to the<br>
existing pci_regs.h and pci_ids.h headers which are a bit different -<br>
how about src/fw/dev-pci.h instead?<br>
<span class="gmail-HOEnZb"><font color="#888888"><br>
-Kevin<br></font></span></blockquote><div> </div></div>No objections, Will do that in v3. </div><div class="gmail_extra">Except 'pci_' prefix - it's still a PCI capability, isn't it?</div><div class="gmail_extra"><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr">Alexander Bezzubikov</div></div></div></div>
</div></div>