Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1621283

Re: [PATCH] clk: add clk_possible_parents debugfs file

From Peter De Schrijver <pdeschrijver@nvidia.com>
Newsgroups linux.kernel
Subject Re: [PATCH] clk: add clk_possible_parents debugfs file
Date 2017-04-11 15:50 +0200
Message-ID <tv4lc-7FU-19@gated-at.bofh.it> (permalink)
References <tns1j-3SO-5@gated-at.bofh.it> <ttI3o-3uy-23@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Fri, Apr 07, 2017 at 12:45:06PM -0700, Stephen Boyd wrote:
> On 03/21, Peter De Schrijver wrote:
> > diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c
> > index f5f2bcd..2fa2fb8 100644
> > --- a/drivers/clk/clk.c
> > +++ b/drivers/clk/clk.c
> > @@ -2128,6 +2128,34 @@ static int clk_dump_open(struct inode *inode, struct file *file)
> >  	.release	= single_release,
> >  };
> >  
> > +static int possible_parents_dump(struct seq_file *s, void *data)
> > +{
> > +	struct clk_core *core;
> > +	int i;
> > +
> > +	core = (struct clk_core *)s->private;
> 
> Useless cast from void.
> 
> > +
> > +	for (i = 0; i < core->num_parents - 1; i++)
> > +		seq_printf(s, "%s ", core->parent_names[i]);
> 
> I hope we don't have spaces in clk names! I don't think we do.
> 

I don't think we do either, at least I've never seen a clk name with a space.

Peter.

Back to linux.kernel | Previous | Next | Find similar | Unroll thread


Thread

Re: [PATCH] clk: add clk_possible_parents debugfs file Peter De Schrijver <pdeschrijver@nvidia.com> - 2017-04-11 15:50 +0200

csiph-web