[Buildroot] [PATCH 1/1] <wireless-regdb>: Adds "regulatory.db[.p7s]" to /lib/firmware
thomas.petazzoni at bootlin.com
Wed Jul 18 11:18:39 UTC 2018
On Mon, 16 Jul 2018 18:13:03 +0300, Christoph Engelbert wrote:
> With kernel version 4.15, Linux started to support loading regulatory.db and regulatory.db.p7s files
> directly from /lib/firmware, without the need of crda. If the kernel can't load those files it'll
> fallback to using crda, but the direct way is preferred.
> This patch copies those those files under /lib/firmware. The use of crda shouldn't be necessary with
> kernels of 4.15 or later.
> Signed-off-by: Christoph Engelbert <me at noctarius.com>
> package/wireless-regdb/wireless-regdb.mk | 4 ++++
> 1 file changed, 4 insertions(+)
Thanks for your contribution, I've applied your patch. However, I would
like to recommend a few improvements for your future contributions, if
- No need to enclose the package name between < and > in the commit
title, i.e just "package: description" is enough.
- When sending new iterations, please change the patch subject prefix
to PATCH v2, PATCH v3, etc. You can do that with "git format-patch
-v2" or git format-patch --subject-prefix="PATCH v2"
- Also when sending new iterations, adding a changelog below the "---"
sign is very nice, because it allows the people reviewing your patch
understand what has changed.
- Finally, the commit log description should be wrapped to 72
But again: I applied your patch after doing minor fixes in the commit
title and commit log, so those recommendations are valid for your
future contributions only :-)
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
More information about the buildroot