Add common interface for clock debug information
Yong Shen
yong.shen at linaro.org
Wed Nov 10 01:21:42 UTC 2010
>
>
> >
> > #define INIT_CLK(name, o) {
> \
> > .ops = &o,
> \
> > .enable_count = 0,
> \
> > .lock.mutex = __MUTEX_INITIALIZER(name.lock.mutex), \
> > .name = #name,
> \
> > }
> >
> > I think that allocating a dentry per clk_lookup is quite heavyweight; we
> could
> > just expose one debugfs file, with the whole set of clocks available
> through
> > this file, one clock per line. The seq_file interface makes this fairly
> > straightforward to do.
> >
>
> Just a few cents, wouldn't it be good as well to add the clk attribute
> to each device
> where clock is associated? It's a bit tricky though as a device could
> have multiple
> clocks in theory.
> As to the name, I agree with Jeremy that we should have a name field.
>
I think the 'struct clock' on most platform has a name field already.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.linaro.org/pipermail/linaro-dev/attachments/20101110/b92a061f/attachment.html>
More information about the linaro-dev
mailing list