summaryrefslogtreecommitdiffstats
path: root/sound/atmel/ac97c.c
diff options
context:
space:
mode:
authorBo Shen <voice.shen@atmel.com>2012-05-11 17:39:28 +0800
committerTakashi Iwai <tiwai@suse.de>2012-05-11 12:10:04 +0200
commitb2522f9262539fc328b4b9344f8a2f7ef2cb18d5 (patch)
treeb3a3ab5a135c32591033ec9842d4103b36d20eb2 /sound/atmel/ac97c.c
parent2abb80176cd80ad8e939ead9c785ac17dc9890a4 (diff)
downloadop-kernel-dev-b2522f9262539fc328b4b9344f8a2f7ef2cb18d5.zip
op-kernel-dev-b2522f9262539fc328b4b9344f8a2f7ef2cb18d5.tar.gz
ALSA: atmel/ac97c: correct the unexpected behavior when using uninitial value for reset pin
When pdata->reset_pin is passed with a negative value (means gpio is invalid), then chip->reset_pin will not be assigned to a vaule, it will use default value 0. This will cause unexpected behavior. So, add this patch to correct. Signed-off-by: Bo Shen <voice.shen@atmel.com> Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com> Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/atmel/ac97c.c')
-rw-r--r--sound/atmel/ac97c.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/sound/atmel/ac97c.c b/sound/atmel/ac97c.c
index 115313e..f5ded64 100644
--- a/sound/atmel/ac97c.c
+++ b/sound/atmel/ac97c.c
@@ -991,6 +991,8 @@ static int __devinit atmel_ac97c_probe(struct platform_device *pdev)
gpio_direction_output(pdata->reset_pin, 1);
chip->reset_pin = pdata->reset_pin;
}
+ } else {
+ chip->reset_pin = -EINVAL;
}
snd_card_set_dev(card, &pdev->dev);
OpenPOWER on IntegriCloud