From: Stephen Boyd <swboyd@xxxxxxxxxxxx> Printks in the kernel have newlines at the end. Add them to the few printks in this driver. Reviewed-by: Daniel Campello <campello@xxxxxxxxxxxx> Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx> Fixes: 72ad02b15d63 ("iio: Add SEMTECH SX9310/9311 sensor driver") Signed-off-by: Stephen Boyd <swboyd@xxxxxxxxxxxx> Signed-off-by: Daniel Campello <campello@xxxxxxxxxxxx> --- drivers/iio/proximity/sx9310.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/iio/proximity/sx9310.c b/drivers/iio/proximity/sx9310.c index 3f981d28ee4056..4553ee83a016a3 100644 --- a/drivers/iio/proximity/sx9310.c +++ b/drivers/iio/proximity/sx9310.c @@ -809,7 +809,7 @@ static int sx9310_init_compensation(struct iio_dev *indio_dev) if (ret) { if (ret == -ETIMEDOUT) dev_err(&data->client->dev, - "0x02 << 3l compensation timed out: 0x%02x", + "0x02 << 3l compensation timed out: 0x%02x\n", val); return ret; } @@ -855,7 +855,7 @@ static int sx9310_set_indio_dev_name(struct device *dev, ddata = (uintptr_t)device_get_match_data(dev); if (ddata != whoami) { - dev_err(dev, "WHOAMI does not match device data: %u", whoami); + dev_err(dev, "WHOAMI does not match device data: %u\n", whoami); return -ENODEV; } @@ -867,7 +867,7 @@ static int sx9310_set_indio_dev_name(struct device *dev, indio_dev->name = "sx9311"; break; default: - dev_err(dev, "unexpected WHOAMI response: %u", whoami); + dev_err(dev, "unexpected WHOAMI response: %u\n", whoami); return -ENODEV; } @@ -896,7 +896,7 @@ static int sx9310_probe(struct i2c_client *client) ret = regmap_read(data->regmap, SX9310_REG_WHOAMI, &data->whoami); if (ret) { - dev_err(dev, "error in reading WHOAMI register: %d", ret); + dev_err(dev, "error in reading WHOAMI register: %d\n", ret); return ret; } -- 2.28.0.rc0.142.g3c755180ce-goog