Re: [PATCH] ASoC: tlv320adcx140: Add controls for PDM clk and edge

From: Mark Brown
Date: Wed May 13 2020 - 11:32:48 EST


On Wed, May 13, 2020 at 09:47:46AM -0500, Dan Murphy wrote:

> +static const char * const pdmclk_text[] = {
> + "2.8224 MHz", "1.4112 MHz", "705.6 kHz", "5.6448 MHz"
> +};
> +
> +static SOC_ENUM_SINGLE_DECL(pdmclk_select_enum, ADCX140_PDMCLK_CFG, 0,
> + pdmclk_text);
> +
> +static const struct snd_kcontrol_new pdmclk_div_controls[] = {
> + SOC_DAPM_ENUM("PDM Clk Divider Select", pdmclk_select_enum),
> +};
> +
> +static const char * const pdm_edge_text[] = {
> + "Negative", "Positive"
> +};

Are these (especially the clock and polarity) things that are going to
vary at runtime? I'd have expected these to come from the hardware
rather than being something that could usefully change.

Attachment: signature.asc
Description: PGP signature