* [PULL] git://git.kernellabs.com/stoth/cx23885-hvr1850.git media-master branch
@ 2012-01-04 15:28 Steven Toth
2012-01-04 22:44 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 4+ messages in thread
From: Steven Toth @ 2012-01-04 15:28 UTC (permalink / raw)
To: Linux-Media; +Cc: Mauro Chehab
Mauro,
I've been adding support to the CX23885 and CX25840 drivers for the
Hauppauge HVR1850
card. These patches enable the use of raw video, audio and/or the mpeg
encoder, via all
video and audio inputs. Support for the HVR1850 is now in pretty good shape.
The card uses the CX23888 PCIe bridge which brings its own complexities and
additional code to the CX25840. I've tested these patches against the
HVR1700, HVR1800
and HVR1850, everything appears to be working correctly.
These also fix a small regression in the HVR1800 driver related to the
work done during
October 2010 on the subdev conversion. Given that nobody has noticed
in the last 12
months it's not too important.
Tree is at git://git.kernellabs.com/stoth/cx23885-hvr1850.git
media-master branch.
Patch series viewable at:
http://git.kernellabs.com/?p=stoth/cx23885-hvr1850.git;a=shortlog;h=refs/heads/media-master
[media] cx25840: Added g_std support to the video decoder driver
[media] cx25840: Hauppauge HVR1850 Analog driver support (patch#4)
[media] cx25840: Add a flag to enable the CX23888 DIF to be enabled or not.
[media] cx23885: Hauppauge HVR1850 Analog driver support (patch#3)
[media] cx23885: Hauppauge HVR1850 Analog driver support (patch#2)
[media] cx23885: Hauppauge HVR1850 Analog driver support (patch#1)
[media] cx23885: Bugfix /sys/class/video4linux/videoX/name truncation
[media] cx23885: Control cleanup on the MPEG Encoder
[media] cx23885: Configure the MPEG encoder early to avoid jerky video
[media] cx23885: Ensure the MPEG encoder height is configured from the norm
[media] cx23885: Cleanup MPEG encoder GPIO handling
[media] cx25840 / cx23885: Fixing audio/volume regression
b/drivers/media/video/cx23885/cx23885-417.c | 4
b/drivers/media/video/cx23885/cx23885-cards.c | 28
b/drivers/media/video/cx23885/cx23885-core.c | 24
b/drivers/media/video/cx23885/cx23885-dvb.c | 14
b/drivers/media/video/cx23885/cx23885-video.c | 153 +
b/drivers/media/video/cx23885/cx23885.h | 12
b/drivers/media/video/cx25840/cx25840-audio.c | 10
b/drivers/media/video/cx25840/cx25840-core.c | 36
b/include/media/cx25840.h | 1
drivers/media/video/cx23885/cx23885-417.c | 137 -
drivers/media/video/cx23885/cx23885-video.c | 10
drivers/media/video/cx25840/cx25840-core.c | 3188 +++++++++++++++++++++++++-
12 files changed, 3487 insertions(+), 130 deletions(-)
Thanks,
--
Steven Toth - Kernel Labs
http://www.kernellabs.com
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PULL] git://git.kernellabs.com/stoth/cx23885-hvr1850.git media-master branch
2012-01-04 15:28 [PULL] git://git.kernellabs.com/stoth/cx23885-hvr1850.git media-master branch Steven Toth
@ 2012-01-04 22:44 ` Mauro Carvalho Chehab
2012-01-04 23:03 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 4+ messages in thread
From: Mauro Carvalho Chehab @ 2012-01-04 22:44 UTC (permalink / raw)
To: Steven Toth; +Cc: Linux-Media
On 04-01-2012 13:28, Steven Toth wrote:
> Mauro,
>
> I've been adding support to the CX23885 and CX25840 drivers for the
> Hauppauge HVR1850
> card. These patches enable the use of raw video, audio and/or the mpeg
> encoder, via all
> video and audio inputs. Support for the HVR1850 is now in pretty good shape.
>
> The card uses the CX23888 PCIe bridge which brings its own complexities and
> additional code to the CX25840. I've tested these patches against the
> HVR1700, HVR1800
> and HVR1850, everything appears to be working correctly.
>
> These also fix a small regression in the HVR1800 driver related to the
> work done during
> October 2010 on the subdev conversion. Given that nobody has noticed
> in the last 12
> months it's not too important.
>
> Tree is at git://git.kernellabs.com/stoth/cx23885-hvr1850.git
> media-master branch.
Steve,
Please, always use git request-pull to generate pull requests, otherwise
patchwork won't catch and I may miss it.
Thanks,
Mauro
>
> Patch series viewable at:
>
> http://git.kernellabs.com/?p=stoth/cx23885-hvr1850.git;a=shortlog;h=refs/heads/media-master
>
> [media] cx25840: Added g_std support to the video decoder driver
> [media] cx25840: Hauppauge HVR1850 Analog driver support (patch#4)
> [media] cx25840: Add a flag to enable the CX23888 DIF to be enabled or not.
> [media] cx23885: Hauppauge HVR1850 Analog driver support (patch#3)
> [media] cx23885: Hauppauge HVR1850 Analog driver support (patch#2)
> [media] cx23885: Hauppauge HVR1850 Analog driver support (patch#1)
> [media] cx23885: Bugfix /sys/class/video4linux/videoX/name truncation
> [media] cx23885: Control cleanup on the MPEG Encoder
> [media] cx23885: Configure the MPEG encoder early to avoid jerky video
> [media] cx23885: Ensure the MPEG encoder height is configured from the norm
> [media] cx23885: Cleanup MPEG encoder GPIO handling
> [media] cx25840 / cx23885: Fixing audio/volume regression
>
> b/drivers/media/video/cx23885/cx23885-417.c | 4
> b/drivers/media/video/cx23885/cx23885-cards.c | 28
> b/drivers/media/video/cx23885/cx23885-core.c | 24
> b/drivers/media/video/cx23885/cx23885-dvb.c | 14
> b/drivers/media/video/cx23885/cx23885-video.c | 153 +
> b/drivers/media/video/cx23885/cx23885.h | 12
> b/drivers/media/video/cx25840/cx25840-audio.c | 10
> b/drivers/media/video/cx25840/cx25840-core.c | 36
> b/include/media/cx25840.h | 1
> drivers/media/video/cx23885/cx23885-417.c | 137 -
> drivers/media/video/cx23885/cx23885-video.c | 10
> drivers/media/video/cx25840/cx25840-core.c | 3188 +++++++++++++++++++++++++-
> 12 files changed, 3487 insertions(+), 130 deletions(-)
>
> Thanks,
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PULL] git://git.kernellabs.com/stoth/cx23885-hvr1850.git media-master branch
2012-01-04 22:44 ` Mauro Carvalho Chehab
@ 2012-01-04 23:03 ` Mauro Carvalho Chehab
2012-01-05 0:30 ` Steven Toth
0 siblings, 1 reply; 4+ messages in thread
From: Mauro Carvalho Chehab @ 2012-01-04 23:03 UTC (permalink / raw)
To: Steven Toth; +Cc: Linux-Media
On 04-01-2012 20:44, Mauro Carvalho Chehab wrote:
> On 04-01-2012 13:28, Steven Toth wrote:
>> Mauro,
>>
>> I've been adding support to the CX23885 and CX25840 drivers for the
>> Hauppauge HVR1850
>> card. These patches enable the use of raw video, audio and/or the mpeg
>> encoder, via all
>> video and audio inputs. Support for the HVR1850 is now in pretty good shape.
>>
>> The card uses the CX23888 PCIe bridge which brings its own complexities and
>> additional code to the CX25840. I've tested these patches against the
>> HVR1700, HVR1800
>> and HVR1850, everything appears to be working correctly.
>>
>> These also fix a small regression in the HVR1800 driver related to the
>> work done during
>> October 2010 on the subdev conversion. Given that nobody has noticed
>> in the last 12
>> months it's not too important.
>>
>> Tree is at git://git.kernellabs.com/stoth/cx23885-hvr1850.git
>> media-master branch.
>
> Steve,
>
> Please, always use git request-pull to generate pull requests, otherwise
> patchwork won't catch and I may miss it.
>
> Thanks,
> Mauro
>
>>
>> Patch series viewable at:
>>
>> http://git.kernellabs.com/?p=stoth/cx23885-hvr1850.git;a=shortlog;h=refs/heads/media-master
>> [media] cx25840: Added g_std support to the video decoder driver
>> [media] cx25840: Hauppauge HVR1850 Analog driver support (patch#4)
>> [media] cx25840: Add a flag to enable the CX23888 DIF to be enabled or not.
>> [media] cx23885: Hauppauge HVR1850 Analog driver support (patch#3)
>> [media] cx23885: Hauppauge HVR1850 Analog driver support (patch#2)
>> [media] cx23885: Hauppauge HVR1850 Analog driver support (patch#1)
>> [media] cx23885: Bugfix /sys/class/video4linux/videoX/name truncation
Not applied the above patches, due to a compilation breakage on the next
patch that are before them.
>> [media] cx23885: Control cleanup on the MPEG Encoder
There's something wrong on this patch. It breaks compilation:
drivers/media/video/cx23885/cx23885-417.c: In function ‘vidioc_s_std’:
drivers/media/video/cx23885/cx23885-417.c:1240:2: error: implicit declaration of function ‘cx23885_set_tvnorm’ [-Werror=implicit-function-declaration]
drivers/media/video/cx23885/cx23885-417.c: In function ‘vidioc_enum_input’:
drivers/media/video/cx23885/cx23885-417.c:1251:2: error: implicit declaration of function ‘cx23885_enum_input’ [-Werror=implicit-function-declaration]
drivers/media/video/cx23885/cx23885-417.c: In function ‘vidioc_g_input’:
drivers/media/video/cx23885/cx23885-417.c:1256:2: error: implicit declaration of function ‘cx23885_get_input’ [-Werror=implicit-function-declaration]
drivers/media/video/cx23885/cx23885-417.c: In function ‘vidioc_s_input’:
drivers/media/video/cx23885/cx23885-417.c:1261:2: error: implicit declaration of function ‘cx23885_set_input’ [-Werror=implicit-function-declaration]
drivers/media/video/cx23885/cx23885-417.c: In function ‘vidioc_s_frequency’:
drivers/media/video/cx23885/cx23885-417.c:1316:2: error: implicit declaration of function ‘cx23885_set_frequency’ [-Werror=implicit-function-declaration]
drivers/media/video/cx23885/cx23885-417.c: In function ‘vidioc_g_ctrl’:
drivers/media/video/cx23885/cx23885-417.c:1324:2: error: implicit declaration of function ‘cx23885_get_control’ [-Werror=implicit-function-declaration]
drivers/media/video/cx23885/cx23885-417.c: In function ‘vidioc_s_ctrl’:
drivers/media/video/cx23885/cx23885-417.c:1332:2: error: implicit declaration of function ‘cx23885_set_control’ [-Werror=implicit-function-declaration]
cc1: some warnings being treated as errors
Please fix it. No patch should break compilation, or it would affect
"git bisect" handling with is bad not only for media developers, but for
everybody else working with Kernel development.
>> [media] cx23885: Configure the MPEG encoder early to avoid jerky video
This one also breaks compilation:
drivers/media/video/cx23885/cx23885-417.c:1351:2: error: too few arguments to function ‘cx23885_initialize_codec’
In this specific case, the fix is trivial, so, I've applied it, and added a
reviewer note about the breakage fix.
>> [media] cx23885: Ensure the MPEG encoder height is configured from the norm
>> [media] cx23885: Cleanup MPEG encoder GPIO handling
>> [media] cx25840 / cx23885: Fixing audio/volume regression
Those were also applied.
Regards,
Mauro
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PULL] git://git.kernellabs.com/stoth/cx23885-hvr1850.git media-master branch
2012-01-04 23:03 ` Mauro Carvalho Chehab
@ 2012-01-05 0:30 ` Steven Toth
0 siblings, 0 replies; 4+ messages in thread
From: Steven Toth @ 2012-01-05 0:30 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: Linux-Media
> There's something wrong on this patch. It breaks compilation:
Mauro,
My mistake, I've corrected the issue:
The following changes since commit 9c9c3d078b0dd81a74e5f531aa1efa30add5b419:
[media] cx23885: Configure the MPEG encoder early to avoid jerky
video (2012-01-04 20:51:18 -0200)
are available in the git repository at:
git://git.kernellabs.com/stoth/cx23885-hvr1850-fixups.git staging/for_v3.3
Steven Toth (6):
[media] cx25840: Add a flag to enable the CX23888 DIF to be
enabled or not.
[media] cx23885: Hauppauge HVR1850 Analog driver support
[media] cx23885: Control cleanup on the MPEG Encoder
[media] cx23885: Bugfix /sys/class/video4linux/videoX/name truncation
[media] cx25840: Hauppauge HVR1850 Analog driver support (patch2)
[media] cx25840: Added g_std support to the video decoder driver
drivers/media/video/cx23885/cx23885-417.c | 105 +-
drivers/media/video/cx23885/cx23885-cards.c | 28 +-
drivers/media/video/cx23885/cx23885-core.c | 24 +-
drivers/media/video/cx23885/cx23885-dvb.c | 14 +
drivers/media/video/cx23885/cx23885-video.c | 157 ++-
drivers/media/video/cx23885/cx23885.h | 12 +
drivers/media/video/cx25840/cx25840-core.c | 3224 ++++++++++++++++++++++++++-
include/media/cx25840.h | 1 +
8 files changed, 3454 insertions(+), 111 deletions(-)
Thanks,
- Steve
--
Steven Toth - Kernel Labs
http://www.kernellabs.com
+1.646.355.8490
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-01-05 0:30 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-01-04 15:28 [PULL] git://git.kernellabs.com/stoth/cx23885-hvr1850.git media-master branch Steven Toth
2012-01-04 22:44 ` Mauro Carvalho Chehab
2012-01-04 23:03 ` Mauro Carvalho Chehab
2012-01-05 0:30 ` Steven Toth
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox