Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1343888

RE: [RFC v1] clk: Add debugfs nodes for enable/disable/set-rate/set-parent

Show all headers | View raw

Hey

The prime motive for clk_set_rate is to set new rate for a clock, since the 'clk_rate' currently available, allows only reading.
To provide reading rate from 'clk_set_rate' is just additional feature.

Regards
Pankaj

>-----Original Message-----
>From: Holger Schurig [mailto:holgerschurig@gmail.com]
>Sent: Friday, February 26, 2016 12:54 PM
>To: Pankaj DEV
>Cc: linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org;
>devicetree@vger.kernel.org; mturquette@linaro.org; kernel@stlinux.com;
>sboyd@codeaurora.org; Laurent MEUNIER; lee.jones@linaro.org; Maxime
>COQUELIN
>Subject: Re: [RFC v1] clk: Add debugfs nodes for enable/disable/set-rate/set-
>parent
>
>Pankaj Dev <pankaj.dev@st.com> writes:
>
>> 1. clk_set_rate : Set new rate to value. Reading returns the current
>> rate
>
>If you can use this to set *and* read it, then "_set_" shouldn't be in the name.
>
>What is wrong with using the existing "clk_rate" for reading/setting the rate?

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[RFC v1] clk: Add debugfs nodes for enable/disable/set-rate/set-parent Pankaj Dev <pankaj.dev@st.com> - 2016-02-26 04:30 +0100
  Re: [RFC v1] clk: Add debugfs nodes for enable/disable/set-rate/set-parent Holger Schurig <holgerschurig@gmail.com> - 2016-02-26 08:30 +0100
    RE: [RFC v1] clk: Add debugfs nodes for  enable/disable/set-rate/set-parent Pankaj DEV <pankaj.dev@st.com> - 2016-02-26 09:10 +0100
      Re: [RFC v1] clk: Add debugfs nodes for enable/disable/set-rate/set-parent Holger Schurig <holgerschurig@gmail.com> - 2016-02-26 09:30 +0100

csiph-web