Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#9247 closed Bug Report (Fixed)

Tuner value not obeyed when using manual IP setting for HDHR

Reported by: Raymond Wagner Owned by: Silicondust
Priority: minor Milestone:
Component: MythTV - HDHomeRun Version: 0.24
Severity: medium Keywords:
Cc: Ticket locked: no

Description (last modified by Raymond Wagner)

When accessing the HDHomeRun using a manual IP address rather than autodetection, the second tuner is not accessible. Setting MythTV to use the second tuner will be ignored, and it will access the first anyway. If MythTV is already using the first tuner, the existing recording will be silently disconnected.

Attachments (1)

hdhrtunerfix.diff (1.1 KB) - added by robertm 13 years ago.
Probable HDHR tuner fix

Download all attachments as: .zip

Change History (8)

comment:1 Changed 13 years ago by Raymond Wagner

Description: modified (diff)
Status: newassigned
Summary: Access to both tuners on HDHR fails when using manual IPTuner value not obeyed when using manual IP setting for HDHR

comment:2 Changed 13 years ago by Silicondust

Acknowledging the receipt of this bug report - will investigate.

Related - MythTV assumes 2 tuners per device whereas some HDHR models have 1 tuner (T1) or 3 tuners (CC).

comment:3 Changed 13 years ago by robertm

Status: assignedinfoneeded

Attaching what I think is the fix here. I don't own an HDHomeRun so I need someone to fix this.

In the lib resync that broke this, the following code was removed:

https://github.com/MythTV/mythtv/commit/baf36e4bab565413c8ca3a4738fc7cf51e0e8a4f#L10L159

Basically, hdhomerun_device_create_from_str_ip stopped even trying to parse the tuner number, and is just hardcoding the tuner number to 0, no matter what is passed to it. Meanwhile, in hdhomerun_device_create_from_str_device_id the tuner number is parsed properly:

if (sscanf(device_str, "%lx-%u", &device_id, &tuner) != 2) {

I have no idea if this was an intentional change, but it is still present in the current libhdhomerun code upstream.

The patch isn't 100% perfect, but I think it should work properly. I just need someone to test it... then we can figure out what patch we can get upstream.

Version 1, edited 13 years ago by robertm (previous) (next) (diff)

Changed 13 years ago by robertm

Attachment: hdhrtunerfix.diff added

Probable HDHR tuner fix

comment:4 Changed 13 years ago by Github

HDHomeRun: Fix capture failures when HDHR is defined by IP.

Around a year ago, SiliconDust? modified the hdhomerun_device_create_from_str_ip to hardcode tuner 0 rather than parse it from the device string. This appears to have been a simple oversight, since the equivalent string parsing function by device ID still does parse for the tuner number.

This restores the logic to parse for the tuner number rather than just unequivocally use tuner 0.

Refs #8193. Refs #9247.

Branch: master Changeset: 0b88b99c23c4c5152fb986163fe31d628ac60173

comment:5 Changed 13 years ago by robertm

Resolution: Fixed
Status: infoneededclosed

comment:6 Changed 13 years ago by Raymond Wagner

Milestone: unknown0.24.2

comment:7 Changed 12 years ago by stuartm

Milestone: 0.24.2

Milestone 0.24.2 deleted

Note: See TracTickets for help on using tickets.