* [PATCH] OMAP4: Clock: Correct OTG clock to use otg_60m_gfclk.
@ 2012-06-29 11:53 Ruslan Bilovol
2012-06-29 20:35 ` Paul Walmsley
0 siblings, 1 reply; 11+ messages in thread
From: Ruslan Bilovol @ 2012-06-29 11:53 UTC (permalink / raw)
To: linux-omap; +Cc: paul, tony
From: Wenbiao Wang <wwang@ti.com>
OTG clock usb_otg_hs_ick used a incorrect parent l3_div_ck.
Correct it to use the right colck otg_60m_gfclk as its
parent.
Signed-off-by: Wenbiao Wang <wwang@ti.com>
Signed-off-by: Ruslan Bilovol <ruslan.bilovol@ti.com>
---
arch/arm/mach-omap2/clock44xx_data.c | 15 ++++++++-------
1 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/arch/arm/mach-omap2/clock44xx_data.c b/arch/arm/mach-omap2/clock44xx_data.c
index b825049..fd43214 100644
--- a/arch/arm/mach-omap2/clock44xx_data.c
+++ b/arch/arm/mach-omap2/clock44xx_data.c
@@ -199,12 +199,6 @@ static struct clk tie_low_clock_ck = {
.ops = &clkops_null,
};
-static struct clk utmi_phy_clkout_ck = {
- .name = "utmi_phy_clkout_ck",
- .rate = 60000000,
- .ops = &clkops_null,
-};
-
static struct clk xclk60mhsp1_ck = {
.name = "xclk60mhsp1_ck",
.rate = 60000000,
@@ -992,6 +986,13 @@ static struct clk dpll_usb_clkdcoldo_ck = {
.recalc = &followparent_recalc,
};
+static struct clk utmi_phy_clkout_ck = {
+ .name = "utmi_phy_clkout_ck",
+ .ops = &clkops_null,
+ .parent = &dpll_usb_clkdcoldo_ck,
+ .recalc = &followparent_recalc,
+};
+
static const struct clksel dpll_usb_m2_div[] = {
{ .parent = &dpll_usb_ck, .rates = div31_1to31_rates },
{ .parent = NULL },
@@ -2685,7 +2686,7 @@ static struct clk usb_otg_hs_ick = {
.enable_reg = OMAP4430_CM_L3INIT_USB_OTG_CLKCTRL,
.enable_bit = OMAP4430_MODULEMODE_HWCTRL,
.clkdm_name = "l3_init_clkdm",
- .parent = &l3_div_ck,
+ .parent = &otg_60m_gfclk,
.recalc = &followparent_recalc,
};
--
1.7.1
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH] OMAP4: Clock: Correct OTG clock to use otg_60m_gfclk.
2012-06-29 11:53 [PATCH] OMAP4: Clock: Correct OTG clock to use otg_60m_gfclk Ruslan Bilovol
@ 2012-06-29 20:35 ` Paul Walmsley
0 siblings, 0 replies; 11+ messages in thread
From: Paul Walmsley @ 2012-06-29 20:35 UTC (permalink / raw)
To: Ruslan Bilovol, b-cousson; +Cc: linux-omap, linux-arm-kernel, tony
[-- Attachment #1: Type: TEXT/PLAIN, Size: 1966 bytes --]
+ Benoît who is the maintainer of this file
+ the linux-arm-kernel mailing list, which should be cc'ed on all OMAP
patches
On Fri, 29 Jun 2012, Ruslan Bilovol wrote:
> From: Wenbiao Wang <wwang@ti.com>
>
> OTG clock usb_otg_hs_ick used a incorrect parent l3_div_ck.
> Correct it to use the right colck otg_60m_gfclk as its
> parent.
>
> Signed-off-by: Wenbiao Wang <wwang@ti.com>
> Signed-off-by: Ruslan Bilovol <ruslan.bilovol@ti.com>
> ---
> arch/arm/mach-omap2/clock44xx_data.c | 15 ++++++++-------
> 1 files changed, 8 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/clock44xx_data.c b/arch/arm/mach-omap2/clock44xx_data.c
> index b825049..fd43214 100644
> --- a/arch/arm/mach-omap2/clock44xx_data.c
> +++ b/arch/arm/mach-omap2/clock44xx_data.c
> @@ -199,12 +199,6 @@ static struct clk tie_low_clock_ck = {
> .ops = &clkops_null,
> };
>
> -static struct clk utmi_phy_clkout_ck = {
> - .name = "utmi_phy_clkout_ck",
> - .rate = 60000000,
> - .ops = &clkops_null,
> -};
> -
> static struct clk xclk60mhsp1_ck = {
> .name = "xclk60mhsp1_ck",
> .rate = 60000000,
> @@ -992,6 +986,13 @@ static struct clk dpll_usb_clkdcoldo_ck = {
> .recalc = &followparent_recalc,
> };
>
> +static struct clk utmi_phy_clkout_ck = {
> + .name = "utmi_phy_clkout_ck",
> + .ops = &clkops_null,
> + .parent = &dpll_usb_clkdcoldo_ck,
> + .recalc = &followparent_recalc,
> +};
> +
> static const struct clksel dpll_usb_m2_div[] = {
> { .parent = &dpll_usb_ck, .rates = div31_1to31_rates },
> { .parent = NULL },
> @@ -2685,7 +2686,7 @@ static struct clk usb_otg_hs_ick = {
> .enable_reg = OMAP4430_CM_L3INIT_USB_OTG_CLKCTRL,
> .enable_bit = OMAP4430_MODULEMODE_HWCTRL,
> .clkdm_name = "l3_init_clkdm",
> - .parent = &l3_div_ck,
> + .parent = &otg_60m_gfclk,
> .recalc = &followparent_recalc,
> };
Benoît should have a look at this one, I think.
- Paul
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH] OMAP4: Clock: Correct OTG clock to use otg_60m_gfclk.
@ 2012-06-29 20:35 ` Paul Walmsley
0 siblings, 0 replies; 11+ messages in thread
From: Paul Walmsley @ 2012-06-29 20:35 UTC (permalink / raw)
To: linux-arm-kernel
+ Beno?t who is the maintainer of this file
+ the linux-arm-kernel mailing list, which should be cc'ed on all OMAP
patches
On Fri, 29 Jun 2012, Ruslan Bilovol wrote:
> From: Wenbiao Wang <wwang@ti.com>
>
> OTG clock usb_otg_hs_ick used a incorrect parent l3_div_ck.
> Correct it to use the right colck otg_60m_gfclk as its
> parent.
>
> Signed-off-by: Wenbiao Wang <wwang@ti.com>
> Signed-off-by: Ruslan Bilovol <ruslan.bilovol@ti.com>
> ---
> arch/arm/mach-omap2/clock44xx_data.c | 15 ++++++++-------
> 1 files changed, 8 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/clock44xx_data.c b/arch/arm/mach-omap2/clock44xx_data.c
> index b825049..fd43214 100644
> --- a/arch/arm/mach-omap2/clock44xx_data.c
> +++ b/arch/arm/mach-omap2/clock44xx_data.c
> @@ -199,12 +199,6 @@ static struct clk tie_low_clock_ck = {
> .ops = &clkops_null,
> };
>
> -static struct clk utmi_phy_clkout_ck = {
> - .name = "utmi_phy_clkout_ck",
> - .rate = 60000000,
> - .ops = &clkops_null,
> -};
> -
> static struct clk xclk60mhsp1_ck = {
> .name = "xclk60mhsp1_ck",
> .rate = 60000000,
> @@ -992,6 +986,13 @@ static struct clk dpll_usb_clkdcoldo_ck = {
> .recalc = &followparent_recalc,
> };
>
> +static struct clk utmi_phy_clkout_ck = {
> + .name = "utmi_phy_clkout_ck",
> + .ops = &clkops_null,
> + .parent = &dpll_usb_clkdcoldo_ck,
> + .recalc = &followparent_recalc,
> +};
> +
> static const struct clksel dpll_usb_m2_div[] = {
> { .parent = &dpll_usb_ck, .rates = div31_1to31_rates },
> { .parent = NULL },
> @@ -2685,7 +2686,7 @@ static struct clk usb_otg_hs_ick = {
> .enable_reg = OMAP4430_CM_L3INIT_USB_OTG_CLKCTRL,
> .enable_bit = OMAP4430_MODULEMODE_HWCTRL,
> .clkdm_name = "l3_init_clkdm",
> - .parent = &l3_div_ck,
> + .parent = &otg_60m_gfclk,
> .recalc = &followparent_recalc,
> };
Beno?t should have a look at this one, I think.
- Paul
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] OMAP4: Clock: Correct OTG clock to use otg_60m_gfclk.
2012-06-29 20:35 ` Paul Walmsley
@ 2012-07-02 12:43 ` Benoit Cousson
-1 siblings, 0 replies; 11+ messages in thread
From: Benoit Cousson @ 2012-07-02 12:43 UTC (permalink / raw)
To: Ruslan Bilovol; +Cc: Paul Walmsley, linux-omap, linux-arm-kernel, tony
On 06/29/2012 10:35 PM, Paul Walmsley wrote:
> + Benoît who is the maintainer of this file
>
> + the linux-arm-kernel mailing list, which should be cc'ed on all OMAP
> patches
>
> On Fri, 29 Jun 2012, Ruslan Bilovol wrote:
>
>> From: Wenbiao Wang <wwang@ti.com>
>>
>> OTG clock usb_otg_hs_ick used a incorrect parent l3_div_ck.
>> Correct it to use the right colck otg_60m_gfclk as its
>> parent.
Mmm, that does not seems to be correct.
otg_60m_gfclk is an optional clock. The interface clock is the main
clock of that module. That's why this is the parent of the fake
MODULEMODE clock node.
Moreover you are changing as well the utmi_phy_clkout_ck. That's not
mentioned at all in the changelog.
I know that there are some non standard stuff in this clock scheme.
The main reason being the utmi_phy_clkout_ck source is generated from
the usb_phy module. Unfortunately the clock fmwk cannot handle module as
a clock node.
So, as of today, this only way to get the OTG_60M_FCLK clock available
is to ensure that the usb_phy module is enabled before the usb_otg_hs
module.
Regards,
Benoit
>>
>> Signed-off-by: Wenbiao Wang <wwang@ti.com>
>> Signed-off-by: Ruslan Bilovol <ruslan.bilovol@ti.com>
>> ---
>> arch/arm/mach-omap2/clock44xx_data.c | 15 ++++++++-------
>> 1 files changed, 8 insertions(+), 7 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/clock44xx_data.c b/arch/arm/mach-omap2/clock44xx_data.c
>> index b825049..fd43214 100644
>> --- a/arch/arm/mach-omap2/clock44xx_data.c
>> +++ b/arch/arm/mach-omap2/clock44xx_data.c
>> @@ -199,12 +199,6 @@ static struct clk tie_low_clock_ck = {
>> .ops = &clkops_null,
>> };
>>
>> -static struct clk utmi_phy_clkout_ck = {
>> - .name = "utmi_phy_clkout_ck",
>> - .rate = 60000000,
>> - .ops = &clkops_null,
>> -};
>> -
>> static struct clk xclk60mhsp1_ck = {
>> .name = "xclk60mhsp1_ck",
>> .rate = 60000000,
>> @@ -992,6 +986,13 @@ static struct clk dpll_usb_clkdcoldo_ck = {
>> .recalc = &followparent_recalc,
>> };
>>
>> +static struct clk utmi_phy_clkout_ck = {
>> + .name = "utmi_phy_clkout_ck",
>> + .ops = &clkops_null,
>> + .parent = &dpll_usb_clkdcoldo_ck,
>> + .recalc = &followparent_recalc,
>> +};
>> +
>> static const struct clksel dpll_usb_m2_div[] = {
>> { .parent = &dpll_usb_ck, .rates = div31_1to31_rates },
>> { .parent = NULL },
>> @@ -2685,7 +2686,7 @@ static struct clk usb_otg_hs_ick = {
>> .enable_reg = OMAP4430_CM_L3INIT_USB_OTG_CLKCTRL,
>> .enable_bit = OMAP4430_MODULEMODE_HWCTRL,
>> .clkdm_name = "l3_init_clkdm",
>> - .parent = &l3_div_ck,
>> + .parent = &otg_60m_gfclk,
>> .recalc = &followparent_recalc,
>> };
>
> Benoît should have a look at this one, I think.
>
>
> - Paul
>
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH] OMAP4: Clock: Correct OTG clock to use otg_60m_gfclk.
@ 2012-07-02 12:43 ` Benoit Cousson
0 siblings, 0 replies; 11+ messages in thread
From: Benoit Cousson @ 2012-07-02 12:43 UTC (permalink / raw)
To: linux-arm-kernel
On 06/29/2012 10:35 PM, Paul Walmsley wrote:
> + Beno?t who is the maintainer of this file
>
> + the linux-arm-kernel mailing list, which should be cc'ed on all OMAP
> patches
>
> On Fri, 29 Jun 2012, Ruslan Bilovol wrote:
>
>> From: Wenbiao Wang <wwang@ti.com>
>>
>> OTG clock usb_otg_hs_ick used a incorrect parent l3_div_ck.
>> Correct it to use the right colck otg_60m_gfclk as its
>> parent.
Mmm, that does not seems to be correct.
otg_60m_gfclk is an optional clock. The interface clock is the main
clock of that module. That's why this is the parent of the fake
MODULEMODE clock node.
Moreover you are changing as well the utmi_phy_clkout_ck. That's not
mentioned at all in the changelog.
I know that there are some non standard stuff in this clock scheme.
The main reason being the utmi_phy_clkout_ck source is generated from
the usb_phy module. Unfortunately the clock fmwk cannot handle module as
a clock node.
So, as of today, this only way to get the OTG_60M_FCLK clock available
is to ensure that the usb_phy module is enabled before the usb_otg_hs
module.
Regards,
Benoit
>>
>> Signed-off-by: Wenbiao Wang <wwang@ti.com>
>> Signed-off-by: Ruslan Bilovol <ruslan.bilovol@ti.com>
>> ---
>> arch/arm/mach-omap2/clock44xx_data.c | 15 ++++++++-------
>> 1 files changed, 8 insertions(+), 7 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/clock44xx_data.c b/arch/arm/mach-omap2/clock44xx_data.c
>> index b825049..fd43214 100644
>> --- a/arch/arm/mach-omap2/clock44xx_data.c
>> +++ b/arch/arm/mach-omap2/clock44xx_data.c
>> @@ -199,12 +199,6 @@ static struct clk tie_low_clock_ck = {
>> .ops = &clkops_null,
>> };
>>
>> -static struct clk utmi_phy_clkout_ck = {
>> - .name = "utmi_phy_clkout_ck",
>> - .rate = 60000000,
>> - .ops = &clkops_null,
>> -};
>> -
>> static struct clk xclk60mhsp1_ck = {
>> .name = "xclk60mhsp1_ck",
>> .rate = 60000000,
>> @@ -992,6 +986,13 @@ static struct clk dpll_usb_clkdcoldo_ck = {
>> .recalc = &followparent_recalc,
>> };
>>
>> +static struct clk utmi_phy_clkout_ck = {
>> + .name = "utmi_phy_clkout_ck",
>> + .ops = &clkops_null,
>> + .parent = &dpll_usb_clkdcoldo_ck,
>> + .recalc = &followparent_recalc,
>> +};
>> +
>> static const struct clksel dpll_usb_m2_div[] = {
>> { .parent = &dpll_usb_ck, .rates = div31_1to31_rates },
>> { .parent = NULL },
>> @@ -2685,7 +2686,7 @@ static struct clk usb_otg_hs_ick = {
>> .enable_reg = OMAP4430_CM_L3INIT_USB_OTG_CLKCTRL,
>> .enable_bit = OMAP4430_MODULEMODE_HWCTRL,
>> .clkdm_name = "l3_init_clkdm",
>> - .parent = &l3_div_ck,
>> + .parent = &otg_60m_gfclk,
>> .recalc = &followparent_recalc,
>> };
>
> Beno?t should have a look at this one, I think.
>
>
> - Paul
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] OMAP4: Clock: Correct OTG clock to use otg_60m_gfclk.
2012-07-02 12:43 ` Benoit Cousson
@ 2012-07-02 16:29 ` Ruslan Bilovol
-1 siblings, 0 replies; 11+ messages in thread
From: Ruslan Bilovol @ 2012-07-02 16:29 UTC (permalink / raw)
To: Benoit Cousson, Wenbiao Wang
Cc: Paul Walmsley, linux-omap, linux-arm-kernel, tony
Thanks Benoit for the explanation. It seems you are right.
+ Wenbiao who is the original patch author that may have another point
of view and some additional explanations.
--
Best regards,
Ruslan Bilovol
On Mon, Jul 2, 2012 at 3:43 PM, Benoit Cousson <b-cousson@ti.com> wrote:
>
> On 06/29/2012 10:35 PM, Paul Walmsley wrote:
>>
>> + Benoît who is the maintainer of this file
>>
>> + the linux-arm-kernel mailing list, which should be cc'ed on all OMAP
>> patches
>>
>> On Fri, 29 Jun 2012, Ruslan Bilovol wrote:
>>
>>> From: Wenbiao Wang <wwang@ti.com>
>>>
>>> OTG clock usb_otg_hs_ick used a incorrect parent l3_div_ck.
>>> Correct it to use the right colck otg_60m_gfclk as its
>>> parent.
>
>
> Mmm, that does not seems to be correct.
>
> otg_60m_gfclk is an optional clock. The interface clock is the main clock
> of that module. That's why this is the parent of the fake MODULEMODE clock
> node.
>
> Moreover you are changing as well the utmi_phy_clkout_ck. That's not
> mentioned at all in the changelog.
>
> I know that there are some non standard stuff in this clock scheme.
> The main reason being the utmi_phy_clkout_ck source is generated from the
> usb_phy module. Unfortunately the clock fmwk cannot handle module as a clock
> node.
>
> So, as of today, this only way to get the OTG_60M_FCLK clock available is
> to ensure that the usb_phy module is enabled before the usb_otg_hs module.
>
> Regards,
> Benoit
>
>>>
>>> Signed-off-by: Wenbiao Wang <wwang@ti.com>
>>> Signed-off-by: Ruslan Bilovol <ruslan.bilovol@ti.com>
>>> ---
>>> arch/arm/mach-omap2/clock44xx_data.c | 15 ++++++++-------
>>> 1 files changed, 8 insertions(+), 7 deletions(-)
>>>
>>> diff --git a/arch/arm/mach-omap2/clock44xx_data.c
>>> b/arch/arm/mach-omap2/clock44xx_data.c
>>> index b825049..fd43214 100644
>>> --- a/arch/arm/mach-omap2/clock44xx_data.c
>>> +++ b/arch/arm/mach-omap2/clock44xx_data.c
>>> @@ -199,12 +199,6 @@ static struct clk tie_low_clock_ck = {
>>> .ops = &clkops_null,
>>> };
>>>
>>> -static struct clk utmi_phy_clkout_ck = {
>>> - .name = "utmi_phy_clkout_ck",
>>> - .rate = 60000000,
>>> - .ops = &clkops_null,
>>> -};
>>> -
>>> static struct clk xclk60mhsp1_ck = {
>>> .name = "xclk60mhsp1_ck",
>>> .rate = 60000000,
>>> @@ -992,6 +986,13 @@ static struct clk dpll_usb_clkdcoldo_ck = {
>>> .recalc = &followparent_recalc,
>>> };
>>>
>>> +static struct clk utmi_phy_clkout_ck = {
>>> + .name = "utmi_phy_clkout_ck",
>>> + .ops = &clkops_null,
>>> + .parent = &dpll_usb_clkdcoldo_ck,
>>> + .recalc = &followparent_recalc,
>>> +};
>>> +
>>> static const struct clksel dpll_usb_m2_div[] = {
>>> { .parent = &dpll_usb_ck, .rates = div31_1to31_rates },
>>> { .parent = NULL },
>>> @@ -2685,7 +2686,7 @@ static struct clk usb_otg_hs_ick = {
>>> .enable_reg = OMAP4430_CM_L3INIT_USB_OTG_CLKCTRL,
>>> .enable_bit = OMAP4430_MODULEMODE_HWCTRL,
>>> .clkdm_name = "l3_init_clkdm",
>>> - .parent = &l3_div_ck,
>>> + .parent = &otg_60m_gfclk,
>>> .recalc = &followparent_recalc,
>>> };
>>
>>
>> Benoît should have a look at this one, I think.
>>
>>
>> - Paul
>>
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH] OMAP4: Clock: Correct OTG clock to use otg_60m_gfclk.
@ 2012-07-02 16:29 ` Ruslan Bilovol
0 siblings, 0 replies; 11+ messages in thread
From: Ruslan Bilovol @ 2012-07-02 16:29 UTC (permalink / raw)
To: linux-arm-kernel
Thanks Benoit for the explanation. It seems you are right.
+ Wenbiao who is the original patch author that may have another point
of view and some additional explanations.
--
Best regards,
Ruslan Bilovol
On Mon, Jul 2, 2012 at 3:43 PM, Benoit Cousson <b-cousson@ti.com> wrote:
>
> On 06/29/2012 10:35 PM, Paul Walmsley wrote:
>>
>> + Beno?t who is the maintainer of this file
>>
>> + the linux-arm-kernel mailing list, which should be cc'ed on all OMAP
>> patches
>>
>> On Fri, 29 Jun 2012, Ruslan Bilovol wrote:
>>
>>> From: Wenbiao Wang <wwang@ti.com>
>>>
>>> OTG clock usb_otg_hs_ick used a incorrect parent l3_div_ck.
>>> Correct it to use the right colck otg_60m_gfclk as its
>>> parent.
>
>
> Mmm, that does not seems to be correct.
>
> otg_60m_gfclk is an optional clock. The interface clock is the main clock
> of that module. That's why this is the parent of the fake MODULEMODE clock
> node.
>
> Moreover you are changing as well the utmi_phy_clkout_ck. That's not
> mentioned at all in the changelog.
>
> I know that there are some non standard stuff in this clock scheme.
> The main reason being the utmi_phy_clkout_ck source is generated from the
> usb_phy module. Unfortunately the clock fmwk cannot handle module as a clock
> node.
>
> So, as of today, this only way to get the OTG_60M_FCLK clock available is
> to ensure that the usb_phy module is enabled before the usb_otg_hs module.
>
> Regards,
> Benoit
>
>>>
>>> Signed-off-by: Wenbiao Wang <wwang@ti.com>
>>> Signed-off-by: Ruslan Bilovol <ruslan.bilovol@ti.com>
>>> ---
>>> arch/arm/mach-omap2/clock44xx_data.c | 15 ++++++++-------
>>> 1 files changed, 8 insertions(+), 7 deletions(-)
>>>
>>> diff --git a/arch/arm/mach-omap2/clock44xx_data.c
>>> b/arch/arm/mach-omap2/clock44xx_data.c
>>> index b825049..fd43214 100644
>>> --- a/arch/arm/mach-omap2/clock44xx_data.c
>>> +++ b/arch/arm/mach-omap2/clock44xx_data.c
>>> @@ -199,12 +199,6 @@ static struct clk tie_low_clock_ck = {
>>> .ops = &clkops_null,
>>> };
>>>
>>> -static struct clk utmi_phy_clkout_ck = {
>>> - .name = "utmi_phy_clkout_ck",
>>> - .rate = 60000000,
>>> - .ops = &clkops_null,
>>> -};
>>> -
>>> static struct clk xclk60mhsp1_ck = {
>>> .name = "xclk60mhsp1_ck",
>>> .rate = 60000000,
>>> @@ -992,6 +986,13 @@ static struct clk dpll_usb_clkdcoldo_ck = {
>>> .recalc = &followparent_recalc,
>>> };
>>>
>>> +static struct clk utmi_phy_clkout_ck = {
>>> + .name = "utmi_phy_clkout_ck",
>>> + .ops = &clkops_null,
>>> + .parent = &dpll_usb_clkdcoldo_ck,
>>> + .recalc = &followparent_recalc,
>>> +};
>>> +
>>> static const struct clksel dpll_usb_m2_div[] = {
>>> { .parent = &dpll_usb_ck, .rates = div31_1to31_rates },
>>> { .parent = NULL },
>>> @@ -2685,7 +2686,7 @@ static struct clk usb_otg_hs_ick = {
>>> .enable_reg = OMAP4430_CM_L3INIT_USB_OTG_CLKCTRL,
>>> .enable_bit = OMAP4430_MODULEMODE_HWCTRL,
>>> .clkdm_name = "l3_init_clkdm",
>>> - .parent = &l3_div_ck,
>>> + .parent = &otg_60m_gfclk,
>>> .recalc = &followparent_recalc,
>>> };
>>
>>
>> Beno?t should have a look at this one, I think.
>>
>>
>> - Paul
>>
>
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] OMAP4: Clock: Correct OTG clock to use otg_60m_gfclk.
2012-07-02 12:43 ` Benoit Cousson
@ 2012-07-04 9:10 ` Paul Walmsley
-1 siblings, 0 replies; 11+ messages in thread
From: Paul Walmsley @ 2012-07-04 9:10 UTC (permalink / raw)
To: Benoit Cousson; +Cc: Ruslan Bilovol, linux-omap, linux-arm-kernel, tony
On Mon, 2 Jul 2012, Benoit Cousson wrote:
> Unfortunately the clock fmwk cannot handle module as a clock node.
Hmm. We have to have similar support in the clockfw for the CLKDIV32K
clock for AM33xx. That uses the modulemode bits to enable and disable the
clock. Or does this require something more complicated?
- Paul
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH] OMAP4: Clock: Correct OTG clock to use otg_60m_gfclk.
@ 2012-07-04 9:10 ` Paul Walmsley
0 siblings, 0 replies; 11+ messages in thread
From: Paul Walmsley @ 2012-07-04 9:10 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, 2 Jul 2012, Benoit Cousson wrote:
> Unfortunately the clock fmwk cannot handle module as a clock node.
Hmm. We have to have similar support in the clockfw for the CLKDIV32K
clock for AM33xx. That uses the modulemode bits to enable and disable the
clock. Or does this require something more complicated?
- Paul
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] OMAP4: Clock: Correct OTG clock to use otg_60m_gfclk.
2012-07-04 9:10 ` Paul Walmsley
@ 2012-07-04 13:09 ` Benoit Cousson
-1 siblings, 0 replies; 11+ messages in thread
From: Benoit Cousson @ 2012-07-04 13:09 UTC (permalink / raw)
To: Paul Walmsley; +Cc: Ruslan Bilovol, linux-omap, linux-arm-kernel, tony
Hi Paul,
On 07/04/2012 11:10 AM, Paul Walmsley wrote:
> On Mon, 2 Jul 2012, Benoit Cousson wrote:
>
>> Unfortunately the clock fmwk cannot handle module as a clock node.
>
> Hmm. We have to have similar support in the clockfw for the CLKDIV32K
> clock for AM33xx. That uses the modulemode bits to enable and disable the
> clock. Or does this require something more complicated?
I don't think so, in that case, that should probably be enough.
But playing with the modulemode in the clock frmwk still looks like a
hack to me.
Ideally it should be represented by a hwmod instead, but then it will
not fit in the clock fmwk.
That being said, using an IP (internal or not) as a source clock should
be supported.
That will allow us to handle the power dependency we have with Phoenix
audio that is the source clock of the McPDM.
Regards,
Benoit
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH] OMAP4: Clock: Correct OTG clock to use otg_60m_gfclk.
@ 2012-07-04 13:09 ` Benoit Cousson
0 siblings, 0 replies; 11+ messages in thread
From: Benoit Cousson @ 2012-07-04 13:09 UTC (permalink / raw)
To: linux-arm-kernel
Hi Paul,
On 07/04/2012 11:10 AM, Paul Walmsley wrote:
> On Mon, 2 Jul 2012, Benoit Cousson wrote:
>
>> Unfortunately the clock fmwk cannot handle module as a clock node.
>
> Hmm. We have to have similar support in the clockfw for the CLKDIV32K
> clock for AM33xx. That uses the modulemode bits to enable and disable the
> clock. Or does this require something more complicated?
I don't think so, in that case, that should probably be enough.
But playing with the modulemode in the clock frmwk still looks like a
hack to me.
Ideally it should be represented by a hwmod instead, but then it will
not fit in the clock fmwk.
That being said, using an IP (internal or not) as a source clock should
be supported.
That will allow us to handle the power dependency we have with Phoenix
audio that is the source clock of the McPDM.
Regards,
Benoit
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2012-07-04 13:09 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-06-29 11:53 [PATCH] OMAP4: Clock: Correct OTG clock to use otg_60m_gfclk Ruslan Bilovol
2012-06-29 20:35 ` Paul Walmsley
2012-06-29 20:35 ` Paul Walmsley
2012-07-02 12:43 ` Benoit Cousson
2012-07-02 12:43 ` Benoit Cousson
2012-07-02 16:29 ` Ruslan Bilovol
2012-07-02 16:29 ` Ruslan Bilovol
2012-07-04 9:10 ` Paul Walmsley
2012-07-04 9:10 ` Paul Walmsley
2012-07-04 13:09 ` Benoit Cousson
2012-07-04 13:09 ` Benoit Cousson
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.