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