On Fri, Dec 19, 2008 at 01:03:59PM +0100, Takashi Iwai wrote: > Mark Brown wrote: > > > kfree(prtd); > > > + runtime->private_data = NULL; > > > + > This looks superfluous. The runtime instance will be anyway freed > soon after the close callback. Even if it wasn't the driver will overwrite it on open(). OTOH it doesn't do any harm either. _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel