From: "Benoît Monin" <benoit.monin@bootlin.com>
To: Jarkko Nikula <jarkko.nikula@linux.intel.com>,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Mika Westerberg <mika.westerberg@linux.intel.com>,
Jan Dabros <jsd@semihalf.com>,
Andi Shyti <andi.shyti@kernel.org>
Cc: "Thomas Petazzoni" <thomas.petazzoni@bootlin.com>,
"Vladimir Kondratiev" <vladimir.kondratiev@mobileye.com>,
"Tawfik Bayouk" <tawfik.bayouk@mobileye.com>,
linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org,
"Benoît Monin" <benoit.monin@bootlin.com>
Subject: [PATCH v2 0/2] Add dev_err_probe() to i2c designware platform probing
Date: Thu, 04 Sep 2025 16:31:05 +0200 [thread overview]
Message-ID: <20250904-i2c-dw-dev-err-probe-v2-0-e59da34815fa@bootlin.com> (raw)
I recently spend some time debugging a case where the i2c controller
never showed up. In the end it was caused by a missing reset controller
due to a typo in the device tree.
While this has nothing to do with the i2c designware driver, not having
any hint about why the device stays in deferred probe state does not
help.
The first patch convert dev_err() to dev_err_probe() on request IRQ
failure, this simplifies error handling.
The second patch add dev_err_probe() in the error paths that can return
-EPROBE_DEFER in dw_i2c_plat_probe() to aid in debugging deferred probe
errors.
Signed-off-by: Benoît Monin <benoit.monin@bootlin.com>
---
Changes in v2:
- Split the patch in two: one to convert dev_err() to dev_err_probe()
on request IRQ failure and the other adding dev_err_probe() to error
paths in dw_i2c_plat_probe()
- Link to v1: https://lore.kernel.org/r/20250904-i2c-dw-dev-err-probe-v1-1-acca6ffd122e@bootlin.com
---
Benoît Monin (2):
i2c: designware: convert to dev_err_probe() on request IRQ error
i2c: designware: use dev_err_probe() when probing platform device
drivers/i2c/busses/i2c-designware-master.c | 9 ++++-----
drivers/i2c/busses/i2c-designware-platdrv.c | 10 ++++++----
drivers/i2c/busses/i2c-designware-slave.c | 9 ++++-----
3 files changed, 14 insertions(+), 14 deletions(-)
---
base-commit: b320789d6883cc00ac78ce83bccbfe7ed58afcf0
change-id: 20250822-i2c-dw-dev-err-probe-eaf9bce3ef4c
Best regards,
--
Benoît Monin, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
next reply other threads:[~2025-09-04 14:31 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-04 14:31 Benoît Monin [this message]
2025-09-04 14:31 ` [PATCH v2 1/2] i2c: designware: convert to dev_err_probe() on request IRQ error Benoît Monin
2025-09-04 15:02 ` Andy Shevchenko
2025-09-05 8:04 ` Jarkko Nikula
2025-09-04 14:31 ` [PATCH v2 2/2] i2c: designware: use dev_err_probe() when probing platform device Benoît Monin
2025-09-04 15:04 ` Andy Shevchenko
2025-09-05 8:04 ` Jarkko Nikula
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250904-i2c-dw-dev-err-probe-v2-0-e59da34815fa@bootlin.com \
--to=benoit.monin@bootlin.com \
--cc=andi.shyti@kernel.org \
--cc=andriy.shevchenko@linux.intel.com \
--cc=jarkko.nikula@linux.intel.com \
--cc=jsd@semihalf.com \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mika.westerberg@linux.intel.com \
--cc=tawfik.bayouk@mobileye.com \
--cc=thomas.petazzoni@bootlin.com \
--cc=vladimir.kondratiev@mobileye.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).