[alsa-devel] [PATCH 2/3] ASoC: TWL4030: Add AVADC Clock Priority

Peter Ujfalusi peter.ujfalusi at nokia.com
Mon Jun 22 08:36:44 CEST 2009


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


More information about the Alsa-devel mailing list