Received: by 2002:a25:ad19:0:0:0:0:0 with SMTP id y25csp941036ybi; Wed, 3 Jul 2019 06:59:27 -0700 (PDT) X-Google-Smtp-Source: APXvYqw21/igNZKBeTXouaLsSKIb8bVM1NPvZjdsLAZJlgv4uMAhrblNa8nKpX4ubn97usHBnluu X-Received: by 2002:a17:902:7248:: with SMTP id c8mr5493235pll.162.1562162367379; Wed, 03 Jul 2019 06:59:27 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1562162367; cv=none; d=google.com; s=arc-20160816; b=v+PMvNRtP2XkjXT7qA4GxT7wo9dWtS77wj0lon7s62RAUvCdADksfD8pHYuVLlJXU8 TdLiZ4UaKJrnWbSYOsVMqoke3h/rLsPFRBn7ouUPPgR8w1g683rjFPjHaGDran7liRn0 U2LZV8B1R5G1Kire8MMgf0c6gkUUrb7rvVlmuNU3hl/bbMtYE1no0Yp2slpeJURnlM8p AN1HNxsSVQVJhl5RSuY9utkW9T0COGIf0hE9/IgmtTuQnCSUyEaMaOH2vFWIGOECpctw ZIF7SgdC52yUEHg92z+R29dieH/LX+PNi7Pz4idVAxiGPmHmu6/WS98yeQrfJ2JuLRna jKjw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:message-id:references :in-reply-to:cc:from:date:content-transfer-encoding:mime-version :subject:to:dkim-signature; bh=UHIUaVsVDuz0yZjRB09x+PshwKLK6UHG+Qvlw7TIC58=; b=kEgXUyUBmYL0FWiAKZnTJlgZYeFthlquDyqCI+22yVXONgPAq08OGbgC6OHHryu4bo v4F8PbndBXvCk+hUbhG4fkVSGxESdg+wCfW3ejIVPHFdsgHQ62IoXbA1DFBUobMMkAZX owuMRyk9mrjBuEgGc3Dt/5VWZzKK2UMqcldcID+a67GQRURig1u6Q5eTe6ajNID7GS/m wg9eU098PVkdJfQUIqF+5BqMurePs7/Qt0E0r/2MfesTgo3xhG7Ys6CukhIp8Gc68B8h j+0wpr0fWQf4xYVzUMLV40dvRXGZF2ezyp8Sgfdyfpxp71y736Lvpwm+6BuQhmi7QoPK Df3w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@akkea.ca header.s=mail header.b=SpfH1iV5; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id k5si2498277pgm.297.2019.07.03.06.59.11; Wed, 03 Jul 2019 06:59:27 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@akkea.ca header.s=mail header.b=SpfH1iV5; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726945AbfGCN6d (ORCPT + 99 others); Wed, 3 Jul 2019 09:58:33 -0400 Received: from node.akkea.ca ([192.155.83.177]:59766 "EHLO node.akkea.ca" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725847AbfGCN6d (ORCPT ); Wed, 3 Jul 2019 09:58:33 -0400 Received: by node.akkea.ca (Postfix, from userid 33) id B606E4E204B; Wed, 3 Jul 2019 13:58:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=akkea.ca; s=mail; t=1562162312; bh=UHIUaVsVDuz0yZjRB09x+PshwKLK6UHG+Qvlw7TIC58=; h=To:Subject:Date:From:Cc:In-Reply-To:References; b=SpfH1iV5gPGx0vLKUhzKfEvLCQNmPLkUyJlyLG3cW8wfWxKgAPFOR+aGMwiCtP5qV S+TSC/nxxbenLnoRMlQrsfUB2MXIUMeQmR2zwJ5dfti+3LJyKCNajnNEm6XcjeReqg E8fKNY+j2dOhPI2vMmPmQD/ssHXJFGIGur51ElBA= To: Daniel Baluta Subject: Re: [PATCH v3] arm64: dts: imx8mq: Add sai3 and sai6 nodes X-PHP-Originating-Script: 1000:rcube.php MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Date: Wed, 03 Jul 2019 07:58:32 -0600 From: Angus Ainslie Cc: Andra Danciu , Shawn Guo , Rob Herring , Mark Rutland , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , dl-linux-imx , Lucas Stach , Abel Vesa , Anson Huang , andrew.smirnov@gmail.com, Carlo Caione , =?UTF-8?Q?Guido_G=C3=BCnther?= , Devicetree List , linux-arm-kernel , Linux Kernel Mailing List In-Reply-To: References: <20190702132353.18632-1-andradanciu1997@gmail.com> <9ea5109f8645c3f27a9e350c5f9b2d4c@www.akkea.ca> <9e196ce51eac9ce9c327198c4a2911a8@www.akkea.ca> Message-ID: <8ebfeaae8541132ad39e4109d135cd5f@www.akkea.ca> X-Sender: angus@akkea.ca User-Agent: Roundcube Webmail/1.1.3 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Daniel, On 2019-07-03 07:25, Daniel Baluta wrote: > On Wed, Jul 3, 2019 at 4:12 PM Angus Ainslie wrote: >> >> Hi Daniel, >> >> On 2019-07-03 07:10, Daniel Baluta wrote: >> > On Wed, Jul 3, 2019 at 4:01 PM Angus Ainslie wrote: >> >> >> >> Hi Andra, >> >> >> >> I tried this out on linux-next and I'm not able to record or play >> >> sound. >> >> >> >> I also added the sai2 entry to test out our devkit and get a PCM >> >> timeout >> >> with that. >> > >> > Hi Angus, >> > >> > There are still lots of SAI patches that need to be upstream. Me and >> > Andra >> > will be working on that over this summer. >> > >> >> >> >> On 2019-07-02 07:23, Andra Danciu wrote: >> >> > SAI3 and SAI6 nodes are used to connect to an external codec. >> >> > They have 1 Tx and 1 Rx dataline. >> >> > >> >> > Cc: Daniel Baluta >> >> > Signed-off-by: Andra Danciu >> >> > --- >> >> > Changes since v2: >> >> > - removed multiple new lines >> >> > >> >> > Changes since v1: >> >> > - Added sai3 node because we need it to enable audio on pico-pi-8m >> >> > - Added commit description >> >> > >> >> > arch/arm64/boot/dts/freescale/imx8mq.dtsi | 29 >> >> > +++++++++++++++++++++++++++++ >> >> > 1 file changed, 29 insertions(+) >> >> > >> >> > diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi >> >> > b/arch/arm64/boot/dts/freescale/imx8mq.dtsi >> >> > index d09b808eff87..736cf81b695e 100644 >> >> > --- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi >> >> > +++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi >> >> > @@ -278,6 +278,20 @@ >> >> > #size-cells = <1>; >> >> > ranges = <0x30000000 0x30000000 0x400000>; >> >> > >> >> > + sai6: sai@30030000 { >> >> > + compatible = "fsl,imx8mq-sai", >> >> >> >> I don't find this compatible string in sound/soc/fsl/fsl_sai.c. Aren't >> >> the registers at a different offset from "fsl,imx6sx-sai". >> > >> > Yes, you are right on this. We are trying to slowly push all our >> > internal-tree >> > patches to mainline. Obviously, with started with low hanging fruits, >> > DTS >> > nodes and small SAI fixes. >> > >> > Soon, we will start to send patches for SAI IP ipgrade for imx8. >> > >> >> >> >> How is this supposed to work ? >> >> >> > >> > For the moment it won't work unless we will upstream all our SAI >> > internal patches. >> > But we will get there hopefully this summer. >> > >> >> Shouldn't a working driver be upstream before enabling it in the >> devicetree ? > > I see your point here and maybe your suggestion is the ideal > way to do things. > > Anyhow, I don't see a problem with adding the node in dts > because CONFIG_FSL_SAI is not set in the default config. > I think the assumption is that devices in the devicetree are tested and working. That's not currently the case for sai on imx8mq. > We try to speedup the upstreaming process giving the fact > that SAI patches will go through audio maintainer's tree and > the DTS patches will most likely go through Shawn's tree. > Ok I can see them being committed concurrently but I haven't even seen a v1 of the sai patches. Thanks Angus > thanks, > Daniel.