From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
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 <muroya@ksk.co.jp>,
Steve Longerbeam <steve_longerbeam@mentor.com>,
Koji Matsuoka <koji.matsuoka.xm@renesas.com>
Subject: Re: [PATCH 2/8] drm: Add DU CMM support boot and clk changes
Date: Thu, 4 Apr 2019 13:12:55 +0300 [thread overview]
Message-ID: <20190404101255.GE5800@pendragon.ideasonboard.com> (raw)
In-Reply-To: <1554297284-14009-3-git-send-email-VenkataRajesh.Kalakodima@in.bosch.com>
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 <venkatarajesh.kalakodima@in.bosch.com>
>
> 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 <koji.matsuoka.xm@renesas.com>
> Signed-off-by: Tsutomu Muroya <muroya@ksk.co.jp>
> Signed-off-by: Steve Longerbeam <steve_longerbeam@mentor.com>
>
> - 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 <venkatarajesh.kalakodima@in.bosch.com>
> ---
> .../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 = <GIC_SPI 256 IRQ_TYPE_LEVEL_HIGH>,
> <GIC_SPI 268 IRQ_TYPE_LEVEL_HIGH>,
> <GIC_SPI 269 IRQ_TYPE_LEVEL_HIGH>,
> @@ -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 = <GIC_SPI 256 IRQ_TYPE_LEVEL_HIGH>,
> <GIC_SPI 268 IRQ_TYPE_LEVEL_HIGH>,
> <GIC_SPI 269 IRQ_TYPE_LEVEL_HIGH>;
> 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 = <GIC_SPI 256 IRQ_TYPE_LEVEL_HIGH>,
> <GIC_SPI 268 IRQ_TYPE_LEVEL_HIGH>,
> <GIC_SPI 270 IRQ_TYPE_LEVEL_HIGH>;
> 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
next prev parent reply other threads:[~2019-04-04 10:13 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-03 13:14 [PATCH 0/8] v4.19.0 Added Color Management Module VenkataRajesh.Kalakodima
2019-04-03 13:14 ` VenkataRajesh.Kalakodima
2019-04-03 13:14 ` [PATCH 1/8] drm: Add DU CMM support functions VenkataRajesh.Kalakodima
2019-04-03 13:14 ` VenkataRajesh.Kalakodima
2019-04-04 10:09 ` Laurent Pinchart
2019-04-04 10:09 ` Laurent Pinchart
2019-04-03 13:14 ` [PATCH 2/8] drm: Add DU CMM support boot and clk changes VenkataRajesh.Kalakodima
2019-04-03 13:14 ` VenkataRajesh.Kalakodima
2019-04-04 10:12 ` Laurent Pinchart [this message]
2019-04-03 13:14 ` [PATCH 3/8] drm: rcar-du: Give a name to clu table samples VenkataRajesh.Kalakodima
2019-04-03 13:14 ` VenkataRajesh.Kalakodima
2019-04-04 10:15 ` Laurent Pinchart
2019-04-03 13:14 ` [PATCH 4/8] drm: rcar-du: Refactor the code with new functions VenkataRajesh.Kalakodima
2019-04-03 13:14 ` VenkataRajesh.Kalakodima
2019-04-03 13:14 ` [PATCH 5/8] drm: rcar-du: Implement interfaces to set clu and lut using drm data structures VenkataRajesh.Kalakodima
2019-04-03 13:14 ` VenkataRajesh.Kalakodima
2019-04-04 7:50 ` Daniel Vetter
2019-04-04 15:40 ` Ville Syrjälä
2019-04-05 8:39 ` Harsha Manjula Mallikarjun (RBEI/ECF3)
2019-04-05 8:39 ` Harsha Manjula Mallikarjun (RBEI/ECF3)
2019-04-03 13:14 ` [PATCH 6/8] drm: rcar-du: Implement atomic_check to check for gamma and ctm properties VenkataRajesh.Kalakodima
2019-04-03 13:14 ` VenkataRajesh.Kalakodima
2019-04-03 13:14 ` [PATCH 7/8] drm: rcar-du: update gamma and ctm properties in commit tail VenkataRajesh.Kalakodima
2019-04-03 13:14 ` VenkataRajesh.Kalakodima
2019-04-04 10:19 ` Laurent Pinchart
2019-04-04 10:19 ` Laurent Pinchart
2019-04-03 13:14 ` [PATCH 8/8] drm: rcar-du: Add shutdown callback function in platform_driver VenkataRajesh.Kalakodima
2019-04-03 13:14 ` VenkataRajesh.Kalakodima
2019-04-04 7:47 ` Daniel Vetter
2019-04-04 7:47 ` Daniel Vetter
2019-04-04 10:26 ` Laurent Pinchart
2019-04-04 10:26 ` Laurent Pinchart
2019-04-04 9:45 ` [PATCH 0/8] v4.19.0 Added Color Management Module Laurent Pinchart
2019-04-04 9:45 ` Laurent Pinchart
2019-04-04 9:46 ` Laurent Pinchart
2019-04-04 9:46 ` Laurent Pinchart
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190404101255.GE5800@pendragon.ideasonboard.com \
--to=laurent.pinchart@ideasonboard.com \
--cc=VenkataRajesh.Kalakodima@in.bosch.com \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=koji.matsuoka.xm@renesas.com \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=muroya@ksk.co.jp \
--cc=steve_longerbeam@mentor.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.