Skip to content

Commit

Permalink
misc: microchip: pci1xxxx: Do not disable the pci device twice in gp_…
Browse files Browse the repository at this point in the history
…aux_bus_remove()

gp_aux_bus_probe() uses pcim_enable_device(), so there is no point in
calling pci_disable_device() explicitly in the remove function.

Fixes: 393fc2f ("misc: microchip: pci1xxxx: load auxiliary bus driver for the PIO function in the multi-function endpoint of pci1xxxx device.")
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: https://lore.kernel.org/r/8a3a385b3ae15ee7497469ec3250302b626a018b.1663482259.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
  • Loading branch information
tititiou36 authored and gregkh committed Sep 22, 2022
1 parent dc2c96a commit c8b4747
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion drivers/misc/mchp_pci1xxxx/mchp_pci1xxxx_gp.c
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,6 @@ static void gp_aux_bus_remove(struct pci_dev *pdev)
auxiliary_device_delete(&aux_bus->aux_device_wrapper[1]->aux_dev);
auxiliary_device_uninit(&aux_bus->aux_device_wrapper[1]->aux_dev);
kfree(aux_bus);
pci_disable_device(pdev);
}

static const struct pci_device_id pci1xxxx_tbl[] = {
Expand Down

0 comments on commit c8b4747

Please sign in to comment.