Enclosed you'll find a link to the agenda, notes and actions from the
Linaro User Platforms Weekly Status meeting dated January 5th held in
#linaro-meeting on irc.freenode.net at 13:00 UTC. This will be the very
last User Platform meeting as the new Multimedia and Graphical teams
will take it's place.
User Platforms Team
"We want great men who, when fortune frowns will not be discouraged."
- Colonel Henry Knox
w) tom.gall att linaro.org
w) tom_gall att vnet.ibm.com
h) tom_gall att mac.com
On Tue, Jan 4, 2011 at 1:18 PM, Yong Shen <yong.shen(a)linaro.org> wrote:
> hi there,
> In last weekly meeting, we talked about the real time display of clock
> information in powerdebug.
What do you mean by real-time display?
powerdebug will only refresh it's information every 'n' seconds. At
that point, the 'show' function for the sysfs rate entry should call
get_rate() for the clock.
> Actually, this feature can be easily supported. if you found some
> clocks' information stay unchanged after your drivers change clock
> setting, it may be caused by the wrong way of clock information
> In the last several weeks, Jeremy and I reviewed the clock debug code
> based on common clock struct. In this code, I used below code to
> expose clock information:
> +static int clk_debug_rate_get(void *data, u64 *val)
> + struct clk *clk = data;
> + *val = (u64)clk_get_rate(clk);
> + return 0;
> +DEFINE_SIMPLE_ATTRIBUTE(clk_debug_rate_fops, clk_debug_rate_get, NULL,
> + "%llu\n");
> + d = debugfs_create_file("rate", S_IRUGO, clk->dentry, (void *)clk,
> + &clk_debug_rate_fops);
> Therefore, whenever the clock information is accessed, it can reflect
> the truth, since it calls clk interface like clk_get_rate() to get the
> right value.
Why is it called clk_debug_rate_get()? Is there not a standard
clk_rate_get() that we can use?
> Previously, some of you might use a variable to store clock
> information like rate or count, and expose these variable to sysfs,
> which is why they are not updated when the real clock information is
Right, storing in variables will give wrong results if you're in the
middle of a rate refresh.
The following changes since commit a581cf741c49c220d159fac1fdbde28255c2e7e3:
Merge branch 'linaro-stable' into linaro-packaged (2011-01-03 14:41:56 -0700)
are available in the git repository at:
John Rigby (2):
debian/rules: remove -Bsymbolic-functions from LDFLAGS
debian/changelog | 17 ++++++++++++-----
debian/rules | 4 ++++
2 files changed, 16 insertions(+), 5 deletions(-)