Message ID | 20230323054452.1543233-1-yung-chuan.liao@linux.intel.com |
---|---|
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:604a:0:0:0:0:0 with SMTP id j10csp2739261wrt; Wed, 22 Mar 2023 22:56:25 -0700 (PDT) X-Google-Smtp-Source: AK7set/nIvZ8HMWA9SDQ4Klhs9WJn2mO+tXvXfp0WYOkpL8V3LHGMjEC4k7fQ4pF1iO1V5ZOGmBH X-Received: by 2002:a17:903:247:b0:1a0:5671:42a6 with SMTP id j7-20020a170903024700b001a0567142a6mr5910994plh.31.1679550984756; Wed, 22 Mar 2023 22:56:24 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1679550984; cv=none; d=google.com; s=arc-20160816; b=SbcTrRQKBzqbsm3TL8OCg7JPEDfN5yIx7EMcCUQlc0bWYuX/84Aw0mbsO2v4/K1YJk ugWU4P0ye/+p/emDonwH6I6p6xhVCnfCc9Tas52d2J2Nxa02mWTzFAI+inau0SqOOZjN Eo60IjcqxZsEsICaH7m5dWnv45LRzPAz+Eq1fyDtPjs4Wh/IKWTzcSbol4W3z+1vDvnW Y0hYn0fRPqwrrO4a5xespP3Wak2vyZ8q6JJtZ2y6GgbSJuoe2h1C0mUw2ngkdiXhXhDs uX/HBNzWE+v7HY5DQpysPZrkKJhSVnKTvOa+2toghs2K+sHT8IfZZ2JM3GrPqGaB19ZW 3CgA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=YK2zjDYHFm35pPzEBasFz9rIuvJIJ18A4exxJJVdMak=; b=HOoTFp9bKJthqs6aSFnmjY9/HyN7+t4/W3WtSCjs+oLWg+frjI1cjyiBjHOSYSpqRX G+KJ0bu31Seqj4v6zM0J7aI7TEszeXPKyouGYvrJIdc55MHnd6Fbzn+xYVB+iW9zdWaU 9vx16tHyUsMsqK1FlVA4n/eVqpVB9FyJNUVbwV6tjk/2R0Hs6MPrczfgVcZ1AwaPzz6x 6Oh8WAAn2MAH3SPbcze8KJlnHbpYBzYst5CwQ+3aNYnW8M2bS5notKkxC6WUUMcGDl/4 41r6xgkv2iJ3r0qhDMNq+h/LeOEj7JeCeX9mylRRm4mKoIBmZgTd2yzgKFDqhD7eWdwl OCpQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=EZD+AHcw; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id i9-20020a17090332c900b001a1a252c4b1si19142100plr.13.2023.03.22.22.56.12; Wed, 22 Mar 2023 22:56:24 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=EZD+AHcw; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230258AbjCWF3y (ORCPT <rfc822;ezelljr.billy@gmail.com> + 99 others); Thu, 23 Mar 2023 01:29:54 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56744 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229517AbjCWF3w (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Thu, 23 Mar 2023 01:29:52 -0400 Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0B8F820050 for <linux-kernel@vger.kernel.org>; Wed, 22 Mar 2023 22:29:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1679549391; x=1711085391; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=9WAW0zepnYSbfRRjixmplrT7V+yloxeDYYdsHsolz/4=; b=EZD+AHcw/5XWfUXVly0ZTq8Cl+c2h0iUf54H1cs/Ekblkg5c2j5Nou6I n9JcxDrE7wdVC/xD4GJ3noMLixUSVcGKYwMYFqgtUF5L2hBajK/e2AkKC DtHi9MP8ylyo0c+r5uPRFVPoYLdaLp7p9XLlPf1MuBuTpuG3G0KTGQhDb sPluE2HpgwiUFQLJPSYfj6VAiioP4YDsIAP64GiU2hZmy6lMe7pcJraL6 OeAT3OHX3bG9C87vMaoX3xvnxTW9ylEMLx19N2uucKfCwpru7CywF5r+s B4cqL70qsVRUJz1os0ZlNfNO468TrtfuiF1ipuLth2Atm/EiTOaoEZGoq w==; X-IronPort-AV: E=McAfee;i="6600,9927,10657"; a="327779265" X-IronPort-AV: E=Sophos;i="5.98,283,1673942400"; d="scan'208";a="327779265" Received: from orsmga007.jf.intel.com ([10.7.209.58]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 22 Mar 2023 22:29:50 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10657"; a="675566983" X-IronPort-AV: E=Sophos;i="5.98,283,1673942400"; d="scan'208";a="675566983" Received: from bard-ubuntu.sh.intel.com ([10.239.185.57]) by orsmga007-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 22 Mar 2023 22:29:48 -0700 From: Bard Liao <yung-chuan.liao@linux.intel.com> To: alsa-devel@alsa-project.org, vkoul@kernel.org, broonie@kernel.org Cc: vinod.koul@linaro.org, linux-kernel@vger.kernel.org, pierre-louis.bossart@linux.intel.com, bard.liao@intel.com, tiwai@suse.de Subject: [PATCH 00/20] ASoC/soundwire: add support for ACE2.x Date: Thu, 23 Mar 2023 13:44:32 +0800 Message-Id: <20230323054452.1543233-1-yung-chuan.liao@linux.intel.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-2.4 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1761136853824141189?= X-GMAIL-MSGID: =?utf-8?q?1761136853824141189?= |
Series |
ASoC/soundwire: add support for ACE2.x
|
|
Message
Bard Liao
March 23, 2023, 5:44 a.m. UTC
This series uses the abstraction added in past kernel cycles to provide support for the ACE2.x integration. The existing SHIM and Cadence registers are now split in 3 (SHIM, IP, SHIM vendor-specific), with some parts also moved to the HDaudio Extended Multi link structures. Nothing fundamentally different except for the register map. This series only provides the basic mechanisms to expose SoundWire-based DAIs. The PCI parts and DSP management will be contributed later, and the DAI ops are now empty as well. The change is mainly on SoundWire. It would be better to go through SoundWire tree. Pierre-Louis Bossart (20): ASoC: SOF: Intel: shim: add enum for ACE 2.0 IP used in LunarLake soundwire: intel: add ACE2.x SHIM definitions soundwire: intel_ace2x: add empty new ops for LunarLake soundwire/ASOC: Intel: update offsets for LunarLake soundwire: intel/cadence: set ip_offset at run-time ASoC/soundwire: intel: pass hdac_bus pointer for link management soundwire: intel: add eml_lock in the interface for new platforms ASoC: SOF: Intel: hda: retrieve SoundWire eml_lock and pass pointer soundwire: intel_init: use eml_lock parameter soundwire: intel_ace2x: add debugfs support soundwire: intel_ace2x: add link power-up/down helpers soundwire: intel_ace2x: set SYNCPRD before powering-up soundwire: intel_ace2x: configure link PHY soundwire: intel_ace2x: add DAI registration soundwire: intel_ace2x: add sync_arm/sync_go helpers soundwire: intel_ace2x: use common helpers for bus start/stop soundwire: intel_ace2x: enable wake support soundwire: intel_ace2x: add check_cmdsync_unlocked helper soundwire: bus: add new manager callback to deal with peripheral enumeration soundwire: intel_ace2x: add new_peripheral_assigned callback drivers/soundwire/Makefile | 3 +- drivers/soundwire/bus.c | 3 + drivers/soundwire/cadence_master.h | 2 + drivers/soundwire/intel.h | 16 + drivers/soundwire/intel_ace2x.c | 390 ++++++++++++++++++++++++ drivers/soundwire/intel_ace2x_debugfs.c | 147 +++++++++ drivers/soundwire/intel_auxdevice.c | 17 ++ drivers/soundwire/intel_init.c | 21 +- include/linux/soundwire/sdw.h | 3 +- include/linux/soundwire/sdw_intel.h | 88 ++++++ sound/soc/sof/intel/hda.c | 31 +- sound/soc/sof/intel/shim.h | 1 + 12 files changed, 711 insertions(+), 11 deletions(-) create mode 100644 drivers/soundwire/intel_ace2x.c create mode 100644 drivers/soundwire/intel_ace2x_debugfs.c
Comments
The patches will not compile due to missed dependency but welcome review on the code changes. > -----Original Message----- > From: Bard Liao <yung-chuan.liao@linux.intel.com> > Sent: Thursday, March 23, 2023 1:45 PM > To: alsa-devel@alsa-project.org; vkoul@kernel.org; broonie@kernel.org > Cc: vinod.koul@linaro.org; linux-kernel@vger.kernel.org; pierre- > louis.bossart@linux.intel.com; Liao, Bard <bard.liao@intel.com>; > tiwai@suse.de > Subject: [PATCH 00/20] ASoC/soundwire: add support for ACE2.x > > This series uses the abstraction added in past kernel cycles to provide > support for the ACE2.x integration. The existing SHIM and Cadence > registers are now split in 3 (SHIM, IP, SHIM vendor-specific), with some > parts also moved to the HDaudio Extended Multi link structures. Nothing > fundamentally different except for the register map. > > This series only provides the basic mechanisms to expose SoundWire-based > DAIs. The PCI parts and DSP management will be contributed later, and the > DAI ops are now empty as well. > > The change is mainly on SoundWire. It would be better to go through > SoundWire tree. > > Pierre-Louis Bossart (20): > ASoC: SOF: Intel: shim: add enum for ACE 2.0 IP used in LunarLake > soundwire: intel: add ACE2.x SHIM definitions > soundwire: intel_ace2x: add empty new ops for LunarLake > soundwire/ASOC: Intel: update offsets for LunarLake > soundwire: intel/cadence: set ip_offset at run-time > ASoC/soundwire: intel: pass hdac_bus pointer for link management > soundwire: intel: add eml_lock in the interface for new platforms > ASoC: SOF: Intel: hda: retrieve SoundWire eml_lock and pass pointer > soundwire: intel_init: use eml_lock parameter > soundwire: intel_ace2x: add debugfs support > soundwire: intel_ace2x: add link power-up/down helpers > soundwire: intel_ace2x: set SYNCPRD before powering-up > soundwire: intel_ace2x: configure link PHY > soundwire: intel_ace2x: add DAI registration > soundwire: intel_ace2x: add sync_arm/sync_go helpers > soundwire: intel_ace2x: use common helpers for bus start/stop > soundwire: intel_ace2x: enable wake support > soundwire: intel_ace2x: add check_cmdsync_unlocked helper > soundwire: bus: add new manager callback to deal with peripheral > enumeration > soundwire: intel_ace2x: add new_peripheral_assigned callback > > drivers/soundwire/Makefile | 3 +- > drivers/soundwire/bus.c | 3 + > drivers/soundwire/cadence_master.h | 2 + > drivers/soundwire/intel.h | 16 + > drivers/soundwire/intel_ace2x.c | 390 ++++++++++++++++++++++++ > drivers/soundwire/intel_ace2x_debugfs.c | 147 +++++++++ > drivers/soundwire/intel_auxdevice.c | 17 ++ > drivers/soundwire/intel_init.c | 21 +- > include/linux/soundwire/sdw.h | 3 +- > include/linux/soundwire/sdw_intel.h | 88 ++++++ > sound/soc/sof/intel/hda.c | 31 +- > sound/soc/sof/intel/shim.h | 1 + > 12 files changed, 711 insertions(+), 11 deletions(-) > create mode 100644 drivers/soundwire/intel_ace2x.c > create mode 100644 drivers/soundwire/intel_ace2x_debugfs.c > > -- > 2.25.1
On 23-03-23, 13:44, Bard Liao wrote: > This series uses the abstraction added in past kernel cycles to provide > support for the ACE2.x integration. The existing SHIM and Cadence ACE2..? > registers are now split in 3 (SHIM, IP, SHIM vendor-specific), with some > parts also moved to the HDaudio Extended Multi link structures. Nothing > fundamentally different except for the register map. > > This series only provides the basic mechanisms to expose SoundWire-based > DAIs. The PCI parts and DSP management will be contributed later, and the > DAI ops are now empty as well. > > The change is mainly on SoundWire. It would be better to go through > SoundWire tree. > > Pierre-Louis Bossart (20): > ASoC: SOF: Intel: shim: add enum for ACE 2.0 IP used in LunarLake > soundwire: intel: add ACE2.x SHIM definitions > soundwire: intel_ace2x: add empty new ops for LunarLake > soundwire/ASOC: Intel: update offsets for LunarLake > soundwire: intel/cadence: set ip_offset at run-time > ASoC/soundwire: intel: pass hdac_bus pointer for link management > soundwire: intel: add eml_lock in the interface for new platforms > ASoC: SOF: Intel: hda: retrieve SoundWire eml_lock and pass pointer > soundwire: intel_init: use eml_lock parameter > soundwire: intel_ace2x: add debugfs support > soundwire: intel_ace2x: add link power-up/down helpers > soundwire: intel_ace2x: set SYNCPRD before powering-up > soundwire: intel_ace2x: configure link PHY > soundwire: intel_ace2x: add DAI registration > soundwire: intel_ace2x: add sync_arm/sync_go helpers > soundwire: intel_ace2x: use common helpers for bus start/stop > soundwire: intel_ace2x: enable wake support > soundwire: intel_ace2x: add check_cmdsync_unlocked helper > soundwire: bus: add new manager callback to deal with peripheral > enumeration > soundwire: intel_ace2x: add new_peripheral_assigned callback > > drivers/soundwire/Makefile | 3 +- > drivers/soundwire/bus.c | 3 + > drivers/soundwire/cadence_master.h | 2 + > drivers/soundwire/intel.h | 16 + > drivers/soundwire/intel_ace2x.c | 390 ++++++++++++++++++++++++ > drivers/soundwire/intel_ace2x_debugfs.c | 147 +++++++++ > drivers/soundwire/intel_auxdevice.c | 17 ++ > drivers/soundwire/intel_init.c | 21 +- > include/linux/soundwire/sdw.h | 3 +- > include/linux/soundwire/sdw_intel.h | 88 ++++++ > sound/soc/sof/intel/hda.c | 31 +- > sound/soc/sof/intel/shim.h | 1 + > 12 files changed, 711 insertions(+), 11 deletions(-) > create mode 100644 drivers/soundwire/intel_ace2x.c > create mode 100644 drivers/soundwire/intel_ace2x_debugfs.c > > -- > 2.25.1
On 4/12/23 05:07, Vinod Koul wrote: > On 23-03-23, 13:44, Bard Liao wrote: >> This series uses the abstraction added in past kernel cycles to provide >> support for the ACE2.x integration. The existing SHIM and Cadence > > ACE2..? Not sure I get the comment, this refers to the IP version. ACE 2.0 IP used in LunarLake, x refers to future TDB versions. BTW this depends on the series "[PATCH v2 00/18] ASoC: SOF: Intel: hda-mlink: HDaudio multi-link extension update" applied yesterday by Mark. It'd be really nice if we can have this in 6.4-rc1 to continue adding the rest of the Lunar Lake patches. We tried to limit the number of changes impacting both ASoC and SoundWire subsystems but there's mechanically 2 merges needed (ASoC -> SoundWire for the hda-mlink dependency and SoundWire->ASoC for the interface changes below). > >> registers are now split in 3 (SHIM, IP, SHIM vendor-specific), with some >> parts also moved to the HDaudio Extended Multi link structures. Nothing >> fundamentally different except for the register map. >> >> This series only provides the basic mechanisms to expose SoundWire-based >> DAIs. The PCI parts and DSP management will be contributed later, and the >> DAI ops are now empty as well. >> >> The change is mainly on SoundWire. It would be better to go through >> SoundWire tree. >> >> Pierre-Louis Bossart (20): >> ASoC: SOF: Intel: shim: add enum for ACE 2.0 IP used in LunarLake >> soundwire: intel: add ACE2.x SHIM definitions >> soundwire: intel_ace2x: add empty new ops for LunarLake >> soundwire/ASOC: Intel: update offsets for LunarLake >> soundwire: intel/cadence: set ip_offset at run-time >> ASoC/soundwire: intel: pass hdac_bus pointer for link management >> soundwire: intel: add eml_lock in the interface for new platforms >> ASoC: SOF: Intel: hda: retrieve SoundWire eml_lock and pass pointer >> soundwire: intel_init: use eml_lock parameter >> soundwire: intel_ace2x: add debugfs support >> soundwire: intel_ace2x: add link power-up/down helpers >> soundwire: intel_ace2x: set SYNCPRD before powering-up >> soundwire: intel_ace2x: configure link PHY >> soundwire: intel_ace2x: add DAI registration >> soundwire: intel_ace2x: add sync_arm/sync_go helpers >> soundwire: intel_ace2x: use common helpers for bus start/stop >> soundwire: intel_ace2x: enable wake support >> soundwire: intel_ace2x: add check_cmdsync_unlocked helper >> soundwire: bus: add new manager callback to deal with peripheral >> enumeration >> soundwire: intel_ace2x: add new_peripheral_assigned callback >> >> drivers/soundwire/Makefile | 3 +- >> drivers/soundwire/bus.c | 3 + >> drivers/soundwire/cadence_master.h | 2 + >> drivers/soundwire/intel.h | 16 + >> drivers/soundwire/intel_ace2x.c | 390 ++++++++++++++++++++++++ >> drivers/soundwire/intel_ace2x_debugfs.c | 147 +++++++++ >> drivers/soundwire/intel_auxdevice.c | 17 ++ >> drivers/soundwire/intel_init.c | 21 +- >> include/linux/soundwire/sdw.h | 3 +- >> include/linux/soundwire/sdw_intel.h | 88 ++++++ >> sound/soc/sof/intel/hda.c | 31 +- >> sound/soc/sof/intel/shim.h | 1 + >> 12 files changed, 711 insertions(+), 11 deletions(-) >> create mode 100644 drivers/soundwire/intel_ace2x.c >> create mode 100644 drivers/soundwire/intel_ace2x_debugfs.c >> >> -- >> 2.25.1 >
On 12-04-23, 09:11, Pierre-Louis Bossart wrote: > > > On 4/12/23 05:07, Vinod Koul wrote: > > On 23-03-23, 13:44, Bard Liao wrote: > >> This series uses the abstraction added in past kernel cycles to provide > >> support for the ACE2.x integration. The existing SHIM and Cadence > > > > ACE2..? > > Not sure I get the comment, this refers to the IP version. What does ACE mean, it should be expanded so that people know what it might refer to..? > > ACE 2.0 IP used in LunarLake, x refers to future TDB versions. > > BTW this depends on the series "[PATCH v2 00/18] ASoC: SOF: Intel: > hda-mlink: HDaudio multi-link extension update" applied yesterday by > Mark. It'd be really nice if we can have this in 6.4-rc1 to continue > adding the rest of the Lunar Lake patches. That should be mentioned in cover, I can ack this if okay and Mark might apply..? or u folks have something else in mind? > > We tried to limit the number of changes impacting both ASoC and > SoundWire subsystems but there's mechanically 2 merges needed (ASoC -> > SoundWire for the hda-mlink dependency and SoundWire->ASoC for the > interface changes below). > > > > >> registers are now split in 3 (SHIM, IP, SHIM vendor-specific), with some > >> parts also moved to the HDaudio Extended Multi link structures. Nothing > >> fundamentally different except for the register map. > >> > >> This series only provides the basic mechanisms to expose SoundWire-based > >> DAIs. The PCI parts and DSP management will be contributed later, and the > >> DAI ops are now empty as well. > >> > >> The change is mainly on SoundWire. It would be better to go through > >> SoundWire tree. > >> > >> Pierre-Louis Bossart (20): > >> ASoC: SOF: Intel: shim: add enum for ACE 2.0 IP used in LunarLake > >> soundwire: intel: add ACE2.x SHIM definitions > >> soundwire: intel_ace2x: add empty new ops for LunarLake > >> soundwire/ASOC: Intel: update offsets for LunarLake > >> soundwire: intel/cadence: set ip_offset at run-time > >> ASoC/soundwire: intel: pass hdac_bus pointer for link management > >> soundwire: intel: add eml_lock in the interface for new platforms > >> ASoC: SOF: Intel: hda: retrieve SoundWire eml_lock and pass pointer > >> soundwire: intel_init: use eml_lock parameter > >> soundwire: intel_ace2x: add debugfs support > >> soundwire: intel_ace2x: add link power-up/down helpers > >> soundwire: intel_ace2x: set SYNCPRD before powering-up > >> soundwire: intel_ace2x: configure link PHY > >> soundwire: intel_ace2x: add DAI registration > >> soundwire: intel_ace2x: add sync_arm/sync_go helpers > >> soundwire: intel_ace2x: use common helpers for bus start/stop > >> soundwire: intel_ace2x: enable wake support > >> soundwire: intel_ace2x: add check_cmdsync_unlocked helper > >> soundwire: bus: add new manager callback to deal with peripheral > >> enumeration > >> soundwire: intel_ace2x: add new_peripheral_assigned callback > >> > >> drivers/soundwire/Makefile | 3 +- > >> drivers/soundwire/bus.c | 3 + > >> drivers/soundwire/cadence_master.h | 2 + > >> drivers/soundwire/intel.h | 16 + > >> drivers/soundwire/intel_ace2x.c | 390 ++++++++++++++++++++++++ > >> drivers/soundwire/intel_ace2x_debugfs.c | 147 +++++++++ > >> drivers/soundwire/intel_auxdevice.c | 17 ++ > >> drivers/soundwire/intel_init.c | 21 +- > >> include/linux/soundwire/sdw.h | 3 +- > >> include/linux/soundwire/sdw_intel.h | 88 ++++++ > >> sound/soc/sof/intel/hda.c | 31 +- > >> sound/soc/sof/intel/shim.h | 1 + > >> 12 files changed, 711 insertions(+), 11 deletions(-) > >> create mode 100644 drivers/soundwire/intel_ace2x.c > >> create mode 100644 drivers/soundwire/intel_ace2x_debugfs.c > >> > >> -- > >> 2.25.1 > >
On 4/12/23 10:22, Vinod Koul wrote: > On 12-04-23, 09:11, Pierre-Louis Bossart wrote: >> >> >> On 4/12/23 05:07, Vinod Koul wrote: >>> On 23-03-23, 13:44, Bard Liao wrote: >>>> This series uses the abstraction added in past kernel cycles to provide >>>> support for the ACE2.x integration. The existing SHIM and Cadence >>> >>> ACE2..? >> >> Not sure I get the comment, this refers to the IP version. > > What does ACE mean, it should be expanded so that people know what it > might refer to..? I think at one point it stood for "Audio Converged Engine", which doesn't really clarify anything, does it? It's easier for most people to just refer to cAVS or ACE as a type of IP followed by a version number, and use the mapping between the IP version and end-user device in the PCI descriptors. >> ACE 2.0 IP used in LunarLake, x refers to future TDB versions. >> >> BTW this depends on the series "[PATCH v2 00/18] ASoC: SOF: Intel: >> hda-mlink: HDaudio multi-link extension update" applied yesterday by >> Mark. It'd be really nice if we can have this in 6.4-rc1 to continue >> adding the rest of the Lunar Lake patches. > > That should be mentioned in cover, Yes the cover and some patches miss details, it's hard when you have such a long series to be clear and avoid repetition at the same time... > I can ack this if okay and Mark might apply..? or u folks have something else in mind? that would be ideal, but unfortunately this series will not apply as is on Mark's for-next branch. I just checked, there are multiple conflicts with the intel_bus_common stuff which is missing in Mark's tree. Mark also provided his Acked-by tag already for patches 1/4/6/8, so I was under the impression that these patches would be merged through the soundwire tree. I think what's missing is a tag from Mark to make sure the hda-mlink dependencies are in the soundwire tree? >> >> We tried to limit the number of changes impacting both ASoC and >> SoundWire subsystems but there's mechanically 2 merges needed (ASoC -> >> SoundWire for the hda-mlink dependency and SoundWire->ASoC for the >> interface changes below). >> >>> >>>> registers are now split in 3 (SHIM, IP, SHIM vendor-specific), with some >>>> parts also moved to the HDaudio Extended Multi link structures. Nothing >>>> fundamentally different except for the register map. >>>> >>>> This series only provides the basic mechanisms to expose SoundWire-based >>>> DAIs. The PCI parts and DSP management will be contributed later, and the >>>> DAI ops are now empty as well. >>>> >>>> The change is mainly on SoundWire. It would be better to go through >>>> SoundWire tree. >>>> >>>> Pierre-Louis Bossart (20): >>>> ASoC: SOF: Intel: shim: add enum for ACE 2.0 IP used in LunarLake >>>> soundwire: intel: add ACE2.x SHIM definitions >>>> soundwire: intel_ace2x: add empty new ops for LunarLake >>>> soundwire/ASOC: Intel: update offsets for LunarLake >>>> soundwire: intel/cadence: set ip_offset at run-time >>>> ASoC/soundwire: intel: pass hdac_bus pointer for link management >>>> soundwire: intel: add eml_lock in the interface for new platforms >>>> ASoC: SOF: Intel: hda: retrieve SoundWire eml_lock and pass pointer >>>> soundwire: intel_init: use eml_lock parameter >>>> soundwire: intel_ace2x: add debugfs support >>>> soundwire: intel_ace2x: add link power-up/down helpers >>>> soundwire: intel_ace2x: set SYNCPRD before powering-up >>>> soundwire: intel_ace2x: configure link PHY >>>> soundwire: intel_ace2x: add DAI registration >>>> soundwire: intel_ace2x: add sync_arm/sync_go helpers >>>> soundwire: intel_ace2x: use common helpers for bus start/stop >>>> soundwire: intel_ace2x: enable wake support >>>> soundwire: intel_ace2x: add check_cmdsync_unlocked helper >>>> soundwire: bus: add new manager callback to deal with peripheral >>>> enumeration >>>> soundwire: intel_ace2x: add new_peripheral_assigned callback >>>> >>>> drivers/soundwire/Makefile | 3 +- >>>> drivers/soundwire/bus.c | 3 + >>>> drivers/soundwire/cadence_master.h | 2 + >>>> drivers/soundwire/intel.h | 16 + >>>> drivers/soundwire/intel_ace2x.c | 390 ++++++++++++++++++++++++ >>>> drivers/soundwire/intel_ace2x_debugfs.c | 147 +++++++++ >>>> drivers/soundwire/intel_auxdevice.c | 17 ++ >>>> drivers/soundwire/intel_init.c | 21 +- >>>> include/linux/soundwire/sdw.h | 3 +- >>>> include/linux/soundwire/sdw_intel.h | 88 ++++++ >>>> sound/soc/sof/intel/hda.c | 31 +- >>>> sound/soc/sof/intel/shim.h | 1 + >>>> 12 files changed, 711 insertions(+), 11 deletions(-) >>>> create mode 100644 drivers/soundwire/intel_ace2x.c >>>> create mode 100644 drivers/soundwire/intel_ace2x_debugfs.c >>>> >>>> -- >>>> 2.25.1 >>> >