[alsa-devel] [PATCH] ASoC: Fix cs4270 error path
timur at freescale.com
Mon Sep 29 15:42:15 CEST 2008
On Sun, Aug 31, 2008 at 7:42 AM, Jean Delvare <khali at linux-fr.org> wrote:
> The error path in cs4270_probe/cs4270_remove is pretty broken:
> * If cs4270_probe fails, codec is leaked.
> * If snd_soc_register_card fails, cs4270_i2c_driver stays registered.
> * If I2C support is enabled but no I2C device is found, i2c_del_driver
> is never called (neither in cs4270_probe nor in cs4270_remove.)
> Fix the first 2 problems by implementing a clean error path in
> cs4270_probe and jumping to its labels as needed. Fix the 3rd problem
> by removing the condition to call i2c_del_driver in cs4270_remove.
> Signed-off-by: Jean Delvare <khali at linux-fr.org>
Acked-By: Timur Tabi <timur at freescale.com>
Takashi, this patch needs to go into 2.6.27 as well. Sorry about
that. I don't know how I missed so many problems with my code.
Linux kernel developer at Freescale
More information about the Alsa-devel