From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Carpenter Subject: Re: ASoC: Intel: mrfld- add ACPI module Date: Tue, 25 Nov 2014 02:52:09 +0300 Message-ID: <20141124235209.GA16222@mwanda> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from userp1040.oracle.com (userp1040.oracle.com [156.151.31.81]) by alsa0.perex.cz (Postfix) with ESMTP id 25C09260692 for ; Tue, 25 Nov 2014 00:52:27 +0100 (CET) Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: vinod.koul@intel.com Cc: alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org Hello Vinod Koul, The patch 336cfbb05edf: "ASoC: Intel: mrfld- add ACPI module" from Nov 11, 2014, leads to the following static checker warning: sound/soc/intel/sst/sst_acpi.c:313 sst_acpi_probe() info: ignoring unreachable code. sound/soc/intel/sst/sst_acpi.c 304 ret = sst_context_init(ctx); 305 if (ret < 0) 306 return ret; 307 308 /* need to save shim registers in BYT */ 309 ctx->shim_regs64 = devm_kzalloc(ctx->dev, sizeof(*ctx->shim_regs64), 310 GFP_KERNEL); 311 if (!ctx->shim_regs64) { 312 return -ENOMEM; ^^^^^^^^^^^^^^^ 313 goto do_sst_cleanup; ^^^^^^^^^^^^^^^^^^^^ 314 } 315 316 sst_configure_runtime_pm(ctx); 317 platform_set_drvdata(pdev, ctx); 318 return ret; 319 320 do_sst_cleanup: 321 sst_context_cleanup(ctx); 322 platform_set_drvdata(pdev, NULL); 323 dev_err(ctx->dev, "failed with %d\n", ret); 324 return ret; 325 } regards, dan carpenter