Re: [PATCH 3/8] ASoC pxa2xx AC97 support

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

 



At Thu, 12 Oct 2006 11:27:19 +0100,
Liam Girdwood wrote:
> 
> On Tue, 2006-10-10 at 19:14 +0200, Takashi Iwai wrote:
> > At Tue, 10 Oct 2006 17:19:36 +0100,
> > Liam Girdwood wrote:
> > > +/* may need to expand this */
> > > +static struct snd_soc_dai_mode pxa2xx_ac97_modes[] = {
> > > +	{0, 0,	SNDRV_PCM_FMTBIT_S16_LE,	AC97_RATES},
> > 
> > Better to use ".field = value" initialization.
> 
> IMHO, this is a better style for the DAI capabilities field because it
> can grow quite large depending on the DAI.

On the contrary, it can reduce the size.
With .field=value style, you can ommit the above 0 initialization, for
example.  Ommitted fields are initialized to 0 as default.

And, the readability is of course much better with .field=value style
:)

Takashi

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/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