Commit 89720264 authored by Wei Yongjun's avatar Wei Yongjun Committed by Lee Jones

mfd: ezx-pcap: Drop devm_free_irq of devm_ allocated irq

The devm_request_irq function allocates irq that is released
when a driver detaches. Thus, there is no reason to explicitly
call devm_free_irq in probe or remove functions.
Signed-off-by: default avatarWei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
parent e90f8754
...@@ -394,16 +394,12 @@ static int pcap_add_subdev(struct pcap_chip *pcap, ...@@ -394,16 +394,12 @@ static int pcap_add_subdev(struct pcap_chip *pcap,
static int ezx_pcap_remove(struct spi_device *spi) static int ezx_pcap_remove(struct spi_device *spi)
{ {
struct pcap_chip *pcap = spi_get_drvdata(spi); struct pcap_chip *pcap = spi_get_drvdata(spi);
struct pcap_platform_data *pdata = dev_get_platdata(&spi->dev); int i;
int i, adc_irq;
/* remove all registered subdevs */ /* remove all registered subdevs */
device_for_each_child(&spi->dev, NULL, pcap_remove_subdev); device_for_each_child(&spi->dev, NULL, pcap_remove_subdev);
/* cleanup ADC */ /* cleanup ADC */
adc_irq = pcap_to_irq(pcap, (pdata->config & PCAP_SECOND_PORT) ?
PCAP_IRQ_ADCDONE2 : PCAP_IRQ_ADCDONE);
devm_free_irq(&spi->dev, adc_irq, pcap);
mutex_lock(&pcap->adc_mutex); mutex_lock(&pcap->adc_mutex);
for (i = 0; i < PCAP_ADC_MAXQ; i++) for (i = 0; i < PCAP_ADC_MAXQ; i++)
kfree(pcap->adc_queue[i]); kfree(pcap->adc_queue[i]);
...@@ -509,8 +505,6 @@ static int ezx_pcap_probe(struct spi_device *spi) ...@@ -509,8 +505,6 @@ static int ezx_pcap_probe(struct spi_device *spi)
remove_subdevs: remove_subdevs:
device_for_each_child(&spi->dev, NULL, pcap_remove_subdev); device_for_each_child(&spi->dev, NULL, pcap_remove_subdev);
/* free_adc: */
devm_free_irq(&spi->dev, adc_irq, pcap);
free_irqchip: free_irqchip:
for (i = pcap->irq_base; i < (pcap->irq_base + PCAP_NIRQS); i++) for (i = pcap->irq_base; i < (pcap->irq_base + PCAP_NIRQS); i++)
irq_set_chip_and_handler(i, NULL, NULL); irq_set_chip_and_handler(i, NULL, NULL);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment