Johannes Berg <[email protected]> writes:

>> +module_param_named(debug_mask, ath11k_debug_mask, uint, 0644);
>> +
>> +MODULE_PARM_DESC(debug_mask, "Debugging mask");
>> +
>> +static const struct ath11k_hw_params ath11k_hw_params = {
>> +                    .name = "ipq8074",
>
> indentation here seems a bit too much
>
>> +MODULE_LICENSE("Dual BSD/GPL");
>
> All your files state "ISC", shouldn't that be reflected here?

The MODULE_LICENSE() macro doesn't accept anything else. AFAIK all ISC
drivers use this.

-- 
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

Reply via email to