On Fri, May 31, 2019 at 12:17:54AM +0800, kbuild test robot wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git
> pci/docs
> head: 5b76337ac1d54eb1c82414f17e29883290ce87de
> commit: fe6cdee86aeb83ee6ed2dc98928fb9fefc138612 [2/12] Documentation: PCI:
> convert pci.txt to reST
> reproduce: make htmldocs
>
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <[email protected]>
>
> All warnings (new ones prefixed by >>):
>
> WARNING: convert(1) not found, for SVG to PDF conversion install
> ImageMagick (https://www.imagemagick.org)
> include/linux/pci.h:158: warning: cannot understand function prototype:
> 'typedef unsigned int __bitwise pci_channel_state_t; '
> >> include/linux/pci.h:831: warning: Function parameter or member 'node' not
> >> described in 'pci_driver'
> >> include/linux/pci.h:831: warning: Function parameter or member 'name' not
> >> described in 'pci_driver'
> >> include/linux/pci.h:831: warning: Function parameter or member
> >> 'sriov_configure' not described in 'pci_driver'
> >> include/linux/pci.h:831: warning: Function parameter or member 'groups'
> >> not described in 'pci_driver'
> >> include/linux/pci.h:831: warning: Function parameter or member 'driver'
> >> not described in 'pci_driver'
> >> include/linux/pci.h:831: warning: Function parameter or member 'dynids'
> >> not described in 'pci_driver'
> include/linux/pci.h:2251: warning: Excess function parameter 'lrdt'
> description in 'pci_vpd_info_field_size'
I fixed these by folding the patch below into fe6cdee86aeb
("Documentation: PCI: convert pci.txt to reST"). A couple of these
fixes are for pre-existing warnings unrelated to fe6cdee86aeb.
diff --git a/include/linux/pci.h b/include/linux/pci.h
index df22373c9164..b74b2a4e6df2 100644
--- a/include/linux/pci.h
+++ b/include/linux/pci.h
@@ -151,6 +151,8 @@ static inline const char *pci_power_name(pci_power_t state)
#define PCI_PM_BUS_WAIT 50
/**
+ * typedef pci_channel_state_t
+ *
* The pci_channel state describes connectivity between the CPU and
* the PCI device. If some PCI bus between here and the PCI device
* has crashed or locked up, this info is reflected here.
@@ -778,6 +780,8 @@ struct module;
/**
* struct pci_driver - PCI driver structure
+ * @node: List of driver structures.
+ * @name: Driver name.
* @id_table: Pointer to table of device IDs the driver is
* interested in. Most drivers should export this
* table using MODULE_DEVICE_TABLE(pci,...).
@@ -786,7 +790,7 @@ struct module;
* devices or later if a new device gets inserted) for
* all PCI devices which match the ID table and are not
* "owned" by the other drivers yet. This function gets
- * passed a "struct pci_dev *" for each device whose
+ * passed a "struct pci_dev \*" for each device whose
* entry in the ID table matches the device. The probe
* function returns zero when the driver chooses to
* take "ownership" of the device or an error code
@@ -810,7 +814,12 @@ struct module;
* Useful for enabling wake-on-lan (NIC) or changing
* the power state of a device before reboot.
* e.g. drivers/net/e100.c.
+ * @sriov_configure: Optional driver callback to allow configuration of
+ * number of VFs to enable via sysfs "sriov_numvfs" file.
* @err_handler: See Documentation/PCI/pci-error-recovery.rst
+ * @groups: Sysfs attribute groups.
+ * @driver: Driver model structure.
+ * @dynids: List of dynamically added device IDs.
*/
struct pci_driver {
struct list_head node;
@@ -2243,7 +2252,7 @@ static inline u8 pci_vpd_srdt_tag(const u8 *srdt)
/**
* pci_vpd_info_field_size - Extracts the information field length
- * @lrdt: Pointer to the beginning of an information field header
+ * @info_field: Pointer to the beginning of an information field header
*
* Returns the extracted information field length.
*/
