Opened 11 years ago

Closed 11 years ago

Last modified 11 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@… 11 years ago.
vbi.2.patch (1.6 KB) - added by dan.littlejohn@… 11 years ago.
vbi.patch
V4L2-VBI.patch (2.2 KB) - added by jpoet 11 years ago.
Updated version with fall back to /dev/videoX

Download all attachments as: .zip

Change History (6)

Changed 11 years ago by dan.littlejohn@…

Attachment: vbi.patch added

Changed 11 years ago by dan.littlejohn@…

Attachment: vbi.2.patch added

vbi.patch

comment:1 Changed 11 years ago by jpoet

Owner: set to jpoet
Status: newaccepted

Changed 11 years ago by jpoet

Attachment: V4L2-VBI.patch added

Updated version with fall back to /dev/videoX

comment:2 Changed 11 years ago by John Poet <jpoet@…>

Resolution: fixed
Status: acceptedclosed

In 25310069a1154213cbc94c903c8b0ace30893ec4/mythtv:

Use the vbi device for VBI releated ioctls. Fall back to using the video
device if the ioctl against the vbi device fails.

This hopefully fixes #11723

comment:3 Changed 11 years ago by jpoet

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