Re: [PATCH 3/3] regulator: 88pm800: Add dev_info to show probe success status

[Vaibhav Hiremath]

On Friday 10 July 2015 01:06 AM, Valdis.Kletnieks@xxxxxx wrote:
> On Thu, 09 Jul 2015 18:11:32 +0530, Vaibhav Hiremath said:
> > This patch adds dev_info line at the end of probe function, to
> > clearly put status of regulator probe on console. Useful during
> > development, specially to check bootlog.
>
> I can see that as a development thing...
>
> > +	dev_info(&pdev->dev, "Regulator registered ...\n");
>
> In which case this should be 'dev_debug()' or similar so it doesn't
> clutter up and slow down a production boot.

Yeup,

Probably dev_debug would have been better here.

Thanks,
Vaibhav
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/