When power_up_sst() fails, stream needs to be freed just like when try_module_get() fails. However, current code is returning directly and ends up leaking memory.

Would an imperative wording be preferred for the change description? https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Docu...

…

+++ b/sound/soc/intel/atom/sst-mfld-platform-pcm.c @@ -330,8 +330,10 @@ static int sst_media_open(struct snd_pcm_substream *substream, runtime->private_data = stream;

ret_val = power_up_sst(stream);

• if (ret_val < 0)

• if (ret_val < 0) {

• kfree(stream);
  

  return ret_val;
• }

…

I propose to add a jump target so that a bit of common exception handling code can be better reused at the end of this function implementation.

Regards, Markus