On Tue, Sep 30, 2014 at 06:11:26PM +0800, arei.gong...@huawei.com wrote:
> From: Gonglei <arei.gong...@huawei.com>
>
> The below functions is not allocation functions, but
> find helper function. The only allocation function is
> pcie_add_capability(), but its argument names is not the
> same with the comments. So remove this comments which
> make people confused.
>
> Signed-off-by: Gonglei <arei.gong...@huawei.com>
The comment applies to all functions below it.
Maybe simply replace
s/capability allocation/capability list management/?
Also pls make this a separate patch, does not belong in
this series.
> ---
> hw/pci/pcie.c | 8 --------
> 1 file changed, 8 deletions(-)
>
> diff --git a/hw/pci/pcie.c b/hw/pci/pcie.c
> index b82211a..f2b9d5d 100644
> --- a/hw/pci/pcie.c
> +++ b/hw/pci/pcie.c
> @@ -528,14 +528,6 @@ bool pcie_cap_is_arifwd_enabled(const PCIDevice *dev)
> PCI_EXP_DEVCTL2_ARI;
> }
>
> -/**************************************************************************
> - * pci express extended capability allocation functions
> - * uint16_t ext_cap_id (16 bit)
> - * uint8_t cap_ver (4 bit)
> - * uint16_t cap_offset (12 bit)
> - * uint16_t ext_cap_size
> - */
> -
> static uint16_t pcie_find_capability_list(PCIDevice *dev, uint16_t cap_id,
> uint16_t *prev_p)
> {
> --
> 1.7.12.4
>
