Linux-Clk Archive on lore.kernel.org
 help / color / Atom feed
* [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch
@ 2019-12-11 14:07 Benoit Parrot
  2019-12-11 14:07 ` [Patch v3 1/3] ARM: dts: am43xx: add support for clkout1 clock Benoit Parrot
                   ` (3 more replies)
  0 siblings, 4 replies; 11+ messages in thread
From: Benoit Parrot @ 2019-12-11 14:07 UTC (permalink / raw)
  To: Tony Lindgren, Tero Kristo, linux-omap, linux-clk
  Cc: Rob Herring, devicetree, linux-kernel, Benoit Parrot

This patch series adds the missing camera endpoint (ov2659) as well as
the required source clocks nodes for the sensor.

On the am437x-sk-evm the camera sensor is sourced from clkout1 but that
clock nodes/tree was removed as it was unsed at the time, we are
re-adding the needed clock nodes here.

Changes since v2:
- Fixed/added Tony's ack

Changes since v1:
- Fix clock name to make it generic
- Add non-standard clock node naming to commit message as per Tony's
  comment
- Rename all clock nodes to use '-' instead of '_'

Benoit Parrot (2):
  ARM: dts: am437x-sk-evm: Add VPFE and OV2659 entries
  ARM: dts: am43x-epos-evm: Add VPFE and OV2659 entries

Tero Kristo (1):
  ARM: dts: am43xx: add support for clkout1 clock

 arch/arm/boot/dts/am437x-sk-evm.dts  | 27 +++++++++++++-
 arch/arm/boot/dts/am43x-epos-evm.dts | 23 +++++++++++-
 arch/arm/boot/dts/am43xx-clocks.dtsi | 54 ++++++++++++++++++++++++++++
 3 files changed, 102 insertions(+), 2 deletions(-)

-- 
2.17.1


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

* [Patch v3 1/3] ARM: dts: am43xx: add support for clkout1 clock
  2019-12-11 14:07 [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch Benoit Parrot
@ 2019-12-11 14:07 ` Benoit Parrot
  2019-12-11 14:07 ` [Patch v3 2/3] ARM: dts: am437x-sk-evm: Add VPFE and OV2659 entries Benoit Parrot
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 11+ messages in thread
From: Benoit Parrot @ 2019-12-11 14:07 UTC (permalink / raw)
  To: Tony Lindgren, Tero Kristo, linux-omap, linux-clk
  Cc: Rob Herring, devicetree, linux-kernel, Benoit Parrot

From: Tero Kristo <t-kristo@ti.com>

clkout1 clock node and its generation tree was missing. Add this based
on the data on TRM and PRCM functional spec.

commit 664ae1ab2536 ("ARM: dts: am43xx: add clkctrl nodes") effectively
reverted this commit 8010f13a40d3 ("ARM: dts: am43xx: add support for
clkout1 clock") which is needed for the ov2659 camera sensor clock
definition hence it is being re-applied here.

Note that because of the current dts node name dependency for mapping to
clock domain, we must still use "clkout1-*ck" naming instead of generic
"clock@" naming for the node. And because of this, it's probably best to
apply the dts node addition together along with the other clock changes.

Fixes: 664ae1ab2536 ("ARM: dts: am43xx: add clkctrl nodes")
Signed-off-by: Tero Kristo <t-kristo@ti.com>
Tested-by: Benoit Parrot <bparrot@ti.com>
Acked-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Benoit Parrot <bparrot@ti.com>
---
 arch/arm/boot/dts/am43xx-clocks.dtsi | 54 ++++++++++++++++++++++++++++
 1 file changed, 54 insertions(+)

diff --git a/arch/arm/boot/dts/am43xx-clocks.dtsi b/arch/arm/boot/dts/am43xx-clocks.dtsi
index 091356f2a8c1..c726cd8dbdf1 100644
--- a/arch/arm/boot/dts/am43xx-clocks.dtsi
+++ b/arch/arm/boot/dts/am43xx-clocks.dtsi
@@ -704,6 +704,60 @@
 		ti,bit-shift = <8>;
 		reg = <0x2a48>;
 	};
+
+	clkout1_osc_div_ck: clkout1-osc-div-ck {
+		#clock-cells = <0>;
+		compatible = "ti,divider-clock";
+		clocks = <&sys_clkin_ck>;
+		ti,bit-shift = <20>;
+		ti,max-div = <4>;
+		reg = <0x4100>;
+	};
+
+	clkout1_src2_mux_ck: clkout1-src2-mux-ck {
+		#clock-cells = <0>;
+		compatible = "ti,mux-clock";
+		clocks = <&clk_rc32k_ck>, <&sysclk_div>, <&dpll_ddr_m2_ck>,
+			 <&dpll_per_m2_ck>, <&dpll_disp_m2_ck>,
+			 <&dpll_mpu_m2_ck>;
+		reg = <0x4100>;
+	};
+
+	clkout1_src2_pre_div_ck: clkout1-src2-pre-div-ck {
+		#clock-cells = <0>;
+		compatible = "ti,divider-clock";
+		clocks = <&clkout1_src2_mux_ck>;
+		ti,bit-shift = <4>;
+		ti,max-div = <8>;
+		reg = <0x4100>;
+	};
+
+	clkout1_src2_post_div_ck: clkout1-src2-post-div-ck {
+		#clock-cells = <0>;
+		compatible = "ti,divider-clock";
+		clocks = <&clkout1_src2_pre_div_ck>;
+		ti,bit-shift = <8>;
+		ti,max-div = <32>;
+		ti,index-power-of-two;
+		reg = <0x4100>;
+	};
+
+	clkout1_mux_ck: clkout1-mux-ck {
+		#clock-cells = <0>;
+		compatible = "ti,mux-clock";
+		clocks = <&clkout1_osc_div_ck>, <&clk_rc32k_ck>,
+			 <&clkout1_src2_post_div_ck>, <&dpll_extdev_m2_ck>;
+		ti,bit-shift = <16>;
+		reg = <0x4100>;
+	};
+
+	clkout1_ck: clkout1-ck {
+		#clock-cells = <0>;
+		compatible = "ti,gate-clock";
+		clocks = <&clkout1_mux_ck>;
+		ti,bit-shift = <23>;
+		reg = <0x4100>;
+	};
 };
 
 &prcm {
-- 
2.17.1


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

* [Patch v3 2/3] ARM: dts: am437x-sk-evm: Add VPFE and OV2659 entries
  2019-12-11 14:07 [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch Benoit Parrot
  2019-12-11 14:07 ` [Patch v3 1/3] ARM: dts: am43xx: add support for clkout1 clock Benoit Parrot
@ 2019-12-11 14:07 ` Benoit Parrot
  2019-12-11 14:07 ` [Patch v3 3/3] ARM: dts: am43x-epos-evm: " Benoit Parrot
  2019-12-12 17:41 ` [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch Tony Lindgren
  3 siblings, 0 replies; 11+ messages in thread
From: Benoit Parrot @ 2019-12-11 14:07 UTC (permalink / raw)
  To: Tony Lindgren, Tero Kristo, linux-omap, linux-clk
  Cc: Rob Herring, devicetree, linux-kernel, Benoit Parrot

Add VPFE device nodes entries.
Add OmniVision OV2659 sensor device nodes and linkage.

The sensor clock (xvclk) is sourced from clkout1.
Add clock entries to properly select clkout1 and set its parent
clock to sys_clkin_ck.

Signed-off-by: Benoit Parrot <bparrot@ti.com>
---
 arch/arm/boot/dts/am437x-sk-evm.dts | 27 ++++++++++++++++++++++++++-
 1 file changed, 26 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/am437x-sk-evm.dts b/arch/arm/boot/dts/am437x-sk-evm.dts
index 74eaa6a3b258..25222497f828 100644
--- a/arch/arm/boot/dts/am437x-sk-evm.dts
+++ b/arch/arm/boot/dts/am437x-sk-evm.dts
@@ -272,6 +272,12 @@
 		>;
 	};
 
+	clkout1_pin: pinmux_clkout1_pin {
+		pinctrl-single,pins = <
+			0x270 (PIN_OUTPUT_PULLDOWN | MUX_MODE3)	/* XDMA_EVENT_INTR0/CLKOUT1 */
+		>;
+	};
+
 	cpsw_default: cpsw_default {
 		pinctrl-single,pins = <
 			/* Slave 1 */
@@ -593,6 +599,25 @@
 	pinctrl-0 = <&i2c1_pins>;
 	clock-frequency = <400000>;
 
+	ov2659@30 {
+		compatible = "ovti,ov2659";
+		reg = <0x30>;
+		pinctrl-names = "default";
+		pinctrl-0 = <&clkout1_pin>;
+
+		clocks = <&clkout1_mux_ck>;
+		clock-names = "xvclk";
+		assigned-clocks = <&clkout1_mux_ck>;
+		assigned-clock-parents = <&clkout1_osc_div_ck>;
+
+		port {
+			ov2659_1: endpoint {
+				remote-endpoint = <&vpfe0_ep>;
+				link-frequencies = /bits/ 64 <70000000>;
+			};
+		};
+	};
+
 	edt-ft5306@38 {
 		status = "okay";
 		compatible = "edt,edt-ft5306", "edt,edt-ft5x06";
@@ -877,7 +902,7 @@
 	/* Camera port */
 	port {
 		vpfe0_ep: endpoint {
-			/* remote-endpoint = <&sensor>; add once we have it */
+			remote-endpoint = <&ov2659_1>;
 			ti,am437x-vpfe-interface = <0>;
 			bus-width = <8>;
 			hsync-active = <0>;
-- 
2.17.1


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

* [Patch v3 3/3] ARM: dts: am43x-epos-evm: Add VPFE and OV2659 entries
  2019-12-11 14:07 [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch Benoit Parrot
  2019-12-11 14:07 ` [Patch v3 1/3] ARM: dts: am43xx: add support for clkout1 clock Benoit Parrot
  2019-12-11 14:07 ` [Patch v3 2/3] ARM: dts: am437x-sk-evm: Add VPFE and OV2659 entries Benoit Parrot
@ 2019-12-11 14:07 ` " Benoit Parrot
  2019-12-12 17:41 ` [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch Tony Lindgren
  3 siblings, 0 replies; 11+ messages in thread
From: Benoit Parrot @ 2019-12-11 14:07 UTC (permalink / raw)
  To: Tony Lindgren, Tero Kristo, linux-omap, linux-clk
  Cc: Rob Herring, devicetree, linux-kernel, Benoit Parrot

Add VPFE device nodes entries.
Add OmniVision OV2659 sensor device nodes and linkage.

Since Rev1.2a on this board the sensor source clock (xvclk) has a
dedicated 12Mhz oscillator instead of using clkout1.
Add 'audio_mstrclk' fixed clock object to represent it.

Signed-off-by: Benoit Parrot <bparrot@ti.com>
---
 arch/arm/boot/dts/am43x-epos-evm.dts | 23 ++++++++++++++++++++++-
 1 file changed, 22 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/am43x-epos-evm.dts b/arch/arm/boot/dts/am43x-epos-evm.dts
index 95314121d111..b0df37014888 100644
--- a/arch/arm/boot/dts/am43x-epos-evm.dts
+++ b/arch/arm/boot/dts/am43x-epos-evm.dts
@@ -145,6 +145,12 @@
 			system-clock-frequency = <12000000>;
 		};
 	};
+
+	audio_mstrclk: clock {
+		compatible = "fixed-clock";
+		#clock-cells = <0>;
+		clock-frequency = <12000000>;
+	};
 };
 
 &am43xx_pinmux {
@@ -696,6 +702,21 @@
 		IOVDD-supply = <&dcdc4>; /* V3_3D -> DCDC4 */
 		DVDD-supply = <&ldo1>; /* V1_8AUD -> V1_8D -> LDO1 */
 	};
+
+	ov2659@30 {
+		compatible = "ovti,ov2659";
+		reg = <0x30>;
+
+		clocks = <&audio_mstrclk>;
+		clock-names = "xvclk";
+
+		port {
+			ov2659_1: endpoint {
+				remote-endpoint = <&vpfe1_ep>;
+				link-frequencies = /bits/ 64 <70000000>;
+			};
+		};
+	};
 };
 
 &i2c2 {
@@ -962,7 +983,7 @@
 
 	port {
 		vpfe1_ep: endpoint {
-			/* remote-endpoint = <&sensor>; add once we have it */
+			remote-endpoint = <&ov2659_1>;
 			ti,am437x-vpfe-interface = <0>;
 			bus-width = <8>;
 			hsync-active = <0>;
-- 
2.17.1


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

* Re: [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch
  2019-12-11 14:07 [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch Benoit Parrot
                   ` (2 preceding siblings ...)
  2019-12-11 14:07 ` [Patch v3 3/3] ARM: dts: am43x-epos-evm: " Benoit Parrot
@ 2019-12-12 17:41 ` Tony Lindgren
  2019-12-13  7:42   ` Tero Kristo
  3 siblings, 1 reply; 11+ messages in thread
From: Tony Lindgren @ 2019-12-12 17:41 UTC (permalink / raw)
  To: Benoit Parrot
  Cc: Tero Kristo, linux-omap, linux-clk, Rob Herring, devicetree,
	linux-kernel

* Benoit Parrot <bparrot@ti.com> [191211 06:04]:
> This patch series adds the missing camera endpoint (ov2659) as well as
> the required source clocks nodes for the sensor.
> 
> On the am437x-sk-evm the camera sensor is sourced from clkout1 but that
> clock nodes/tree was removed as it was unsed at the time, we are
> re-adding the needed clock nodes here.

Tero, it seems I can already pick this series?

Or ou want to queue the changes to am43xx-clocks.dtsi along with all
your other clock patches?

Regards,

Tony

 
> Changes since v2:
> - Fixed/added Tony's ack
> 
> Changes since v1:
> - Fix clock name to make it generic
> - Add non-standard clock node naming to commit message as per Tony's
>   comment
> - Rename all clock nodes to use '-' instead of '_'
> 
> Benoit Parrot (2):
>   ARM: dts: am437x-sk-evm: Add VPFE and OV2659 entries
>   ARM: dts: am43x-epos-evm: Add VPFE and OV2659 entries
> 
> Tero Kristo (1):
>   ARM: dts: am43xx: add support for clkout1 clock
> 
>  arch/arm/boot/dts/am437x-sk-evm.dts  | 27 +++++++++++++-
>  arch/arm/boot/dts/am43x-epos-evm.dts | 23 +++++++++++-
>  arch/arm/boot/dts/am43xx-clocks.dtsi | 54 ++++++++++++++++++++++++++++
>  3 files changed, 102 insertions(+), 2 deletions(-)
> 
> -- 
> 2.17.1
> 

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

* Re: [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch
  2019-12-12 17:41 ` [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch Tony Lindgren
@ 2019-12-13  7:42   ` Tero Kristo
  2019-12-13 15:29     ` Tony Lindgren
  0 siblings, 1 reply; 11+ messages in thread
From: Tero Kristo @ 2019-12-13  7:42 UTC (permalink / raw)
  To: Tony Lindgren, Benoit Parrot
  Cc: linux-omap, linux-clk, Rob Herring, devicetree, linux-kernel

On 12/12/2019 19:41, Tony Lindgren wrote:
> * Benoit Parrot <bparrot@ti.com> [191211 06:04]:
>> This patch series adds the missing camera endpoint (ov2659) as well as
>> the required source clocks nodes for the sensor.
>>
>> On the am437x-sk-evm the camera sensor is sourced from clkout1 but that
>> clock nodes/tree was removed as it was unsed at the time, we are
>> re-adding the needed clock nodes here.
> 
> Tero, it seems I can already pick this series?

I believe it is ready if you approve the clkout1 clock patch.

> Or ou want to queue the changes to am43xx-clocks.dtsi along with all
> your other clock patches?

Well, I have actually never queued any omap2+ dts patches myself, and I 
don't think there would be too many of those coming for next merge either.

-Tero

> 
> Regards,
> 
> Tony
> 
>   
>> Changes since v2:
>> - Fixed/added Tony's ack
>>
>> Changes since v1:
>> - Fix clock name to make it generic
>> - Add non-standard clock node naming to commit message as per Tony's
>>    comment
>> - Rename all clock nodes to use '-' instead of '_'
>>
>> Benoit Parrot (2):
>>    ARM: dts: am437x-sk-evm: Add VPFE and OV2659 entries
>>    ARM: dts: am43x-epos-evm: Add VPFE and OV2659 entries
>>
>> Tero Kristo (1):
>>    ARM: dts: am43xx: add support for clkout1 clock
>>
>>   arch/arm/boot/dts/am437x-sk-evm.dts  | 27 +++++++++++++-
>>   arch/arm/boot/dts/am43x-epos-evm.dts | 23 +++++++++++-
>>   arch/arm/boot/dts/am43xx-clocks.dtsi | 54 ++++++++++++++++++++++++++++
>>   3 files changed, 102 insertions(+), 2 deletions(-)
>>
>> -- 
>> 2.17.1
>>

--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

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

* Re: [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch
  2019-12-13  7:42   ` Tero Kristo
@ 2019-12-13 15:29     ` Tony Lindgren
  2020-01-13 16:54       ` Benoit Parrot
  0 siblings, 1 reply; 11+ messages in thread
From: Tony Lindgren @ 2019-12-13 15:29 UTC (permalink / raw)
  To: Tero Kristo
  Cc: Benoit Parrot, linux-omap, linux-clk, Rob Herring, devicetree,
	linux-kernel

* Tero Kristo <t-kristo@ti.com> [191213 07:43]:
> On 12/12/2019 19:41, Tony Lindgren wrote:
> > * Benoit Parrot <bparrot@ti.com> [191211 06:04]:
> > > This patch series adds the missing camera endpoint (ov2659) as well as
> > > the required source clocks nodes for the sensor.
> > > 
> > > On the am437x-sk-evm the camera sensor is sourced from clkout1 but that
> > > clock nodes/tree was removed as it was unsed at the time, we are
> > > re-adding the needed clock nodes here.
> > 
> > Tero, it seems I can already pick this series?
> 
> I believe it is ready if you approve the clkout1 clock patch.

OK yeah looks fine.

> > Or ou want to queue the changes to am43xx-clocks.dtsi along with all
> > your other clock patches?
> 
> Well, I have actually never queued any omap2+ dts patches myself, and I
> don't think there would be too many of those coming for next merge either.

OK will queue this series then. For the other ones from Benoit
looks like we need an immutable clock branch before I can apply
anything.

Regards,

Tony

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

* Re: [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch
  2019-12-13 15:29     ` Tony Lindgren
@ 2020-01-13 16:54       ` Benoit Parrot
  2020-01-14 16:04         ` Tony Lindgren
  0 siblings, 1 reply; 11+ messages in thread
From: Benoit Parrot @ 2020-01-13 16:54 UTC (permalink / raw)
  To: Tony Lindgren
  Cc: Tero Kristo, linux-omap, linux-clk, Rob Herring, devicetree,
	linux-kernel

Tony Lindgren <tony@atomide.com> wrote on Fri [2019-Dec-13 07:29:38 -0800]:
> * Tero Kristo <t-kristo@ti.com> [191213 07:43]:
> > On 12/12/2019 19:41, Tony Lindgren wrote:
> > > * Benoit Parrot <bparrot@ti.com> [191211 06:04]:
> > > > This patch series adds the missing camera endpoint (ov2659) as well as
> > > > the required source clocks nodes for the sensor.
> > > > 
> > > > On the am437x-sk-evm the camera sensor is sourced from clkout1 but that
> > > > clock nodes/tree was removed as it was unsed at the time, we are
> > > > re-adding the needed clock nodes here.
> > > 
> > > Tero, it seems I can already pick this series?
> > 
> > I believe it is ready if you approve the clkout1 clock patch.
> 
> OK yeah looks fine.
> 
> > > Or ou want to queue the changes to am43xx-clocks.dtsi along with all
> > > your other clock patches?
> > 
> > Well, I have actually never queued any omap2+ dts patches myself, and I
> > don't think there would be too many of those coming for next merge either.
> 
> OK will queue this series then. For the other ones from Benoit
> looks like we need an immutable clock branch before I can apply
> anything.

Tony, Tero,

Are these merged anyware now?
I still don't see any of these on linux-next?

And by "these" I mean this one and both
 ARM: dts: dra7: add vpe nodes
 ARM: dts: dra7: add cal nodes

Regards,
Benoit

> 
> Regards,
> 
> Tony

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

* Re: [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch
  2020-01-13 16:54       ` Benoit Parrot
@ 2020-01-14 16:04         ` Tony Lindgren
  2020-01-17 14:15           ` Tero Kristo
  0 siblings, 1 reply; 11+ messages in thread
From: Tony Lindgren @ 2020-01-14 16:04 UTC (permalink / raw)
  To: Benoit Parrot
  Cc: Tero Kristo, linux-omap, linux-clk, Rob Herring, devicetree,
	linux-kernel

* Benoit Parrot <bparrot@ti.com> [200113 16:51]:
> Tony Lindgren <tony@atomide.com> wrote on Fri [2019-Dec-13 07:29:38 -0800]:
> > * Tero Kristo <t-kristo@ti.com> [191213 07:43]:
> > > On 12/12/2019 19:41, Tony Lindgren wrote:
> > > > * Benoit Parrot <bparrot@ti.com> [191211 06:04]:
> > > > > This patch series adds the missing camera endpoint (ov2659) as well as
> > > > > the required source clocks nodes for the sensor.
> > > > > 
> > > > > On the am437x-sk-evm the camera sensor is sourced from clkout1 but that
> > > > > clock nodes/tree was removed as it was unsed at the time, we are
> > > > > re-adding the needed clock nodes here.
> > > > 
> > > > Tero, it seems I can already pick this series?
> > > 
> > > I believe it is ready if you approve the clkout1 clock patch.
> > 
> > OK yeah looks fine.
> > 
> > > > Or ou want to queue the changes to am43xx-clocks.dtsi along with all
> > > > your other clock patches?
> > > 
> > > Well, I have actually never queued any omap2+ dts patches myself, and I
> > > don't think there would be too many of those coming for next merge either.
> > 
> > OK will queue this series then. For the other ones from Benoit
> > looks like we need an immutable clock branch before I can apply
> > anything.
> 
> Tony, Tero,
> 
> Are these merged anyware now?
> I still don't see any of these on linux-next?

Tero any news on getting some immutable clock changes branch done?
Looks like there are quite a few pending clock patches right now,
probably best to set them all into a branch that I can also merge
in.

> And by "these" I mean this one and both
>  ARM: dts: dra7: add vpe nodes
>  ARM: dts: dra7: add cal nodes

Yeah looks good to me other than the clock dependency.

Regards,

Tony

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

* Re: [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch
  2020-01-14 16:04         ` Tony Lindgren
@ 2020-01-17 14:15           ` Tero Kristo
  2020-01-23 17:20             ` Tony Lindgren
  0 siblings, 1 reply; 11+ messages in thread
From: Tero Kristo @ 2020-01-17 14:15 UTC (permalink / raw)
  To: Tony Lindgren, Benoit Parrot
  Cc: linux-omap, linux-clk, Rob Herring, devicetree, linux-kernel

On 14/01/2020 18:04, Tony Lindgren wrote:
> * Benoit Parrot <bparrot@ti.com> [200113 16:51]:
>> Tony Lindgren <tony@atomide.com> wrote on Fri [2019-Dec-13 07:29:38 -0800]:
>>> * Tero Kristo <t-kristo@ti.com> [191213 07:43]:
>>>> On 12/12/2019 19:41, Tony Lindgren wrote:
>>>>> * Benoit Parrot <bparrot@ti.com> [191211 06:04]:
>>>>>> This patch series adds the missing camera endpoint (ov2659) as well as
>>>>>> the required source clocks nodes for the sensor.
>>>>>>
>>>>>> On the am437x-sk-evm the camera sensor is sourced from clkout1 but that
>>>>>> clock nodes/tree was removed as it was unsed at the time, we are
>>>>>> re-adding the needed clock nodes here.
>>>>>
>>>>> Tero, it seems I can already pick this series?
>>>>
>>>> I believe it is ready if you approve the clkout1 clock patch.
>>>
>>> OK yeah looks fine.
>>>
>>>>> Or ou want to queue the changes to am43xx-clocks.dtsi along with all
>>>>> your other clock patches?
>>>>
>>>> Well, I have actually never queued any omap2+ dts patches myself, and I
>>>> don't think there would be too many of those coming for next merge either.
>>>
>>> OK will queue this series then. For the other ones from Benoit
>>> looks like we need an immutable clock branch before I can apply
>>> anything.
>>
>> Tony, Tero,
>>
>> Are these merged anyware now?
>> I still don't see any of these on linux-next?
> 
> Tero any news on getting some immutable clock changes branch done?
> Looks like there are quite a few pending clock patches right now,
> probably best to set them all into a branch that I can also merge
> in.

Working on that now, have been forced to prioritize work lately. I can 
probably get the branch set-up on Monday and send a pull-req out 
assuming it is not too late for 5.6 already.

-Tero

> 
>> And by "these" I mean this one and both
>>   ARM: dts: dra7: add vpe nodes
>>   ARM: dts: dra7: add cal nodes
> 
> Yeah looks good to me other than the clock dependency.
> 
> Regards,
> 
> Tony
> 

--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

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

* Re: [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch
  2020-01-17 14:15           ` Tero Kristo
@ 2020-01-23 17:20             ` Tony Lindgren
  0 siblings, 0 replies; 11+ messages in thread
From: Tony Lindgren @ 2020-01-23 17:20 UTC (permalink / raw)
  To: Tero Kristo
  Cc: Benoit Parrot, linux-omap, linux-clk, Rob Herring, devicetree,
	linux-kernel

* Tero Kristo <t-kristo@ti.com> [200117 14:16]:
> On 14/01/2020 18:04, Tony Lindgren wrote:
> > * Benoit Parrot <bparrot@ti.com> [200113 16:51]:
> > > Tony Lindgren <tony@atomide.com> wrote on Fri [2019-Dec-13 07:29:38 -0800]:
> > > > * Tero Kristo <t-kristo@ti.com> [191213 07:43]:
> > > > > On 12/12/2019 19:41, Tony Lindgren wrote:
> > > > > > * Benoit Parrot <bparrot@ti.com> [191211 06:04]:
> > > > > > > This patch series adds the missing camera endpoint (ov2659) as well as
> > > > > > > the required source clocks nodes for the sensor.
> > > > > > > 
> > > > > > > On the am437x-sk-evm the camera sensor is sourced from clkout1 but that
> > > > > > > clock nodes/tree was removed as it was unsed at the time, we are
> > > > > > > re-adding the needed clock nodes here.
> > > > > > 
> > > > > > Tero, it seems I can already pick this series?
> > > > > 
> > > > > I believe it is ready if you approve the clkout1 clock patch.
> > > > 
> > > > OK yeah looks fine.
> > > > 
> > > > > > Or ou want to queue the changes to am43xx-clocks.dtsi along with all
> > > > > > your other clock patches?
> > > > > 
> > > > > Well, I have actually never queued any omap2+ dts patches myself, and I
> > > > > don't think there would be too many of those coming for next merge either.
> > > > 
> > > > OK will queue this series then. For the other ones from Benoit
> > > > looks like we need an immutable clock branch before I can apply
> > > > anything.
> > > 
> > > Tony, Tero,
> > > 
> > > Are these merged anyware now?
> > > I still don't see any of these on linux-next?
> > 
> > Tero any news on getting some immutable clock changes branch done?
> > Looks like there are quite a few pending clock patches right now,
> > probably best to set them all into a branch that I can also merge
> > in.
> 
> Working on that now, have been forced to prioritize work lately. I can
> probably get the branch set-up on Monday and send a pull-req out assuming it
> is not too late for 5.6 already.

Yes it might be too late, but I'm applying these too into
omap-for-v5.6/ti-sysc-dt-cam.

Regards,

Tony

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

end of thread, back to index

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-11 14:07 [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch Benoit Parrot
2019-12-11 14:07 ` [Patch v3 1/3] ARM: dts: am43xx: add support for clkout1 clock Benoit Parrot
2019-12-11 14:07 ` [Patch v3 2/3] ARM: dts: am437x-sk-evm: Add VPFE and OV2659 entries Benoit Parrot
2019-12-11 14:07 ` [Patch v3 3/3] ARM: dts: am43x-epos-evm: " Benoit Parrot
2019-12-12 17:41 ` [Patch v3 0/3] ARM: dts: am43x-vpfe/ov2659.patch Tony Lindgren
2019-12-13  7:42   ` Tero Kristo
2019-12-13 15:29     ` Tony Lindgren
2020-01-13 16:54       ` Benoit Parrot
2020-01-14 16:04         ` Tony Lindgren
2020-01-17 14:15           ` Tero Kristo
2020-01-23 17:20             ` Tony Lindgren

Linux-Clk Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-clk/0 linux-clk/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-clk linux-clk/ https://lore.kernel.org/linux-clk \
		linux-clk@vger.kernel.org
	public-inbox-index linux-clk

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-clk


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git