<p>harsha p has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21195">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">driver/i2c/max98927: Add imon and vmon params<br><br>This change list adds imon and vmon slot numbers as params for<br>Maxim 98927 driver. These values are looked up in the kernel driver<br>to confiure IV feedback for audio playback on speakers.<br><br>BUG=b:36724448<br>TEST=After boot, the register dump for  Max98927 codecs should have<br>imon and vmon slots numbers set in 0x1e register.<br><br>Change-Id: I21d72ba91af83782587f11018b2d1d1c8d4f676c<br>---<br>M src/drivers/i2c/max98927/chip.h<br>M src/drivers/i2c/max98927/max98927.c<br>2 files changed, 5 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/95/21195/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/drivers/i2c/max98927/chip.h b/src/drivers/i2c/max98927/chip.h<br>index c7101bc..195b35d 100644<br>--- a/src/drivers/i2c/max98927/chip.h<br>+++ b/src/drivers/i2c/max98927/chip.h<br>@@ -27,4 +27,7 @@<br>        const char *desc;<br>     /* ACPI Device Name */<br>        const char *name;<br>+    unsigned int vmon_slot_no;<br>+   unsigned int imon_slot_no;<br>+<br> };<br>diff --git a/src/drivers/i2c/max98927/max98927.c b/src/drivers/i2c/max98927/max98927.c<br>index 976c2f2..e9fc00d 100644<br>--- a/src/drivers/i2c/max98927/max98927.c<br>+++ b/src/drivers/i2c/max98927/max98927.c<br>@@ -61,6 +61,8 @@<br>    dp = acpi_dp_new_table("_DSD");<br> <br>  acpi_dp_add_integer(dp, "interleave_mode", config->interleave_mode);<br>+    acpi_dp_add_integer(dp, "vmon-slot-no", config->vmon_slot_no);<br>+  acpi_dp_add_integer(dp, "imon-slot-no", config->imon_slot_no);<br> <br>        acpi_dp_write(dp);<br> <br></pre><p>To view, visit <a href="https://review.coreboot.org/21195">change 21195</a>. To unsubscribe, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/21195"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I21d72ba91af83782587f11018b2d1d1c8d4f676c </div>
<div style="display:none"> Gerrit-Change-Number: 21195 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: harsha p <harsha.p.n@intel.corp-partner.google.com> </div>
<div style="display:none"> Gerrit-Reviewer: HARSHAPRIYA N <harshapriya.n@intel.com> </div>