Commit a08e3518 authored by Johannes Berg's avatar Johannes Berg Committed by Kalle Valo
Browse files

wifi: libertas: fix a couple of sparse warnings 



 - endian swapping is required in one place, use the
   already swapped 'bsssize' local
 - lbs_disablemesh need not be exported and can be static

Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
Signed-off-by: default avatarKalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20220904212910.76c34b2ae7a0.Ieb97c72b6d26f9d695cc4ab10fa7af5c3509612b@changeid
parent 76a8c54c

drivers/net/wireless/marvell/libertas/cfg.c

+1 −1
Original line number Diff line number Diff line
@@ -546,7 +546,7 @@ static int lbs_ret_scan(struct lbs_private *priv, unsigned long dummy, 
	pos = scanresp->bssdesc_and_tlvbuffer;



	lbs_deb_hex(LBS_DEB_SCAN, "SCAN_RSP", scanresp->bssdesc_and_tlvbuffer,

			scanresp->bssdescriptsize);

		    bsssize);



	tsfdesc = pos + bsssize;

	tsfsize = 4 + 8 * scanresp->nr_sets;

drivers/net/wireless/marvell/libertas/main.c

+1 −2
Original line number Diff line number Diff line
@@ -39,8 +39,7 @@ unsigned int lbs_debug; 
EXPORT_SYMBOL_GPL(lbs_debug);

module_param_named(libertas_debug, lbs_debug, int, 0644);



unsigned int lbs_disablemesh;

EXPORT_SYMBOL_GPL(lbs_disablemesh);

static unsigned int lbs_disablemesh;

module_param_named(libertas_disablemesh, lbs_disablemesh, int, 0644);