linux-kernel - Re: [PATCH 2/4 V2] Staging: media: atomisp: i2c: removed redundand debug message

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <6213cdfb-4638-4ff5-adc2-71fdd78b2604@kernel.org>
Date: Sun, 6 Jul 2025 17:01:01 +0200
From: Hans de Goede <hansg@...nel.org>
To: Thomas Andreatta <thomasandreatta2000@...il.com>
Cc: linux-kernel@...r.kernel.org, linux-staging@...ts.linux.dev,
 Thomas Andreatta <thomas.andreatta2000@...il.com>
Subject: Re: [PATCH 2/4 V2] Staging: media: atomisp: i2c: removed redundand
 debug message

Hi,

On 19-Jun-25 10:44 AM, Thomas Andreatta wrote:
> Checkpatch fix: deleted `dev_dbg()` printing the name of the function.
>                 ftrace can be used instead.
> 
> Signed-off-by: Thomas Andreatta <thomas.andreatta2000@...il.com>

Thank you for your patch.

I have merged this in my media-atomisp branch:
https://git.kernel.org/pub/scm/linux/kernel/git/hansg/linux.git/log/?h=media-atomisp

And this patch will be included in my next
pull-request to Mauro (to media subsystem maintainer)

Regards,

Hans




> ---
>  drivers/staging/media/atomisp/i2c/atomisp-gc2235.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/staging/media/atomisp/i2c/atomisp-gc2235.c b/drivers/staging/media/atomisp/i2c/atomisp-gc2235.c
> index 857d7175942c..6fc39ab95e46 100644
> --- a/drivers/staging/media/atomisp/i2c/atomisp-gc2235.c
> +++ b/drivers/staging/media/atomisp/i2c/atomisp-gc2235.c
> @@ -779,8 +779,6 @@ static void gc2235_remove(struct i2c_client *client)
>  	struct v4l2_subdev *sd = i2c_get_clientdata(client);
>  	struct gc2235_device *dev = to_gc2235_sensor(sd);
>  
> -	dev_dbg(&client->dev, "gc2235_remove...\n");
> -
>  	dev->platform_data->csi_cfg(sd, 0);
>  
>  	v4l2_device_unregister_subdev(sd);