[PATCH 3/3] regulator: 88pm800: Add dev_info to show probe success status
Vaibhav Hiremath
vaibhav.hiremath at linaro.org
Fri Jul 10 05:12:17 PDT 2015
On Friday 10 July 2015 01:06 AM, Valdis.Kletnieks at vt.edu 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
More information about the linux-arm-kernel
mailing list