[RFC] [PATCH] arm & sh: factorised duplicated clkdev.c

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Wed Sep 1 07:18:46 EDT 2010


> >  
> > -struct clk_lookup *clkdev_alloc(struct clk *clk, const char *con_id,
> > -	const char *dev_fmt, ...)
> > +struct clk_lookup * __init_refok
> Why do you need __init_refok?  __init_refok is defined as follows in
> include/linux/init.h:
> 
> 	/* compatibility defines */
> 	#define __init_refok     __ref
> 
> So if really needed better use __ref.  Couldn't this be __init?  
ok for __ref but __init_refok is more clear IMHO
and yes we need it as for early device we may use bootmem as slab allocators
may no be availlable in some case

Best Regards,
J.



More information about the linux-arm-kernel mailing list