Skip to contentSkip to conversation

omap3 clock framework for clock set_rate

Question Wang Sawsd-A24013 2 participants 3 messages Jun 18, 2009 – Jun 19, 2009 1 message not in the archive

lore
Share & tools
lore

A message that is not in the archive (sent off-list or not yet received): [email protected]

#1 lore raw

Paul,

In linux-omap clock framework, most clocks have no set_rate in their definition in clock34xx.h, but defined DEFAULT_RATE in clksel_rate structures, this Will only allow some pre-set rates for some clocks and Can not get a more flexible rates if driver wants to Change the clock rate usng clk_set_rate.

What is the benefit of this implementation?

We want to understand this before we made the change In case we missed some imprtant background of This implementation.

Thanks,
Chunqiu

#2 lore raw

Hello Chunqiu,

Quoted 7 lines · On Fri, 19 Jun 2009, Wang Sawsd-A24013 wrote:

In linux-omap clock framework, most clocks have no set_rate in their definition in clock34xx.h, but defined DEFAULT_RATE in clksel_rate structures, this Will only allow some pre-set rates for some clocks and Can not get a more flexible rates if driver wants to Change the clock rate usng clk_set_rate.

Quoted 5 lines

What is the benefit of this implementation?

We want to understand this before we made the change In case we missed some imprtant background of This implementation.

Any driver that wishes to change its clock rate should add .set_rate and .round_rate function pointers to its corresponding clocks. There's no good reason why those function pointers aren't there; they probably should have been added with the original clock tree.

Hope this helps,

- Paul

#3 lore raw
Quoted 26 lines

-----Original Message-----
From: Paul Walmsley [mailto:[email protected]]
Sent: 2009年6月19日 8:18
To: Wang Sawsd-A24013
Cc: [email protected]
Subject: Re: [Question] omap3 clock framework for clock set_rate

Hello Chunqiu,

Quoted 7 lines · On Fri, 19 Jun 2009, Wang Sawsd-A24013 wrote:

In linux-omap clock framework, most clocks have no set_rate in their definition in clock34xx.h, but defined DEFAULT_RATE in clksel_rate structures, this Will only allow some pre-set rates for some clocks and Can not get a more flexible rates if driver wants to Change the clock rate usng clk_set_rate.

Quoted 5 lines

What is the benefit of this implementation?

We want to understand this before we made the change In case we missed some imprtant background of This implementation.

Quoted 8 lines

Any driver that wishes to change its clock rate should add .set_rate and
.round_rate function pointers to its corresponding clocks. There's no good reason why those function pointers aren't there; they probably
should have been added with the original clock tree.

Hope this helps,

Thanks, this helps a lot. :-)

Quoted 3 lines

- Paul

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