Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#11723 closed Bug Report - General (fixed)

pvr150 subtitle fix

Reported by: dan.littlejohn@… Owned by: jpoet
Priority: minor Milestone: 0.27
Component: MythTV - General Version: Master Head
Severity: low Keywords:
Cc: Ticket locked: no

Description

John Poet provided a patch that fixes missing pvr150 subtitles in mythtv 26 fixes. He figured out a kernel change requires that /dev/vbi be used instead of /dev/video. I am using 3.7.10 and the patch works. Would appreciate if it could be patch in. Thanks!

Below is the mailing list back and forth

+++

I recently bought an old pvr150 to try and get closed captions to work on our dish recordings. It records fine, but still no VBI and I am not sure what this error is telling me.

2013-05-28 23:06:48.204570 I [2533/2585] TVRecEvent tv_rec.cpp:1043 (HandleStateChange) - TVRec(3): Changing from None to RecordingOnly 2013-05-28 23:06:48.209599 I [2533/2585] TVRecEvent tv_rec.cpp:3562 (TuningCheckForHWChange) - TVRec(3): HW Tuner: 3->3 2013-05-28 23:06:48.237386 I [2533/2585] TVRecEvent v4lchannel.cpp:661 (SetInputAndFormat) - V4LChannel(/dev/video0): SetInputAndFormat(3, NTSC) (v4l v2) input_switch: 0 mode_switch: 0 2013-05-28 23:06:52.568117 I [2533/2585] TVRecEvent tv_rec.cpp:4056 (TuningNewRecorder) - TVRec(3): rec->GetPathname(): '/store/tv/2118_20130529040700.mpg' 2013-05-28 23:06:52.619924 W [2533/2756] RecThread mpegrecorder.cpp:864 (SetVBIOptions) - MPEGRec(/dev/video0): Unable to enable VBI embedding

eno: Invalid argument (22)

2013-05-28 23:06:56.168135 I [2533/2586] HouseKeeping housekeeper.cpp:221 (RunHouseKeeping) - Running housekeeping thread 2013-05-28 23:07:02.228775 I [2533/2695] ProcessRequest mainserver.cpp:1395 (HandleAnnounce) - MainServer::ANN Playback 2013-05-28 23:07:02.228789 I [2533/2695] ProcessRequest mainserver.cpp:1397 (HandleAnnounce) - adding: one as a client (events: 0) 2013-05-28 23:07:03.111910 I [2533/2695] ProcessRequest recorderbase.cpp:395 (GetKeyframePositions) - RecBase(3:/dev/video0): GetKeyframePositions(241,9223372036854775807,#3) out of 20 2013-05-28 23:07:10.111152 I [2533/2695] ProcessRequest mainserver.cpp:1395 (HandleAnnounce) - MainServer::ANN Playback 2013-05-28 23:07:10.111161 I [2533/2695] ProcessRequest mainserver.cpp:1397 (HandleAnnounce) - adding: one as a client (events: 0) 2013-05-28 23:07:10.458248 I [2533/2695] ProcessRequest recorderbase.cpp:395 (GetKeyframePositions) - RecBase(3:/dev/video0): GetKeyframePositions(376,9223372036854775807,#9) out of 35

I have tried changing the recording profile

DVD 48000 720 x 480 layer II - 320

Does anyone have a tip?

Dan

Are you using a recent kernel? Recently something changed and the ioctl's to setup VBI now need to be made again /dev/vbi? instead of /dev/video?. Myth probably needs updated to do that -- but only for the right kernel versions. I have not done the research to figure which kernel version that started with. If anyone knows we might be able to update Myth appropriately.

I don't try and use my pvr-500 to record into myth, I *just* use it to grab CC for my HD-PVR recordings. CC stopped working a couple of months ago, and I finally figured out that I had to change my CC script to talk to /dev/vbi? instead. I am using a modified version of the script from here http://www.mythtv.org/wiki/Captions_with_HD-PVR .

John

Bummer. I am using the 3.7.10 kernel on Gentoo.

I don't see anywhere in the mythtv-setup to update the CC source and I am guessing there is no script I can manually change.

If there is no easy fix maybe I will try these HD-PVR scripts. Where there any other major changes that would be helpful to know besides the /dev/vbi VBI setup?

Dan

The attached patch is completely untested, doesn't check for kernel version, etc., but may be worth trying...

John

I finally got a chance to try this out and your patch works, there are subtitles for my pvr150 recordings.

I will be putting in a bug fix into the myth tracker to hopefully have this fixed in the main tree. Thanks John!

Attachments (3)

vbi.patch (1.6 KB ) - added by dan.littlejohn@… 12 years ago.
vbi.2.patch (1.6 KB ) - added by dan.littlejohn@… 12 years ago.
vbi.patch
V4L2-VBI.patch (2.2 KB ) - added by jpoet 12 years ago.
Updated version with fall back to /dev/videoX

Download all attachments as: .zip

Change History (6)

by dan.littlejohn@…, 12 years ago

Attachment: vbi.patch added

by dan.littlejohn@…, 12 years ago

Attachment: vbi.2.patch added

vbi.patch

comment:1 by jpoet, 12 years ago

Owner: set to jpoet
Status: newaccepted

by jpoet, 12 years ago

Attachment: V4L2-VBI.patch added

Updated version with fall back to /dev/videoX

comment:2 by John Poet <jpoet@…>, 12 years ago

Resolution: fixed
Status: acceptedclosed

In 25310069a1154213cbc94c903c8b0ace30893ec4/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:3 by jpoet, 12 years ago

Milestone: unknown0.27
Version: 0.26-fixesMaster Head
Note: See TracTickets for help on using tickets.