Re: [patch] PCI hotplug: cpqphp: hex vs decimal typo in cpqhpc_probe()

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

 



On Mon, Feb 10, 2014 at 06:23:50PM +0300, Dan Carpenter wrote:
> This is a static checker fix and I can't test it, but from the context
> it definitely looks like hexidecimal 0x20 was intended here instead of
> decimal 20.
> 
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

Applied to pci/misc for v3.15, thanks a lot!

Bjorn

> diff --git a/drivers/pci/hotplug/cpqphp_core.c b/drivers/pci/hotplug/cpqphp_core.c
> index 31273e155e6c..037e2612c5bd 100644
> --- a/drivers/pci/hotplug/cpqphp_core.c
> +++ b/drivers/pci/hotplug/cpqphp_core.c
> @@ -920,12 +920,12 @@ static int cpqhpc_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>  				bus->max_bus_speed = PCI_SPEED_100MHz_PCIX;
>  				break;
>  			}
> -			if (bus_cap & 20) {
> +			if (bus_cap & 0x20) {
>  				dbg("bus max supports 66MHz PCI-X\n");
>  				bus->max_bus_speed = PCI_SPEED_66MHz_PCIX;
>  				break;
>  			}
> -			if (bus_cap & 10) {
> +			if (bus_cap & 0x10) {
>  				dbg("bus max supports 66MHz PCI\n");
>  				bus->max_bus_speed = PCI_SPEED_66MHz;
>  				break;
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[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