Hi Sakari, Thank you for the patch. On Thu, Dec 21, 2023 at 03:58:22PM +0200, Sakari Ailus wrote: > Drop a debug print in media_release(), which is a release callback for a > file handle. Printing a debug message here is simply not necessary. > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx> > --- > drivers/media/mc/mc-devnode.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/media/mc/mc-devnode.c b/drivers/media/mc/mc-devnode.c > index 680fbb3a9340..9c8fe9335dc1 100644 > --- a/drivers/media/mc/mc-devnode.c > +++ b/drivers/media/mc/mc-devnode.c > @@ -190,7 +190,6 @@ static int media_release(struct inode *inode, struct file *filp) > return value is ignored. */ > put_device(&devnode->dev); > > - pr_debug("%s: Media Release\n", __func__); > return 0; > } > -- Regards, Laurent Pinchart