On Friday, May 20, 2011 03:27:49 Jeremy Kerr wrote: > int clk_set_rate(struct clk *clk, unsigned long rate) > { > - /* not yet implemented */ > - return -ENOSYS; > + unsigned long parent_rate, new_rate; > + int ret; > + > + if (!clk->ops->set_rate) > + return -ENOSYS; i thought ENOSYS is only for syscalls. shouldnt this be ENODEV or perhaps EOPNOTSUPP ? -mike