Re: [PATCH 2/3] ASoC: TWL4030: Add AVADC Clock Priority

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

 



On Friday 19 June 2009 20:57:39 ext Lopez Cruz, Misael wrote:
> > > +/* AVADC clock priority */
> > > +static const char *twl4030_avadc_clk_priority_texts[] = {
> > > +	"Voice high priority", "Audio high priority"
> > > +};
> > > +
> >
> > It'd probably be better to say "Hifi" (or whatever spelling
> > the rest of the driver uses) rather than "Audio" here.  Audio
> > is a bit confusing since obviously the voice data is audio
> > data too, it's just a different path through the CODEC with
> > different characteristics.  The same thing has been done
> > elsewhere in the driver.
>
> Agree, "HiFi" is being used in codec DAI definition so I'll
> use that too.
>
> I'll wait for Peter's comments before resending the patch.

Please change the Audio to HiFi, otherwise it is OK.

>
> Thanks,
> Misa

-- 
Péter
_______________________________________________
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