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

From: Pankaj DEV
Date: Fri Feb 26 2016 - 03:07:31 EST


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.


>-----Original Message-----
>From: Holger Schurig [mailto:holgerschurig@xxxxxxxxx]
>Sent: Friday, February 26, 2016 12:54 PM
>To: Pankaj DEV
>Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
>devicetree@xxxxxxxxxxxxxxx; mturquette@xxxxxxxxxx; kernel@xxxxxxxxxxx;
>sboyd@xxxxxxxxxxxxxx; Laurent MEUNIER; lee.jones@xxxxxxxxxx; Maxime
>Subject: Re: [RFC v1] clk: Add debugfs nodes for enable/disable/set-rate/set-
>Pankaj Dev <> 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?