From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 97441C43441 for ; Wed, 28 Nov 2018 18:09:22 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 69932206B2 for ; Wed, 28 Nov 2018 18:09:22 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 69932206B2 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.intel.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727177AbeK2FLu (ORCPT ); Thu, 29 Nov 2018 00:11:50 -0500 Received: from mga12.intel.com ([192.55.52.136]:19774 "EHLO mga12.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725760AbeK2FLu (ORCPT ); Thu, 29 Nov 2018 00:11:50 -0500 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 28 Nov 2018 10:09:20 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.56,291,1539673200"; d="scan'208";a="288160878" Received: from jahealey-mobl.amr.corp.intel.com (HELO [10.251.138.254]) ([10.251.138.254]) by fmsmga006.fm.intel.com with ESMTP; 28 Nov 2018 10:09:20 -0800 Subject: Re: [RFC PATCH 0/6] ASoC:Intel:Skylake: Enable HDaudio legacy fallback To: Takashi Iwai Cc: alsa-devel@alsa-project.org, broonie@kernel.org, vkoul@kernel.org, liam.r.girdwood@linux.intel.com, andriy.shevchenko@linux.intel.com, arnd@arndb.de, linux-kernel@vger.kernel.org References: <20181120213644.19103-1-pierre-louis.bossart@linux.intel.com> From: Pierre-Louis Bossart Message-ID: <5854417f-5e36-4397-5644-99c9b5f4ed40@linux.intel.com> Date: Wed, 28 Nov 2018 12:09:19 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.2.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/21/18 4:32 AM, Takashi Iwai wrote: > On Tue, 20 Nov 2018 22:36:38 +0100, > Pierre-Louis Bossart wrote: >> This patchset is provided as an RFC and should not be merged as is >> (Turkey break in the USA and more validation needed). This is however >> a good time to gather comments. This work is the result of multiple >> email discussions to finally provide more flexibility for >> distributions to chose whether to stick with the legacy HDaudio driver >> or to enable the SST/Skylake driver for newer platforms (required >> e.g. for digital microphone support) >> >> The patches add support for CoffeeLake, simplify the probe for the >> Skylake driver, introduce more compile-time granularity so that >> platforms can be selected individually and last provide a dynamic >> fallback mechanism when two drivers are registered for the same PCI >> ID. >> >> When the SOF driver is released, the same mechanism will be used to >> enable the SOF-legacy fallback. There will be no plans to enable an >> SOF->SST falback. > This looks like a sensible way to go, thanks for working on this! > > While the fallback stuff might need more testing, the other patches > (addition of CFL-S and split of configs) are rather systematic, so we > can merge this at first soonish. > > And we may need a bit more comments in Kconfig help for the fallback > behavior. Or document it properly and refer to it from Kconfig help. > The git commit log isn't present in the released kernel code, after > all. I will indeed put the dynamic fallback on the back-burner, of course the tests work on NUCs and recent devices but fail on a HP SKL device I tried on, so additional code needs to be added to check if the DSP is present or not, remove silly dependencies on NHLT that make no sense for HDaudio, etc. I'd like however to follow your idea of a 'shared' caps but use it for static mutual-exclusion to start, e.g. to let distributions use legacy for Broadwell but SST for SKL/KBL without requiring the user to play with blacklists.  Once we have this in place, and the additional code added for DSP detection the dynamic fallback is an 'easy' transition. > > > thanks, > > Takashi > >> Pierre-Louis Bossart (4): >> ASoC: Intel: Skylake: stop init/probe if DSP is not present >> ASoC: Intel: Skylake: remove useless tests on DSP presence >> ASoC: Intel: Skylake: Add more platform granularity >> ALSA: hda: add fallback capabilities for SKL+ platforms >> >> Takashi Iwai (2): >> ASoC: Intel: Skylake: Add CFL-S support >> ALSA: hda: Allow fallback binding with legacy HD-audio for Intel SKL+ >> >> include/sound/hdaudio.h | 6 + >> sound/pci/hda/Kconfig | 40 +++++++ >> sound/pci/hda/hda_controller.h | 2 +- >> sound/pci/hda/hda_intel.c | 51 ++++++-- >> sound/soc/intel/Kconfig | 86 ++++++++++++-- >> sound/soc/intel/boards/Kconfig | 16 ++- >> sound/soc/intel/skylake/skl-messages.c | 8 ++ >> sound/soc/intel/skylake/skl.c | 154 +++++++++++++++++++------ >> 8 files changed, 311 insertions(+), 52 deletions(-) >> >> -- >> 2.17.1 >>