[PULL] git://git.kernellabs.com/stoth/cx23885-hvr1850.git media-master branch

[PULL] git://git.kernellabs.com/stoth/cx23885-hvr1850.git media-master branch

[Steven Toth]

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

Re: [PULL] git://git.kernellabs.com/stoth/cx23885-hvr1850.git media-master branch

[Mauro Carvalho Chehab]

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,
> 

Re: [PULL] git://git.kernellabs.com/stoth/cx23885-hvr1850.git media-master branch

[Mauro Carvalho Chehab]

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

Re: [PULL] git://git.kernellabs.com/stoth/cx23885-hvr1850.git media-master branch

[Steven Toth]

> 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