[asterisk-dev] [Code Review] Dahdi FXS line polarity reversal when remote party Answers and/or Hangups

[rmudgett at digium.com]

> On 2010-07-22 16:59:35, rmudgett wrote:
> > Other than the quibble below, I do not see anything.  Although, I have not spent much time in sig_analog code.

Forgot to mention that the sample config file should be updated to document the new functionality of the answeronpolarityswitch and hanguponpolarityswitch options.


- rmudgett


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/797/#review2451
-----------------------------------------------------------


On 2010-07-22 16:06:38, Alec Davis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/797/
> -----------------------------------------------------------
> 
> (Updated 2010-07-22 16:06:38)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> Quoting armeniki:
> "FXS lines normally connect to a telephone. However, when FXS lines are routed to an external PBX or Key System to act as "external" or "CO" lines, it is extremely difficult, if not impossible for the external PBX to know when the call has been disconnected without receiving a polarity reversal on the line."
> 
> Implementation was to use existing keywords as used for FXO modules 'answeronpolityswitch' and 'hanguponpolarityswitch', with a similar function for the FXS module but from the Legacy PBX's point of view.
> 
> 
> This addresses bug 17318.
>     https://issues.asterisk.org/view.php?id=17318
> 
> 
> Diffs
> -----
> 
>   trunk/channels/chan_dahdi.c 278424 
>   trunk/channels/sig_analog.h 278424 
>   trunk/channels/sig_analog.c 278424 
> 
> Diff: https://reviewboard.asterisk.org/r/797/diff
> 
> 
> Testing
> -------
> 
> Using TDM800P FXS modules:
> 
> Inbound and outbound calls, using answeronpolarityswitch and hanguponpolarityswitch 
> 
> 
> Thanks,
> 
> Alec
> 
>