diff options
| author | Darshan Rathod <darshanrathod475@gmail.com> | 2025-08-12 18:06:36 +0530 |
|---|---|---|
| committer | Johannes Berg <johannes.berg@intel.com> | 2025-09-03 10:02:09 +0200 |
| commit | b662bc503d95058b1cd640941edc48588849d9ce (patch) | |
| tree | ce17917a675768df304520ffd94181f688c28bf5 /rust/helpers/build_bug.c | |
| parent | 1373f94148a5adac2f42c8ba9771105624fe4af0 (diff) | |
wifi: brcmfmac: avoid assignment in if/else-if conditions in NVRAM load path
The NVRAM selection logic in brcmf_fw_request_nvram_done() used
patterns like:
if ((data = bcm47xx_nvram_get_contents(&data_len)))
free_bcm47xx_nvram = true;
else if ((data = brcmf_fw_nvram_from_efi(&data_len)))
kfree_nvram = true;
This style violates kernel coding style guidelines and triggers
checkpatch.pl errors. It also slightly reduces readability.
Refactor these cases by separating the assignment and the check,
ensuring behavior remains identical while complying with coding
standards.
Signed-off-by: Darshan Rathod <darshanrathod475@gmail.com>
Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com>
Link: https://patch.msgid.link/20250812123636.2142292-1-darshanrathod475@gmail.com
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'rust/helpers/build_bug.c')
0 files changed, 0 insertions, 0 deletions