提交 a8d12007 编写于 作者: A Alan Cox 提交者: Linus Torvalds

n_gsm: Fix timings

Alek Du reported that the code erroneously applies time to jiffies
conversions twice to the t1 and t2 values. In normal use on a modem link
this cases no visible problem but on a slower link it will break as with
HZ=1000 as is typical we are running t1/t2 ten times too fast.

Alek's original patch removed the conversion from the timer setting but we
in fact have to be more careful as the contents of t1/t2 are visible via
the device API and we thus need to correct the constants.
Signed-off-by: NAlan Cox <alan@linux.intel.com>
Signed-off-by: NLinus Torvalds <torvalds@linux-foundation.org>
上级 1ea6b8f4
...@@ -66,14 +66,16 @@ ...@@ -66,14 +66,16 @@
static int debug; static int debug;
module_param(debug, int, 0600); module_param(debug, int, 0600);
#define T1 (HZ/10) /* Defaults: these are from the specification */
#define T2 (HZ/3)
#define N2 3 #define T1 10 /* 100mS */
#define T2 34 /* 333mS */
#define N2 3 /* Retry 3 times */
/* Use long timers for testing at low speed with debug on */ /* Use long timers for testing at low speed with debug on */
#ifdef DEBUG_TIMING #ifdef DEBUG_TIMING
#define T1 HZ #define T1 100
#define T2 (2 * HZ) #define T2 200
#endif #endif
/* /*
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册