* [PATCH] Revert "media: em28xx: add missing em28xx_close_extension"
[not found] <a0ba3685-8c59-cb89-7f81-280c38a92c40@elbmurf.de>
@ 2022-01-20 19:37 ` Pavel Skripkin
2022-01-21 18:30 ` Maximilian Böhm
0 siblings, 1 reply; 2+ messages in thread
From: Pavel Skripkin @ 2022-01-20 19:37 UTC (permalink / raw)
To: mchehab
Cc: linux-media, linux-kernel, Pavel Skripkin, Hans Verkuil,
Maximilian Böhm
This reverts commit 2c98b8a3458df03abdc6945bbef67ef91d181938.
Reverted patch causes problems with Hauppauge WinTV dualHD as Maximilian
reported [1]. Since quick solution didn't come up let's just revert it
to make this device work with upstream kernels.
Cc: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Link: https://lore.kernel.org/all/6a72a37b-e972-187d-0322-16336e12bdc5@elbmurf.de/ [1]
Reported-by: Maximilian Böhm <maximilian.boehm@elbmurf.de>
Signed-off-by: Pavel Skripkin <paskripkin@gmail.com>
---
drivers/media/usb/em28xx/em28xx-cards.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/media/usb/em28xx/em28xx-cards.c b/drivers/media/usb/em28xx/em28xx-cards.c
index b451ce3cb169..4a46ef50baf9 100644
--- a/drivers/media/usb/em28xx/em28xx-cards.c
+++ b/drivers/media/usb/em28xx/em28xx-cards.c
@@ -4150,11 +4150,8 @@ static void em28xx_usb_disconnect(struct usb_interface *intf)
em28xx_close_extension(dev);
- if (dev->dev_next) {
- em28xx_close_extension(dev->dev_next);
+ if (dev->dev_next)
em28xx_release_resources(dev->dev_next);
- }
-
em28xx_release_resources(dev);
if (dev->dev_next) {
--
2.34.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] Revert "media: em28xx: add missing em28xx_close_extension"
2022-01-20 19:37 ` [PATCH] Revert "media: em28xx: add missing em28xx_close_extension" Pavel Skripkin
@ 2022-01-21 18:30 ` Maximilian Böhm
0 siblings, 0 replies; 2+ messages in thread
From: Maximilian Böhm @ 2022-01-21 18:30 UTC (permalink / raw)
To: Pavel Skripkin, mchehab; +Cc: linux-media, linux-kernel, Hans Verkuil
Tested-by: Maximilian Böhm, kernelml@elbmurf.de
Am 20.01.22 um 20:37 schrieb Pavel Skripkin:
> This reverts commit 2c98b8a3458df03abdc6945bbef67ef91d181938.
>
> Reverted patch causes problems with Hauppauge WinTV dualHD as Maximilian
> reported [1]. Since quick solution didn't come up let's just revert it
> to make this device work with upstream kernels.
>
> Cc: Hans Verkuil <hverkuil-cisco@xs4all.nl>
> Link: https://lore.kernel.org/all/6a72a37b-e972-187d-0322-16336e12bdc5@elbmurf.de/ [1]
> Reported-by: Maximilian Böhm <maximilian.boehm@elbmurf.de>
> Signed-off-by: Pavel Skripkin <paskripkin@gmail.com>
> ---
> drivers/media/usb/em28xx/em28xx-cards.c | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/drivers/media/usb/em28xx/em28xx-cards.c b/drivers/media/usb/em28xx/em28xx-cards.c
> index b451ce3cb169..4a46ef50baf9 100644
> --- a/drivers/media/usb/em28xx/em28xx-cards.c
> +++ b/drivers/media/usb/em28xx/em28xx-cards.c
> @@ -4150,11 +4150,8 @@ static void em28xx_usb_disconnect(struct usb_interface *intf)
>
> em28xx_close_extension(dev);
>
> - if (dev->dev_next) {
> - em28xx_close_extension(dev->dev_next);
> + if (dev->dev_next)
> em28xx_release_resources(dev->dev_next);
> - }
> -
> em28xx_release_resources(dev);
>
> if (dev->dev_next) {
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-01-21 18:30 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <a0ba3685-8c59-cb89-7f81-280c38a92c40@elbmurf.de>
2022-01-20 19:37 ` [PATCH] Revert "media: em28xx: add missing em28xx_close_extension" Pavel Skripkin
2022-01-21 18:30 ` Maximilian Böhm
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).