Hi Girish, On Fri, Sep 23 2011, Girish K S wrote: >> I suggested replacing your BUG(); here with a return, but you just have >> a break here. Is having the function continue in this case intentional? > > sorry for that. > will update with a return -EBADMSG Thanks; let's go with -EINVAL, though. - Chris. -- Chris Ball <cjb@xxxxxxxxxx> <http://printf.net/> One Laptop Per Child -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html