From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755600AbcA2LeC (ORCPT ); Fri, 29 Jan 2016 06:34:02 -0500 Received: from mail.karo-electronics.de ([81.173.242.67]:52720 "EHLO mail.karo-electronics.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751536AbcA2Ld7 convert rfc822-to-8bit (ORCPT ); Fri, 29 Jan 2016 06:33:59 -0500 Date: Fri, 29 Jan 2016 12:33:41 +0100 From: Lothar =?UTF-8?B?V2HDn21hbm4=?= To: Nicolin Chen Cc: Mark Brown , Arnd Bergmann , Fabio Estevam , Jaroslav Kysela , Liam Girdwood , Takashi Iwai , alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] ASoC: fsl: select SND_SOC_FSL_SAI or SND_SOC_FSL_SSI depending on SoC type Message-ID: <20160129123341.64699d68@ipc1.ka-ro> In-Reply-To: <20160129062529.GA3370@Asurada-CZ80> References: <1453293038-16032-1-git-send-email-LW@KARO-electronics.de> <20160128223352.GB4130@sirena.org.uk> <20160128230846.GA32388@Asurada-Nvidia> <20160129065133.7338cf2c@ipc1.ka-ro> <20160129062529.GA3370@Asurada-CZ80> Organization: Ka-Ro electronics GmbH MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On Thu, 28 Jan 2016 22:25:30 -0800 Nicolin Chen wrote: > On Fri, Jan 29, 2016 at 06:51:33AM +0100, Lothar Waßmann wrote: > > Hi, > > > > On Thu, 28 Jan 2016 15:08:47 -0800 Nicolin Chen wrote: > > > On Thu, Jan 28, 2016 at 11:33:52PM +0100, Mark Brown wrote: > > > > On Wed, Jan 20, 2016 at 01:30:38PM +0100, Lothar Waßmann wrote: > > > > > > > > > - select SND_SOC_FSL_SSI > > > > > + select SND_SOC_FSL_SAI if SOC_IMX6UL > > > > > + select SND_SOC_FSL_SSI if SOC_IMX6Q || SOC_IMX6SL || SOC_IMX6SX > > > > > > > > Does this card not work for older i.MXs (which had the same SSI/AUDMUX > > > > combination as the majority of the i.MX6 family) as well? > > > > > > It's widely used in older i.MXs according to their DTS files. So > > > it should be safer to just leave SSI over here. > > > > > Nothing prevents you to enable the SSI driver on any i.MX6 module. > > What about i.MX2, 3 and 5 series? Will they still be able to enable > SSI without changing .config file as usual? > > > The only change I made is to select the SAI driver instead of the SSI > > driver on i.MX6UL because the i.MX6UL does not have an SSI unit! > > > > > And I actually doubt the feasibility of running this driver with > > > i.MX6UL as there might not be an AUDMUX on i.MX6UL since it does > > > not have SSI any more while the driver always touches the address > > > space of AUDMUX which may not exist on i.MX6UL. > > > > > #define this driver > > Your change is applied to "SND_SOC_IMX_SGTL5000" whose corresponding > driver is the imx-sgtl5000.c file. > > > The FSL_SAI driver does not touch the audmux address space in any way. > > This imx-sgtl5000 driver does access AUDMUX registers. And I thought > you tried to use this one which made me very confused. > That change is obsolete, because as Mark Brown suggested, the simple-card driver works perfectly well. > > The simple-card audio driver works perfectly well with the FSL_SAI > > driver on i.MX6UL. > > It seems like you are using simple-card while letting the other driver > (imx-sgtl5000) select SAI for you. It doesn't sound so right to me. > That was before I found out that the simple-card driver works. Lothar Waßmann