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=-13.6 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SIGNED_OFF_BY,SPF_PASS,URIBL_BLOCKED,USER_AGENT_MUTT autolearn=unavailable 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 323FBC10F0E for ; Thu, 4 Apr 2019 10:13:11 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id D6DD32147C for ; Thu, 4 Apr 2019 10:13:10 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="a4IZ28/o" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729014AbfDDKNJ (ORCPT ); Thu, 4 Apr 2019 06:13:09 -0400 Received: from perceval.ideasonboard.com ([213.167.242.64]:49154 "EHLO perceval.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727053AbfDDKNI (ORCPT ); Thu, 4 Apr 2019 06:13:08 -0400 Received: from pendragon.ideasonboard.com (dfj612yhrgyx302h3jwwy-3.rev.dnainternet.fi [IPv6:2001:14ba:21f5:5b00:ce28:277f:58d7:3ca4]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id A2F2554B; Thu, 4 Apr 2019 12:13:05 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1554372785; bh=hKzRtg7j8R+9gQ49MCbnqzx9w8lTPjSCPmG5Scs6gUg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=a4IZ28/obyhVm/s4OLfaVkRZM7WoHaFznkR6GG2w6vCeFB/ViY/KVZ1F1xlGKeXsX GcDfIdSZzJ/lsGA778CVzrxqDx3M993Tqst+wJBdm+YyDwaDXSWW6g0XwOlFaV1M9V eIGfL/CC/mf4lwG0p4+7M/qN2ktw1YK5pKO0s0S0= Date: Thu, 4 Apr 2019 13:12:55 +0300 From: Laurent Pinchart To: VenkataRajesh.Kalakodima@in.bosch.com Cc: linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, dri-devel@lists.freedesktop.org, Tsutomu Muroya , Steve Longerbeam , Koji Matsuoka Subject: Re: [PATCH 2/8] drm: Add DU CMM support boot and clk changes Message-ID: <20190404101255.GE5800@pendragon.ideasonboard.com> References: <1554297284-14009-1-git-send-email-VenkataRajesh.Kalakodima@in.bosch.com> <1554297284-14009-3-git-send-email-VenkataRajesh.Kalakodima@in.bosch.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1554297284-14009-3-git-send-email-VenkataRajesh.Kalakodima@in.bosch.com> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Kalakodima, Thank you for the patch. On Wed, Apr 03, 2019 at 06:44:38PM +0530, VenkataRajesh.Kalakodima@in.bosch.com wrote: > From: kalakodima venkata rajesh > > This is the out-of-tree patch for DU CMM driver support from > Yocto release v3.4.0. > > Link: https://github.com/renesas-rcar/du_cmm/commit/2d8ea2b667ad4616aa639c54ecc11f7c4b58959d.patch > > Following is from the patch description: > > du_cmm: Release for Yocto v3.4.0 > > This patch made the following correspondence. > > - Corresponds to kernel v 4.14. > - Double buffer only is supported. > - Fix CLU / LUT update timing. > - Add CMM Channel occupation mode. > - Fix Close process. > > Signed-off-by: Koji Matsuoka > Signed-off-by: Tsutomu Muroya > Signed-off-by: Steve Longerbeam > > - Resolved checkpatch errors > - Resolved merge conflicts according to latest version > - In patch included boot and clock files from base patch As for patch 1/8, and all other patches in this series, the commit message needs an overhaul. > Signed-off-by: kalakodima venkata rajesh > --- > .../boot/dts/renesas/r8a7795-es1-salvator-x.dts | 5 + > arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 5 + > .../arm64/boot/dts/renesas/r8a7795-salvator-xs.dts | 5 + > arch/arm64/boot/dts/renesas/r8a7795.dtsi | 29 +++++- > arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts | 6 +- > .../arm64/boot/dts/renesas/r8a7796-salvator-xs.dts | 4 + > arch/arm64/boot/dts/renesas/r8a7796.dtsi | 25 ++++- > .../arm64/boot/dts/renesas/r8a77965-salvator-x.dts | 7 +- > .../boot/dts/renesas/r8a77965-salvator-xs.dts | 7 +- > arch/arm64/boot/dts/renesas/r8a77965.dtsi | 27 +++++- > drivers/clk/renesas/r8a7795-cpg-mssr.c | 4 + > drivers/clk/renesas/r8a7796-cpg-mssr.c | 3 + > drivers/clk/renesas/r8a77965-cpg-mssr.c | 106 ++++++++++++++++++++- > 13 files changed, 217 insertions(+), 16 deletions(-) > > diff --git a/arch/arm64/boot/dts/renesas/r8a7795-es1-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-es1-salvator-x.dts > index 6b5fa91..45c1f8a 100644 > --- a/arch/arm64/boot/dts/renesas/r8a7795-es1-salvator-x.dts > +++ b/arch/arm64/boot/dts/renesas/r8a7795-es1-salvator-x.dts > @@ -41,11 +41,16 @@ > <&cpg CPG_MOD 722>, > <&cpg CPG_MOD 721>, > <&cpg CPG_MOD 727>, > + <&cpg CPG_MOD 711>, > + <&cpg CPG_MOD 710>, > + <&cpg CPG_MOD 709>, > + <&cpg CPG_MOD 708>, > <&versaclock5 1>, > <&x21_clk>, > <&x22_clk>, > <&versaclock5 2>; > clock-names = "du.0", "du.1", "du.2", "du.3", "lvds.0", > + "cmm.0", "cmm.1", "cmm.2", "cmm.3", > "dclkin.0", "dclkin.1", "dclkin.2", "dclkin.3"; I think the CMM should be modeled as a separate DT node, as it's a separate IP core. In any case DT bindings need to be updated. > }; > > diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts > index 446822f..67b64e7 100644 > --- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts > +++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts > @@ -41,11 +41,16 @@ > <&cpg CPG_MOD 722>, > <&cpg CPG_MOD 721>, > <&cpg CPG_MOD 727>, > + <&cpg CPG_MOD 711>, > + <&cpg CPG_MOD 710>, > + <&cpg CPG_MOD 709>, > + <&cpg CPG_MOD 708>, > <&versaclock5 1>, > <&x21_clk>, > <&x22_clk>, > <&versaclock5 2>; > clock-names = "du.0", "du.1", "du.2", "du.3", "lvds.0", > + "cmm.0", "cmm.1", "cmm.2", "cmm.3", > "dclkin.0", "dclkin.1", "dclkin.2", "dclkin.3"; > }; > > diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-xs.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-xs.dts > index 8ded64d0..adcacea 100644 > --- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-xs.dts > +++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-xs.dts > @@ -41,11 +41,16 @@ > <&cpg CPG_MOD 722>, > <&cpg CPG_MOD 721>, > <&cpg CPG_MOD 727>, > + <&cpg CPG_MOD 711>, > + <&cpg CPG_MOD 710>, > + <&cpg CPG_MOD 709>, > + <&cpg CPG_MOD 708>, > <&versaclock6 1>, > <&x21_clk>, > <&x22_clk>, > <&versaclock6 2>; > clock-names = "du.0", "du.1", "du.2", "du.3", "lvds.0", > + "cmm.0", "cmm.1", "cmm.2", "cmm.3", > "dclkin.0", "dclkin.1", "dclkin.2", "dclkin.3"; > }; > > diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi b/arch/arm64/boot/dts/renesas/r8a7795.dtsi > index fb9d08a..aa5fc3c 100644 > --- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi > @@ -2783,8 +2783,13 @@ > du: display@feb00000 { > compatible = "renesas,du-r8a7795"; > reg = <0 0xfeb00000 0 0x80000>, > - <0 0xfeb90000 0 0x14>; > - reg-names = "du", "lvds.0"; > + <0 0xfeb90000 0 0x14>, > + <0 0xfea40000 0 0x00001000>, > + <0 0xfea50000 0 0x00001000>, > + <0 0xfea60000 0 0x00001000>, > + <0 0xfea70000 0 0x00001000>; > + reg-names = "du", "lvds.0", > + "cmm.0", "cmm.1", "cmm.2", "cmm.3"; > interrupts = , > , > , > @@ -2793,8 +2798,24 @@ > <&cpg CPG_MOD 723>, > <&cpg CPG_MOD 722>, > <&cpg CPG_MOD 721>, > - <&cpg CPG_MOD 727>; > - clock-names = "du.0", "du.1", "du.2", "du.3", "lvds.0"; > + <&cpg CPG_MOD 727>, > + <&cpg CPG_MOD 711>, > + <&cpg CPG_MOD 710>, > + <&cpg CPG_MOD 709>, > + <&cpg CPG_MOD 708>; > + clock-names = "du.0", "du.1", "du.2", "du.3", "lvds.0", > + "cmm.0", "cmm.1", "cmm.2", "cmm.3"; > + resets = <&cpg 724>, > + <&cpg 724>, > + <&cpg 722>, > + <&cpg 722>, > + <&cpg 727>, > + <&cpg 711>, > + <&cpg 710>, > + <&cpg 709>, > + <&cpg 708>; > + reset-names = "du.0", "du.1", "du.2", "du.3", "lvds.0", > + "cmm.0", "cmm.1", "cmm.2", "cmm.3"; > vsps = <&vspd0 0 &vspd1 0 &vspd2 0 &vspd0 1>; > status = "disabled"; > > diff --git a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts > index 052d72a..abe24e5 100644 > --- a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts > +++ b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts > @@ -30,18 +30,22 @@ > <&cpg CPG_MOD 723>, > <&cpg CPG_MOD 722>, > <&cpg CPG_MOD 727>, > + <&cpg CPG_MOD 711>, > + <&cpg CPG_MOD 710>, > + <&cpg CPG_MOD 709>, > <&versaclock5 1>, > <&x21_clk>, > <&versaclock5 2>; > clock-names = "du.0", "du.1", "du.2", "lvds.0", > + "cmm.0", "cmm.1", "cmm.2", > "dclkin.0", "dclkin.1", "dclkin.2"; > + > }; > > &sound_card { > dais = <&rsnd_port0 /* ak4613 */ > &rsnd_port1>; /* HDMI0 */ > }; > - > &hdmi0 { > status = "okay"; > > diff --git a/arch/arm64/boot/dts/renesas/r8a7796-salvator-xs.dts b/arch/arm64/boot/dts/renesas/r8a7796-salvator-xs.dts > index 8860be6..6c3af11 100644 > --- a/arch/arm64/boot/dts/renesas/r8a7796-salvator-xs.dts > +++ b/arch/arm64/boot/dts/renesas/r8a7796-salvator-xs.dts > @@ -30,10 +30,14 @@ > <&cpg CPG_MOD 723>, > <&cpg CPG_MOD 722>, > <&cpg CPG_MOD 727>, > + <&cpg CPG_MOD 711>, > + <&cpg CPG_MOD 710>, > + <&cpg CPG_MOD 709>, > <&versaclock6 1>, > <&x21_clk>, > <&versaclock6 2>; > clock-names = "du.0", "du.1", "du.2", "lvds.0", > + "cmm.0", "cmm.1", "cmm.2", > "dclkin.0", "dclkin.1", "dclkin.2"; > }; > > diff --git a/arch/arm64/boot/dts/renesas/r8a7796.dtsi b/arch/arm64/boot/dts/renesas/r8a7796.dtsi > index cbd35c0..729b2b6 100644 > --- a/arch/arm64/boot/dts/renesas/r8a7796.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a7796.dtsi > @@ -2438,16 +2438,33 @@ > du: display@feb00000 { > compatible = "renesas,du-r8a7796"; > reg = <0 0xfeb00000 0 0x70000>, > - <0 0xfeb90000 0 0x14>; > - reg-names = "du", "lvds.0"; > + <0 0xfeb90000 0 0x14>, > + <0 0xfea40000 0 0x00001000>, > + <0 0xfea50000 0 0x00001000>, > + <0 0xfea60000 0 0x00001000>; > + reg-names = "du", "lvds.0", > + "cmm.0", "cmm.1", "cmm.2"; > interrupts = , > , > ; > clocks = <&cpg CPG_MOD 724>, > <&cpg CPG_MOD 723>, > <&cpg CPG_MOD 722>, > - <&cpg CPG_MOD 727>; > - clock-names = "du.0", "du.1", "du.2", "lvds.0"; > + <&cpg CPG_MOD 727>, > + <&cpg CPG_MOD 711>, > + <&cpg CPG_MOD 710>, > + <&cpg CPG_MOD 709>; > + clock-names = "du.0", "du.1", "du.2", "lvds.0", > + "cmm.0", "cmm.1", "cmm.2"; > + resets = <&cpg 724>, > + <&cpg 724>, > + <&cpg 722>, > + <&cpg 727>, > + <&cpg 711>, > + <&cpg 710>, > + <&cpg 709>; > + reset-names = "du.0", "du.1", "du.2", "lvds.0", > + "cmm.0", "cmm.1", "cmm.2"; > status = "disabled"; > > vsps = <&vspd0 &vspd1 &vspd2>; > diff --git a/arch/arm64/boot/dts/renesas/r8a77965-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a77965-salvator-x.dts > index 340a3c7..20992e2 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77965-salvator-x.dts > +++ b/arch/arm64/boot/dts/renesas/r8a77965-salvator-x.dts > @@ -24,10 +24,15 @@ > clocks = <&cpg CPG_MOD 724>, > <&cpg CPG_MOD 723>, > <&cpg CPG_MOD 721>, > + <&cpg CPG_MOD 727>, > + <&cpg CPG_MOD 711>, > + <&cpg CPG_MOD 710>, > + <&cpg CPG_MOD 708>, > <&versaclock5 1>, > <&x21_clk>, > <&versaclock5 2>; > - clock-names = "du.0", "du.1", "du.3", > + clock-names = "du.0", "du.1", "du.3", "lvds.0", > + "cmm.0", "cmm.1", "cmm.3", > "dclkin.0", "dclkin.1", "dclkin.3"; > }; > > diff --git a/arch/arm64/boot/dts/renesas/r8a77965-salvator-xs.dts b/arch/arm64/boot/dts/renesas/r8a77965-salvator-xs.dts > index 9de4e3d..eb02075 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77965-salvator-xs.dts > +++ b/arch/arm64/boot/dts/renesas/r8a77965-salvator-xs.dts > @@ -24,10 +24,15 @@ > clocks = <&cpg CPG_MOD 724>, > <&cpg CPG_MOD 723>, > <&cpg CPG_MOD 721>, > + <&cpg CPG_MOD 727>, > + <&cpg CPG_MOD 711>, > + <&cpg CPG_MOD 710>, > + <&cpg CPG_MOD 708>, > <&versaclock6 1>, > <&x21_clk>, > <&versaclock6 2>; > - clock-names = "du.0", "du.1", "du.3", > + clock-names = "du.0", "du.1", "du.3", "lvds.0", > + "cmm.0", "cmm.1", "cmm.3", > "dclkin.0", "dclkin.1", "dclkin.3"; > }; > > diff --git a/arch/arm64/boot/dts/renesas/r8a77965.dtsi b/arch/arm64/boot/dts/renesas/r8a77965.dtsi > index 0cd4446..37382b7 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77965.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a77965.dtsi > @@ -1801,15 +1801,34 @@ > > du: display@feb00000 { > compatible = "renesas,du-r8a77965"; > - reg = <0 0xfeb00000 0 0x80000>; > - reg-names = "du"; > + reg = <0 0xfeb00000 0 0x80000>, > + <0 0xfeb90000 0 0x14>, > + <0 0xfea40000 0 0x00001000>, > + <0 0xfea50000 0 0x00001000>, > + <0 0xfea70000 0 0x00001000>; > + reg-names = "du", "lvds.0", > + "cmm.0", "cmm.1", "cmm.3"; > interrupts = , > , > ; > clocks = <&cpg CPG_MOD 724>, > <&cpg CPG_MOD 723>, > - <&cpg CPG_MOD 721>; > - clock-names = "du.0", "du.1", "du.3"; > + <&cpg CPG_MOD 721>, > + <&cpg CPG_MOD 727>, > + <&cpg CPG_MOD 711>, > + <&cpg CPG_MOD 710>, > + <&cpg CPG_MOD 708>; > + clock-names = "du.0", "du.1", "du.3", "lvds.0", > + "cmm.0", "cmm.1", "cmm.3"; > + resets = <&cpg 724>, > + <&cpg 724>, > + <&cpg 722>, > + <&cpg 727>, > + <&cpg 711>, > + <&cpg 710>, > + <&cpg 708>; > + reset-names = "du.0", "du.1", "du.3", "lvds.0", > + "cmm.0", "cmm.1", "cmm.3"; > status = "disabled"; > > vsps = <&vspd0 0 &vspd1 0 &vspd0 1>; > diff --git a/drivers/clk/renesas/r8a7795-cpg-mssr.c b/drivers/clk/renesas/r8a7795-cpg-mssr.c > index a85dd50..ba9e595 100644 > --- a/drivers/clk/renesas/r8a7795-cpg-mssr.c > +++ b/drivers/clk/renesas/r8a7795-cpg-mssr.c > @@ -201,6 +201,10 @@ static struct mssr_mod_clk r8a7795_mod_clks[] __initdata = { > DEF_MOD("ehci0", 703, R8A7795_CLK_S3D4), > DEF_MOD("hsusb", 704, R8A7795_CLK_S3D4), > DEF_MOD("hsusb3", 705, R8A7795_CLK_S3D4), > + DEF_MOD("cmm3", 708, R8A7795_CLK_S2D1), > + DEF_MOD("cmm2", 709, R8A7795_CLK_S2D1), > + DEF_MOD("cmm1", 710, R8A7795_CLK_S2D1), > + DEF_MOD("cmm0", 711, R8A7795_CLK_S2D1), > DEF_MOD("csi21", 713, R8A7795_CLK_CSI0), /* ES1.x */ > DEF_MOD("csi20", 714, R8A7795_CLK_CSI0), > DEF_MOD("csi41", 715, R8A7795_CLK_CSI0), > diff --git a/drivers/clk/renesas/r8a7796-cpg-mssr.c b/drivers/clk/renesas/r8a7796-cpg-mssr.c > index dfb267a..9f01fda 100644 > --- a/drivers/clk/renesas/r8a7796-cpg-mssr.c > +++ b/drivers/clk/renesas/r8a7796-cpg-mssr.c > @@ -180,6 +180,9 @@ static const struct mssr_mod_clk r8a7796_mod_clks[] __initconst = { > DEF_MOD("ehci1", 702, R8A7796_CLK_S3D4), > DEF_MOD("ehci0", 703, R8A7796_CLK_S3D4), > DEF_MOD("hsusb", 704, R8A7796_CLK_S3D4), > + DEF_MOD("cmm2", 709, R8A7796_CLK_S2D1), > + DEF_MOD("cmm1", 710, R8A7796_CLK_S2D1), > + DEF_MOD("cmm0", 711, R8A7796_CLK_S2D1), > DEF_MOD("csi20", 714, R8A7796_CLK_CSI0), > DEF_MOD("csi40", 716, R8A7796_CLK_CSI0), > DEF_MOD("du2", 722, R8A7796_CLK_S2D1), > diff --git a/drivers/clk/renesas/r8a77965-cpg-mssr.c b/drivers/clk/renesas/r8a77965-cpg-mssr.c > index 8fae5e9..cac4570 100644 > --- a/drivers/clk/renesas/r8a77965-cpg-mssr.c > +++ b/drivers/clk/renesas/r8a77965-cpg-mssr.c > @@ -123,7 +123,6 @@ static const struct mssr_mod_clk r8a77965_mod_clks[] __initconst = { > DEF_MOD("sys-dmac2", 217, R8A77965_CLK_S0D3), > DEF_MOD("sys-dmac1", 218, R8A77965_CLK_S0D3), > DEF_MOD("sys-dmac0", 219, R8A77965_CLK_S0D3), > - > DEF_MOD("cmt3", 300, R8A77965_CLK_R), > DEF_MOD("cmt2", 301, R8A77965_CLK_R), > DEF_MOD("cmt1", 302, R8A77965_CLK_R), > @@ -215,6 +214,111 @@ static const struct mssr_mod_clk r8a77965_mod_clks[] __initconst = { > DEF_MOD("i2c1", 930, R8A77965_CLK_S3D2), > DEF_MOD("i2c0", 931, R8A77965_CLK_S3D2), > > + DEF_MOD("3dge", 112, R8A77965_CLK_ZG), > + DEF_MOD("fdp0", 119, R8A77965_CLK_S0D1), > + DEF_MOD("vcplf", 130, R8A77965_CLK_S0D2), > + DEF_MOD("vdpb", 131, R8A77965_CLK_S0D2), > + DEF_MOD("scif5", 202, R8A77965_CLK_S3D4), > + DEF_MOD("scif4", 203, R8A77965_CLK_S3D4), > + DEF_MOD("scif3", 204, R8A77965_CLK_S3D4), > + DEF_MOD("scif1", 206, R8A77965_CLK_S3D4), > + DEF_MOD("scif0", 207, R8A77965_CLK_S3D4), > + DEF_MOD("msiof3", 208, R8A77965_CLK_MSO), > + DEF_MOD("msiof2", 209, R8A77965_CLK_MSO), > + DEF_MOD("msiof1", 210, R8A77965_CLK_MSO), > + DEF_MOD("msiof0", 211, R8A77965_CLK_MSO), > + DEF_MOD("sys-dmac2", 217, R8A77965_CLK_S0D3), > + DEF_MOD("sys-dmac1", 218, R8A77965_CLK_S0D3), > + DEF_MOD("sys-dmac0", 219, R8A77965_CLK_S0D3), > + DEF_MOD("cmt3", 300, R8A77965_CLK_R), > + DEF_MOD("cmt2", 301, R8A77965_CLK_R), > + DEF_MOD("cmt1", 302, R8A77965_CLK_R), > + DEF_MOD("cmt0", 303, R8A77965_CLK_R), > + DEF_MOD("scif2", 310, R8A77965_CLK_S3D4), > + DEF_MOD("sdif3", 311, R8A77965_CLK_SD3), > + DEF_MOD("sdif2", 312, R8A77965_CLK_SD2), > + DEF_MOD("sdif1", 313, R8A77965_CLK_SD1), > + DEF_MOD("sdif0", 314, R8A77965_CLK_SD0), > + DEF_MOD("pcie1", 318, R8A77965_CLK_S3D1), > + DEF_MOD("pcie0", 319, R8A77965_CLK_S3D1), > + DEF_MOD("usb3-if0", 328, R8A77965_CLK_S3D1), > + DEF_MOD("usb-dmac0", 330, R8A77965_CLK_S3D1), > + DEF_MOD("usb-dmac1", 331, R8A77965_CLK_S3D1), > + DEF_MOD("rwdt0", 402, R8A77965_CLK_R), > + DEF_MOD("intc-ex", 407, R8A77965_CLK_CP), > + DEF_MOD("intc-ap", 408, R8A77965_CLK_S3D1), > + DEF_MOD("audmac0", 502, R8A77965_CLK_S3D4), > + DEF_MOD("audmac1", 501, R8A77965_CLK_S3D4), > + DEF_MOD("adsp", 506, R8A77965_CLK_S1D1), > + DEF_MOD("drif7", 508, R8A77965_CLK_S3D2), > + DEF_MOD("drif6", 509, R8A77965_CLK_S3D2), > + DEF_MOD("drif5", 510, R8A77965_CLK_S3D2), > + DEF_MOD("drif4", 511, R8A77965_CLK_S3D2), > + DEF_MOD("drif3", 512, R8A77965_CLK_S3D2), > + DEF_MOD("drif2", 513, R8A77965_CLK_S3D2), > + DEF_MOD("drif1", 514, R8A77965_CLK_S3D2), > + DEF_MOD("drif0", 515, R8A77965_CLK_S3D2), > + DEF_MOD("hscif4", 516, R8A77965_CLK_S3D1), > + DEF_MOD("hscif3", 517, R8A77965_CLK_S3D1), > + DEF_MOD("hscif2", 518, R8A77965_CLK_S3D1), > + DEF_MOD("hscif1", 519, R8A77965_CLK_S3D1), > + DEF_MOD("hscif0", 520, R8A77965_CLK_S3D1), > + DEF_MOD("thermal", 522, R8A77965_CLK_CP), > + DEF_MOD("pwm", 523, R8A77965_CLK_S3D4), > + DEF_MOD("fcpvd1", 602, R8A77965_CLK_S0D2), > + DEF_MOD("fcpvd0", 603, R8A77965_CLK_S0D2), > + DEF_MOD("fcpvb0", 607, R8A77965_CLK_S0D1), > + DEF_MOD("fcpvi0", 611, R8A77965_CLK_S0D1), > + DEF_MOD("fcpf0", 615, R8A77965_CLK_S0D1), > + DEF_MOD("fcpcs", 619, R8A77965_CLK_S0D2), > + DEF_MOD("vspd1", 622, R8A77965_CLK_S0D2), > + DEF_MOD("vspd0", 623, R8A77965_CLK_S0D2), > + DEF_MOD("vspb", 626, R8A77965_CLK_S0D1), > + DEF_MOD("vspi0", 631, R8A77965_CLK_S0D1), > + DEF_MOD("ehci1", 702, R8A77965_CLK_S3D4), > + DEF_MOD("ehci0", 703, R8A77965_CLK_S3D4), > + DEF_MOD("hsusb", 704, R8A77965_CLK_S3D4), > + DEF_MOD("cmm3", 708, R8A77965_CLK_S2D1), > + DEF_MOD("cmm1", 710, R8A77965_CLK_S2D1), > + DEF_MOD("cmm0", 711, R8A77965_CLK_S2D1), > + DEF_MOD("csi20", 714, R8A77965_CLK_CSI0), > + DEF_MOD("csi40", 716, R8A77965_CLK_CSI0), > + DEF_MOD("du3", 721, R8A77965_CLK_S2D1), > + DEF_MOD("du1", 723, R8A77965_CLK_S2D1), > + DEF_MOD("du0", 724, R8A77965_CLK_S2D1), > + DEF_MOD("lvds", 727, R8A77965_CLK_S2D1), > + DEF_MOD("hdmi0", 729, R8A77965_CLK_HDMI), > + DEF_MOD("vin7", 804, R8A77965_CLK_S0D2), > + DEF_MOD("vin6", 805, R8A77965_CLK_S0D2), > + DEF_MOD("vin5", 806, R8A77965_CLK_S0D2), > + DEF_MOD("vin4", 807, R8A77965_CLK_S0D2), > + DEF_MOD("vin3", 808, R8A77965_CLK_S0D2), > + DEF_MOD("vin2", 809, R8A77965_CLK_S0D2), > + DEF_MOD("vin1", 810, R8A77965_CLK_S0D2), > + DEF_MOD("vin0", 811, R8A77965_CLK_S0D2), > + DEF_MOD("etheravb", 812, R8A77965_CLK_S0D6), > + DEF_MOD("sata0", 815, R8A77965_CLK_S3D2), > + DEF_MOD("gpio7", 905, R8A77965_CLK_S3D4), > + DEF_MOD("gpio6", 906, R8A77965_CLK_S3D4), > + DEF_MOD("gpio5", 907, R8A77965_CLK_S3D4), > + DEF_MOD("gpio4", 908, R8A77965_CLK_S3D4), > + DEF_MOD("gpio3", 909, R8A77965_CLK_S3D4), > + DEF_MOD("gpio2", 910, R8A77965_CLK_S3D4), > + DEF_MOD("gpio1", 911, R8A77965_CLK_S3D4), > + DEF_MOD("gpio0", 912, R8A77965_CLK_S3D4), > + DEF_MOD("can-fd", 914, R8A77965_CLK_S3D2), > + DEF_MOD("can-if1", 915, R8A77965_CLK_S3D4), > + DEF_MOD("can-if0", 916, R8A77965_CLK_S3D4), > + DEF_MOD("i2c6", 918, R8A77965_CLK_S0D6), > + DEF_MOD("i2c5", 919, R8A77965_CLK_S0D6), > + DEF_MOD("adg", 922, R8A77965_CLK_S0D1), > + DEF_MOD("i2c-dvfs", 926, R8A77965_CLK_CP), > + DEF_MOD("i2c4", 927, R8A77965_CLK_S0D6), > + DEF_MOD("i2c3", 928, R8A77965_CLK_S0D6), > + DEF_MOD("i2c2", 929, R8A77965_CLK_S3D2), > + DEF_MOD("i2c1", 930, R8A77965_CLK_S3D2), > + DEF_MOD("i2c0", 931, R8A77965_CLK_S3D2), > + That's much more than adding CMM clocks... Please split the clocks changes to separate patches. You can even repost them separately, as they can be merged before the rest of the series. > DEF_MOD("ssi-all", 1005, R8A77965_CLK_S3D4), > DEF_MOD("ssi9", 1006, MOD_CLK_ID(1005)), > DEF_MOD("ssi8", 1007, MOD_CLK_ID(1005)), -- Regards, Laurent Pinchart