drivers/i2c/cs42l42: Make HS_BIAS_SENSE_EN optional

HSBIAS_SENSE_EN configures HSBIAS output current sense through
the external 2.21-k resistor. HSBIAS_SENSE is hardware feature to reduce
the potential pop noise during the headset plug out slowly. But on some
platforms ESD voltage will affect it causing test to fail, especially
with CTIA headset type. For different hardware setups, a designer might
want to tweak default behavior.

Signed-off-by: Vitaly Rodionov <vitaly.rodionov@cirrus.corp-partner.google.com>
Change-Id: I87c6f01af1bdb5b1cb8e399191519598d7fbe9ea
Reviewed-on: https://review.coreboot.org/c/coreboot/+/52981
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Vitaly Rodionov 2021-05-06 19:50:36 +01:00 committed by Patrick Georgi
parent 5d24231427
commit 2be6da1d49
2 changed files with 12 additions and 0 deletions

View File

@ -106,4 +106,14 @@ struct drivers_i2c_cs42l42_config {
uint64_t bias_lvls[4];
/* headset bias ramp rate */
enum cs42l42_hs_bias_ramp_rate hs_bias_ramp_rate;
/*
* cirrus,hs-bias-sense-disable: This is boolean property. If present the
* HSBIAS sense is disabled. Configures HSBIAS output current sense through
* the external 2.21-k resistor. HSBIAS_SENSE is hardware feature to reduce
* the potential pop noise during the headset plug out slowly. But on some
* platforms ESD voltage will affect it causing test to fail, especially
* with CTIA headset type. For different hardware setups, a designer might
* want to tweak default behavior.
*/
bool hs_bias_sense_disable;
};

View File

@ -87,6 +87,8 @@ static void cs42l42_fill_ssdt(const struct device *dev)
acpi_dp_add_integer_array(dsd, "cirrus,bias-lvls", config->bias_lvls, 4);
acpi_dp_add_integer(dsd, "cirrus,hs-bias-ramp-rate", config->hs_bias_ramp_rate);
if (config->hs_bias_sense_disable)
acpi_dp_add_integer(dsd, "cirrus,hs-bias-sense-disable", 1);
/* Write Device Property Hierarchy */
acpi_dp_write(dsd);