From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752355AbcHLLhO (ORCPT ); Fri, 12 Aug 2016 07:37:14 -0400 Received: from mga04.intel.com ([192.55.52.120]:24742 "EHLO mga04.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751895AbcHLLhN (ORCPT ); Fri, 12 Aug 2016 07:37:13 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.28,510,1464678000"; d="scan'208";a="1013198338" Subject: Re: Correct modules for Bay Trail MAX98090 soc? To: Mark Brown References: <88774f89-ce4a-1877-db49-b018d725f1cd@redhat.com> <3a548725-5b52-4b3e-457a-f6ca93645b15@linux.intel.com> <20160812095317.GZ9347@sirena.org.uk> Cc: Laura Abbott , Jie Yang , Takashi Iwai , Vinod Koul , Liam Girdwood , alsa-devel@alsa-project.org, Linux Kernel Mailing List From: Pierre-Louis Bossart Message-ID: <4cba0e60-2dd4-38bd-6353-a6bf5b404d75@linux.intel.com> Date: Fri, 12 Aug 2016 06:37:11 -0500 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <20160812095317.GZ9347@sirena.org.uk> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 8/12/16 4:53 AM, Mark Brown wrote: > On Thu, Aug 11, 2016 at 06:31:27PM -0500, Pierre-Louis Bossart wrote: >> On 8/11/16 3:42 PM, Laura Abbott wrote: > >>> which changed the dependencies for CONFIG_SND_SOC_INTEL_BYT_MAX98090_MACH. >>> The set of options Fedora selects means that >>> CONFIG_SND_SOC_INTEL_BYT_MAX98090_MACH >>> can't be selected. Is there another driver that's supposed to replace >>> CONFIG_SND_SOC_INTEL_BYT_MAX98090_MACH on Bay Trail or do the dependencies >>> need to be updated? The bugzilla has alsa-info for working and non- >>> working cases and the Fedora config is attached. > >> If you remove support for all other baytrail options this driver should >> still be there and selectable. We just can't support both this driver for >> Chromebooks and the rest for other machines with the same distribution at >> the moment. > > That sounds like a regression, what's the plan to fix it. The simple fix is easy: disable all other codecs and the BYT_MAX98090 option will be enabled. BYT_MAX98090 relies on the 'old' non-dpcm driver which is used only for Chromebooks with Baytrail, which never enable any other codecs, so there was never any issue before. If there is a need for concurrency, then a new machine driver based on the dpcm Atom driver needs to be created. I don't have a Baytrail chromebook so don't want to commit on the change.