[asterisk-bugs] [Zaptel 0009264]: [patch] wctdm sometimes misses callerid

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=9264 
====================================================================== 
Reported By:                mjagdis
Assigned To:                kpfleming
====================================================================== 
Project:                    Zaptel
Issue ID:                   9264
Category:                   wctdm
Reproducibility:            sometimes
Severity:                   minor
Priority:                   normal
Status:                     closed
Zaptel Version:             I did not set the version :( 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!):  
Disclaimer on File?:        Yes 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             03-13-2007 08:12 CDT
Last Modified:              06-07-2008 13:47 CDT
====================================================================== 
Summary:                    [patch] wctdm sometimes misses callerid
Description: 
When caller ID is sent before the first ring and preceeded by a polarity
reversal (e.g. UK) the reversal sometimes causes the wctdm to report a
"phantom" ring which aborts/prevents the read of the callerid.

This is documented in the Si3050 specs. The recommended method of handling
ring recognition in the presence of polarity reverals is to put the ring
detector in full-wave rectification mode and watch for a leading _and_
trailing edge in the received ring tone.

The attached patch implements the recommended method. Without it I could
trigger a missed clid in just 3 or 4 attempts. With it I can't trigger a
missed clid at all.
====================================================================== 

---------------------------------------------------------------------- 
 svnbot - 06-07-08 13:47  
---------------------------------------------------------------------- 
Repository: dahdi
Revision: 3701

U   branches/1.4/wctdm.c
U   branches/1.4/wctdm24xxp/base.c
U   branches/1.4/wctdm24xxp/wctdm24xxp.h

------------------------------------------------------------------------
r3701 | dbailey | 2008-06-07 13:47:48 -0500 (Sat, 07 Jun 2008) | 7 lines

Added modified patch so that the ring detect method can be set using a
module parameter.  
Setting fwringdetect=1 enables the method submitted by this user. 

(closes issue http://bugs.digium.com/view.php?id=9264)
Reported by: mjagdis
Tested by: benbrown

------------------------------------------------------------------------

http://svn.digium.com/view/dahdi?view=rev&revision=3701 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
06-07-08 13:47  svnbot         Checkin                                      
06-07-08 13:47  svnbot         Note Added: 0088364                          
======================================================================