Commit 9b3bc20d authored by Jérôme Pouiller's avatar Jérôme Pouiller Committed by Greg Kroah-Hartman
Browse files

staging: wfx: fix wrong bytes order 



The field wakeup_period_max from struct hif_mib_beacon_wake_up_period is
a u8. So, assigning it a __le16 produces a nasty bug on big-endian
architectures.

Signed-off-by: default avatarJérôme Pouiller <jerome.pouiller@silabs.com>
Link: https://lore.kernel.org/r/20200512150414.267198-5-Jerome.Pouiller@silabs.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 095e86c8

drivers/staging/wfx/hif_tx_mib.c

+1 −1
Original line number Diff line number Diff line
@@ -32,7 +32,7 @@ int hif_set_beacon_wakeup_period(struct wfx_vif *wvif, 
	struct hif_mib_beacon_wake_up_period val = {

		.wakeup_period_min = dtim_interval,

		.receive_dtim = 0,

		.wakeup_period_max = cpu_to_le16(listen_interval),

		.wakeup_period_max = listen_interval,

	};



	if (dtim_interval > 0xFF || listen_interval > 0xFFFF)