On Monday, August 12, 2013 10:49:46 AM Viresh Kumar wrote: > 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> Queued up for 3.12. Thanks, Rafael -- 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