You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
> +/* control */
> +static inline int snd_sof_dsp_run(struct snd_sof_dev *sdev)
> +{
> + if (sdev->ops->run)
> + return sdev->ops->run(sdev);
> +
> + return 0;
> +}
Do we really want to return 0 for all these ops if they're not
implemented? For some that seems sensible but there's others where it
seems like the caller might want to know they got ignored and an error
code like -ENOTSUPP might be better.
Need to scrub all ops routines and mark which ones are required and which ones are optional, and add -ENOTSUPP if relevant?
Activity
ranj063 commentedon Jan 14, 2019
#539 fixes this issue
plbossart commentedon Feb 8, 2019
done