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:[email protected]] >Sent: Friday, February 26, 2016 12:54 PM >To: Pankaj DEV >Cc: [email protected]; [email protected]; >[email protected]; [email protected]; [email protected]; >[email protected]; Laurent MEUNIER; [email protected]; Maxime >COQUELIN >Subject: Re: [RFC v1] clk: Add debugfs nodes for enable/disable/set-rate/set- >parent > >Pankaj Dev <[email protected]> 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?
