[arm:clearfog 12/13] drivers/pci/pcie/portdrv_core.c:325 pcie_port_device_register() warn: inconsistent indenting

kernel test robot lkp at intel.com
Thu Jun 3 09:08:16 PDT 2021


tree:   git://git.armlinux.org.uk/~rmk/linux-arm.git clearfog
head:   b895d4f82428028f93d7005bb6676362d2dcf8cc
commit: 9b4378e3df2f3f1d9a162c30ca2c45929ca258a5 [12/13] mvebu/clearfog pcie updates
config: i386-randconfig-m021-20210603 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp at intel.com>

New smatch warnings:
drivers/pci/pcie/portdrv_core.c:325 pcie_port_device_register() warn: inconsistent indenting
drivers/pci/pcie/aspm.c:581 pcie_aspm_cap_init() warn: inconsistent indenting

Old smatch warnings:
drivers/pci/pcie/portdrv_core.c:338 pcie_port_device_register() warn: inconsistent indenting

vim +325 drivers/pci/pcie/portdrv_core.c

   305	
   306	/**
   307	 * pcie_port_device_register - register PCI Express port
   308	 * @dev: PCI Express port to register
   309	 *
   310	 * Allocate the port extension structure and register services associated with
   311	 * the port.
   312	 */
   313	int pcie_port_device_register(struct pci_dev *dev)
   314	{
   315		int status, capabilities, i, nr_service;
   316		int irqs[PCIE_PORT_DEVICE_MAXSERVICES];
   317	
   318		/* Enable PCI Express port device */
   319		status = pci_enable_device(dev);
   320		if (status)
   321			return status;
   322	
   323		/* Get and check PCI Express port services */
   324		capabilities = get_port_device_capability(dev);
 > 325	dev_info(&dev->dev, "PCIe capabilities: 0x%x\n", capabilities);
   326		if (!capabilities)
   327			return 0;
   328	
   329		pci_set_master(dev);
   330		/*
   331		 * Initialize service irqs. Don't use service devices that
   332		 * require interrupts if there is no way to generate them.
   333		 * However, some drivers may have a polling mode (e.g. pciehp_poll_mode)
   334		 * that can be used in the absence of irqs.  Allow them to determine
   335		 * if that is to be used.
   336		 */
   337		status = pcie_init_service_irqs(dev, irqs, capabilities);
   338	dev_info(&dev->dev, "init_service_irqs: %d\n", status);
   339		if (status) {
   340			capabilities &= PCIE_PORT_SERVICE_HP;
   341			if (!capabilities)
   342				goto error_disable;
   343		}
   344	
   345		/* Allocate child services if any */
   346		status = -ENODEV;
   347		nr_service = 0;
   348		for (i = 0; i < PCIE_PORT_DEVICE_MAXSERVICES; i++) {
   349			int service = 1 << i;
   350			if (!(capabilities & service))
   351				continue;
   352			if (!pcie_device_init(dev, service, irqs[i]))
   353				nr_service++;
   354		}
   355		if (!nr_service)
   356			goto error_cleanup_irqs;
   357	
   358		return 0;
   359	
   360	error_cleanup_irqs:
   361		pci_free_irq_vectors(dev);
   362	error_disable:
   363		pci_disable_device(dev);
   364		return status;
   365	}
   366	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: .config.gz
Type: application/gzip
Size: 44320 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20210604/05bab348/attachment-0001.gz>


More information about the linux-arm-kernel mailing list