> > + ret = snd_soc_jack_new(&snd_soc_sdp3430, "SDP3430 headset jack", > > + SND_JACK_HEADSET, hs_jack); > > + if (ret) > > + return ret; > This leaks the jack. If hs_jack is declared as a direct static variable instead, then no mem leaks, isn't it? Or do you mean to clean the hs_jack if any error? -Misa-- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html