Re: [PATCH 12/25] ASoC: meson: t9015: add support for A1 SoC family
From: Mark Brown <broonie@kernel.org>
Date: 2024-03-15 13:36:55
Also in:
alsa-devel, linux-amlogic, linux-clk, linux-devicetree, linux-gpio, linux-sound, lkml
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?