[PATCH 3/3] regulator: 88pm800: Add dev_info to show probe success status
Vaibhav Hiremath
vaibhav.hiremath at linaro.org
Fri Jul 10 05:11:11 PDT 2015
On Friday 10 July 2015 12:35 AM, Mark Brown wrote:
> On Thu, Jul 09, 2015 at 06:11:32PM +0530, Vaibhav Hiremath wrote:
>> 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.
>
>> + dev_info(&pdev->dev, "Regulator registered ...\n");
>> return 0;
>
> No, this is pure noise - all you're doing is slowing down boot for users
> with no content. Think about what would happen if every driver did
> this. The core already displays the constraints when things are
> registered.
>
I was just trying my luck here :)
And honestly, this was useful for me atleast.
Thanks,
Vaibhav
More information about the linux-arm-kernel
mailing list