Hello Paul Fagerburg, Duncan Laurie, Tim Wawrzynczak, Jimmy Cheng-Yi Chiang, Shelley Chen, Hung-Te Lin, build bot (Jenkins), Furquan Shaikh, Patrick Georgi, Martin Roth,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/36029
to look at the new patch set (#8).
Change subject: drivers/i2c/rt1011: Add a driver for RT1011 ......................................................................
drivers/i2c/rt1011: Add a driver for RT1011
RT1011 is a smart amplifier. It needs to know speaker related parameters including speaker resistor value and temperature when the calibration is done in order to run Dynamic Speaker Management(DSM) algorithm on chip. The purpose of DSM is to protect speaker when the volume is large.
The calibration data of speaker is stored in VPD in factory. This driver is needed to read data from VPD and write to ACPI _DSD when config CHROMEOS_DSM_CALIB is turned on.
Kernel rt1011 codec driver will read these device properties to set up codec accordingly on boot.
The reason to prepare these parameters in coreboot is because kernel codec driver expects to read per-device parameters directly from device properties. Another benefit is that other OS can also take these parameters through ACPI _DSD table and take benefit of DSM on RT1011.
The kernel driver device properties of RT1011 are documented at linux/Documentation/devicetree/bindings/sound/rt1011.txt It is currently in ASoC maintainer's tree at https://kernel.googlesource.com/pub/scm/linux/kernel/git/broonie/sound/+/for... and hopefully should be merged to mainline kernel in the next merge window.
BUG=b:140397934 BRANCH=none TEST=On Helios, with patch series, check realtek,r0_calib and realtek,temperature_calib are available to rt1011 codec driver.
Signed-off-by: Cheng-Yi Chiang cychiang@chromium.org Change-Id: I9550b9890ce2cae787f4f17779a5ade77f619171 --- A src/drivers/i2c/rt1011/Kconfig A src/drivers/i2c/rt1011/Makefile.inc A src/drivers/i2c/rt1011/chip.h A src/drivers/i2c/rt1011/rt1011.c 4 files changed, 160 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/29/36029/8