Re: [PATCH] RFC: ASoC: Davinci: Fix data_type divide by zero

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Sep 11, 2009 at 02:27:18PM -0700, Troy Kisky wrote:
> Mark Brown wrote:

> >> +	 * dma_params must be first because rtd->dai->cpu_dai->private_data
> >> +	 * is cast to an array of struct davinci_pcm_dma_params ** in
> >> +	 * davinci_pcm_open.
> >> +	 */

> > Is it not possible to deal with this by going through the types
> > properly?  The comment sets off all sorts of alarm bells.

> private_data will be either struct davinci_mcbsp_dev or
> struct davinci_audio_dev

Hrm.  Might be nice to change that :/

> Btw, I'm about to submit another rev of this against your for32
> branch. But your branch currently give compile errors, so it
> my compile test just doesn't generate new errors.

You'll need to merge it with Kevin's branch to build - or wait for -rc1
when everything should've made it into mainline.  Kevin's git has a
branch temp/asoc with the merge done in it.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux