> - return (ret == 0) ? ret : -ETIMEDOUT ; > + if (ret == 0) > + return ret; > + else > + return -ETIMEDOUT; I actually like more the original version. If you write it like this: return !ret ? ret : -ETIMEDOUT; checkpatch shouldn't complain. Andi _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel