From f780b7c576bc7df13b706c649ac8b6879c9e4958 Mon Sep 17 00:00:00 2001 From: Jeremy Compostella Date: Thu, 26 Feb 2026 09:17:26 -0800 Subject: [PATCH] soc/intel/tigerlake: Use common SoundWire driver Migrate Tiger Lake to use the common SoundWire driver implementation from the Intel common feature code. This change eliminates platform- specific code by leveraging the shared soundwire.c driver. This commit: - Selects SOC_INTEL_COMMON_FEATURE_SOUNDWIRE Kconfig - Removes src/soc/intel/tigerlake/soundwire.c - Updates Makefile to remove soundwire.c compilation Tiger Lake uses the default values (4 SoundWire master links with ACPI address 0x40000000). Change-Id: Ife743d28c8760d9de3f593f7d8caafd7a73efe3a Signed-off-by: Jeremy Compostella Reviewed-on: https://review.coreboot.org/c/coreboot/+/91280 Tested-by: build bot (Jenkins) Reviewed-by: Huang, Cliff --- src/soc/intel/tigerlake/Kconfig | 1 + src/soc/intel/tigerlake/Makefile.mk | 1 - src/soc/intel/tigerlake/soundwire.c | 70 ----------------------------- 3 files changed, 1 insertion(+), 71 deletions(-) delete mode 100644 src/soc/intel/tigerlake/soundwire.c diff --git a/src/soc/intel/tigerlake/Kconfig b/src/soc/intel/tigerlake/Kconfig index 5168546661..b136c3354b 100644 --- a/src/soc/intel/tigerlake/Kconfig +++ b/src/soc/intel/tigerlake/Kconfig @@ -74,6 +74,7 @@ config SOC_INTEL_TIGERLAKE select SOC_INTEL_COMMON_BLOCK_XHCI_ELOG select SOC_INTEL_COMMON_FEATURE select SOC_INTEL_COMMON_FEATURE_GSPI_DEVFN + select SOC_INTEL_COMMON_FEATURE_SOUNDWIRE select SOC_INTEL_COMMON_FEATURE_SPI_DEVFN select SOC_INTEL_COMMON_FEATURE_SPI_DEVFN_PSF select SOC_INTEL_COMMON_FEATURE_I2C_DEVFN diff --git a/src/soc/intel/tigerlake/Makefile.mk b/src/soc/intel/tigerlake/Makefile.mk index 074f172d38..973ad0f015 100644 --- a/src/soc/intel/tigerlake/Makefile.mk +++ b/src/soc/intel/tigerlake/Makefile.mk @@ -34,7 +34,6 @@ ramstage-y += pcie_rp.c ramstage-y += pmc.c ramstage-y += reset.c ramstage-y += retimer.c -ramstage-y += soundwire.c ramstage-y += systemagent.c ramstage-y += tcss.c ramstage-y += xhci.c diff --git a/src/soc/intel/tigerlake/soundwire.c b/src/soc/intel/tigerlake/soundwire.c deleted file mode 100644 index 878ce1973c..0000000000 --- a/src/soc/intel/tigerlake/soundwire.c +++ /dev/null @@ -1,70 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ - -#include -#include -#include -#include -#include -#include -#include - -static const struct soundwire_link link_xtal_38_4 = { - .clock_stop_mode0_supported = 1, - .clock_stop_mode1_supported = 1, - .clock_frequencies_supported_count = 1, - .clock_frequencies_supported = { 4800 * KHz }, - .default_frame_rate = 48 * KHz, - .default_frame_row_size = 50, - .default_frame_col_size = 4, - .dynamic_frame_shape = 1, - .command_error_threshold = 16, -}; - -static const struct soundwire_link link_xtal_24 = { - .clock_stop_mode0_supported = 1, - .clock_stop_mode1_supported = 1, - .clock_frequencies_supported_count = 1, - .clock_frequencies_supported = { 6 * MHz }, - .default_frame_rate = 48 * KHz, - .default_frame_row_size = 125, - .default_frame_col_size = 2, - .dynamic_frame_shape = 1, - .command_error_threshold = 16, -}; - -static struct intel_soundwire_controller intel_controller = { - .acpi_address = 0x40000000, - .sdw = { - .master_list_count = 4 - } -}; - -int soc_fill_soundwire_controller(struct intel_soundwire_controller **controller) -{ - const struct soundwire_link *link; - enum pch_pmc_xtal xtal = pmc_get_xtal_freq(); - size_t i; - - /* Select link config based on XTAL frequency and set IP clock. */ - switch (xtal) { - case XTAL_24_MHZ: - link = &link_xtal_24; - intel_controller.ip_clock = 24 * MHz; - break; - case XTAL_38_4_MHZ: - link = &link_xtal_38_4; - intel_controller.ip_clock = 38400 * KHz; - break; - case XTAL_19_2_MHZ: - default: - printk(BIOS_ERR, "%s: XTAL not supported: 0x%x\n", __func__, xtal); - return -1; - } - - /* Fill link config in controller map based on selected XTAL. */ - for (i = 0; i < intel_controller.sdw.master_list_count; i++) - memcpy(&intel_controller.sdw.master_list[i], link, sizeof(*link)); - - *controller = &intel_controller; - return 0; -}