staging: ks7010: Fix the initialization of the 'sleep_status' structure
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Wed, 21 Jul 2021 08:45:11 +0000 (10:45 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 27 Jul 2021 13:07:43 +0000 (15:07 +0200)
'sleep_status' has 3 atomic_t members. Initialize the 3 of them instead of
initializing only 2 of them and setting 0 twice to the same variable.

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: https://lore.kernel.org/r/d2e52a33a9beab41879551d0ae2fdfc99970adab.1626856991.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ks7010/ks7010_sdio.c

index cbc0032c160452bd93319e567ec6ae4ae5fa62df..98d759e7cc957c7e6a4c0a2e5e9962462009f620 100644 (file)
@@ -939,9 +939,9 @@ static void ks7010_private_init(struct ks_wlan_private *priv,
        memset(&priv->wstats, 0, sizeof(priv->wstats));
 
        /* sleep mode */
+       atomic_set(&priv->sleepstatus.status, 0);
        atomic_set(&priv->sleepstatus.doze_request, 0);
        atomic_set(&priv->sleepstatus.wakeup_request, 0);
-       atomic_set(&priv->sleepstatus.wakeup_request, 0);
 
        trx_device_init(priv);
        hostif_init(priv);