| Message ID | 20231221135822.152448-4-sakari.ailus@linux.intel.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Miscellaneous small things | expand |
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@linux.intel.com> Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > --- > 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; > } >
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; }
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@linux.intel.com> --- drivers/media/mc/mc-devnode.c | 1 - 1 file changed, 1 deletion(-)