<p>Duncan Laurie has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/25367">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/skylake: Add NHLT config for max98373 codec<br><br>Add the NHLT configuration for the max98373 codec to skylake,<br>taken directly from cannonlake.<br><br>This will allow skylake/kabylake boards to use this codec.<br><br>Change-Id: Ifb6bf2d31fda25b18d9b1ce2bb721255335d55e4<br>Signed-off-by: Duncan Laurie <dlaurie@google.com><br>---<br>M src/soc/intel/skylake/Kconfig<br>M src/soc/intel/skylake/include/soc/nhlt.h<br>M src/soc/intel/skylake/nhlt/Makefile.inc<br>A src/soc/intel/skylake/nhlt/max98373.c<br>4 files changed, 100 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/67/25367/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/soc/intel/skylake/Kconfig b/src/soc/intel/skylake/Kconfig</span><br><span>index aac4a8f..f76cf72 100644</span><br><span>--- a/src/soc/intel/skylake/Kconfig</span><br><span>+++ b/src/soc/intel/skylake/Kconfig</span><br><span>@@ -253,6 +253,12 @@</span><br><span>       help</span><br><span>           Include DSP firmware settings for max98357 amplifier.</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+config NHLT_MAX98373</span><br><span style="color: hsl(120, 100%, 40%);">+     bool</span><br><span style="color: hsl(120, 100%, 40%);">+  default n</span><br><span style="color: hsl(120, 100%, 40%);">+     help</span><br><span style="color: hsl(120, 100%, 40%);">+    Include DSP firmware settings for max98373 amplifier.</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> config NHLT_SSM4567</span><br><span>     bool</span><br><span>         default n</span><br><span>diff --git a/src/soc/intel/skylake/include/soc/nhlt.h b/src/soc/intel/skylake/include/soc/nhlt.h</span><br><span>index 1af27e8..a25e5ac 100644</span><br><span>--- a/src/soc/intel/skylake/include/soc/nhlt.h</span><br><span>+++ b/src/soc/intel/skylake/include/soc/nhlt.h</span><br><span>@@ -61,6 +61,12 @@</span><br><span> int nhlt_soc_add_max98357(struct nhlt *nhlt, int hwlink);</span><br><span> </span><br><span> /*</span><br><span style="color: hsl(120, 100%, 40%);">+ * Add max98373 amplifier in stereo configuration on provided SSP link.</span><br><span style="color: hsl(120, 100%, 40%);">+ * Return 0 on success, < 0 on error.</span><br><span style="color: hsl(120, 100%, 40%);">+ */</span><br><span style="color: hsl(120, 100%, 40%);">+int nhlt_soc_add_max98373(struct nhlt *nhlt, int hwlink);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+/*</span><br><span>  * Add rt5514 DSP on provided SSP link. Return 0 on success, < 0 on error.</span><br><span>  */</span><br><span> int nhlt_soc_add_rt5514(struct nhlt *nhlt, int hwlink, int num_channels);</span><br><span>diff --git a/src/soc/intel/skylake/nhlt/Makefile.inc b/src/soc/intel/skylake/nhlt/Makefile.inc</span><br><span>index 9c9b4c8..5c8bd80 100644</span><br><span>--- a/src/soc/intel/skylake/nhlt/Makefile.inc</span><br><span>+++ b/src/soc/intel/skylake/nhlt/Makefile.inc</span><br><span>@@ -1,6 +1,7 @@</span><br><span> ramstage-y += dmic.c</span><br><span> ramstage-y += nau88l25.c</span><br><span> ramstage-y += max98357.c</span><br><span style="color: hsl(120, 100%, 40%);">+ramstage-y += max98373.c</span><br><span> ramstage-y += ssm4567.c</span><br><span> ramstage-y += rt5514.c</span><br><span> ramstage-y += rt5663.c</span><br><span>@@ -20,6 +21,8 @@</span><br><span> DMIC_4CH_48KHZ_32B = dmic-4ch-48khz-32b.bin</span><br><span> NAU88L25 = nau88l25-2ch-48khz-24b.bin</span><br><span> MAX98357_RENDER = max98357-render-2ch-48khz-24b.bin</span><br><span style="color: hsl(120, 100%, 40%);">+MAX98373_RENDER_16B = max98373-render-2ch-48khz-16b.bin</span><br><span style="color: hsl(120, 100%, 40%);">+MAX98373_RENDER_24B = max98373-render-2ch-48khz-24b.bin</span><br><span> MAX98927_RENDER_24B = max98927-render-2ch-48khz-24b.bin</span><br><span> MAX98927_RENDER_16B = max98927-render-2ch-48khz-16b.bin</span><br><span> RT5514_CAPTURE = rt5514-capture-4ch-48khz-16b.bin</span><br><span>@@ -52,6 +55,14 @@</span><br><span> $(MAX98357_RENDER)-file := $(NHLT_BLOB_PATH)/$(MAX98357_RENDER)</span><br><span> $(MAX98357_RENDER)-type := raw</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+cbfs-files-$(CONFIG_NHLT_MAX98373) += $(MAX98373_RENDER_16B)</span><br><span style="color: hsl(120, 100%, 40%);">+$(MAX98373_RENDER_16B)-file := $(NHLT_BLOB_PATH)/$(MAX98373_RENDER_16B)</span><br><span style="color: hsl(120, 100%, 40%);">+$(MAX98373_RENDER_16B)-type := raw</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+cbfs-files-$(CONFIG_NHLT_MAX98373) += $(MAX98373_RENDER_24B)</span><br><span style="color: hsl(120, 100%, 40%);">+$(MAX98373_RENDER_24B)-file := $(NHLT_BLOB_PATH)/$(MAX98373_RENDER_24B)</span><br><span style="color: hsl(120, 100%, 40%);">+$(MAX98373_RENDER_24B)-type := raw</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> cbfs-files-$(CONFIG_NHLT_SSM4567) += $(SSM4567_RENDER)</span><br><span> $(SSM4567_RENDER)-file := $(NHLT_BLOB_PATH)/$(SSM4567_RENDER)</span><br><span> $(SSM4567_RENDER)-type := raw</span><br><span>diff --git a/src/soc/intel/skylake/nhlt/max98373.c b/src/soc/intel/skylake/nhlt/max98373.c</span><br><span>new file mode 100644</span><br><span>index 0000000..6d357f6</span><br><span>--- /dev/null</span><br><span>+++ b/src/soc/intel/skylake/nhlt/max98373.c</span><br><span>@@ -0,0 +1,77 @@</span><br><span style="color: hsl(120, 100%, 40%);">+/*</span><br><span style="color: hsl(120, 100%, 40%);">+ * This file is part of the coreboot project.</span><br><span style="color: hsl(120, 100%, 40%);">+ *</span><br><span style="color: hsl(120, 100%, 40%);">+ * Copyright 2017 Intel Corporation.</span><br><span style="color: hsl(120, 100%, 40%);">+ *</span><br><span style="color: hsl(120, 100%, 40%);">+ * This program is free software; you can redistribute it and/or modify</span><br><span style="color: hsl(120, 100%, 40%);">+ * it under the terms of the GNU General Public License as published by</span><br><span style="color: hsl(120, 100%, 40%);">+ * the Free Software Foundation; version 2 of the License.</span><br><span style="color: hsl(120, 100%, 40%);">+ *</span><br><span style="color: hsl(120, 100%, 40%);">+ * This program is distributed in the hope that it will be useful,</span><br><span style="color: hsl(120, 100%, 40%);">+ * but WITHOUT ANY WARRANTY; without even the implied warranty of</span><br><span style="color: hsl(120, 100%, 40%);">+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the</span><br><span style="color: hsl(120, 100%, 40%);">+ * GNU General Public License for more details.</span><br><span style="color: hsl(120, 100%, 40%);">+ */</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+#include <soc/nhlt.h></span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+static const struct nhlt_format_config max98373_render_formats[] = {</span><br><span style="color: hsl(120, 100%, 40%);">+       /* 48 KHz 24-bits per sample. */</span><br><span style="color: hsl(120, 100%, 40%);">+      {</span><br><span style="color: hsl(120, 100%, 40%);">+             .num_channels = 2,</span><br><span style="color: hsl(120, 100%, 40%);">+            .sample_freq_khz = 48,</span><br><span style="color: hsl(120, 100%, 40%);">+                .container_bits_per_sample = 32,</span><br><span style="color: hsl(120, 100%, 40%);">+              .valid_bits_per_sample = 24,</span><br><span style="color: hsl(120, 100%, 40%);">+          .speaker_mask = SPEAKER_FRONT_LEFT | SPEAKER_FRONT_RIGHT,</span><br><span style="color: hsl(120, 100%, 40%);">+             .settings_file = "max98373-render-2ch-48khz-24b.bin",</span><br><span style="color: hsl(120, 100%, 40%);">+       },</span><br><span style="color: hsl(120, 100%, 40%);">+    /* 48 KHz 16-bits per sample. */</span><br><span style="color: hsl(120, 100%, 40%);">+      {</span><br><span style="color: hsl(120, 100%, 40%);">+             .num_channels = 2,</span><br><span style="color: hsl(120, 100%, 40%);">+            .sample_freq_khz = 48,</span><br><span style="color: hsl(120, 100%, 40%);">+                .container_bits_per_sample = 16,</span><br><span style="color: hsl(120, 100%, 40%);">+              .valid_bits_per_sample = 16,</span><br><span style="color: hsl(120, 100%, 40%);">+          .speaker_mask = SPEAKER_FRONT_LEFT | SPEAKER_FRONT_RIGHT,</span><br><span style="color: hsl(120, 100%, 40%);">+             .settings_file = "max98373-render-2ch-48khz-16b.bin",</span><br><span style="color: hsl(120, 100%, 40%);">+       }</span><br><span style="color: hsl(120, 100%, 40%);">+};</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+static const struct nhlt_format_config max98373_capture_formats[] = {</span><br><span style="color: hsl(120, 100%, 40%);">+        /* 48 KHz 16-bits per sample. */</span><br><span style="color: hsl(120, 100%, 40%);">+      {</span><br><span style="color: hsl(120, 100%, 40%);">+             .num_channels = 2,</span><br><span style="color: hsl(120, 100%, 40%);">+            .sample_freq_khz = 48,</span><br><span style="color: hsl(120, 100%, 40%);">+                .container_bits_per_sample = 16,</span><br><span style="color: hsl(120, 100%, 40%);">+              .valid_bits_per_sample = 16,</span><br><span style="color: hsl(120, 100%, 40%);">+          .speaker_mask = SPEAKER_FRONT_LEFT | SPEAKER_FRONT_RIGHT,</span><br><span style="color: hsl(120, 100%, 40%);">+             .settings_file = "max98373-render-2ch-48khz-16b.bin",</span><br><span style="color: hsl(120, 100%, 40%);">+       },</span><br><span style="color: hsl(120, 100%, 40%);">+};</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+static const struct nhlt_endp_descriptor max98373_descriptors[] = {</span><br><span style="color: hsl(120, 100%, 40%);">+ {</span><br><span style="color: hsl(120, 100%, 40%);">+             .link = NHLT_LINK_SSP,</span><br><span style="color: hsl(120, 100%, 40%);">+                .device = NHLT_SSP_DEV_I2S,</span><br><span style="color: hsl(120, 100%, 40%);">+           .direction = NHLT_DIR_RENDER,</span><br><span style="color: hsl(120, 100%, 40%);">+         .vid = NHLT_VID,</span><br><span style="color: hsl(120, 100%, 40%);">+              .did = NHLT_DID_SSP,</span><br><span style="color: hsl(120, 100%, 40%);">+          .formats = max98373_render_formats,</span><br><span style="color: hsl(120, 100%, 40%);">+           .num_formats = ARRAY_SIZE(max98373_render_formats),</span><br><span style="color: hsl(120, 100%, 40%);">+   },</span><br><span style="color: hsl(120, 100%, 40%);">+    {</span><br><span style="color: hsl(120, 100%, 40%);">+             .link = NHLT_LINK_SSP,</span><br><span style="color: hsl(120, 100%, 40%);">+                .device = NHLT_SSP_DEV_I2S,</span><br><span style="color: hsl(120, 100%, 40%);">+           .direction = NHLT_DIR_CAPTURE,</span><br><span style="color: hsl(120, 100%, 40%);">+                .vid = NHLT_VID,</span><br><span style="color: hsl(120, 100%, 40%);">+              .did = NHLT_DID_SSP,</span><br><span style="color: hsl(120, 100%, 40%);">+          .formats = max98373_capture_formats,</span><br><span style="color: hsl(120, 100%, 40%);">+          .num_formats = ARRAY_SIZE(max98373_capture_formats),</span><br><span style="color: hsl(120, 100%, 40%);">+       },</span><br><span style="color: hsl(120, 100%, 40%);">+};</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+int nhlt_soc_add_max98373(struct nhlt *nhlt, int hwlink)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+    /* Virtual bus id of SSP links are the hardware port ids proper. */</span><br><span style="color: hsl(120, 100%, 40%);">+   return nhlt_add_ssp_endpoints(nhlt, hwlink, max98373_descriptors,</span><br><span style="color: hsl(120, 100%, 40%);">+                                   ARRAY_SIZE(max98373_descriptors));</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/25367">change 25367</a>. To unsubscribe, or for help writing mail filters, 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/25367"/><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: Ifb6bf2d31fda25b18d9b1ce2bb721255335d55e4 </div>
<div style="display:none"> Gerrit-Change-Number: 25367 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Duncan Laurie <dlaurie@chromium.org> </div>