diff options
author | John Baldwin <jhb@FreeBSD.org> | 2023-06-20 16:28:59 +0000 |
---|---|---|
committer | John Baldwin <jhb@FreeBSD.org> | 2023-06-20 16:28:59 +0000 |
commit | 5f0bd29180616c726ed3ded5e746f23aaf8aabc4 (patch) | |
tree | 0350da9af3702ba8efb279f88f524fada1f727fb /usr.sbin/bluetooth | |
parent | 98f7dec5084e832648c0ff8a93820f644b01a73a (diff) | |
download | src-5f0bd29180616c726ed3ded5e746f23aaf8aabc4.tar.gz src-5f0bd29180616c726ed3ded5e746f23aaf8aabc4.zip |
bluetooth/ath3kfw: Propagate return value from ath3k_load_fwfile.
The caller ignores the return value so this is a no-op, but
ath3k_init_ar3012 returns the return values of its internal functions,
so this is more consistent.
Fix some misleading indentation while here.
Reported by: GCC -Wunused-but-set-variable (1)
Diffstat (limited to 'usr.sbin/bluetooth')
-rw-r--r-- | usr.sbin/bluetooth/ath3kfw/main.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/usr.sbin/bluetooth/ath3kfw/main.c b/usr.sbin/bluetooth/ath3kfw/main.c index 6ca5206b1507..25dca398c902 100644 --- a/usr.sbin/bluetooth/ath3kfw/main.c +++ b/usr.sbin/bluetooth/ath3kfw/main.c @@ -155,7 +155,7 @@ ath3k_init_ar3012(libusb_device_handle *hdl, const char *fw_path) ret = ath3k_load_patch(hdl, fw_path); if (ret < 0) { ath3k_err("Loading patch file failed\n"); - return (ret); + return (ret); } ret = ath3k_load_syscfg(hdl, fw_path); @@ -199,7 +199,7 @@ ath3k_init_firmware(libusb_device_handle *hdl, const char *file_prefix) /* free it */ ath3k_fw_free(&fw); - return (0); + return (ret); } /* |