diff options
| author | Malathi A <malathi.a2000@gmail.com> | 2026-09-18 07:45:46 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2026-09-23 14:49:42 +0200 |
| commit | e04afb34a1f35b494fc8ca6b3b81832bbb6b794f (patch) | |
| tree | ae36aae331d1a721ca1fd43242666ccfc98649b9 /tools/perf/scripts/python | |
| parent | 9b0bc6f8b31432f466915119b5e9c2baef2552f8 (diff) | |
| download | linux-next-e04afb34a1f35b494fc8ca6b3b81832bbb6b794f.tar.gz linux-next-e04afb34a1f35b494fc8ca6b3b81832bbb6b794f.zip | |
serial: 8250_uniphier: Use dev_err_probe() in probe error paths
devm_clk_get_enabled() returns -EPROBE_DEFER while the clock provider is
not registered yet, and uniphier_uart_probe() reports that with
dev_err(). Every retry of an intermittently deferred probe therefore
prints "failed to get and enable clock" at error level, which is only
dmesg noise.
Use dev_err_probe() instead: it demotes -EPROBE_DEFER to dev_dbg() and
records the reason for /sys/kernel/debug/devices_deferred. Convert the
other two error messages in probe as well, so the whole function follows
the "return dev_err_probe()" pattern already used by other 8250 drivers
such as 8250_em.c. Those two codes are never -EPROBE_DEFER, so there the
conversion is a readability change only.
Errors other than a deferral are still logged at error level, now with
the error code added by the helper. No functional change otherwise.
Suggested-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Malathi A <malathi.a2000@gmail.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://patch.msgid.link/20260918074546.245342-1-malathi.a2000@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'tools/perf/scripts/python')
0 files changed, 0 insertions, 0 deletions
