Re: [PATCH] PCI: probe: Fix spacing around operators

["Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>]

On Tue, 3 Mar 2026, hechushiguitu666@gmail.com wrote:

> From: "haoyu.lu" <hechushiguitu666@gmail.com>
> 
> Fix checkpatch.pl warnings for missing spaces around operators:
> 
> - size-1 -> size - 1
> 
> - PCI_BRIDGE_RESOURCES+i -> PCI_BRIDGE_RESOURCES + i
> 
> - 60*1000 -> 60 * 1000
> 
> Signed-off-by: haoyu.lu <hechushiguitu666@gmail.com>
> ---
>  drivers/pci/probe.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/pci/probe.c b/drivers/pci/probe.c
> index bccc7a4bdd79..e54fa8a9f377 100644
> --- a/drivers/pci/probe.c
> +++ b/drivers/pci/probe.c
> @@ -119,7 +119,7 @@ static u64 pci_size(u64 base, u64 maxbase, u64 mask)
>  	 * Get the lowest of them to find the decode size, and from that
>  	 * the extent.
>  	 */
> -	size = size & ~(size-1);
> +	size = size & ~(size - 1);
>  	/*
>  	 * base == maxbase can be valid only if the BAR has already been
> @@ -594,7 +594,7 @@ void pci_read_bridge_bases(struct pci_bus *child)
>  
>  	pci_bus_remove_resources(child);
>  	for (i = 0; i < PCI_BRIDGE_RESOURCE_NUM; i++)
> -		child->resource[i] = &dev->resource[PCI_BRIDGE_RESOURCES+i];
> +		child->resource[i] = &dev->resource[PCI_BRIDGE_RESOURCES + i];

Convert to pci_resource_n().

>  
>  	pci_read_bridge_io(child->self,
>  			   child->resource[PCI_BUS_BRIDGE_IO_WINDOW], false);
> @@ -2600,7 +2600,7 @@ static struct pci_dev *pci_scan_device(struct pci_bus *bus, int devfn)
>  	struct pci_dev *dev;
>  	u32 l;
>  
> -	if (!pci_bus_read_dev_vendor_id(bus, devfn, &l, 60*1000))
> +	if (!pci_bus_read_dev_vendor_id(bus, devfn, &l, 60 * 1000))

This should use linux/units.h *_PER_* constant instead of 1000.

>  		return NULL;
>  
>  	dev = pci_alloc_dev(bus);
> 

-- 
 i.