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>
Wed, 22 Sep 2021 09:43:06 +0000 (11:43 +0200)
[ Upstream commit 56315e55119c0ea57e142b6efb7c31208628ad86 ]

'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>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/staging/ks7010/ks7010_sdio.c

index 81c46f4d0935abac2b8d15f848099ba52ba376b7..16d2036018e278dc4bf3168bcf81f7ff8dc3f881 100644 (file)
@@ -1037,9 +1037,9 @@ static int ks7010_sdio_probe(struct sdio_func *func,
        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);