pm8001: Use pci_enable_msix_exact() instead of pci_enable_msix()
As result of deprecation of MSI-X/MSI enablement functions pci_enable_msix() and pci_enable_msi_block() all drivers using these two interfaces need to be updated to use the new pci_enable_msi_range() or pci_enable_msi_exact() and pci_enable_msix_range() or pci_enable_msix_exact() interfaces. Signed-off-by: Alexander Gordeev <agordeev@redhat.com> Reviewed-by: Jack Wang <xjtuwjp@gmail.com> Signed-off-by: Christoph Hellwig <hch@lst.de>
This commit is contained in:
parent
db1924d08d
commit
b4d511e5c7
|
@ -748,34 +748,35 @@ static u32 pm8001_setup_msix(struct pm8001_hba_info *pm8001_ha)
|
||||||
sizeof(pm8001_ha->msix_entries[0]);
|
sizeof(pm8001_ha->msix_entries[0]);
|
||||||
for (i = 0; i < max_entry ; i++)
|
for (i = 0; i < max_entry ; i++)
|
||||||
pm8001_ha->msix_entries[i].entry = i;
|
pm8001_ha->msix_entries[i].entry = i;
|
||||||
rc = pci_enable_msix(pm8001_ha->pdev, pm8001_ha->msix_entries,
|
rc = pci_enable_msix_exact(pm8001_ha->pdev, pm8001_ha->msix_entries,
|
||||||
number_of_intr);
|
number_of_intr);
|
||||||
pm8001_ha->number_of_intr = number_of_intr;
|
pm8001_ha->number_of_intr = number_of_intr;
|
||||||
if (!rc) {
|
if (rc)
|
||||||
PM8001_INIT_DBG(pm8001_ha, pm8001_printk(
|
return rc;
|
||||||
"pci_enable_msix request ret:%d no of intr %d\n",
|
|
||||||
rc, pm8001_ha->number_of_intr));
|
|
||||||
|
|
||||||
|
PM8001_INIT_DBG(pm8001_ha, pm8001_printk(
|
||||||
|
"pci_enable_msix_exact request ret:%d no of intr %d\n",
|
||||||
|
rc, pm8001_ha->number_of_intr));
|
||||||
|
|
||||||
for (i = 0; i < number_of_intr; i++) {
|
for (i = 0; i < number_of_intr; i++) {
|
||||||
snprintf(intr_drvname[i], sizeof(intr_drvname[0]),
|
snprintf(intr_drvname[i], sizeof(intr_drvname[0]),
|
||||||
DRV_NAME"%d", i);
|
DRV_NAME"%d", i);
|
||||||
pm8001_ha->irq_vector[i].irq_id = i;
|
pm8001_ha->irq_vector[i].irq_id = i;
|
||||||
pm8001_ha->irq_vector[i].drv_inst = pm8001_ha;
|
pm8001_ha->irq_vector[i].drv_inst = pm8001_ha;
|
||||||
|
|
||||||
rc = request_irq(pm8001_ha->msix_entries[i].vector,
|
rc = request_irq(pm8001_ha->msix_entries[i].vector,
|
||||||
pm8001_interrupt_handler_msix, flag,
|
pm8001_interrupt_handler_msix, flag,
|
||||||
intr_drvname[i], &(pm8001_ha->irq_vector[i]));
|
intr_drvname[i], &(pm8001_ha->irq_vector[i]));
|
||||||
if (rc) {
|
if (rc) {
|
||||||
for (j = 0; j < i; j++)
|
for (j = 0; j < i; j++) {
|
||||||
free_irq(
|
free_irq(pm8001_ha->msix_entries[j].vector,
|
||||||
pm8001_ha->msix_entries[j].vector,
|
|
||||||
&(pm8001_ha->irq_vector[i]));
|
&(pm8001_ha->irq_vector[i]));
|
||||||
pci_disable_msix(pm8001_ha->pdev);
|
|
||||||
break;
|
|
||||||
}
|
}
|
||||||
|
pci_disable_msix(pm8001_ha->pdev);
|
||||||
|
break;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
return rc;
|
return rc;
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|
Loading…
Reference in New Issue