]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
ASoC: intel: Kill BUG_ON() usage
authorTakashi Iwai <tiwai@suse.de>
Thu, 7 Sep 2017 08:59:18 +0000 (10:59 +0200)
committerMark Brown <broonie@kernel.org>
Wed, 20 Sep 2017 11:45:50 +0000 (12:45 +0100)
Don't use BUG_ON() for a non-critical sanity check on production
systems.  This patch either removes useless BUG_ON() calls.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Acked-By: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/atom/sst/sst_loader.c
sound/soc/intel/atom/sst/sst_stream.c

index 33917146d9c445d9919c3dacad2bf904f04b1d9a..a686eef2cf7f7a1a4a7f7f23a00808122734a850 100644 (file)
@@ -415,7 +415,6 @@ int sst_load_fw(struct intel_sst_drv *sst_drv_ctx)
                        return ret_val;
        }
 
-       BUG_ON(!sst_drv_ctx->fw_in_mem);
        block = sst_create_block(sst_drv_ctx, 0, FW_DWNL_ID);
        if (block == NULL)
                return -ENOMEM;
index 83d8dda152331054650f6fca0d5d09d160d73108..65e257b17a7efe69ecf8810622db647f373b820a 100644 (file)
@@ -45,7 +45,6 @@ int sst_alloc_stream_mrfld(struct intel_sst_drv *sst_drv_ctx, void *params)
        void *data = NULL;
 
        dev_dbg(sst_drv_ctx->dev, "Enter\n");
-       BUG_ON(!params);
 
        str_params = (struct snd_sst_params *)params;
        memset(&alloc_param, 0, sizeof(alloc_param));