public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
* Cropping with Hauppauge HVR-1900
@ 2009-06-26  9:46 jmdk
  2009-06-26 16:33 ` Mike Isely
  0 siblings, 1 reply; 4+ messages in thread
From: jmdk @ 2009-06-26  9:46 UTC (permalink / raw)
  To: linux-media

Hello,

I have a Hauppauge HVR-1900 which works fine with the pvrusb2 driver.
However because most TV channels now air with 16:9 content inside 4:3
images, I would like to crop out the top and bottom black bars before
encoding via the hardware MPEG2 encoder. I tried using the ctl_crop_top
and ctl_crop_height in  /sysfs as well as the --set-crop option of
v4l2-ctrl but only received error messages indicating a value out of range.

The card has a cx25843 which should support cropping. Does anyone know
how to get this feature to work ?

Thanks in advance for the help,

Joseph

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Cropping with Hauppauge HVR-1900
  2009-06-26  9:46 Cropping with Hauppauge HVR-1900 jmdk
@ 2009-06-26 16:33 ` Mike Isely
  2009-06-26 17:44   ` Hans Verkuil
  2009-06-26 18:27   ` Andy Walls
  0 siblings, 2 replies; 4+ messages in thread
From: Mike Isely @ 2009-06-26 16:33 UTC (permalink / raw)
  To: jmdk; +Cc: linux-media

On Fri, 26 Jun 2009, jmdk wrote:

> Hello,
> 
> I have a Hauppauge HVR-1900 which works fine with the pvrusb2 driver.
> However because most TV channels now air with 16:9 content inside 4:3
> images, I would like to crop out the top and bottom black bars before
> encoding via the hardware MPEG2 encoder. I tried using the ctl_crop_top
> and ctl_crop_height in  /sysfs as well as the --set-crop option of
> v4l2-ctrl but only received error messages indicating a value out of range.
> 
> The card has a cx25843 which should support cropping. Does anyone know
> how to get this feature to work ?
> 
> Thanks in advance for the help,

