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

Path csiph.com!news.freedyn.net!aioe.org!bofh.it!news.nic.it!robomod
From Pankaj DEV <pankaj.dev@st.com>
Newsgroups linux.kernel
Subject RE: [RFC v1] clk: Add debugfs nodes for enable/disable/set-rate/set-parent
Date Fri, 26 Feb 2016 09:10:01 +0100
Message-ID <r6lDj-eK-1@gated-at.bofh.it> (permalink)
References <r6hgm-5Bc-13@gated-at.bofh.it> <r6l0D-8aG-25@gated-at.bofh.it>
X-Original-To Holger Schurig <holgerschurig@gmail.com>
Thread-Topic [RFC v1] clk: Add debugfs nodes for enable/disable/set-rate/set-parent
Thread-Index AdFwZrfEARnwf2pHR0WY4AQnywFpMwABYcMg
Accept-Language en-US, en-IN
Content-Language en-US
Acceptlanguage en-US, en-IN
Content-Type text/plain; charset="us-ascii"
Content-Transfer-Encoding 8BIT
MIME-Version 1.0
X-Proofpoint-Virus-Version vendor=fsecure engine=2.50.10432:,, definitions=2016-02-26_07:,, signatures=0
Sender robomod@news.nic.it
List-ID <linux-kernel.vger.kernel.org>
X-Mailing-List linux-kernel@vger.kernel.org
Approved robomod@news.nic.it
Lines 27
Organization linux.* mail to news gateway
X-Original-Cc "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "mturquette@linaro.org" <mturquette@linaro.org>, "kernel@stlinux.com" <kernel@stlinux.com>, "sboyd@codeaurora.org" <sboyd@codeaurora.org>, Laurent MEUNIER <laurent.meunier@st.com>, "lee.jones@linaro.org" <lee.jones@linaro.org>, Maxime COQUELIN <maxime.coquelin@st.com>
X-Original-Date Fri, 26 Feb 2016 16:06:48 +0800
X-Original-Message-ID <8B5B4FB66A982549A83D90C7791B761007F362D2E1@EAPEX1MAIL2.st.com>
X-Original-References <1456454309-25037-1-git-send-email-pankaj.dev@st.com> <87fuwg9bog.fsf@gmail.com>
X-Original-Sender linux-kernel-owner@vger.kernel.org
Xref csiph.com linux.kernel:1343888

Show key headers only | 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 | NextPrevious 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