linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Hans Verkuil <hverkuil-cisco@xs4all.nl>
To: dac6a072-39f4-96c5-cb13-171c54213937@xs4all.nl,
	Corinna Vinschen <vinschen@redhat.com>,
	linux-media@vger.kernel.org
Cc: Rudy Zijlstra <rudy@grumpydevil.homelinux.org>
Subject: Re: [PATCHv2 00/16] staging/media: remove most deprecated drivers
Date: Tue, 17 Jan 2023 16:48:52 +0100	[thread overview]
Message-ID: <c75b0172-fc2c-90ab-8138-8f916d20be63@xs4all.nl> (raw)
In-Reply-To: <7bc7316b-6d4a-cac0-bc70-e8b4d2b4b1fd@gmail.com>

On 17/01/2023 16:38, Tomasz Maciej Nowak wrote:
> Hi.
> 
> W dniu 16.01.2023 o 15:39, Hans Verkuil pisze:
>> On 16/01/2023 14:35, Corinna Vinschen wrote:
>>> Hi Hans,
>>>
>>> On Jan 16 11:08, Hans Verkuil wrote:
>>>> Hi Corinna,
>>>>
>>>> On 16/01/2023 10:53, Corinna Vinschen wrote:
>>>>> Hi Hans,
>>>>>
>>>>> I only learned about this yesterday, so I hope I'm not too late.
>>>>>
>>>>> Please don't do that.  You're about to remove working drivers used by a
>>>>> lot of people.
>>>>>
>>>>> If you remove them from the Linux kernel, you will leave *lots* of DVB-C
>>>>> and DVB-S card users behind.  They will update their TV recording
>>>>> machines to a newer system at one point and suddenly their ability to
>>>>> record from TV is gone forever.
>>>>>
>>>>> I'm personally affected by this as well.  We're using a machine with
>>>>> four Technotrend S2-3200 Budget DVB-S2 cards for TV recordings using
>>>>> the VDR package.  This card is apparently handled by the code under
>>>>> drivers/staging/media/deprecated/saa7146.
>>>>>
>>>>> If this code goes away, we will have to keep the machine running on
>>>>> an old kernel for a long time.
>>>>>
>>>>> I'm fortunate that I even learned about this developement, being a
>>>>> developer myself, but how's a normal user to know that a Linux driver
>>>>> they are using every day is about to be removed from the kernel?  Again,
>>>>> please don't break the equipment of us users of these DVB-C and DVB-S
>>>>> cards for the future.  
>>>>
>>>> I've dropped the PR for now.
>>>
>>> I'm really glad to read that.
> 
> Thanks from me as well.
> 
>>>> Is the concern specifically for the saa7146
>>>> based hardware?
>>>>
>>>> I.e., from Red Hat's point of view, are there any concerns about removing
>>>> vpfe_capture, tm6000, zr364xx, stkwebcam, fsl-viu, cpia2 and meye?
>>>
>>> I'm not part of the Red Hat media team, so I can't answer that, sorry.  
>>>
>>> The problem is that there are no usage numbers anywhere.  I don't know
>>> which of these drivers are still in use and which aren't.  I only know
>>> for certain for our saa7146 based cards @home.  Terratec was the market
>>> leader for quite some time, so there are lots of Terratec cards and
>>> budget cards from third party vendors in the wild.
> 
> I've got six of saa7146 based DVB-S2 budget cards still in use.
> Apparently You can still buy a new one[1] which is supported by budget
> driver.
> 
>>>> The core problem with saa7146 (and the other deprecated drivers) is that
>>>> it is using the old videobuf framework, which has known problems and we
>>>> want (need!) to get rid of it, either by dropping drivers or converting
>>>> them.
>>>>
>>>> One partial solution would be to drop analog video support from saa7146,
>>>> since that's the bit that uses this framework. DVB would remain working,
>>>> but analog video support would die, unless someone steps up to do the
>>>> conversion from vb1 to vb2.
> 
> There is ongoing discussion in VDR community about converting to vb2,
> albeit it's very slow and I don't know yet if it'll reach a conclusion.
> And, as proposed, keeping it in staging area or keeping the driver with
> stripped analog video capabilities would definitely help.
> 
>>> Maybe I'm biased, but who has actually still access to analog TV?  I'm
>>> not sure about that, but in Germany, analog TV over terrestrial antenna
>>> has been dropped about 2005.  Analog satellite has gone 2012.  Analog
>>> cable took until 2019.  I don't know about other regions.
>>
>> Analog TV is almost certainly still used in places.
>>
>> But it's not so much about TV as it is about analog video in general, so
>> also capturing from S-Video or composite connectors. That's definitely
>> still used.
>>
>>>
>>>> So support for the old MXB, Hexium Gemini and Hexium Orion would die, but
>>>> for the other DVB devices it would stay alive.
>>>
>>> So, yeah, from my POV this might be an option.
>>>
>>>> I'm honestly quite surprised that these old DVB PCI cards are still in
>>>> use, I did not expect that.
>>>
>>> There isn't a really well-working alternative, unless you plug in lots
>>> of USB devices instead.  Even more important these days, the cards are
>>> still working, despite their age.  There's really no good reason to
>>> throw them into the dustbin and buy something new, instead of trying to
>>> use working equipment as long as possible to save waste.
>>
>> OK, I'll make a new PR that keeps the saa7146 for now, and work on removing
>> analog video support from saa7146.
> 
> Thank You, that would certainly make me satisfied. Would that mean that
> after removing analog video support, driver would go out of staging, back
> to maintained area?

