Re: [PATCH 1/7] media: atomisp: Fix missing v4l2_fh_release() in atomisp_open() error exit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Hans de Goede <hdegoede@xxxxxxxxxx>
Subject: Re: [PATCH 1/7] media: atomisp: Fix missing v4l2_fh_release() in atomisp_open() error exit
From: Andy Shevchenko <andy@xxxxxxxxxx>
Date: Mon, 19 Jun 2023 14:12:41 +0300
Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>, Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>, Tsuchiya Yuto <kitakar@xxxxxxxxx>, Yury Luneff <yury.lunev@xxxxxxxxx>, Nable <nable.maininbox@xxxxxxxxxxxxxx>, andrey.i.trufanov@xxxxxxxxx, Fabio Aiuto <fabioaiuto83@xxxxxxxxx>, Kate Hsuan <hpa@xxxxxxxxxx>, linux-media@xxxxxxxxxxxxxxx, linux-staging@xxxxxxxxxxxxxxx, kernel test robot <lkp@xxxxxxxxx>, Dan Carpenter <dan.carpenter@xxxxxxxxxx>
In-reply-to: <20230619105212.303653-1-hdegoede@redhat.com>
Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
References: <20230619105212.303653-1-hdegoede@redhat.com>

On Mon, Jun 19, 2023 at 12:52:06PM +0200, Hans de Goede wrote:
> Fix missing v4l2_fh_release() in atomisp_open()'s
> "if (pipe->users)" error exit path.

All LGTM,
Reviewed-by: Andy Shevchenko <andy@xxxxxxxxxx>

-- 
With Best Regards,
Andy Shevchenko

Follow-Ups:
- Re: [PATCH 1/7] media: atomisp: Fix missing v4l2_fh_release() in atomisp_open() error exit
  - From: Hans de Goede

References:
- [PATCH 1/7] media: atomisp: Fix missing v4l2_fh_release() in atomisp_open() error exit
  - From: Hans de Goede

Prev by Date: Re: [PATCH 6/7] media: atomisp: Remove support for custom run-mode v4l2-ctrl on sensors
Next by Date: Re: [PATCH v3 0/4] Trivial code cleanup patches
Previous by thread: [PATCH 7/7] media: atomisp: Remove v4l2_ctrl_s_ctrl(asd->run_mode) calls from atomisp_open()
Next by thread: Re: [PATCH 1/7] media: atomisp: Fix missing v4l2_fh_release() in atomisp_open() error exit
Index(es):
- Date
- Thread

[Index of Archives] [Linux Driver Development] [Linux Driver Backports] [DMA Engine] [Linux GPIO] [Linux SPI] [Video for Linux] [Linux USB Devel] [Linux Coverity] [Linux Audio Users] [Linux Kernel] [Linux SCSI] [Yosemite Backpacking]