Re: [bug report] ASoC: dapm: Implement and instantiate DAI widgets

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jan 19, 2017 at 10:38:26AM +0300, Dan Carpenter wrote:
> Hello Mark Brown,

Dan you need to update Mark's email in your address book, its been a while
since he moved away from wolfson..

> 
> The patch 888df395ebc5: "ASoC: dapm: Implement and instantiate DAI
> widgets" from Feb 16, 2012, leads to the following static checker
> warning:
> 
> 	sound/soc/soc-dapm.c:3944 snd_soc_dapm_new_dai_widgets()
> 	error: 'w' dereferencing possible ERR_PTR()
> 
> sound/soc/soc-dapm.c
>   3929          if (dai->driver->playback.stream_name) {
>   3930                  template.id = snd_soc_dapm_dai_in;
>   3931                  template.name = dai->driver->playback.stream_name;
>   3932                  template.sname = dai->driver->playback.stream_name;
>   3933  
>   3934                  dev_dbg(dai->dev, "ASoC: adding %s widget\n",
>   3935                          template.name);
>   3936  
>   3937                  w = snd_soc_dapm_new_control_unlocked(dapm, &template);
> 
> This can either return NULL or ERR_PTR(-EPROBE_DEFER).
> 
>   3938                  if (!w) {
>   3939                          dev_err(dapm->dev, "ASoC: Failed to create %s widget\n",
>   3940                                  dai->driver->playback.stream_name);
>   3941                          return -ENOMEM;
>   3942                  }
>   3943  
>   3944                  w->priv = dai;
>                         ^^^^^^^^^^^^^
> 
>   3945                  dai->playback_widget = w;
>   3946          }
> 
> regards,
> dan carpenter
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@xxxxxxxxxxxxxxxx
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

-- 
~Vinod
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux