lkml.org 
[lkml]   [2004]   [Oct]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [Kernel-janitors] [PATCH 2.6.9-rc2-mm4 ibmphp_core.c][6/8] replace pci_get_device with pci_dev_present
On Wed, 29 Sep 2004, Hanna Linder wrote:

>
> This can be converted to pci_dev_present as the dev returned is never used.
> Compile tested.
>
> Hanna Linder
> IBM Linux Technology Center
>
> Signed-off-by: Hanna Linder <hannal@us.ibm.com>
>
> diff -Nrup linux-2.6.9-rc2-mm4cln/drivers/pci/hotplug/ibmphp_core.c linux-2.6.9-rc2-mm4patch2/drivers/pci/hotplug/ibmphp_core.c
> --- linux-2.6.9-rc2-mm4cln/drivers/pci/hotplug/ibmphp_core.c 2004-09-28 14:58:50.000000000 -0700
> +++ linux-2.6.9-rc2-mm4patch2/drivers/pci/hotplug/ibmphp_core.c 2004-09-29 15:39:39.385406240 -0700
> @@ -838,8 +838,11 @@ static int set_bus (struct slot * slot_c
> int rc;
> u8 speed;
> u8 cmd = 0x0;
> - struct pci_dev *dev = NULL;
> int retval;
> + static struct pci_device_id ciobx[] = {
> + { PCI_DEVICE(PCI_VENDOR_ID_SERVERWORKS, 0x0101) },
> + { },
> + };
^^^^
white space damaged

>
> debug ("%s - entry slot # %d\n", __FUNCTION__, slot_cur->number);
> if (SET_BUS_STATUS (slot_cur->ctrl) && is_bus_empty (slot_cur)) {
> @@ -886,8 +889,7 @@ static int set_bus (struct slot * slot_c
> break;
> case BUS_SPEED_133:
> /* This is to take care of the bug in CIOBX chip */
> - while ((dev = pci_get_device(PCI_VENDOR_ID_SERVERWORKS,
> - 0x0101, dev)) != NULL)
> + if(pci_dev_present(ciobx))
> ibmphp_hpc_writeslot (slot_cur, HPC_BUS_100PCIXMODE);
> cmd = HPC_BUS_133PCIXMODE;
> break;
>

first timer for your patches.
corrected for next kjt.


--
maks
kernel janitor http://janitor.kernelnewbies.org/

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:07    [W:0.034 / U:0.040 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site