Message ID | 1257351063-5616-1-git-send-email-jirislaby@gmail.com |
---|---|
State | Not Applicable |
Delegated to: | David Miller |
Headers | show |
On Wed, 4 Nov 2009 17:11:03 +0100 Jiri Slaby <jirislaby@gmail.com> wrote: > In via_init_one, when via_isa_bridges iterator reaches > PCI_DEVICE_ID_VIA_ANON and last but one via_isa_bridges bridge is > found but rev doesn't match, pci_dev_put(isa) is called twice. > > Do pci_dev_put only once. > > Signed-off-by: Jiri Slaby <jirislaby@gmail.com> > Cc: Jeff Garzik <jgarzik@pobox.com> Acked-by: Alan Cox <alan@linux.intel.com> Definitely a far saner way to do it. -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 11/04/2009 11:11 AM, Jiri Slaby wrote: > In via_init_one, when via_isa_bridges iterator reaches > PCI_DEVICE_ID_VIA_ANON and last but one via_isa_bridges bridge is > found but rev doesn't match, pci_dev_put(isa) is called twice. > > Do pci_dev_put only once. > > Signed-off-by: Jiri Slaby<jirislaby@gmail.com> > Cc: Jeff Garzik<jgarzik@pobox.com> > --- > drivers/ata/pata_via.c | 10 ++++------ > 1 files changed, 4 insertions(+), 6 deletions(-) applied -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/ata/pata_via.c b/drivers/ata/pata_via.c index 88984b8..520d5a3 100644 --- a/drivers/ata/pata_via.c +++ b/drivers/ata/pata_via.c @@ -526,7 +526,7 @@ static int via_init_one(struct pci_dev *pdev, const struct pci_device_id *id) .port_ops = &via_port_ops }; const struct ata_port_info *ppi[] = { NULL, NULL }; - struct pci_dev *isa = NULL; + struct pci_dev *isa; const struct via_isa_bridge *config; static int printed_version; u8 enable; @@ -551,15 +551,13 @@ static int via_init_one(struct pci_dev *pdev, const struct pci_device_id *id) if ((isa = pci_get_device(PCI_VENDOR_ID_VIA + !!(config->flags & VIA_BAD_ID), config->id, NULL))) { + u8 rev = isa->revision; + pci_dev_put(isa); - if (isa->revision >= config->rev_min && - isa->revision <= config->rev_max) + if (rev >= config->rev_min && rev <= config->rev_max) break; - pci_dev_put(isa); } - pci_dev_put(isa); - if (!(config->flags & VIA_NO_ENABLES)) { /* 0x40 low bits indicate enabled channels */ pci_read_config_byte(pdev, 0x40 , &enable);
In via_init_one, when via_isa_bridges iterator reaches PCI_DEVICE_ID_VIA_ANON and last but one via_isa_bridges bridge is found but rev doesn't match, pci_dev_put(isa) is called twice. Do pci_dev_put only once. Signed-off-by: Jiri Slaby <jirislaby@gmail.com> Cc: Jeff Garzik <jgarzik@pobox.com> --- drivers/ata/pata_via.c | 10 ++++------ 1 files changed, 4 insertions(+), 6 deletions(-)