* [PATCH] platform/chrome: cros_ec_sensorhub: Add missing '\n' in log messages
@ 2020-04-11 14:58 Christophe JAILLET
2020-04-13 14:35 ` Enric Balletbo i Serra
0 siblings, 1 reply; 2+ messages in thread
From: Christophe JAILLET @ 2020-04-11 14:58 UTC (permalink / raw)
To: bleung, enric.balletbo, groeck, andy.shevchenko, Jonathan.Cameron
Cc: gwendal, linux-kernel, kernel-janitors, Christophe JAILLET
Message logged by 'dev_xxx()' or 'pr_xxx()' should end with a '\n'.
Fixes: 145d59baff59 ("platform/chrome: cros_ec_sensorhub: Add FIFO support")
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
---
drivers/platform/chrome/cros_ec_sensorhub_ring.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/platform/chrome/cros_ec_sensorhub_ring.c b/drivers/platform/chrome/cros_ec_sensorhub_ring.c
index 230e6cf3da2f..c51af569708f 100644
--- a/drivers/platform/chrome/cros_ec_sensorhub_ring.c
+++ b/drivers/platform/chrome/cros_ec_sensorhub_ring.c
@@ -820,7 +820,7 @@ static void cros_ec_sensorhub_ring_handler(struct cros_ec_sensorhub *sensorhub)
if (fifo_info->count > sensorhub->fifo_size ||
fifo_info->size != sensorhub->fifo_size) {
dev_warn(sensorhub->dev,
- "Mismatch EC data: count %d, size %d - expected %d",
+ "Mismatch EC data: count %d, size %d - expected %d\n",
fifo_info->count, fifo_info->size,
sensorhub->fifo_size);
goto error;
@@ -851,14 +851,14 @@ static void cros_ec_sensorhub_ring_handler(struct cros_ec_sensorhub *sensorhub)
}
if (number_data > fifo_info->count - i) {
dev_warn(sensorhub->dev,
- "Invalid EC data: too many entry received: %d, expected %d",
+ "Invalid EC data: too many entry received: %d, expected %d\n",
number_data, fifo_info->count - i);
break;
}
if (out + number_data >
sensorhub->ring + fifo_info->count) {
dev_warn(sensorhub->dev,
- "Too many samples: %d (%zd data) to %d entries for expected %d entries",
+ "Too many samples: %d (%zd data) to %d entries for expected %d entries\n",
i, out - sensorhub->ring, i + number_data,
fifo_info->count);
break;
--
2.20.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] platform/chrome: cros_ec_sensorhub: Add missing '\n' in log messages
2020-04-11 14:58 [PATCH] platform/chrome: cros_ec_sensorhub: Add missing '\n' in log messages Christophe JAILLET
@ 2020-04-13 14:35 ` Enric Balletbo i Serra
0 siblings, 0 replies; 2+ messages in thread
From: Enric Balletbo i Serra @ 2020-04-13 14:35 UTC (permalink / raw)
To: Christophe JAILLET, bleung, groeck, andy.shevchenko, Jonathan.Cameron
Cc: gwendal, linux-kernel, kernel-janitors
Hi Christophe,
Thank you for your patch.
On 11/4/20 16:58, Christophe JAILLET wrote:
> Message logged by 'dev_xxx()' or 'pr_xxx()' should end with a '\n'.
>
> Fixes: 145d59baff59 ("platform/chrome: cros_ec_sensorhub: Add FIFO support")
> Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
> ---
Applied as a fix for 5.7
> drivers/platform/chrome/cros_ec_sensorhub_ring.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/platform/chrome/cros_ec_sensorhub_ring.c b/drivers/platform/chrome/cros_ec_sensorhub_ring.c
> index 230e6cf3da2f..c51af569708f 100644
> --- a/drivers/platform/chrome/cros_ec_sensorhub_ring.c
> +++ b/drivers/platform/chrome/cros_ec_sensorhub_ring.c
> @@ -820,7 +820,7 @@ static void cros_ec_sensorhub_ring_handler(struct cros_ec_sensorhub *sensorhub)
> if (fifo_info->count > sensorhub->fifo_size ||
> fifo_info->size != sensorhub->fifo_size) {
> dev_warn(sensorhub->dev,
> - "Mismatch EC data: count %d, size %d - expected %d",
> + "Mismatch EC data: count %d, size %d - expected %d\n",
> fifo_info->count, fifo_info->size,
> sensorhub->fifo_size);
> goto error;
> @@ -851,14 +851,14 @@ static void cros_ec_sensorhub_ring_handler(struct cros_ec_sensorhub *sensorhub)
> }
> if (number_data > fifo_info->count - i) {
> dev_warn(sensorhub->dev,
> - "Invalid EC data: too many entry received: %d, expected %d",
> + "Invalid EC data: too many entry received: %d, expected %d\n",
> number_data, fifo_info->count - i);
> break;
> }
> if (out + number_data >
> sensorhub->ring + fifo_info->count) {
> dev_warn(sensorhub->dev,
> - "Too many samples: %d (%zd data) to %d entries for expected %d entries",
> + "Too many samples: %d (%zd data) to %d entries for expected %d entries\n",
> i, out - sensorhub->ring, i + number_data,
> fifo_info->count);
> break;
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-04-13 14:35 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-11 14:58 [PATCH] platform/chrome: cros_ec_sensorhub: Add missing '\n' in log messages Christophe JAILLET
2020-04-13 14:35 ` Enric Balletbo i Serra
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).