2021-07-26 10:44:00

by Krzysztof Hałasa

[permalink] [raw]
Subject: [PATCH] Fix cosmetic error in TDA1997x driver

The colon isn't followed by anything here.

Signed-off-by: Krzysztof Hałasa <[email protected]>

diff --git a/drivers/media/i2c/tda1997x.c b/drivers/media/i2c/tda1997x.c
index 91e6db847bb5..e55e4afaae8a 100644
--- a/drivers/media/i2c/tda1997x.c
+++ b/drivers/media/i2c/tda1997x.c
@@ -563,7 +563,7 @@ static void tda1997x_delayed_work_enable_hpd(struct work_struct *work)
delayed_work_enable_hpd);
struct v4l2_subdev *sd = &state->sd;

- v4l2_dbg(2, debug, sd, "%s:\n", __func__);
+ v4l2_dbg(2, debug, sd, "%s\n", __func__);

/* Set HPD high */
tda1997x_manual_hpd(sd, HPD_HIGH_OTHER);


2021-08-19 16:49:05

by Tim Harvey

[permalink] [raw]
Subject: Re: [PATCH] Fix cosmetic error in TDA1997x driver

On Mon, Jul 26, 2021 at 3:42 AM Krzysztof Hałasa <[email protected]> wrote:
>
> The colon isn't followed by anything here.
>
> Signed-off-by: Krzysztof Hałasa <[email protected]>
>
> diff --git a/drivers/media/i2c/tda1997x.c b/drivers/media/i2c/tda1997x.c
> index 91e6db847bb5..e55e4afaae8a 100644
> --- a/drivers/media/i2c/tda1997x.c
> +++ b/drivers/media/i2c/tda1997x.c
> @@ -563,7 +563,7 @@ static void tda1997x_delayed_work_enable_hpd(struct work_struct *work)
> delayed_work_enable_hpd);
> struct v4l2_subdev *sd = &state->sd;
>
> - v4l2_dbg(2, debug, sd, "%s:\n", __func__);
> + v4l2_dbg(2, debug, sd, "%s\n", __func__);
>
> /* Set HPD high */
> tda1997x_manual_hpd(sd, HPD_HIGH_OTHER);

Acked-By: Tim Harvey <[email protected]>