Re: Patch: Schedule obsolete v4l1 quickcam_messenger and ov511 drivers for removal

Re: Patch: Schedule obsolete v4l1 quickcam_messenger and ov511 drivers for removal

[Mauro Carvalho Chehab]

Hi Hans,

Em Tue, 23 Jun 2009 13:39:38 +0200
Hans de Goede <hdegoede@redhat.com> escreveu:

> Schedule obsolete v4l1 quickcam_messenger and ov511 drivers for removal
> 

It would be better to add the "Files:" field to explicitly indicate what files
will be removed, like the modified version of your patch. Please check if those
are the files you're meaning to remove:

---

From: Hans de Goede <hdegoede@redhat.com>

Schedule obsolete v4l1 quickcam_messenger and ov511 drivers for removal

[mchehab@redhat.com: add the files: tag to indicate what will be removed]

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

diff --git a/Documentation/feature-removal-schedule.txt b/Documentation/feature-removal-schedule.txt
index f8cd450..8a8c045 100644
--- a/Documentation/feature-removal-schedule.txt
+++ b/Documentation/feature-removal-schedule.txt
@@ -458,3 +458,19 @@ Why:	Remove the old legacy 32bit machine check code. This has been
 	but the old version has been kept around for easier testing. Note this
 	doesn't impact the old P5 and WinChip machine check handlers.
 Who:	Andi Kleen <andi@firstfloor.org>
+
+----------------------------
+
+What:	usbvideo quickcam_messenger driver
+When:	2.6.32
+Why:	obsolete v4l1 driver replaced by gspca_stv06xx
+Who:	Hans de Goede <hdegoede@redhat.com>
+Files:	drivers/media/video/c-qcam.c drivers/media/video/bw-qcam.c
+
+----------------------------
+
+What:	ov511 v4l1 driver
+When:	2.6.32
+Why:	obsolete v4l1 driver replaced by gspca_ov519
+Who:	Hans de Goede <hdegoede@redhat.com>
+Files:	drivers/media/video/ov511.[ch]




Cheers,
Mauro

Re: Patch: Schedule obsolete v4l1 quickcam_messenger and ov511 drivers for removal

[Hans de Goede]

Hi,

On 07/02/2009 07:23 PM, Mauro Carvalho Chehab wrote:
> Hi Hans,
>
> Em Tue, 23 Jun 2009 13:39:38 +0200
> Hans de Goede<hdegoede@redhat.com>  escreveu:
>
>> Schedule obsolete v4l1 quickcam_messenger and ov511 drivers for removal
>>
>
> It would be better to add the "Files:" field to explicitly indicate what files
> will be removed, like the modified version of your patch. Please check if those
> are the files you're meaning to remove:
>

Hi,

Good idea, but the files listed for the quickcam_messenger obsolete are wrong, those
should be:
drivers/media/video/usbvideo/quickcam_messenger.[c,h]

Regards,

Hans


> ---
>
> From: Hans de Goede<hdegoede@redhat.com>
>
> Schedule obsolete v4l1 quickcam_messenger and ov511 drivers for removal
>
> [mchehab@redhat.com: add the files: tag to indicate what will be removed]
>
> Signed-off-by: Hans de Goede<hdegoede@redhat.com>
> Signed-off-by: Mauro Carvalho Chehab<mchehab@redhat.com>
>
> diff --git a/Documentation/feature-removal-schedule.txt b/Documentation/feature-removal-schedule.txt
> index f8cd450..8a8c045 100644
> --- a/Documentation/feature-removal-schedule.txt
> +++ b/Documentation/feature-removal-schedule.txt
> @@ -458,3 +458,19 @@ Why:	Remove the old legacy 32bit machine check code. This has been
>   	but the old version has been kept around for easier testing. Note this
>   	doesn't impact the old P5 and WinChip machine check handlers.
>   Who:	Andi Kleen<andi@firstfloor.org>
> +
> +----------------------------
> +
> +What:	usbvideo quickcam_messenger driver
> +When:	2.6.32
> +Why:	obsolete v4l1 driver replaced by gspca_stv06xx
> +Who:	Hans de Goede<hdegoede@redhat.com>
> +Files:	drivers/media/video/c-qcam.c drivers/media/video/bw-qcam.c
> +
> +----------------------------
> +
> +What:	ov511 v4l1 driver
> +When:	2.6.32
> +Why:	obsolete v4l1 driver replaced by gspca_ov519
> +Who:	Hans de Goede<hdegoede@redhat.com>
> +Files:	drivers/media/video/ov511.[ch]
>
>
>
>
> Cheers,
> Mauro

cx23885 HVR-1700 broken in v4l drivers vs standard kernel

[ldone]

Hi

if I use stock kernel (Suse)  2.6.27.23-0.1 the card works, scans and
plays DVB-T channels.
if I use the latest v4l drivers the card is recognised, and picked up,
tunes a channel, but cannot find any DVB-T channels.

I'm afraid I'm not sure what to look for

Thanks

cx23885 HVR-1700 broken in v4l drivers vs standard kernel

[ldone]

Hi - apologies for posting twice, posted this as a reply rather than a new thread - doing well with mailing lists!

if I use stock kernel (Suse)  2.6.27.23-0.1 the card works, scans and
plays DVB-T channels.
if I use the latest v4l drivers the card is recognised, and picked up,
tune a channel, but cannot find any DVB-T channels.

I'm afraid I'm not sure what to look for there are quite a few structural changes to the code.

Thanks