* [PATCH] ath: fix incorrect PPB on FCC radar type 5 @ 2014-12-15 18:55 ` Peter Oh 0 siblings, 0 replies; 8+ messages in thread From: Peter Oh @ 2014-12-15 18:55 UTC (permalink / raw) To: ath10k; +Cc: linux-wireless The minimum number of pulses per burst on FCC radar type 5 is 1. Use this number for correct radar detection. Signed-off-by: Peter Oh <poh@qca.qualcomm.com> --- drivers/net/wireless/ath/dfs_pattern_detector.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/ath/dfs_pattern_detector.c b/drivers/net/wireless/ath/dfs_pattern_detector.c index cfd0554..3d57f87 100644 --- a/drivers/net/wireless/ath/dfs_pattern_detector.c +++ b/drivers/net/wireless/ath/dfs_pattern_detector.c @@ -86,7 +86,7 @@ static const struct radar_detector_specs fcc_radar_ref_types[] = { FCC_PATTERN(1, 0, 5, 150, 230, 1, 23), FCC_PATTERN(2, 6, 10, 200, 500, 1, 16), FCC_PATTERN(3, 11, 20, 200, 500, 1, 12), - FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 20), + FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 1), FCC_PATTERN(5, 0, 1, 333, 333, 1, 9), }; -- 1.9.1 _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH] ath: fix incorrect PPB on FCC radar type 5 @ 2014-12-15 18:55 ` Peter Oh 0 siblings, 0 replies; 8+ messages in thread From: Peter Oh @ 2014-12-15 18:55 UTC (permalink / raw) To: ath10k; +Cc: linux-wireless The minimum number of pulses per burst on FCC radar type 5 is 1. Use this number for correct radar detection. Signed-off-by: Peter Oh <poh@qca.qualcomm.com> --- drivers/net/wireless/ath/dfs_pattern_detector.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/ath/dfs_pattern_detector.c b/drivers/net/wireless/ath/dfs_pattern_detector.c index cfd0554..3d57f87 100644 --- a/drivers/net/wireless/ath/dfs_pattern_detector.c +++ b/drivers/net/wireless/ath/dfs_pattern_detector.c @@ -86,7 +86,7 @@ static const struct radar_detector_specs fcc_radar_ref_types[] = { FCC_PATTERN(1, 0, 5, 150, 230, 1, 23), FCC_PATTERN(2, 6, 10, 200, 500, 1, 16), FCC_PATTERN(3, 11, 20, 200, 500, 1, 12), - FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 20), + FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 1), FCC_PATTERN(5, 0, 1, 333, 333, 1, 9), }; -- 1.9.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] ath: fix incorrect PPB on FCC radar type 5 2014-12-15 18:55 ` Peter Oh @ 2014-12-15 20:42 ` Felix Fietkau -1 siblings, 0 replies; 8+ messages in thread From: Felix Fietkau @ 2014-12-15 20:42 UTC (permalink / raw) To: Peter Oh, ath10k; +Cc: linux-wireless On 2014-12-15 19:55, Peter Oh wrote: > The minimum number of pulses per burst on FCC radar type 5 is 1. > Use this number for correct radar detection. > > Signed-off-by: Peter Oh <poh@qca.qualcomm.com> > --- > drivers/net/wireless/ath/dfs_pattern_detector.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/wireless/ath/dfs_pattern_detector.c b/drivers/net/wireless/ath/dfs_pattern_detector.c > index cfd0554..3d57f87 100644 > --- a/drivers/net/wireless/ath/dfs_pattern_detector.c > +++ b/drivers/net/wireless/ath/dfs_pattern_detector.c > @@ -86,7 +86,7 @@ static const struct radar_detector_specs fcc_radar_ref_types[] = { > FCC_PATTERN(1, 0, 5, 150, 230, 1, 23), > FCC_PATTERN(2, 6, 10, 200, 500, 1, 16), > FCC_PATTERN(3, 11, 20, 200, 500, 1, 12), > - FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 20), > + FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 1), Won't that significantly increase the false positive rate? Here's the previous change on that pattern: commit 562930f8baee7a1cb5b9f382b4338c9b6fbf1139 Author: Vivek Natarajan <nataraja@qti.qualcomm.com> Date: Mon Nov 24 17:26:22 2014 +0530 ath: Fix a false radar detection pattern For FCC and JP, in one of the radar patterns, PPB and PRF seems to be interchanged leading to frequent incorrect radar detections. Signed-off-by: Vivek Natarajan <nataraja@qti.qualcomm.com> Signed-off-by: John W. Linville <linville@tuxdriver.com> diff --git a/drivers/net/wireless/ath/dfs_pattern_detector.c b/drivers/net/wireless/ath/dfs_pattern_detector.c index 650be79..cfd0554 100644 --- a/drivers/net/wireless/ath/dfs_pattern_detector.c +++ b/drivers/net/wireless/ath/dfs_pattern_detector.c @@ -86,7 +86,7 @@ static const struct radar_detector_specs fcc_radar_ref_types[] = { FCC_PATTERN(1, 0, 5, 150, 230, 1, 23), FCC_PATTERN(2, 6, 10, 200, 500, 1, 16), FCC_PATTERN(3, 11, 20, 200, 500, 1, 12), - FCC_PATTERN(4, 50, 100, 1000, 2000, 20, 1), + FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 20), FCC_PATTERN(5, 0, 1, 333, 333, 1, 9), }; @@ -105,7 +105,7 @@ static const struct radar_detector_specs jp_radar_ref_types[] = { JP_PATTERN(4, 0, 5, 150, 230, 1, 23), JP_PATTERN(5, 6, 10, 200, 500, 1, 16), JP_PATTERN(6, 11, 20, 200, 500, 1, 12), - JP_PATTERN(7, 50, 100, 1000, 2000, 20, 1), + JP_PATTERN(7, 50, 100, 1000, 2000, 1, 20), JP_PATTERN(5, 0, 1, 333, 333, 1, 9), }; _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] ath: fix incorrect PPB on FCC radar type 5 @ 2014-12-15 20:42 ` Felix Fietkau 0 siblings, 0 replies; 8+ messages in thread From: Felix Fietkau @ 2014-12-15 20:42 UTC (permalink / raw) To: Peter Oh, ath10k; +Cc: linux-wireless On 2014-12-15 19:55, Peter Oh wrote: > The minimum number of pulses per burst on FCC radar type 5 is 1. > Use this number for correct radar detection. > > Signed-off-by: Peter Oh <poh@qca.qualcomm.com> > --- > drivers/net/wireless/ath/dfs_pattern_detector.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/wireless/ath/dfs_pattern_detector.c b/drivers/net/wireless/ath/dfs_pattern_detector.c > index cfd0554..3d57f87 100644 > --- a/drivers/net/wireless/ath/dfs_pattern_detector.c > +++ b/drivers/net/wireless/ath/dfs_pattern_detector.c > @@ -86,7 +86,7 @@ static const struct radar_detector_specs fcc_radar_ref_types[] = { > FCC_PATTERN(1, 0, 5, 150, 230, 1, 23), > FCC_PATTERN(2, 6, 10, 200, 500, 1, 16), > FCC_PATTERN(3, 11, 20, 200, 500, 1, 12), > - FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 20), > + FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 1), Won't that significantly increase the false positive rate? Here's the previous change on that pattern: commit 562930f8baee7a1cb5b9f382b4338c9b6fbf1139 Author: Vivek Natarajan <nataraja@qti.qualcomm.com> Date: Mon Nov 24 17:26:22 2014 +0530 ath: Fix a false radar detection pattern For FCC and JP, in one of the radar patterns, PPB and PRF seems to be interchanged leading to frequent incorrect radar detections. Signed-off-by: Vivek Natarajan <nataraja@qti.qualcomm.com> Signed-off-by: John W. Linville <linville@tuxdriver.com> diff --git a/drivers/net/wireless/ath/dfs_pattern_detector.c b/drivers/net/wireless/ath/dfs_pattern_detector.c index 650be79..cfd0554 100644 --- a/drivers/net/wireless/ath/dfs_pattern_detector.c +++ b/drivers/net/wireless/ath/dfs_pattern_detector.c @@ -86,7 +86,7 @@ static const struct radar_detector_specs fcc_radar_ref_types[] = { FCC_PATTERN(1, 0, 5, 150, 230, 1, 23), FCC_PATTERN(2, 6, 10, 200, 500, 1, 16), FCC_PATTERN(3, 11, 20, 200, 500, 1, 12), - FCC_PATTERN(4, 50, 100, 1000, 2000, 20, 1), + FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 20), FCC_PATTERN(5, 0, 1, 333, 333, 1, 9), }; @@ -105,7 +105,7 @@ static const struct radar_detector_specs jp_radar_ref_types[] = { JP_PATTERN(4, 0, 5, 150, 230, 1, 23), JP_PATTERN(5, 6, 10, 200, 500, 1, 16), JP_PATTERN(6, 11, 20, 200, 500, 1, 12), - JP_PATTERN(7, 50, 100, 1000, 2000, 20, 1), + JP_PATTERN(7, 50, 100, 1000, 2000, 1, 20), JP_PATTERN(5, 0, 1, 333, 333, 1, 9), }; ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] ath: fix incorrect PPB on FCC radar type 5 2014-12-15 20:42 ` Felix Fietkau @ 2014-12-15 23:38 ` Peter Oh -1 siblings, 0 replies; 8+ messages in thread From: Peter Oh @ 2014-12-15 23:38 UTC (permalink / raw) To: Felix Fietkau, ath10k; +Cc: linux-wireless On 12/15/2014 12:42 PM, Felix Fietkau wrote: > On 2014-12-15 19:55, Peter Oh wrote: >> The minimum number of pulses per burst on FCC radar type 5 is 1. >> Use this number for correct radar detection. >> >> Signed-off-by: Peter Oh <poh@qca.qualcomm.com> >> --- >> drivers/net/wireless/ath/dfs_pattern_detector.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/net/wireless/ath/dfs_pattern_detector.c b/drivers/net/wireless/ath/dfs_pattern_detector.c >> index cfd0554..3d57f87 100644 >> --- a/drivers/net/wireless/ath/dfs_pattern_detector.c >> +++ b/drivers/net/wireless/ath/dfs_pattern_detector.c >> @@ -86,7 +86,7 @@ static const struct radar_detector_specs fcc_radar_ref_types[] = { >> FCC_PATTERN(1, 0, 5, 150, 230, 1, 23), >> FCC_PATTERN(2, 6, 10, 200, 500, 1, 16), >> FCC_PATTERN(3, 11, 20, 200, 500, 1, 12), >> - FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 20), >> + FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 1), > Won't that significantly increase the false positive rate? No. FCC type 5 radar's PPB is in between 1~3 and DFS simulators such as VeriWave machine pick one among the number (typically 1), hence to detect the type 5 radar we should pick the minimum number which is 1 unless we won't detect the radar if pulse comes only once. > Here's the previous change on that pattern: Actually their assumption (?) on FCC is incorrect. PPB and PRF was not interchanged, the previous parameters simply picked wrong PRF for FCC instead (FCC type 5 radar's PRF is 1). For Japan those parameters look quite different from what I've seen from Japan's Communications Ministry (MIC, www.tele.soumu.go.jp/e/sys/equ/tech/5ghz/5ghz.htm), but I couldn't say if it's correct or not at this moment since I haven't run DFS on Japan yet. > commit 562930f8baee7a1cb5b9f382b4338c9b6fbf1139 > Author: Vivek Natarajan <nataraja@qti.qualcomm.com> > Date: Mon Nov 24 17:26:22 2014 +0530 > > ath: Fix a false radar detection pattern > > For FCC and JP, in one of the radar patterns, PPB and PRF seems to be > interchanged leading to frequent incorrect radar detections. > > Signed-off-by: Vivek Natarajan <nataraja@qti.qualcomm.com> > Signed-off-by: John W. Linville <linville@tuxdriver.com> > > diff --git a/drivers/net/wireless/ath/dfs_pattern_detector.c b/drivers/net/wireless/ath/dfs_pattern_detector.c > index 650be79..cfd0554 100644 > --- a/drivers/net/wireless/ath/dfs_pattern_detector.c > +++ b/drivers/net/wireless/ath/dfs_pattern_detector.c > @@ -86,7 +86,7 @@ static const struct radar_detector_specs fcc_radar_ref_types[] = { > FCC_PATTERN(1, 0, 5, 150, 230, 1, 23), > FCC_PATTERN(2, 6, 10, 200, 500, 1, 16), > FCC_PATTERN(3, 11, 20, 200, 500, 1, 12), > - FCC_PATTERN(4, 50, 100, 1000, 2000, 20, 1), > + FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 20), > FCC_PATTERN(5, 0, 1, 333, 333, 1, 9), > }; > > @@ -105,7 +105,7 @@ static const struct radar_detector_specs jp_radar_ref_types[] = { > JP_PATTERN(4, 0, 5, 150, 230, 1, 23), > JP_PATTERN(5, 6, 10, 200, 500, 1, 16), > JP_PATTERN(6, 11, 20, 200, 500, 1, 12), > - JP_PATTERN(7, 50, 100, 1000, 2000, 20, 1), > + JP_PATTERN(7, 50, 100, 1000, 2000, 1, 20), > JP_PATTERN(5, 0, 1, 333, 333, 1, 9), > }; _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ath: fix incorrect PPB on FCC radar type 5 @ 2014-12-15 23:38 ` Peter Oh 0 siblings, 0 replies; 8+ messages in thread From: Peter Oh @ 2014-12-15 23:38 UTC (permalink / raw) To: Felix Fietkau, ath10k; +Cc: linux-wireless On 12/15/2014 12:42 PM, Felix Fietkau wrote: > On 2014-12-15 19:55, Peter Oh wrote: >> The minimum number of pulses per burst on FCC radar type 5 is 1. >> Use this number for correct radar detection. >> >> Signed-off-by: Peter Oh <poh@qca.qualcomm.com> >> --- >> drivers/net/wireless/ath/dfs_pattern_detector.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/net/wireless/ath/dfs_pattern_detector.c b/drivers/net/wireless/ath/dfs_pattern_detector.c >> index cfd0554..3d57f87 100644 >> --- a/drivers/net/wireless/ath/dfs_pattern_detector.c >> +++ b/drivers/net/wireless/ath/dfs_pattern_detector.c >> @@ -86,7 +86,7 @@ static const struct radar_detector_specs fcc_radar_ref_types[] = { >> FCC_PATTERN(1, 0, 5, 150, 230, 1, 23), >> FCC_PATTERN(2, 6, 10, 200, 500, 1, 16), >> FCC_PATTERN(3, 11, 20, 200, 500, 1, 12), >> - FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 20), >> + FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 1), > Won't that significantly increase the false positive rate? No. FCC type 5 radar's PPB is in between 1~3 and DFS simulators such as VeriWave machine pick one among the number (typically 1), hence to detect the type 5 radar we should pick the minimum number which is 1 unless we won't detect the radar if pulse comes only once. > Here's the previous change on that pattern: Actually their assumption (?) on FCC is incorrect. PPB and PRF was not interchanged, the previous parameters simply picked wrong PRF for FCC instead (FCC type 5 radar's PRF is 1). For Japan those parameters look quite different from what I've seen from Japan's Communications Ministry (MIC, www.tele.soumu.go.jp/e/sys/equ/tech/5ghz/5ghz.htm), but I couldn't say if it's correct or not at this moment since I haven't run DFS on Japan yet. > commit 562930f8baee7a1cb5b9f382b4338c9b6fbf1139 > Author: Vivek Natarajan <nataraja@qti.qualcomm.com> > Date: Mon Nov 24 17:26:22 2014 +0530 > > ath: Fix a false radar detection pattern > > For FCC and JP, in one of the radar patterns, PPB and PRF seems to be > interchanged leading to frequent incorrect radar detections. > > Signed-off-by: Vivek Natarajan <nataraja@qti.qualcomm.com> > Signed-off-by: John W. Linville <linville@tuxdriver.com> > > diff --git a/drivers/net/wireless/ath/dfs_pattern_detector.c b/drivers/net/wireless/ath/dfs_pattern_detector.c > index 650be79..cfd0554 100644 > --- a/drivers/net/wireless/ath/dfs_pattern_detector.c > +++ b/drivers/net/wireless/ath/dfs_pattern_detector.c > @@ -86,7 +86,7 @@ static const struct radar_detector_specs fcc_radar_ref_types[] = { > FCC_PATTERN(1, 0, 5, 150, 230, 1, 23), > FCC_PATTERN(2, 6, 10, 200, 500, 1, 16), > FCC_PATTERN(3, 11, 20, 200, 500, 1, 12), > - FCC_PATTERN(4, 50, 100, 1000, 2000, 20, 1), > + FCC_PATTERN(4, 50, 100, 1000, 2000, 1, 20), > FCC_PATTERN(5, 0, 1, 333, 333, 1, 9), > }; > > @@ -105,7 +105,7 @@ static const struct radar_detector_specs jp_radar_ref_types[] = { > JP_PATTERN(4, 0, 5, 150, 230, 1, 23), > JP_PATTERN(5, 6, 10, 200, 500, 1, 16), > JP_PATTERN(6, 11, 20, 200, 500, 1, 12), > - JP_PATTERN(7, 50, 100, 1000, 2000, 20, 1), > + JP_PATTERN(7, 50, 100, 1000, 2000, 1, 20), > JP_PATTERN(5, 0, 1, 333, 333, 1, 9), > }; ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ath: fix incorrect PPB on FCC radar type 5 2014-12-15 18:55 ` Peter Oh @ 2014-12-24 15:29 ` Kalle Valo -1 siblings, 0 replies; 8+ messages in thread From: Kalle Valo @ 2014-12-24 15:29 UTC (permalink / raw) To: Peter Oh; +Cc: linux-wireless, ath10k Peter Oh <poh@qca.qualcomm.com> writes: > The minimum number of pulses per burst on FCC radar type 5 is 1. > Use this number for correct radar detection. > > Signed-off-by: Peter Oh <poh@qca.qualcomm.com> Thanks, applied to wireless-drivers-next.git. -- Kalle Valo _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ath: fix incorrect PPB on FCC radar type 5 @ 2014-12-24 15:29 ` Kalle Valo 0 siblings, 0 replies; 8+ messages in thread From: Kalle Valo @ 2014-12-24 15:29 UTC (permalink / raw) To: Peter Oh; +Cc: ath10k, linux-wireless Peter Oh <poh@qca.qualcomm.com> writes: > The minimum number of pulses per burst on FCC radar type 5 is 1. > Use this number for correct radar detection. > > Signed-off-by: Peter Oh <poh@qca.qualcomm.com> Thanks, applied to wireless-drivers-next.git. -- Kalle Valo ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2014-12-24 15:29 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-12-15 18:55 [PATCH] ath: fix incorrect PPB on FCC radar type 5 Peter Oh 2014-12-15 18:55 ` Peter Oh 2014-12-15 20:42 ` Felix Fietkau 2014-12-15 20:42 ` Felix Fietkau 2014-12-15 23:38 ` Peter Oh 2014-12-15 23:38 ` Peter Oh 2014-12-24 15:29 ` Kalle Valo 2014-12-24 15:29 ` Kalle Valo
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.