From patchwork Mon May 15 05:53:16 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simon Horman X-Patchwork-Id: 9726097 X-Patchwork-Delegate: horms@verge.net.au Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 98C6B60380 for ; Mon, 15 May 2017 05:53:23 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 8803928935 for ; Mon, 15 May 2017 05:53:23 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 7B97F28953; Mon, 15 May 2017 05:53:23 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.3 required=2.0 tests=BAYES_00,DKIM_SIGNED, FAKE_REPLY_C, RCVD_IN_DNSWL_HI, T_DKIM_INVALID autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id C5BC428935 for ; Mon, 15 May 2017 05:53:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758953AbdEOFxV (ORCPT ); Mon, 15 May 2017 01:53:21 -0400 Received: from kirsty.vergenet.net ([202.4.237.240]:48835 "EHLO kirsty.vergenet.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758945AbdEOFxV (ORCPT ); Mon, 15 May 2017 01:53:21 -0400 Received: from penelope.horms.nl (52D9BC73.cm-11-1c.dynamic.ziggo.nl [82.217.188.115]) by kirsty.vergenet.net (Postfix) with ESMTPA id 17D1225AF5F; Mon, 15 May 2017 15:53:19 +1000 (AEST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=verge.net.au; s=mail; t=1494827599; bh=9BNaoFJzRmsD/w/NM+7jHnQBSRqhDospBuQyyMLoqkw=; h=Date:From:To:Cc:Subject:In-Reply-To:From; b=SRjVkxZuwB5fRXaI+8mqHrrN97AIPYTTUXWyiUsMus7tj9DfElH8xRuQEz9YQmna8 A5MuRtU4jWg1dCAZ9RP34hewkniE6Tbe00NWPM1D29BF1QlT6MwwVIY5X+jRiN5iU7 YXY6Xk0u0BKSSVCp0XGGgyqcRnS1DM+oqG5jItEw= Received: by penelope.horms.nl (Postfix, from userid 7100) id DB757E22360; Mon, 15 May 2017 07:53:16 +0200 (CEST) Date: Mon, 15 May 2017 07:53:16 +0200 From: Simon Horman To: Geert Uytterhoeven , Kuninori Morimoto Cc: Magnus , Linux-Renesas Subject: Re: [PATCH v2 05/10] arm64: renesas: r8a7796: add Sound SSI PIO support Message-ID: <20170515055316.GD16545@verge.net.au> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <871srrj5tl.wl%kuninori.morimoto.gx@renesas.com> Organisation: Horms Solutions BV User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-renesas-soc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-renesas-soc@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On Fri, May 12, 2017 at 11:43:39AM +0200, Geert Uytterhoeven wrote: > Hi Morimoto-san, Simon, > > On Fri, May 12, 2017 at 2:14 AM, Kuninori Morimoto > wrote: > > --- a/arch/arm64/boot/dts/renesas/r8a7796.dtsi > > +++ b/arch/arm64/boot/dts/renesas/r8a7796.dtsi > > @@ -1259,7 +1259,42 @@ > > }; > > > > rcar_sound: sound@ec500000 { > > [...] > > > + power-domains = <&cpg>; > > power-domains = <&sysc R8A7796_PD_ALWAYS_ON>; > > (by comparing r8a779[66].dtsi). > > Simon: As you have already applied this, can you please fix it up in your tree? Sure, I have squashed the batch below into the relevant patch in my local tree which I plan to push later today. Morimoto-san, FYI, I did a s/renesas/dts/ in the subject of the patches in this series. On Mon, May 15, 2017 at 01:01:23AM +0000, Kuninori Morimoto wrote: > > From: Kuninori Morimoto > > commit a32681b13157f("arm64: dts: r8a7796: add Sound SSI PIO support") > added R-Car sound, but it used old style power-domains. > This patch tidyup it. > > Signed-off-by: Kuninori Morimoto > --- > arch/arm64/boot/dts/renesas/r8a7796.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/renesas/r8a7796.dtsi b/arch/arm64/boot/dts/renesas/r8a7796.dtsi > index f815532..f21c5ce 100644 > --- a/arch/arm64/boot/dts/renesas/r8a7796.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a7796.dtsi > @@ -1323,7 +1323,7 @@ > "ctu.1", "ctu.0", > "dvc.0", "dvc.1", > "clk_a", "clk_b", "clk_c", "clk_i"; > - power-domains = <&cpg>; > + power-domains = <&sysc R8A7796_PD_ALWAYS_ON>; > status = "disabled"; > > rcar_sound,dvc { The patch I now have queued-up is as follows: From: Kuninori Morimoto arm64: dts: r8a7796: add Sound SSI PIO support Signed-off-by: Kuninori Morimoto Signed-off-by: Simon Horman --- arch/arm64/boot/dts/renesas/r8a7796.dtsi | 64 +++++++++++++++++++++++++++++++- 1 file changed, 62 insertions(+), 2 deletions(-) diff --git a/arch/arm64/boot/dts/renesas/r8a7796.dtsi b/arch/arm64/boot/dts/renesas/r8a7796.dtsi index 0a76a1c53082..5c6e54a1e27a 100644 --- a/arch/arm64/boot/dts/renesas/r8a7796.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a7796.dtsi @@ -1165,7 +1165,42 @@ }; rcar_sound: sound@ec500000 { - /* placeholder */ + /* + * #sound-dai-cells is required + * + * Single DAI : #sound-dai-cells = <0>; <&rcar_sound>; + * Multi DAI : #sound-dai-cells = <1>; <&rcar_sound N>; + */ + /* + * #clock-cells is required for audio_clkout0/1/2/3 + * + * clkout : #clock-cells = <0>; <&rcar_sound>; + * clkout0/1/2/3: #clock-cells = <1>; <&rcar_sound N>; + */ + compatible = "renesas,rcar_sound-r8a7796", "renesas,rcar_sound-gen3"; + reg = <0 0xec500000 0 0x1000>, /* SCU */ + <0 0xec5a0000 0 0x100>, /* ADG */ + <0 0xec540000 0 0x1000>, /* SSIU */ + <0 0xec541000 0 0x280>, /* SSI */ + <0 0xec740000 0 0x200>; /* Audio DMAC peri peri*/ + reg-names = "scu", "adg", "ssiu", "ssi", "audmapp"; + + clocks = <&cpg CPG_MOD 1005>, + <&cpg CPG_MOD 1006>, <&cpg CPG_MOD 1007>, + <&cpg CPG_MOD 1008>, <&cpg CPG_MOD 1009>, + <&cpg CPG_MOD 1010>, <&cpg CPG_MOD 1011>, + <&cpg CPG_MOD 1012>, <&cpg CPG_MOD 1013>, + <&cpg CPG_MOD 1014>, <&cpg CPG_MOD 1015>, + <&audio_clk_a>, <&audio_clk_b>, + <&audio_clk_c>, + <&cpg CPG_CORE R8A7796_CLK_S0D4>; + clock-names = "ssi-all", + "ssi.9", "ssi.8", "ssi.7", "ssi.6", + "ssi.5", "ssi.4", "ssi.3", "ssi.2", + "ssi.1", "ssi.0", + "clk_a", "clk_b", "clk_c", "clk_i"; + power-domains = <&sysc R8A7796_PD_ALWAYS_ON>; + status = "disabled"; rcar_sound,dvc { dvc0: dvc-0 { @@ -1184,9 +1219,34 @@ rcar_sound,ssi { ssi0: ssi-0 { + interrupts = ; }; - ssi1: ssi-1 { + interrupts = ; + }; + ssi2: ssi-2 { + interrupts = ; + }; + ssi3: ssi-3 { + interrupts = ; + }; + ssi4: ssi-4 { + interrupts = ; + }; + ssi5: ssi-5 { + interrupts = ; + }; + ssi6: ssi-6 { + interrupts = ; + }; + ssi7: ssi-7 { + interrupts = ; + }; + ssi8: ssi-8 { + interrupts = ; + }; + ssi9: ssi-9 { + interrupts = ; }; }; };