ASoC: wm5100: Use WARN_ON() instead of BUG_ON()
authorTakashi Iwai <tiwai@suse.de>
Tue, 5 Nov 2013 17:39:55 +0000 (18:39 +0100)
committerMark Brown <broonie@linaro.org>
Wed, 6 Nov 2013 10:26:36 +0000 (10:26 +0000)
Use WARN_ON() and handle the error cases accordingly.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Mark Brown <broonie@linaro.org>
sound/soc/codecs/wm5100.c

index ac1745d030d6256525f8d05aebe14cd542de89b3..4cf91deabc0261dde5d470e5bef375dab415b0b7 100644 (file)
@@ -1972,7 +1972,8 @@ static void wm5100_set_detect_mode(struct wm5100_priv *wm5100, int the_mode)
 {
        struct wm5100_jack_mode *mode = &wm5100->pdata.jack_modes[the_mode];
 
-       BUG_ON(the_mode >= ARRAY_SIZE(wm5100->pdata.jack_modes));
+       if (WARN_ON(the_mode >= ARRAY_SIZE(wm5100->pdata.jack_modes)))
+               return;
 
        gpio_set_value_cansleep(wm5100->pdata.hp_pol, mode->hp_pol);
        regmap_update_bits(wm5100->regmap, WM5100_ACCESSORY_DETECT_MODE_1,