On Fri, 14 Aug 2020 10:34:36 +0200, Brent Lu wrote: > @@ -1784,6 +1913,16 @@ int hdac_hdmi_jack_init(struct snd_soc_dai *dai, int device, > } > } > > + /* add control for ELD Bytes */ > + err = hdac_hdmi_create_eld_ctl(component, pcm); > + if (err < 0) { > + dev_err(&hdev->dev, > + "eld control add failed with err: %d for pcm: %d\n", > + err, device); > + kfree(pcm); pcm is allocated vida devm_kzalloc(), hence you shoudn't free it explicitly. thanks, Takashi