ARM: PNX4008: i2c-pnx: use the same dev_id for request_irq and free_irq
This allows i2c-pnx to free its interrupt handler when the module is removed or if an error occurs; using the same dev_id for both request_irq and free_irq is desirable. Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk> Acked-by: Ben Dooks <ben-linux@fluff.org>
This commit is contained in:
parent
1508c99506
commit
f8d5e5a8f5
1 changed files with 2 additions and 2 deletions
|
@ -650,7 +650,7 @@ static int __devinit i2c_pnx_probe(struct platform_device *pdev)
|
|||
return 0;
|
||||
|
||||
out_irq:
|
||||
free_irq(alg_data->irq, alg_data);
|
||||
free_irq(alg_data->irq, i2c_pnx->adapter);
|
||||
out_clock:
|
||||
i2c_pnx->set_clock_stop(pdev);
|
||||
out_unmap:
|
||||
|
@ -669,7 +669,7 @@ static int __devexit i2c_pnx_remove(struct platform_device *pdev)
|
|||
struct i2c_adapter *adap = i2c_pnx->adapter;
|
||||
struct i2c_pnx_algo_data *alg_data = adap->algo_data;
|
||||
|
||||
free_irq(alg_data->irq, alg_data);
|
||||
free_irq(alg_data->irq, i2c_pnx->adapter);
|
||||
i2c_del_adapter(adap);
|
||||
i2c_pnx->set_clock_stop(pdev);
|
||||
iounmap((void *)alg_data->ioaddr);
|
||||
|
|
Reference in a new issue