[prev in list] [next in list] [prev in thread] [next in thread] 

List:       mythtv-dev
Subject:    [mythtv] Re: [mythtv-commits] Re: Ticket #610: Add a seperate
From:       "Michael T. Dean" <mtdean () thirdcontact ! com>
Date:       2005-11-30 17:48:06
Message-ID: 438DE5D6.2020906 () thirdcontact ! com
[Download RAW message or body]

MythTV wrote:

>#610: Add a seperate frequency list for US-Cable-IRC
>-----------------------+----------------------------------------------------
> Reporter:  anonymous  |        Owner:  ijr   
>     Type:  patch      |       Status:  closed
> Priority:  minor      |    Milestone:        
>Component:  mythtv     |      Version:        
> Severity:  low        |   Resolution:  fixed 
>-----------------------+----------------------------------------------------
>Comment (by anonymous):
>
> The frequencies in the patch are all still low by 125 Khz. This is so
> trivial to fix that it's silly not to. c.f. -
> http://www.jneuhaus.com/fccindex/cablech.html
>  
>
Anonymous,

I'm sending this directly to you...  Oh, wait.  I guess I can't.  I just 
hope you read the lists...

First, most channels are off by 12.5kHz, not by 125kHz.  Second, when I 
edited the 0.18.1 patch that jrandall posted, I left the values as they 
were even though I know they were wrong.  I also explicitly mentioned 
the fact that the frequencies are off (see my 11/09/05 comment on the 
ticket).  As a matter of fact, I posted the same link you've quoted in 
an on-list discussion of the patch ( 
http://www.gossamer-threads.com/lists/mythtv/dev/159917#159917 ), and 
Daniel even added the link as a comment above the IRC frequency listing.

I wasn't willing to just change the values because I don't have RF 
modulated sources, and, therefore no IRC source, to test with.  Since 
jrandall posted a configuration that worked for him, I figured a 
known-working, tested patch would be much better than an 
"academically-correct" patch.  Therefore, I'm taking your addition to 
the ticket to be your volunteering to test your IRC source with an 
"academically-correct" IRC frequency listing...

So, are you mentioning the frequencies are off simply because they're 
not as published or is it actually causing you problems?  Your tuner's 
built-in fine-tuning mechanism should automatically find the appropriate 
center frequency even when the channel is off by 12.5kHz, so there 
shouldn't be any user-visible effect of changing the values.  If you're 
the same anonymous that posted the original ticket, then you may be 
experiencing audio problems.  If so, saying so would be very useful.  
There are too many people named "anonymous" posting tickets on Trac, so 
it's hard to tell you all apart.

However, it's rather unlikely that the frequency difference is causing 
audio problems.  Since a large number of Myth users are using the 
ntsc_cable (us-cable) frequency list, we've had a large number of 
testers of that list.  However, on that list, channels 14-16, 25-41 and 
43-53 are off by the same 12.5kHz and 98 and 99 are off by 25kHz, and I 
don't remember seeing any reports of audio problems on these channels 
for us-cable users that were traced back to the frequency list 
(otherwise, the frequency list probably would have been changed).  As a 
matter of fact, most of the HRC channel frequencies are off by 5-15kHz, 
and they also seem to work.

The attached patch corrects the frequencies (and even leaves 42 as is 
and corrects the 25kHz offset on 98 and 99).  Please test the patch and 
report back whether or not it works.  If it works, I'd be happy to post 
ths patch for inclusion.  Oh, and, yes, I know that we're still 
0.0005MHz (=0.5kHz=500Hz) low, but try representing 500Hz as an integer 
number of kHz.  I truncated instead of rounding up, but it shouldn't matter.

Mike

["mythtv-irc_frequency_offset.patch" (text/plain)]

Index: libs/libmythtv/frequencies.c
===================================================================
--- libs/libmythtv/frequencies.c	(revision 8088)
+++ libs/libmythtv/frequencies.c	(working copy)
@@ -379,134 +379,134 @@
 
 /** US IRC http://www.jneuhaus.com/fccindex/cablech.html */
 static struct CHANLIST ntsc_irc[] = {
-    { "1",	 73250 },
-    { "2",	 55250 },
-    { "3",	 61250 },
-    { "4",	 67250 },
-    { "5",	 79250 },
-    { "6",	 85250 },
-    { "7",	175250 },
-    { "8",	181250 },
-    { "9",	187250 },
-    { "10",	193250 },
-    { "11",	199250 },
-    { "12",	205250 },
+    { "1",	 73262 },
+    { "2",	 55262 },
+    { "3",	 61262 },
+    { "4",	 67262 },
+    { "5",	 79262 },
+    { "6",	 85262 },
+    { "7",	175262 },
+    { "8",	181262 },
+    { "9",	187262 },
+    { "10",	193262 },
+    { "11",	199262 },
+    { "12",	205262 },
 
-    { "13",	211250 },
-    { "14",	121250 },
-    { "15",	127250 },
-    { "16",	133250 },
-    { "17",	139250 },
-    { "18",	145250 },
-    { "19",	151250 },
-    { "20",	157250 },
+    { "13",	211262 },
+    { "14",	121262 },
+    { "15",	127262 },
+    { "16",	133262 },
+    { "17",	139262 },
+    { "18",	145262 },
+    { "19",	151262 },
+    { "20",	157262 },
  
-    { "21",	163250 },
-    { "22",	169250 },
-    { "23",	217250 },
-    { "24",	223250 },
-    { "25",	229250 },
-    { "26",	235250 },
-    { "27",	241250 },
-    { "28",	247250 },
-    { "29",	253250 },
-    { "30",	259250 },
-    { "31",	265250 },
-    { "32",	271250 },
-    { "33",	277250 },
-    { "34",	283250 },
-    { "35",	289250 },
-    { "36",	295250 },
-    { "37",	301250 },
-    { "38",	307250 },
-    { "39",	313250 },
-    { "40",	319250 },
-    { "41",	325250 },
+    { "21",	163262 },
+    { "22",	169262 },
+    { "23",	217262 },
+    { "24",	223262 },
+    { "25",	229262 },
+    { "26",	235262 },
+    { "27",	241262 },
+    { "28",	247262 },
+    { "29",	253262 },
+    { "30",	259262 },
+    { "31",	265262 },
+    { "32",	271262 },
+    { "33",	277262 },
+    { "34",	283262 },
+    { "35",	289262 },
+    { "36",	295262 },
+    { "37",	301262 },
+    { "38",	307262 },
+    { "39",	313262 },
+    { "40",	319262 },
+    { "41",	325262 },
     { "42",	331250 },
-    { "43",	337250 },
-    { "44",	343250 },
-    { "45",	349250 },
-    { "46",	355250 },
-    { "47",	361250 },
-    { "48",	367250 },
-    { "49",	373250 },
-    { "50",	379250 },
-    { "51",	385250 },
-    { "52",	391250 },
-    { "53",	397250 },
-    { "54",	403250 },
-    { "55",	409250 },
-    { "56",	415250 },
-    { "57",	421250 },
-    { "58",	427250 },
-    { "59",	433250 },
-    { "60",	439250 },
-    { "61",	445250 },
-    { "62",	451250 },
-    { "63",	457250 },
-    { "64",	463250 },
-    { "65",	469250 },
-    { "66",	475250 },
-    { "67",	481250 },
-    { "68",	487250 },
-    { "69",	493250 },
+    { "43",	337262 },
+    { "44",	343262 },
+    { "45",	349262 },
+    { "46",	355262 },
+    { "47",	361262 },
+    { "48",	367262 },
+    { "49",	373262 },
+    { "50",	379262 },
+    { "51",	385262 },
+    { "52",	391262 },
+    { "53",	397262 },
+    { "54",	403262 },
+    { "55",	409262 },
+    { "56",	415262 },
+    { "57",	421262 },
+    { "58",	427262 },
+    { "59",	433262 },
+    { "60",	439262 },
+    { "61",	445262 },
+    { "62",	451262 },
+    { "63",	457262 },
+    { "64",	463262 },
+    { "65",	469262 },
+    { "66",	475262 },
+    { "67",	481262 },
+    { "68",	487262 },
+    { "69",	493262 },
  
-    { "70",	499250 },
-    { "71",	505250 },
-    { "72",	511250 },
-    { "73",	517250 },
-    { "74",	523250 },
-    { "75",	529250 },
-    { "76",	535250 },
-    { "77",	541250 },
-    { "78",	547250 },
-    { "79",	553250 },
-    { "80",	559250 },
-    { "81",	565250 },
-    { "82",	571250 },
-    { "83",	577250 },
-    { "84",	583250 },
-    { "85",	589250 },
-    { "86",	595250 },
-    { "87",	601250 },
-    { "88",	607250 },
-    { "89",	613250 },
-    { "90",	619250 },
-    { "91",	625250 },
-    { "92",	631250 },
-    { "93",	637250 },
-    { "94",	643250 },
-    { "95",	 91250 },
-    { "96",	 97250 },
-    { "97",	103250 },
-    { "98",	109250 },
-    { "99",	115250 },
-    { "100",	649250 },
-    { "101",	655250 },
-    { "102",	661250 },
-    { "103",	667250 },
-    { "104",	673250 },
-    { "105",	679250 },
-    { "106",	685250 },
-    { "107",	691250 },
-    { "108",	697250 },
-    { "109",	703250 },
-    { "110",	709250 },
-    { "111",	715250 },
-    { "112",	721250 },
-    { "113",	727250 },
-    { "114",	733250 },
-    { "115",	739250 },
-    { "116",	745250 },
-    { "117",	751250 },
-    { "118",	757250 },
-    { "119",	763250 },
-    { "120",	769250 },
-    { "121",	775250 },
-    { "122",	781250 },
-    { "123",	787250 },
-    { "124",	793250 },
-    { "125",	799250 },
+    { "70",	499262 },
+    { "71",	505262 },
+    { "72",	511262 },
+    { "73",	517262 },
+    { "74",	523262 },
+    { "75",	529262 },
+    { "76",	535262 },
+    { "77",	541262 },
+    { "78",	547262 },
+    { "79",	553262 },
+    { "80",	559262 },
+    { "81",	565262 },
+    { "82",	571262 },
+    { "83",	577262 },
+    { "84",	583262 },
+    { "85",	589262 },
+    { "86",	595262 },
+    { "87",	601262 },
+    { "88",	607262 },
+    { "89",	613262 },
+    { "90",	619262 },
+    { "91",	625262 },
+    { "92",	631262 },
+    { "93",	637262 },
+    { "94",	643262 },
+    { "95",	 91262 },
+    { "96",	 97262 },
+    { "97",	103262 },
+    { "98",	109275 },
+    { "99",	115275 },
+    { "100",	649262 },
+    { "101",	655262 },
+    { "102",	661262 },
+    { "103",	667262 },
+    { "104",	673262 },
+    { "105",	679262 },
+    { "106",	685262 },
+    { "107",	691262 },
+    { "108",	697262 },
+    { "109",	703262 },
+    { "110",	709262 },
+    { "111",	715262 },
+    { "112",	721262 },
+    { "113",	727262 },
+    { "114",	733262 },
+    { "115",	739262 },
+    { "116",	745262 },
+    { "117",	751262 },
+    { "118",	757262 },
+    { "119",	763262 },
+    { "120",	769262 },
+    { "121",	775262 },
+    { "122",	781262 },
+    { "123",	787262 },
+    { "124",	793262 },
+    { "125",	799262 },
  
     { "T7", 	  8250 },
     { "T8",	 14250 },


_______________________________________________
mythtv-dev mailing list
mythtv-dev@mythtv.org
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic