Re: [PATCH] wireless-regdb: Fix ranges of EU countries as they are harmonized since 2014...

Seth Forshee
2019-09-07T23:56:41Z

On Mon, Aug 05, 2019 at 04:19:16PM +0200, Emil Petersky wrote:
> This patch unites entries for EU countries, as they have been harmonized
> latest by July 2014...
> EU decision 2005/513/EC:
> https://eur-lex.europa.eu/legal-content/EN/TXT/?uri=CELEX:02005D0513-20070213
> EU decision 2006/771/EC:
> https://eur-lex.europa.eu/legal-content/EN/TXT/?uri=CELEX:02008D0432-20080611
>
> Signed-off-by: Emil Petersky <emil.petersky@streamunlimited.com>

Thanks for this patch, and especially for all the references you
provided. Sorry it's taken a while to get back -- I've been checking the
changes against the links which took quite some time, and I have pretty
limited time for reviewing these patches.

Overall this looks good, however when I try to apply it I get an error
that the patch is corrupt. Can you try resending?

I also get warnings from git about trailing whitespace, and I've noted a
couple other trivial whitespace issues below, if you wouldn't mind
fixing those up before resending.

> @@ -167,23 +185,30 @@ country BF: DFS-FCC
> #
> # Note: The transmit power limits in the 5250-5350 MHz and 5470-5725 MHz bands
> # can be raised by 3 dBm if TPC is enabled. Refer to BDS EN 301 893 for details.
> +#
> +# BG as part of EU/CEPT accepted decisions 2005/513/EC (5GHz RLAN, EN 301 893)
> +# and 2006/771/EC (amended by 2008/432/EC, Short-Range Devices, EN 300 440)
> +# EU decision 2005/513/EC: https://eur-lex.europa.eu/legal-content/EN/TXT/?uri=CELEX:02005D0513-20070213
> +# EU decision 2006/771/EC: https://eur-lex.europa.eu/legal-content/EN/TXT/?uri=CELEX:02008D0432-20080611
> +# BG: https://crc.bg/files/_en/Electronic_Communications_Revised_EN1.pdf
> +# BG: acceptance of 2006/771/EC https://crc.bg/files/Pravila_06_12_2018.pdf
> +
> country BG: DFS-ETSI

This is the only place where you added a blank line between the comment
and the country. Let's remove that for consistency.

> # Wideband data transmission systems (WDTS) in the 2.4GHz ISM band, ref:
> # I.22 of the List, BDS EN 300 328
> - (2402 - 2482 @ 40), (20)
> + (2400 - 2483.5 @ 40), (100 mW)
> # 5 GHz Radio Local Area Networks (RLANs), ref:
> # II.H01 of the List, BDS EN 301 893
> - (5170 - 5250 @ 80), (23), AUTO-BW, wmmrule=ETSI
> - (5250 - 5330 @ 80), (20), DFS, AUTO-BW, wmmrule=ETSI
> + (5150 - 5250 @ 80), (200 mW), NO-OUTDOOR, AUTO-BW, wmmrule=ETSI
> + (5250 - 5350 @ 80), (100 mW), NO-OUTDOOR, DFS, AUTO-BW, wmmrule=ETSI
> # II.H01 of the List, I.54 from the List, BDS EN 301 893
> - (5490 - 5710 @ 160), (27), DFS, wmmrule=ETSI
> - # Short range devices (SRDs) in the 5725-5875 MHz frequency range, ref:
> + (5470 - 5725 @ 160), (500 mW), DFS, wmmrule=ETSI
> + # short range devices (ETSI EN 300 440-1)
> # I.43 of the List, BDS EN 300 440-2, BDS EN 300 440-1
> - (5725 - 5875 @ 80), (14)
> - # 60 GHz Multiple-Gigabit RLAN Systems, ref:
> + (5725 - 5875 @ 80), (25 mW)
> + # 60 GHz band channels 1-4 (ETSI EN 302 567)
> # II.H03 of the List, BDS EN 302 567-2
> - (57000 - 66000 @ 2160), (40), NO-OUTDOOR
> -
> + (57000 - 66000 @ 2160), (40)
> country BH: DFS-JP

You removed the blank line between the BG rules and BH here.

Thanks,
Seth