Hi Tomas, > use strict_strtoul as suggested by checkpatch.pl > > Signed-off-by: Tomas Winkler <tomas.winkler@xxxxxxxxx> > --- > V2: > 1. more verbose commit message > 2. return the error code that was produced by strict_strtoul why do you bother actually. Reading the comment about struct_strtoul it will only return -EINVAL or 0. So using my proposal would be just fine. I also don't prefer to differ the return value to user space until it actually makes sense. Invalid argument is just fine for all error cases. Regards Marcel -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html