Yes, since there is no reason to keep it in staging once that's done.

Regards,

	Hans

> 
> Best Regards
> 
> 1. https://www.omicom.info/PayPal_Omicom_S2_PCIr3.html
> 


  reply	other threads:[~2023-01-17 15:51 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-01-11 11:24 [PATCHv2 00/16] staging/media: remove most deprecated drivers Hans Verkuil
2023-01-11 11:24 ` [PATCHv2 01/16] meye: remove this deprecated driver Hans Verkuil
2023-01-11 11:24 ` [PATCHv2 02/16] cpia2: remove " Hans Verkuil
2023-01-11 11:24 ` [PATCHv2 03/16] fsl-viu: " Hans Verkuil
2023-01-11 11:24 ` [PATCHv2 04/16] stkwebcam: " Hans Verkuil
2023-01-11 11:24 ` [PATCHv2 05/16] zr364xx: " Hans Verkuil
2023-01-11 11:24 ` [PATCHv2 06/16] vpfe_capture: remove deprecated davinci drivers Hans Verkuil
2023-01-13 11:39   ` Prabhakar Mahadev Lad
2023-01-11 11:24 ` [PATCHv2 07/16] tm6000: remove deprecated driver Hans Verkuil
2023-01-11 11:24 ` [PATCHv2 08/16] av7110: stop building this " Hans Verkuil
2023-01-11 11:24 ` [PATCHv2 09/16] av7110: remove deprecated documentation Hans Verkuil
2023-01-11 11:24 ` [PATCHv2 10/16] av7110: remove av7110* sources Hans Verkuil
2023-01-11 11:24 ` [PATCHv2 11/16] av7110: remove deprecated driver Hans Verkuil
2023-01-11 11:24 ` [PATCHv2 12/16] ttpci: " Hans Verkuil
2023-01-11 11:24 ` [PATCHv2 13/16] saa7146: remove deprecated drivers Hans Verkuil
2023-01-11 11:24 ` [PATCHv2 14/16] dvbdev.h: remove DVB_DEVICE_VIDEO/AUDIO/OSD Hans Verkuil
2023-01-11 11:24 ` [PATCHv2 15/16] dvb_demux.h: remove av7110-private fields Hans Verkuil
2023-01-11 11:24 ` [PATCHv2 16/16] dvb_demux.h: remove write_to_decoder Hans Verkuil
2023-01-16  9:53 ` [PATCHv2 00/16] staging/media: remove most deprecated drivers Corinna Vinschen
2023-01-16 10:05   ` Rudy Zijlstra
2023-01-16 10:08   ` Hans Verkuil
2023-01-16 13:35     ` Corinna Vinschen
2023-01-16 14:39       ` Hans Verkuil
2023-01-16 17:17         ` Corinna Vinschen
2023-01-17 15:38         ` Tomasz Maciej Nowak
2023-01-17 15:48           ` Hans Verkuil [this message]
2023-01-18  2:06 ` Stefan Herdler

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=c75b0172-fc2c-90ab-8138-8f916d20be63@xs4all.nl \
    --to=hverkuil-cisco@xs4all.nl \
    --cc=dac6a072-39f4-96c5-cb13-171c54213937@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    --cc=rudy@grumpydevil.homelinux.org \
    --cc=vinschen@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).