Re: [PATCH] ARM: dts: set 'ti,set-rate-parent' for dpll4_m5x2 clock

2014-07-30 Thread Tero Kristo

On 07/16/2014 03:29 AM, Laurent Pinchart wrote:

Hi Stefan,

Thank you for the patch.

On Tuesday 15 July 2014 12:02:35 Stefan Herbrechtsmeier wrote:

Set 'ti,set-rate-parent' property for the dpll4_m5x2_ck clock, which
is used for the ISP functional clock. This fixes the OMAP3 ISP driver's
clock rate configuration on OMAP34xx, which needs the rate to be
propagated properly to the divider node (dpll4_m5_ck).

Signed-off-by: Stefan Herbrechtsmeier ste...@herbrechtsmeier.net
Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com
Cc: Tony Lindgren t...@atomide.com
Cc: Tero Kristo t-kri...@ti.com
Cc: linux-me...@vger.kernel.org
Cc: linux-omap@vger.kernel.org


Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com

Tero, could you please process it for v3.17 if time still permits ?


This is too late for 3.17 merge window as I was on holiday last few 
weeks. Queued for 3.17-rc early fixes though.


-Tero




---
  arch/arm/boot/dts/omap3xxx-clocks.dtsi | 1 +
  1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/omap3xxx-clocks.dtsi
b/arch/arm/boot/dts/omap3xxx-clocks.dtsi index e47ff69..5c37500 100644
--- a/arch/arm/boot/dts/omap3xxx-clocks.dtsi
+++ b/arch/arm/boot/dts/omap3xxx-clocks.dtsi
@@ -467,6 +467,7 @@
ti,bit-shift = 0x1e;
reg = 0x0d00;
ti,set-bit-to-disable;
+   ti,set-rate-parent;
};

dpll4_m6_ck: dpll4_m6_ck {




--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] ARM: dts: set 'ti,set-rate-parent' for dpll4_m5x2 clock

2014-07-30 Thread Laurent Pinchart
On Wednesday 30 July 2014 15:40:33 Tero Kristo wrote:
 On 07/16/2014 03:29 AM, Laurent Pinchart wrote:
  On Tuesday 15 July 2014 12:02:35 Stefan Herbrechtsmeier wrote:
  Set 'ti,set-rate-parent' property for the dpll4_m5x2_ck clock, which
  is used for the ISP functional clock. This fixes the OMAP3 ISP driver's
  clock rate configuration on OMAP34xx, which needs the rate to be
  propagated properly to the divider node (dpll4_m5_ck).
  
  Signed-off-by: Stefan Herbrechtsmeier ste...@herbrechtsmeier.net
  Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com
  Cc: Tony Lindgren t...@atomide.com
  Cc: Tero Kristo t-kri...@ti.com
  Cc: linux-me...@vger.kernel.org
  Cc: linux-omap@vger.kernel.org
  
  Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
  
  Tero, could you please process it for v3.17 if time still permits ?
 
 This is too late for 3.17 merge window as I was on holiday last few
 weeks. Queued for 3.17-rc early fixes though.

Kiitos.

  ---
  
arch/arm/boot/dts/omap3xxx-clocks.dtsi | 1 +
1 file changed, 1 insertion(+)
  
  diff --git a/arch/arm/boot/dts/omap3xxx-clocks.dtsi
  b/arch/arm/boot/dts/omap3xxx-clocks.dtsi index e47ff69..5c37500 100644
  --- a/arch/arm/boot/dts/omap3xxx-clocks.dtsi
  +++ b/arch/arm/boot/dts/omap3xxx-clocks.dtsi
  @@ -467,6 +467,7 @@
 ti,bit-shift = 0x1e;
 reg = 0x0d00;
 ti,set-bit-to-disable;
  +  ti,set-rate-parent;
 };
 
 dpll4_m6_ck: dpll4_m6_ck {

-- 
Regards,

Laurent Pinchart

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] ARM: dts: set 'ti,set-rate-parent' for dpll4_m5x2 clock

2014-07-15 Thread Stefan Herbrechtsmeier
Set 'ti,set-rate-parent' property for the dpll4_m5x2_ck clock, which
is used for the ISP functional clock. This fixes the OMAP3 ISP driver's
clock rate configuration on OMAP34xx, which needs the rate to be
propagated properly to the divider node (dpll4_m5_ck).

Signed-off-by: Stefan Herbrechtsmeier ste...@herbrechtsmeier.net
Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com
Cc: Tony Lindgren t...@atomide.com
Cc: Tero Kristo t-kri...@ti.com
Cc: linux-me...@vger.kernel.org
Cc: linux-omap@vger.kernel.org
---
 arch/arm/boot/dts/omap3xxx-clocks.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/omap3xxx-clocks.dtsi 
b/arch/arm/boot/dts/omap3xxx-clocks.dtsi
index e47ff69..5c37500 100644
--- a/arch/arm/boot/dts/omap3xxx-clocks.dtsi
+++ b/arch/arm/boot/dts/omap3xxx-clocks.dtsi
@@ -467,6 +467,7 @@
ti,bit-shift = 0x1e;
reg = 0x0d00;
ti,set-bit-to-disable;
+   ti,set-rate-parent;
};
 
dpll4_m6_ck: dpll4_m6_ck {
-- 
2.0.1

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] ARM: dts: set 'ti,set-rate-parent' for dpll4_m5x2 clock

2014-07-15 Thread Laurent Pinchart
Hi Stefan,

Thank you for the patch.

On Tuesday 15 July 2014 12:02:35 Stefan Herbrechtsmeier wrote:
 Set 'ti,set-rate-parent' property for the dpll4_m5x2_ck clock, which
 is used for the ISP functional clock. This fixes the OMAP3 ISP driver's
 clock rate configuration on OMAP34xx, which needs the rate to be
 propagated properly to the divider node (dpll4_m5_ck).
 
 Signed-off-by: Stefan Herbrechtsmeier ste...@herbrechtsmeier.net
 Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com
 Cc: Tony Lindgren t...@atomide.com
 Cc: Tero Kristo t-kri...@ti.com
 Cc: linux-me...@vger.kernel.org
 Cc: linux-omap@vger.kernel.org

Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com

Tero, could you please process it for v3.17 if time still permits ?

 ---
  arch/arm/boot/dts/omap3xxx-clocks.dtsi | 1 +
  1 file changed, 1 insertion(+)
 
 diff --git a/arch/arm/boot/dts/omap3xxx-clocks.dtsi
 b/arch/arm/boot/dts/omap3xxx-clocks.dtsi index e47ff69..5c37500 100644
 --- a/arch/arm/boot/dts/omap3xxx-clocks.dtsi
 +++ b/arch/arm/boot/dts/omap3xxx-clocks.dtsi
 @@ -467,6 +467,7 @@
   ti,bit-shift = 0x1e;
   reg = 0x0d00;
   ti,set-bit-to-disable;
 + ti,set-rate-parent;
   };
 
   dpll4_m6_ck: dpll4_m6_ck {

-- 
Regards,

Laurent Pinchart

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html