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=-0.8 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no 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 DFDBAC11D0C for ; Thu, 20 Feb 2020 20:14:59 +0000 (UTC) Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 6B5F220722 for ; Thu, 20 Feb 2020 20:14:59 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=alsa-project.org header.i=@alsa-project.org header.b="mR/K4qoc" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6B5F220722 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=atomide.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=alsa-devel-bounces@alsa-project.org Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id CE653166D; Thu, 20 Feb 2020 21:14:07 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz CE653166D DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1582229697; bh=+BZJa9jz17pMSHqRVluYuLLqh+qPRjpdYrpH4fC9no4=; h=Date:From:To:Subject:References:In-Reply-To:Cc:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=mR/K4qocpWDlp+zAOzuCZ8J5wlDFC45FpEpeTQYC8dd4OVaOJjGDl10K9ElQEFEqR MZKlR+VG1R/h/9ImtCTSsyextz5p6zOAtXYI1VshceeTMd8E4P+MEaEhzKwD4PiLNF TkvYXDpUejgsDqOvFSE9qio/NJLy1CZ5CRTO8xhU= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id 420DCF80101; Thu, 20 Feb 2020 21:14:07 +0100 (CET) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 6BE38F80148; Thu, 20 Feb 2020 21:14:05 +0100 (CET) Received: from muru.com (muru.com [72.249.23.125]) by alsa1.perex.cz (Postfix) with ESMTP id 1AEAAF8011D for ; Thu, 20 Feb 2020 21:13:44 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 1AEAAF8011D Received: from atomide.com (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id 055CE8080; Thu, 20 Feb 2020 20:14:25 +0000 (UTC) Date: Thu, 20 Feb 2020 12:13:38 -0800 From: Tony Lindgren To: Peter Ujfalusi Subject: Re: [PATCH] ASoC: ti: Allocate dais dynamically for TDM and audio graph card Message-ID: <20200220201338.GW37466@atomide.com> References: <20200211171645.41990-1-tony@atomide.com> <20200212143543.GI64767@atomide.com> <346dfd2b-23f8-87e0-6f45-27a5099b1066@ti.com> <20200214170322.GZ64767@atomide.com> <20200217231001.GC35972@atomide.com> <5402eba8-4f84-0973-e11b-6ab2667ada85@ti.com> <20200218152833.GH35972@atomide.com> <40a803e2-6a08-2c73-0312-666441716daa@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <40a803e2-6a08-2c73-0312-666441716daa@ti.com> Cc: alsa-devel@alsa-project.org, linux-omap@vger.kernel.org, Kuninori Morimoto , Aaro Koskinen , linux-kernel@vger.kernel.org, Merlijn Wajer , Takashi Iwai , Liam Girdwood , Mark Brown , Pavel Machek , Sebastian Reichel , "Arthur D ." , Jarkko Nikula X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" * Peter Ujfalusi [200220 14:08]: > On 18/02/2020 17.28, Tony Lindgren wrote: > > Right. I'm not attached to the dummy dai, but looks like currently > > snd-soc-audio-graph-card won't work without it. > > The generic cards will link up a dummy dai/codec when it is needed by DPMC. Not sure what should be fixed here.. > > And we potentially > > do need a place to configure TDM slot specific stuff for mcbsp. > > Yes, but you still have one port and one endpoint should not change the > configuration which is already in used for the other endpoint. OK so what's the fix for snd-soc-audio-graph-card expecting a separate DAI then? > > Oh, I think there are Android apps to do that though.. Never tried > > if they work on droid4. But if they do, doing a register dump of > > mcbsp3 would show up how it's configured. > > I don't see how you could record the data from the line which is > connected to McBSP_DX pin (the pin is output). > > But I might be missing something. Yeah I don't know either, but the pins we have muxed for mcbsp3 are: /* 0x4a100106 abe_pdm_ul_data.abe_mcbsp3_dr ag25 */ OMAP4_IOPAD(0x106, PIN_INPUT | MUX_MODE1) /* 0x4a100108 abe_pdm_dl_data.abe_mcbsp3_dx af25 */ OMAP4_IOPAD(0x108, PIN_OUTPUT | MUX_MODE1) /* 0x4a10010a abe_pdm_frame.abe_mcbsp3_clkx ae25 */ OMAP4_IOPAD(0x10a, PIN_INPUT | MUX_MODE1) /* 0x4a10010c abe_pdm_lb_clk.abe_mcbsp3_fsx af26 */ OMAP4_IOPAD(0x10c, PIN_INPUT | MUX_MODE1) Isn't the data receive there as mcbsp3_dr? > > I think the link for the patches you posted is patching the > > snd-soc-audio-graph-card already? > > Yes it does, but the functionality is there via custom machine drivers. > What I afraid is that such a complex wiring as the Droid4 have it might > be not possible to use a generic - fits everything - driver without > making it a customized one ;) > > Otho, if the only thing is the machine level DAPM switching and linking > the paths then it might be relatively straight forward to extend the > simple-card family. Yeah or maybe it just needs to be handled directly in the cpcap, mdm6600 codec drivers? > > Right. So right now it seems that for snd-soc-audio-graph-card > > needs the dummy dai, but it's unclear what would need to be > > changed to not use a dummy dai for mcbsp. > > Since simple-card family can and will connect up dummy dai/codec when > needed based on the setup, I would look at that and make it do so. Oh so make simple-card spin up the dummy dai instead of mcbsp? > > The dts snippets I posted earlier do follow the graph bindings > > as far as I know. But just to confirm, do you see any need to > > move things around there? > > It also states that a port is a physical port which can have multiple > endpoints. But multiple endpoint != DAI. port == dai. I guess I'm getting really confused now.. Are you saying the dts needs to be changed too now? Regards, Tony