I see a number of occurrences of code like the following: if (IS_ERR(alg)) return ERR_PTR(PTR_ERR(alg)); Is there any reason why the second line couldn't just be return alg? julia -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html