Unfortunately the cx25840 driver does not implement any cropping 
capability and the recent change to the v4l-subdevice architecture 
currently lacks the API entrypoints to make such a thing possible :-(

A pvrusb2 user a while back (it's in the pvrusb2 list archives) came up 
with some patches to the cx25840 driver that implement cropping.  He 
also generated corresponding patches for the pvrusb2 driver.  I merged 
his pvrusb2 patches (which is why you see that support in the driver) 
but not the cx25840 patches since I'm not an expert on that chip and I 
don't have the ability to verify that the patches preserve correct 
behavior for all the other devices that use the same chip.  I did pass 
the cx25840 patches to Hans Verkuil and he tried them out at the time 
and basically pronounced them "ok" but I think even he wasn't sure if it 
was doing everything right.

That was a while ago and the cx25840 driver has undergone enough other 
changes to make merging those patches a lot more difficult - plus the 
v4l-subdevice stuff needs additional changes to support the extra 
cropping API.

  -Mike


-- 

Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Cropping with Hauppauge HVR-1900
  2009-06-26 16:33 ` Mike Isely
@ 2009-06-26 17:44   ` Hans Verkuil
  2009-06-26 18:27   ` Andy Walls
  1 sibling, 0 replies; 4+ messages in thread
From: Hans Verkuil @ 2009-06-26 17:44 UTC (permalink / raw)
  To: Mike Isely; +Cc: jmdk, linux-media

On Friday 26 June 2009 18:33:11 Mike Isely wrote:
> On Fri, 26 Jun 2009, jmdk wrote:
> 
> > Hello,
> > 
> > I have a Hauppauge HVR-1900 which works fine with the pvrusb2 driver.
> > However because most TV channels now air with 16:9 content inside 4:3
> > images, I would like to crop out the top and bottom black bars before
> > encoding via the hardware MPEG2 encoder. I tried using the ctl_crop_top
> > and ctl_crop_height in  /sysfs as well as the --set-crop option of
> > v4l2-ctrl but only received error messages indicating a value out of range.
> > 
> > The card has a cx25843 which should support cropping. Does anyone know
> > how to get this feature to work ?
> > 
> > Thanks in advance for the help,
> 
> Unfortunately the cx25840 driver does not implement any cropping 
> capability and the recent change to the v4l-subdevice architecture 
> currently lacks the API entrypoints to make such a thing possible :-(
> 
> A pvrusb2 user a while back (it's in the pvrusb2 list archives) came up 
> with some patches to the cx25840 driver that implement cropping.  He 
> also generated corresponding patches for the pvrusb2 driver.  I merged 
> his pvrusb2 patches (which is why you see that support in the driver) 
> but not the cx25840 patches since I'm not an expert on that chip and I 
> don't have the ability to verify that the patches preserve correct 
> behavior for all the other devices that use the same chip.  I did pass 
> the cx25840 patches to Hans Verkuil and he tried them out at the time 
> and basically pronounced them "ok" but I think even he wasn't sure if it 
> was doing everything right.
> 
> That was a while ago and the cx25840 driver has undergone enough other 
> changes to make merging those patches a lot more difficult - plus the 
> v4l-subdevice stuff needs additional changes to support the extra 
> cropping API.

Just a quick heads-up: the cropping API will be added to the v4l2_subdev API
soon. A pull request from Guennadi that does this is already posted to the
list.

Regards,

	Hans

-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Cropping with Hauppauge HVR-1900
  2009-06-26 16:33 ` Mike Isely
  2009-06-26 17:44   ` Hans Verkuil
@ 2009-06-26 18:27   ` Andy Walls
  1 sibling, 0 replies; 4+ messages in thread
From: Andy Walls @ 2009-06-26 18:27 UTC (permalink / raw)
  To: Mike Isely; +Cc: jmdk, linux-media

On Fri, 2009-06-26 at 11:33 -0500, Mike Isely wrote:
> On Fri, 26 Jun 2009, jmdk wrote:
> 
> > Hello,
> > 
> > I have a Hauppauge HVR-1900 which works fine with the pvrusb2 driver.
> > However because most TV channels now air with 16:9 content inside 4:3
> > images, I would like to crop out the top and bottom black bars before
> > encoding via the hardware MPEG2 encoder. I tried using the ctl_crop_top
> > and ctl_crop_height in  /sysfs as well as the --set-crop option of
> > v4l2-ctrl but only received error messages indicating a value out of range.
> > 
> > The card has a cx25843 which should support cropping. Does anyone know
> > how to get this feature to work ?
> > 
> > Thanks in advance for the help,
> 
> Unfortunately the cx25840 driver does not implement any cropping 
> capability and the recent change to the v4l-subdevice architecture 
> currently lacks the API entrypoints to make such a thing possible :-(
> 
> A pvrusb2 user a while back (it's in the pvrusb2 list archives) came up 
> with some patches to the cx25840 driver that implement cropping.  He 
> also generated corresponding patches for the pvrusb2 driver.  I merged 
> his pvrusb2 patches (which is why you see that support in the driver) 
> but not the cx25840 patches since I'm not an expert on that chip and I 
> don't have the ability to verify that the patches preserve correct 
> behavior for all the other devices that use the same chip.  I did pass 
> the cx25840 patches to Hans Verkuil and he tried them out at the time 
> and basically pronounced them "ok" but I think even he wasn't sure if it 
> was doing everything right.
> 
> That was a while ago and the cx25840 driver has undergone enough other 
> changes to make merging those patches a lot more difficult - plus the 
> v4l-subdevice stuff needs additional changes to support the extra 
> cropping API.

Not only that, doing cropping with the CX2584x chips and friends appears
to break VBI extraction in interaction with the CX2341x encoding engine.

A good bit of experimentation would need to be done to keep VBI working
with cropping.

Regards,
Andy

>   -Mike



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2009-06-26 18:25 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-06-26  9:46 Cropping with Hauppauge HVR-1900 jmdk
2009-06-26 16:33 ` Mike Isely
2009-06-26 17:44   ` Hans Verkuil
2009-06-26 18:27   ` Andy Walls

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox