On 09/05/2016 11:14 AM, Crt Mori wrote: > This patch is inspired by a comment of Jonathan Cameron on patch of > Linus Walleij commit aeb55fff3891834e07a3144159a7298a19696af8 ("iio: > st_sensors: fetch and enable regulators unconditionally"). Because > changes made in this patch are actually reference generators they should > be using devm_regulator_get_optional, but if they do not explicitly set > the reference to NULL they should not be using IS_ERR_OR_NULL, but > simple IS_ERR check. > > Suggested-by: Lars-Peter Clausen <lars@xxxxxxxxxx> > Signed-off-by: Crt Mori <cmo@xxxxxxxxxxx> Looks good, thanks. Acked-by: Lars-Peter Clausen <lars@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html