linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: renesas: eagle: Add x1 clock
@ 2021-05-04  9:14 Geert Uytterhoeven
  2021-05-04 16:47 ` Kieran Bingham
  2021-05-04 22:49 ` Laurent Pinchart
  0 siblings, 2 replies; 3+ messages in thread
From: Geert Uytterhoeven @ 2021-05-04  9:14 UTC (permalink / raw)
  To: Magnus Damm, Laurent Pinchart, Kieran Bingham
  Cc: linux-renesas-soc, linux-arm-kernel, Valentine Barshak,
	Geert Uytterhoeven

From: Valentine Barshak <valentine.barshak@cogentembedded.com>

This adds X1 clock which supplies a frequency of 148.5 MHz.
This clock is connected to the external dot clock input signal.

Signed-off-by: Valentine Barshak <valentine.barshak@cogentembedded.com>
[geert: Verified schematics]
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Untested due to lack of hardware
---
 arch/arm64/boot/dts/renesas/r8a77970-eagle.dts | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
index 874a7fc2730b00db..5c84681703edad2e 100644
--- a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
+++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
@@ -73,6 +73,12 @@ memory@48000000 {
 		/* first 128MB is reserved for secure area. */
 		reg = <0x0 0x48000000 0x0 0x38000000>;
 	};
+
+	x1_clk: x1-clock {
+		compatible = "fixed-clock";
+		#clock-cells = <0>;
+		clock-frequency = <148500000>;
+	};
 };
 
 &avb {
@@ -104,6 +110,8 @@ channel0 {
 };
 
 &du {
+	clocks = <&cpg CPG_MOD 724>, <&x1_clk>;
+	clock-names = "du.0", "dclkin.0";
 	status = "okay";
 };
 
-- 
2.25.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: dts: renesas: eagle: Add x1 clock
  2021-05-04  9:14 [PATCH] arm64: dts: renesas: eagle: Add x1 clock Geert Uytterhoeven
@ 2021-05-04 16:47 ` Kieran Bingham
  2021-05-04 22:49 ` Laurent Pinchart
  1 sibling, 0 replies; 3+ messages in thread
From: Kieran Bingham @ 2021-05-04 16:47 UTC (permalink / raw)
  To: Geert Uytterhoeven, Magnus Damm, Laurent Pinchart
  Cc: linux-renesas-soc, linux-arm-kernel, Valentine Barshak

Hi Geert, Valentine,

On 04/05/2021 10:14, Geert Uytterhoeven wrote:
> From: Valentine Barshak <valentine.barshak@cogentembedded.com>
> 
> This adds X1 clock which supplies a frequency of 148.5 MHz.
> This clock is connected to the external dot clock input signal.
> 
> Signed-off-by: Valentine Barshak <valentine.barshak@cogentembedded.com>
> [geert: Verified schematics]
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> Untested due to lack of hardware

Tested on my Eagle-V3M with running the DU tests:

  Testing composition on CRTC 46: SUCCESS
  Testing connector HDMI-A-1: SUCCESS
  Testing plane formats: SUCCESS
  Testing legacy mode set on connector HDMI-A-1: SUCCESS
  Testing modes on connector HDMI-A-1: SUCCESS
  Testing atomic mode set on connector HDMI-A-1: SUCCESS
  Testing page flip on connector HDMI-A-1: SUCCESS
  Testing plane positioning boundaries: SUCCESS

and verifying that the output visible was as expected.

Tested-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>

> ---
>  arch/arm64/boot/dts/renesas/r8a77970-eagle.dts | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> index 874a7fc2730b00db..5c84681703edad2e 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> @@ -73,6 +73,12 @@ memory@48000000 {
>  		/* first 128MB is reserved for secure area. */
>  		reg = <0x0 0x48000000 0x0 0x38000000>;
>  	};
> +
> +	x1_clk: x1-clock {
> +		compatible = "fixed-clock";
> +		#clock-cells = <0>;
> +		clock-frequency = <148500000>;
> +	};
>  };
>  
>  &avb {
> @@ -104,6 +110,8 @@ channel0 {
>  };
>  
>  &du {
> +	clocks = <&cpg CPG_MOD 724>, <&x1_clk>;
> +	clock-names = "du.0", "dclkin.0";
>  	status = "okay";
>  };
>  
> 


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: dts: renesas: eagle: Add x1 clock
  2021-05-04  9:14 [PATCH] arm64: dts: renesas: eagle: Add x1 clock Geert Uytterhoeven
  2021-05-04 16:47 ` Kieran Bingham
@ 2021-05-04 22:49 ` Laurent Pinchart
  1 sibling, 0 replies; 3+ messages in thread
From: Laurent Pinchart @ 2021-05-04 22:49 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Magnus Damm, Kieran Bingham, linux-renesas-soc, linux-arm-kernel,
	Valentine Barshak

Hi Geert and Valentine,

Thank you for the patch.

On Tue, May 04, 2021 at 11:14:34AM +0200, Geert Uytterhoeven wrote:
> From: Valentine Barshak <valentine.barshak@cogentembedded.com>
> 
> This adds X1 clock which supplies a frequency of 148.5 MHz.
> This clock is connected to the external dot clock input signal.
> 
> Signed-off-by: Valentine Barshak <valentine.barshak@cogentembedded.com>
> [geert: Verified schematics]
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
> Untested due to lack of hardware
> ---
>  arch/arm64/boot/dts/renesas/r8a77970-eagle.dts | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> index 874a7fc2730b00db..5c84681703edad2e 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> @@ -73,6 +73,12 @@ memory@48000000 {
>  		/* first 128MB is reserved for secure area. */
>  		reg = <0x0 0x48000000 0x0 0x38000000>;
>  	};
> +
> +	x1_clk: x1-clock {
> +		compatible = "fixed-clock";
> +		#clock-cells = <0>;
> +		clock-frequency = <148500000>;
> +	};
>  };
>  
>  &avb {
> @@ -104,6 +110,8 @@ channel0 {
>  };
>  
>  &du {
> +	clocks = <&cpg CPG_MOD 724>, <&x1_clk>;
> +	clock-names = "du.0", "dclkin.0";
>  	status = "okay";
>  };
>  

-- 
Regards,

Laurent Pinchart

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-05-04 22:52 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-04  9:14 [PATCH] arm64: dts: renesas: eagle: Add x1 clock Geert Uytterhoeven
2021-05-04 16:47 ` Kieran Bingham
2021-05-04 22:49 ` Laurent Pinchart

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).