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@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 >COQUELIN >Subject: Re: [RFC v1] clk: Add debugfs nodes for enable/disable/set-rate/set- >parent > >Pankaj Dev <pankaj.dev@xxxxxx> 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? -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html