On Fri, Mar 15, 2024 at 02:21:48AM +0300, Jan Dakinevich wrote:
> +static const char * const a1_adc_mic_bias_level_txt[] = { "2.0V", "2.1V",
> + "2.3V", "2.5V", "2.8V" };
> +static const unsigned int a1_adc_mic_bias_level_values[] = { 0, 1, 2, 3, 7 };
Why would this be varied at runtime rather than being something fixed
when the system is designed?
> +static const char * const a1_adc_pga_txt[] = { "None", "Differential",
> + "Positive", "Negative" };
> +static const unsigned int a1_adc_pga_right_values[] = { 0, PGAR_DIFF,
> + PGAR_POSITIVE, PGAR_NEGATIVE };
> +static const unsigned int a1_adc_pga_left_values[] = { 0, PGAL_DIFF,
> + PGAL_POSITIVE, PGAL_NEGATIVE };
Similarly here.
> + SOC_SINGLE("ADC Mic Bias Switch", LINEIN_CFG, MICBIAS_EN, 1, 0),
> + SOC_ENUM("ADC Mic Bias Level", a1_adc_mic_bias_level),
Why would micbias be user controlled rather than a DAPM widget as
normal?
Attachment:
signature.asc
Description: PGP signature
