diff mbox

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

Message ID 1405418556-7030-1-git-send-email-stefan@herbrechtsmeier.net (mailing list archive)
State New, archived
Headers show

Commit Message

Stefan Herbrechtsmeier July 15, 2014, 10:02 a.m. UTC
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 <stefan@herbrechtsmeier.net>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Tony Lindgren <tony@atomide.com>
Cc: Tero Kristo <t-kristo@ti.com>
Cc: <linux-media@vger.kernel.org>
Cc: <linux-omap@vger.kernel.org>
---
 arch/arm/boot/dts/omap3xxx-clocks.dtsi | 1 +
 1 file changed, 1 insertion(+)

Comments

Laurent Pinchart July 16, 2014, 12:29 a.m. UTC | #1
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 <stefan@herbrechtsmeier.net>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: Tony Lindgren <tony@atomide.com>
> Cc: Tero Kristo <t-kristo@ti.com>
> Cc: <linux-media@vger.kernel.org>
> Cc: <linux-omap@vger.kernel.org>

Acked-by: Laurent Pinchart <laurent.pinchart@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 {
Tero Kristo July 30, 2014, 12:40 p.m. UTC | #2
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 <stefan@herbrechtsmeier.net>
>> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>> Cc: Tony Lindgren <tony@atomide.com>
>> Cc: Tero Kristo <t-kristo@ti.com>
>> Cc: <linux-media@vger.kernel.org>
>> Cc: <linux-omap@vger.kernel.org>
>
> Acked-by: Laurent Pinchart <laurent.pinchart@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-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Laurent Pinchart July 30, 2014, 12:44 p.m. UTC | #3
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 <stefan@herbrechtsmeier.net>
> >> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> >> Cc: Tony Lindgren <tony@atomide.com>
> >> Cc: Tero Kristo <t-kristo@ti.com>
> >> Cc: <linux-media@vger.kernel.org>
> >> Cc: <linux-omap@vger.kernel.org>
> > 
> > Acked-by: Laurent Pinchart <laurent.pinchart@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 {
diff mbox

Patch

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 {