Re: [PATCH v4] brcmfmac: add CLM download support

On 28-08-17 16:52, Steve deRosier wrote:

On Mon, Aug 28, 2017 at 2:25 AM, Wright Feng [off-list ref] wrote:

quoted

From: Chung-Hsien Hsu <redacted>

The firmware for brcmfmac devices includes information regarding
regulatory constraints. For certain devices this information is kept
separately in a binary form that needs to be downloaded to the device.
This patch adds support to download this so-called CLM blob file. It
uses the same naming scheme as the other firmware files with extension
of .clm_blob.

Not to bikeshed this, but why not just ".clm"?  ".clm_blob" seems
unnecessary and could feel "unprofessional" to users.

To me simple seems better.

Hi Steve,

Glad you ask ;-) It was one of my comments pre-reviewing this 
"internally"*. The problem with calling it .clm is that it means 
something else within Broadcom and Cypress for that matter. The 
.clm_blob is the output of tool compiling the regulatory data according 
an input specification which uses the .clm extension. It is similar to 
regdb.txt and regulatory.bin.

Regards,
Arend

* internally does not quite cover Cypress+Broadcom collaboration.