Re: ASoC: Intel: mrfld- add ACPI module

[Vinod Koul <vinod.koul@intel.com>]

On Wed, Mar 18, 2015 at 02:09:32PM +0300, Dan Carpenter wrote:
> Ping?
Sory dan, this seems to have slipped thru, I will send the patch for this.

Thanks a bunch for reporting

-- 
~Vinod

> 
> regards,
> dan carpenter
> 
> On Tue, Nov 25, 2014 at 02:52:09AM +0300, Dan Carpenter wrote:
> > 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

--