Re: [PATCH 1/1] regulator: pwm: suppress EPROBE_DEFER error message

From: Ben Dooks
Date: Wed Jan 23 2019 - 11:14:59 EST




On 2019-01-23 16:04, Mark Brown wrote:
On Mon, Jan 21, 2019 at 07:37:23PM +0100, Martin Blumenstingl wrote:
Suppress the "Failed to get PWM" error output if the actual error code
is EPROBE_DEFER. This makes the behavior of the pwm-regulator driver
consistent with what most other drivers do (which is: print all errors
except EPROBE_DEFER).

An example where this cleans up the kernel log are the 32-bit Amlogic
Meson boards:
multi_v7_defconfig has CONFIG_REGULATOR_PWM=y and CONFIG_PWM_MESON=m.

This also cleans up the kernel log in the case where you've not got a
driver enabled that you need (or it's not loading for some reason) which
isn't super helpful when you're trying to figure out why the driver
won't probe. There's not even anything at debug level, that would
probably be fine.

The ideal thing here would be to work on setting up the dependency
information based on DT and using that to try to sort initialization
order so we try things in an order that minimizes the number of failed
tries.

Hmm, would it be useful to have a devfs file which holds the last probe code?