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

From: Holger Schurig
Date: Fri Feb 26 2016 - 02:24:23 EST


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?