Re: [PATCH 2/2] pci: Update VPD size with correct length

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Hannes,

[auto build test WARNING on pci/next]
[also build test WARNING on v4.4-rc5 next-20151217]

url:    https://github.com/0day-ci/linux/commits/Hannes-Reinecke/pci-Update-VPD-definitions/20151217-160050
base:   https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git next
reproduce: make htmldocs

All warnings (new ones prefixed by >>):

   lib/crc32.c:148: warning: No description found for parameter 'tab)[256]'
   lib/crc32.c:148: warning: Excess function parameter 'tab' description in 'crc32_le_generic'
   lib/crc32.c:293: warning: No description found for parameter 'tab)[256]'
   lib/crc32.c:293: warning: Excess function parameter 'tab' description in 'crc32_be_generic'
   lib/crc32.c:1: warning: no structured comments found
>> drivers/pci/access.c:486: warning: Excess function parameter 'old_size' description in 'pci_vpd_pci22_size'

vim +486 drivers/pci/access.c

   470	}
   471	
   472	static const struct pci_vpd_ops pci_vpd_f0_ops = {
   473		.read = pci_vpd_f0_read,
   474		.write = pci_vpd_f0_write,
   475		.release = pci_vpd_pci22_release,
   476	};
   477	
   478	/**
   479	 * pci_vpd_size - determine actual size of Vital Product Data
   480	 * @dev:	pci device struct
   481	 * @old_size:	current assumed size, also maximum allowed size
   482	 *
   483	 */
   484	static size_t
   485	pci_vpd_pci22_size(struct pci_dev *dev)
 > 486	{
   487		size_t off = 0;
   488		unsigned char header[1+2];	/* 1 byte tag, 2 bytes length */
   489	
   490		while (off < PCI_VPD_PCI22_SIZE &&
   491		       pci_read_vpd(dev, off, 1, header) == 1) {
   492			unsigned char tag;
   493	
   494			if (header[0] & PCI_VPD_LRDT) {

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: Binary data


[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux