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

From	Holger Schurig <holgerschurig@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [RFC v1] clk: Add debugfs nodes for enable/disable/set-rate/set-parent
Date	2016-02-26 09:30 +0100
Message-ID	<r6lWH-lR-15@gated-at.bofh.it> (permalink)
References	<r6hgm-5Bc-13@gated-at.bofh.it> <r6l0D-8aG-25@gated-at.bofh.it> <r6lDj-eK-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Pankaj DEV <pankaj.dev@st.com> writes:

> 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.

Sure.

But my point is to combine things.


After your patch:

- read via clk_rate
- read via clk_set_rate
- write via clk_set_rate

My proposal:

- read via clk_rate
- write via clk_rate

Back to linux.kernel | Previous | Next — Previous 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