ALSA: ctxfi: Use WARN_ON() instead of BUG_ON()
authorTakashi Iwai <tiwai@suse.de>
Tue, 5 Nov 2013 14:00:02 +0000 (15:00 +0100)
committerTakashi Iwai <tiwai@suse.de>
Wed, 6 Nov 2013 07:34:49 +0000 (08:34 +0100)
BUG_ON() is rather useless for debugging as it leads to panic().
Use WARN_ON() and handle the error cases accordingly.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/ctxfi/cthardware.c

index 110b8ace6d8a46f24cec3225682eb4952e8c6ef1..a689f255270641ab21f17474e8dbfe8ced37ae19 100644 (file)
@@ -69,7 +69,8 @@ unsigned int get_field(unsigned int data, unsigned int field)
 {
        int i;
 
-       BUG_ON(!field);
+       if (WARN_ON(!field))
+               return 0;
        /* @field should always be greater than 0 */
        for (i = 0; !(field & (1 << i)); )
                i++;
@@ -81,7 +82,8 @@ void set_field(unsigned int *data, unsigned int field, unsigned int value)
 {
        int i;
 
-       BUG_ON(!field);
+       if (WARN_ON(!field))
+               return;
        /* @field should always be greater than 0 */
        for (i = 0; !(field & (1 << i)); )
                i++;