At Fri, 19 Dec 2008 11:57:38 +0000, Mark Brown wrote: > > > @@ -266,6 +271,8 @@ static int pxa2xx_pcm_close(struct snd_p > > dma_free_writecombine(substream->pcm->card->dev, PAGE_SIZE, > > prtd->dma_desc_array, prtd->dma_desc_array_phys); > > kfree(prtd); > > + runtime->private_data = NULL; > > + > > return 0; > > } This looks superfluous. The runtime instance will be anyway freed soon after the close callback. Just a subtle thing, though. Takashi _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel