From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH v2 0/2] ALSA: hda: add module flag snd_adsp_enable Date: Fri, 19 Jun 2015 13:37:22 +0200 Message-ID: References: <1434609505-776-1-git-send-email-vinod.koul@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 (cantor2.suse.de [195.135.220.15]) by alsa0.perex.cz (Postfix) with ESMTP id 76304260525 for ; Fri, 19 Jun 2015 13:37:23 +0200 (CEST) In-Reply-To: <1434609505-776-1-git-send-email-vinod.koul@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: Vinod Koul Cc: liam.r.girdwood@linux.intel.com, patches.audio@intel.com, alsa-devel@alsa-project.org, broonie@kernel.org List-Id: alsa-devel@alsa-project.org At Thu, 18 Jun 2015 12:08:23 +0530, Vinod Koul wrote: > > This is second attempt for creating a module option for hda core to > select either legacy HDA driver or upcoming ASoC driver. This series > still relies on module flag (with feedback from previous series baked > in). The eventual plan is to rely on HW advertised capabilities and > BIOS configuration to load either driver, but since ASoC driver has > still some time to get fully merged and matured on various system, we > will rely on module flag till the who ASoC HDA systems is completed > and available is all distros Well, somehow I prefer postponing this. The method would work but is indeed hackish, so let's put this together with other pieces as a last resort unless any better solution is found. For the driver development itself, this isn't really needed; you can put blacklist at any time. thanks, Takashi > > Vinod Koul (2): > ALSA: hda: add snd_adsp_enable module flag > ALSA: hda: use snd_adsp_enable in HDA driver > > include/sound/hdaudio.h | 11 +++++++++++ > sound/hda/hda_bus_type.c | 6 ++++++ > sound/pci/hda/hda_controller.h | 1 + > sound/pci/hda/hda_intel.c | 10 +++++++++- > 4 files changed, 27 insertions(+), 1 deletion(-) > > -- > 1.9.1 >