ahci: cleanup ahci_host_activate_multi_irqs
With devm there is no need to explicitly free irqs on error. Signed-off-by: Dan Williams <dan.j.williams@intel.com> Signed-off-by: Tejun Heo <tj@kernel.org>
This commit is contained in:
		
							parent
							
								
									899a63c3eb
								
							
						
					
					
						commit
						0a142b2692
					
				| @ -2496,25 +2496,10 @@ static int ahci_host_activate_multi_irqs(struct ata_host *host, int irq, | ||||
| 					       ahci_port_thread_fn, 0, | ||||
| 					       pp->irq_desc, host->ports[i]); | ||||
| 		if (rc) | ||||
| 			goto out_free_irqs; | ||||
| 	} | ||||
| 
 | ||||
| 	for (i = 0; i < host->n_ports; i++) | ||||
| 			return rc; | ||||
| 		ata_port_desc(host->ports[i], "irq %d", irq + i); | ||||
| 
 | ||||
| 	rc = ata_host_register(host, sht); | ||||
| 	if (rc) | ||||
| 		goto out_free_all_irqs; | ||||
| 
 | ||||
| 	return 0; | ||||
| 
 | ||||
| out_free_all_irqs: | ||||
| 	i = host->n_ports; | ||||
| out_free_irqs: | ||||
| 	for (i--; i >= 0; i--) | ||||
| 		devm_free_irq(host->dev, irq + i, host->ports[i]); | ||||
| 
 | ||||
| 	return rc; | ||||
| 	} | ||||
| 	return ata_host_register(host, sht); | ||||
| } | ||||
| 
 | ||||
| /**
 | ||||
|  | ||||
		Loading…
	
		Reference in New Issue
	
	Block a user