From: Sanjay Chitroda <sanjayembeddedse@gmail.com>
To: jic23@kernel.org
Cc: dlechner@baylibre.com, nuno.sa@analog.com, andy@kernel.org,
sanjayembeddedse@gmail.com, tglx@kernel.org,
christophe.jaillet@wanadoo.fr, mingo@kernel.org,
nabijaczleweli@nabijaczleweli.xyz, kees@kernel.org,
linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [PATCH v6 6/7] iio: ssp_sensors: Use dev_err_probe
Date: Wed, 15 Apr 2026 10:37:48 +0530 [thread overview]
Message-ID: <20260415050749.3858046-7-sanjayembedded@gmail.com> (raw)
In-Reply-To: <20260415050749.3858046-1-sanjayembedded@gmail.com>
From: Sanjay Chitroda <sanjayembeddedse@gmail.com>
dev_err_probe() makes error code handling simpler and handle
deferred probe nicely (avoid spamming logs).
Signed-off-by: Sanjay Chitroda <sanjayembeddedse@gmail.com>
---
drivers/iio/common/ssp_sensors/ssp_dev.c | 36 ++++++++++--------------
1 file changed, 15 insertions(+), 21 deletions(-)
diff --git a/drivers/iio/common/ssp_sensors/ssp_dev.c b/drivers/iio/common/ssp_sensors/ssp_dev.c
index 94ecc794f926..0a00ead3195d 100644
--- a/drivers/iio/common/ssp_sensors/ssp_dev.c
+++ b/drivers/iio/common/ssp_sensors/ssp_dev.c
@@ -496,25 +496,21 @@ static int ssp_probe(struct spi_device *spi)
struct ssp_data *data;
data = ssp_parse_dt(&spi->dev);
- if (!data) {
- dev_err(&spi->dev, "Failed to find platform data\n");
- return -ENODEV;
- }
+ if (!data)
+ return dev_err_probe(&spi->dev, -ENODEV,
+ "Failed to find platform data\n");
ret = devm_mfd_add_devices(&spi->dev, PLATFORM_DEVID_NONE,
sensorhub_sensor_devs,
- ARRAY_SIZE(sensorhub_sensor_devs), NULL, 0, NULL);
- if (ret < 0) {
- dev_err(&spi->dev, "mfd add devices fail\n");
- return ret;
- }
+ ARRAY_SIZE(sensorhub_sensor_devs),
+ NULL, 0, NULL);
+ if (ret < 0)
+ return dev_err_probe(&spi->dev, ret, "mfd add devices fail\n");
spi->mode = SPI_MODE_1;
ret = spi_setup(spi);
- if (ret < 0) {
- dev_err(&spi->dev, "Failed to setup spi\n");
- return ret;
- }
+ if (ret < 0)
+ return dev_err_probe(&spi->dev, ret, "Failed to setup spi\n");
data->fw_dl_state = SSP_FW_DL_STATE_NONE;
data->spi = spi;
@@ -573,14 +569,12 @@ static int ssp_probe(struct spi_device *spi)
data->fw_dl_state = ssp_check_fwbl(data);
if (data->fw_dl_state == SSP_FW_DL_STATE_NONE) {
ret = ssp_initialize_mcu(data);
- if (ret < 0) {
- dev_err(&spi->dev, "Initialize_mcu failed\n");
- return ret;
- }
- } else {
- dev_err(&spi->dev, "Firmware version not supported\n");
- return -EPERM;
- }
+ if (ret < 0)
+ return dev_err_probe(&spi->dev, ret,
+ "Initialize_mcu failed\n");
+ } else
+ return dev_err_probe(&spi->dev, -EPERM,
+ "Firmware version not supported\n");
return 0;
}
--
2.34.1
next prev parent reply other threads:[~2026-04-15 5:08 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-15 5:07 [PATCH v6 0/7] iio: ssp_sensors: improve resource cleanup Sanjay Chitroda
2026-04-15 5:07 ` [PATCH v6 1/7] iio: ssp_sensors: cleanup codestyle warning Sanjay Chitroda
2026-04-19 17:28 ` Jonathan Cameron
2026-04-20 4:28 ` Sanjay Chitroda
2026-04-15 5:07 ` [PATCH v6 2/7] iio: ssp_sensors: cleanup codestyle check Sanjay Chitroda
2026-04-15 9:14 ` Andy Shevchenko
2026-04-19 17:33 ` Jonathan Cameron
2026-04-20 4:29 ` Sanjay Chitroda
2026-04-15 5:07 ` [PATCH v6 3/7] iio: ssp_sensors: factor out pending list add/remove helper(s) Sanjay Chitroda
2026-04-15 9:22 ` Andy Shevchenko
2026-04-15 9:25 ` Andy Shevchenko
2026-04-15 5:07 ` [PATCH v6 4/7] iio: ssp_sensors: drop duplicated wdt timer and work cleanup Sanjay Chitroda
2026-04-15 9:29 ` Andy Shevchenko
2026-04-19 12:32 ` Sanjay Chitroda
2026-04-15 5:07 ` [PATCH v6 5/7] iio: ssp_sensors: convert probe and teardown to devm-managed resources Sanjay Chitroda
2026-04-15 9:33 ` Andy Shevchenko
2026-04-19 12:33 ` Sanjay Chitroda
2026-04-19 17:39 ` Jonathan Cameron
2026-04-25 11:05 ` Sanjay Chitroda
2026-04-15 5:07 ` Sanjay Chitroda [this message]
2026-04-15 9:39 ` [PATCH v6 6/7] iio: ssp_sensors: Use dev_err_probe Andy Shevchenko
2026-04-19 12:40 ` Sanjay Chitroda
2026-04-15 5:07 ` [PATCH v6 7/7] iio: ssp_sensors: reuse embedded RX buffer for SPI transfers Sanjay Chitroda
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=20260415050749.3858046-7-sanjayembedded@gmail.com \
--to=sanjayembeddedse@gmail.com \
--cc=andy@kernel.org \
--cc=christophe.jaillet@wanadoo.fr \
--cc=dlechner@baylibre.com \
--cc=jic23@kernel.org \
--cc=kees@kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@kernel.org \
--cc=nabijaczleweli@nabijaczleweli.xyz \
--cc=nuno.sa@analog.com \
--cc=tglx@kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.