[alsa-devel] [PATCH] ASoC: max98090: Fix missing free_irq (part 2)

Jarkko Nikula jarkko.nikula at linux.intel.com
Fri Sep 19 09:51:13 CEST 2014


On 09/18/2014 08:54 PM, Mark Brown wrote:
> On Thu, Sep 18, 2014 at 01:48:44PM +0300, Jarkko Nikula wrote:
>
>> ---
>> for-next, I don't think this is critical enough for stable.
>> ---
>> @@ -2413,6 +2413,7 @@ static int max98090_remove(struct snd_soc_codec *codec)
>>   {
>>   	struct max98090_priv *max98090 = snd_soc_codec_get_drvdata(codec);
>>   
>> +	devm_free_irq(codec->dev, max98090->irq, codec);
> If this is a fix for -next shouldn't we be doing the proper fix and
> moving the resource acquisition to the bus probe?
Fair enough. I'll cook another version.

-- 
Jarkko


More information about the Alsa-devel mailing list