2021-03-25 07:53:46

by Xiongfeng Wang

[permalink] [raw]
Subject: [PATCH 0/4] PCI: Correct function names in the header


Xiongfeng Wang (4):
PCI: acpi_pcihp: Correct acpi_pci_check_ejectable() function name in
the header
PCI/AER: Correct function names in the header
PCI/PME: Correct pcie_pme_init() function name in the header
PCI/ATS: Correct pci_max_pasids() function name in header

drivers/pci/ats.c | 2 +-
drivers/pci/hotplug/acpi_pcihp.c | 2 +-
drivers/pci/pcie/aer.c | 6 +++---
drivers/pci/pcie/pme.c | 2 +-
4 files changed, 6 insertions(+), 6 deletions(-)

--
2.20.1


2021-03-25 07:53:53

by Xiongfeng Wang

[permalink] [raw]
Subject: [PATCH 4/4] PCI/ATS: Correct pci_max_pasids() function name in header

Fixes the following W=1 kernel build warning(s):

drivers/pci/ats.c:490: warning: expecting prototype for pci_max_pasid(). Prototype was for pci_max_pasids() instead

Reported-by: Hulk Robot <[email protected]>
Signed-off-by: Xiongfeng Wang <[email protected]>
---
drivers/pci/ats.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pci/ats.c b/drivers/pci/ats.c
index 0d3719407b8b..6d7d64939f82 100644
--- a/drivers/pci/ats.c
+++ b/drivers/pci/ats.c
@@ -480,7 +480,7 @@ EXPORT_SYMBOL_GPL(pci_pasid_features);
#define PASID_NUMBER_SHIFT 8
#define PASID_NUMBER_MASK (0x1f << PASID_NUMBER_SHIFT)
/**
- * pci_max_pasid - Get maximum number of PASIDs supported by device
+ * pci_max_pasids - Get maximum number of PASIDs supported by device
* @pdev: PCI device structure
*
* Returns negative value when PASID capability is not present.
--
2.20.1

2021-03-25 07:54:07

by Xiongfeng Wang

[permalink] [raw]
Subject: [PATCH 3/4] PCI/PME: Correct pcie_pme_init() function name in the header

Fixes the following W=1 kernel build warning(s):

drivers/pci/pcie/pme.c:469: warning: expecting prototype for pcie_pme_service_init(). Prototype was for pcie_pme_init() instead

Reported-by: Hulk Robot <[email protected]>
Signed-off-by: Xiongfeng Wang <[email protected]>
---
drivers/pci/pcie/pme.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pci/pcie/pme.c b/drivers/pci/pcie/pme.c
index 3fc08488d65f..1d0dd77fed3a 100644
--- a/drivers/pci/pcie/pme.c
+++ b/drivers/pci/pcie/pme.c
@@ -463,7 +463,7 @@ static struct pcie_port_service_driver pcie_pme_driver = {
};

/**
- * pcie_pme_service_init - Register the PCIe PME service driver.
+ * pcie_pme_init - Register the PCIe PME service driver.
*/
int __init pcie_pme_init(void)
{
--
2.20.1

2021-03-26 01:11:05

by Krzysztof Wilczyński

[permalink] [raw]
Subject: Re: [PATCH 0/4] PCI: Correct function names in the header

Hi,

[...]
> PCI: acpi_pcihp: Correct acpi_pci_check_ejectable() function name in
> the header
> PCI/AER: Correct function names in the header
> PCI/PME: Correct pcie_pme_init() function name in the header
> PCI/ATS: Correct pci_max_pasids() function name in header
[...]

All of this has already been fixed in the following series:

https://lore.kernel.org/linux-pci/[email protected]/

Krzysztof

2021-03-27 08:22:21

by Xiongfeng Wang

[permalink] [raw]
Subject: Re: [PATCH 0/4] PCI: Correct function names in the header

Hi, Krzysztof

On 2021/3/26 9:07, Krzysztof Wilczyński wrote:
> Hi,
>
> [...]
>> PCI: acpi_pcihp: Correct acpi_pci_check_ejectable() function name in
>> the header
>> PCI/AER: Correct function names in the header
>> PCI/PME: Correct pcie_pme_init() function name in the header
>> PCI/ATS: Correct pci_max_pasids() function name in header
> [...]
>
> All of this has already been fixed in the following series:
>
> https://lore.kernel.org/linux-pci/[email protected]/

Thanks for your remind. I will skip this series in the future.

Thanks,
Xiongfeng

>
> Krzysztof
> .
>