On 9 August 2013 23:36, Mark Brown <broonie@xxxxxxxxxx> wrote: > From: Mark Brown <broonie@xxxxxxxxxx> > > Since NULL could in theory be a valid regulator we ought to check for > IS_ERR() rather than for NULL. In practice this is unlikely to be an > issue but it's better for neatness. > > Signed-off-by: Mark Brown <broonie@xxxxxxxxxx> > --- > drivers/cpufreq/cpufreq-cpu0.c | 9 ++++----- > 1 file changed, 4 insertions(+), 5 deletions(-) Makes sense. Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html