From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH v3 00/13] Enable HDA Codec support on Intel Platforms Date: Sat, 02 Jun 2018 11:00:01 +0200 Message-ID: References: <20180602035401.26934-1-pierre-louis.bossart@linux.intel.com> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mx2.suse.de (mx2.suse.de [195.135.220.15]) by alsa0.perex.cz (Postfix) with ESMTP id CC5702678B3 for ; Sat, 2 Jun 2018 11:00:02 +0200 (CEST) In-Reply-To: <20180602035401.26934-1-pierre-louis.bossart@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Pierre-Louis Bossart Cc: liam.r.girdwood@linux.intel.com, alsa-devel@alsa-project.org, broonie@kernel.org, rakesh.a.ughreja@intel.com, vkoul@kernel.org List-Id: alsa-devel@alsa-project.org On Sat, 02 Jun 2018 05:53:48 +0200, Pierre-Louis Bossart wrote: > > Many Intel platforms (SKL, KBL) etc. in the market supports enhanced > audio capabilities which also includes DSP processing. The default > HDaudio legacy driver does not allow for the use of the DSP, this > patch set makes it possible while reusing existing code for HDAudio > codecs and without significant changes to the legacy driver. > > This v3 is not split into two batches as done for v1 and v2, but keeps > the same logical progression. The first three patches are mostly data > structure changes, the DSP support capability is added then with an > ASoC HDA driver and the last patches are fixes required for > Skylake+. The changes to the HDAudio legacy driver are minimal. > > Tests were run successfully on multiple platforms (Dell XPS13, KBL > NUC, APL NUC and LeafHill reference board). > > Credits: all the initial code was written by Rakesh Ughreja, the > rebase to broonie/for-next, cleanups and additional tests were done by > Pierre Bossart. > > Changes v3: > - port to component model > - additional tests on ApolloLake and KabyLake NUC devices > - cleanups (alignment, typos, etc) > > Changes v2: > - Resolved review comments and rebased to latest kernel. > - added module load support for codec drivers. > > Rakesh Ughreja (13): > ALSA: hdac: Remove usage of struct hdac_ext_device and use hdac_device > instead > ALSA: hdac: Remove usage of struct hdac_ext_bus and use hdac_bus > instead > ALSA: hdac: Remove usage of struct hdac_ext_driver, use hdac_driver > instead > ASoC: Intel: Boards: Machine driver for SKL+ w/ HDAudio codecs > ASoC: Intel: Skylake: Add entry in sst_acpi_mach for HDA codecs > ASoC: Intel: Skylake: add HDA BE DAIs > ASoC: Intel: Skylake: use hda_bus instead of hdac_bus > ALSA: hda: split snd_hda_codec_new function > ALSA: hdac: remove memory allocation from snd_hdac_ext_bus_device_init > ALSA: hdac: add extended ops in the hdac_bus > ASoC: hdac_hda: add asoc extension for legacy HDA codec drivers > ALSA: hdac: ext: add wait for codec to respond after link reset > ASoC: Intel: Skylake: fix widget handling Thanks for the patches. But, from obvious reasons, we can't take these for the next merge window. So it's really no good timing for such a big series... In anyways, the patch series look like a mixture of lots of different things. Maybe it's better to split to multiple series, namely: - Individual fixes Patches 12 and 13 are irrelevant with the whole story, and they are merely fixes, can be applied individually. (the patch13 might be depending on others, though; didn't take a deeper look at it yet.) - The removal of hdac_ext_* structs These are basically no functional changes and local in hdac_ext. The preliminary work. The regression test is mandatory. - Some code split / exports in snd_hda_codec_*(), adaption of hdac_bus extended ops Another preliminary work and a setup for the new binding. They should be also no functional changes, and the existing setup should work as is. The regression test is mandatory. - The new hdac_hdmi codec support and the new board codes These are the new stuff, and applied after the preparations above. ... and these patches are touching both ASoC and HD-audio legacy, we'd need the coordinated patch application. That is, we'd need a topic branch that will be merged to both Mark and my trees. We can branch it off after 4.18-rc1 release, for example, as a clean start point. thanks, Takashi