On Mon, May 26, 2014 at 06:01:44PM +0530, chaitanya.mgit@xxxxxxxxx wrote:
> Generalize the power conversion from mW to dBm
> using log. This should fix the below compilation
> error for country NO which adds a new power value
> 2000mW which is not handled earlier.
>
> CC [M] net/wireless/wext-sme.o
> CC [M] net/wireless/regdb.o
> net/wireless/regdb.c:1130:1: error: Unknown undeclared here (not in
> a function)
> net/wireless/regdb.c:1130:9: error: expected } before power
> make[2]: *** [net/wireless/regdb.o] Error 1
> make[1]: *** [net/wireless] Error 2
> make: *** [net] Error 2
>
> Reported-By: John Walker <john@xxxxxxxx>
> Signed-off-by: Chaitanya T K <chaitanya.mgit@xxxxxxxxx>
>
> ---
> net/wireless/genregdb.awk | 12 +-----------
> 1 file changed, 1 insertion(+), 11 deletions(-)
>
> diff --git a/net/wireless/genregdb.awk b/net/wireless/genregdb.awk
> index 4119833..949ee38 100644
> --- a/net/wireless/genregdb.awk
> +++ b/net/wireless/genregdb.awk
> @@ -68,17 +68,7 @@ function parse_reg_rule()
> sub(/,/, "", units)
> dfs_cac = $9
> if (units == "mW") {
> - if (power == 100) {
> - power = 20
> - } else if (power == 200) {
> - power = 23
> - } else if (power == 500) {
> - power = 27
> - } else if (power == 1000) {
> - power = 30
> - } else {
> - print "Unknown power value in database!"
> - }
> + power = 10 * (log(power)/log(10))
> } else {
> dfs_cac = $8
> }
I'm not fond of the unnecessary parenthesis. But otherwise it seems great.
Acked-by: John W. Linville <linville@xxxxxxxxxxxxx>
--
John W. Linville Someday the world will need a hero, and you
linville@xxxxxxxxxxxxx might be all we have. Be ready.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
