[PATCH 2/7 v5] Add the clock framework for Telechips TCC8xxx processors.

Hans J. Koch hjk at linutronix.de
Wed May 26 13:31:11 EDT 2010


On Thu, Apr 22, 2010 at 12:19:17PM +0100, Russell King - ARM Linux wrote:
> On Wed, Apr 21, 2010 at 02:42:01PM +0200, Hans J. Koch wrote:
> > +static int __set_parent(struct clk *clock, struct clk *parent)
> > +{
> > +	if (clock->parent && (clock->parent != parent)) {
> > +		clock->parent->refcount--;
> > +		if ((clock->parent->refcount == 0) && clock->parent->disable)
> > +			clock->parent->disable(clock->parent);
> > +	}
> > +
> > +	clock->parent = parent;
> > +	if (clock->parent)
> > +		clock->parent->refcount++;
> 
> Is this left over from the old clk_set_parent() implementation - with
> the handling of refcounting in the main clk_set_parent(), this refcounting
> seems unnecessary (and seems buggy.)

Right. All refcounting is already done in plat-tcc/clock.c, so this
__set_parent() became obsolete. I removed it completely, because the initial
parent setting is already done elsewhere.

Thanks,
Hans





More information about the linux-arm-kernel mailing list