linux-kernel - Re: ath10k: sdio: fix compile warning

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Thu, 3 Aug 2017 13:35:27 +0200
From:   Kalle Valo <kvalo@....qualcomm.com>
To:     Erik Stromdahl <erik.stromdahl@...il.com>
CC:     <linux-wireless@...r.kernel.org>, <ath10k@...ts.infradead.org>,
        <linux-kernel@...r.kernel.org>,
        Erik Stromdahl <erik.stromdahl@...il.com>
Subject: Re: ath10k: sdio: fix compile warning

Erik Stromdahl <erik.stromdahl@...il.com> wrote:

> As suggested by Arnd Bergmann, replace
> "while (time_before_...) {}"
> with
> "do {} while (time_before_...)"
> 
> This fixes the following warnings detected by gcc 4.1.2:
> 
> drivers/net/wireless/ath/ath10k/sdio.c: In function
> ‘ath10k_sdio_mbox_rxmsg_pending_handler’:
> drivers/net/wireless/ath/ath10k/sdio.c:676: warning: ‘ret’ may be used uninitialized in this function
> 
> ...
> 
> drivers/net/wireless/ath/ath10k/sdio.c: In function
> ‘ath10k_sdio_irq_handler’:
> drivers/net/wireless/ath/ath10k/sdio.c:1331: warning: ‘ret’ may be used uninitialized in this function
> 
> Signed-off-by: Erik Stromdahl <erik.stromdahl@...il.com>
> Signed-off-by: Kalle Valo <kvalo@....qualcomm.com>

Patch applied to ath-next branch of ath.git, thanks.

433ef1b22678 ath10k: sdio: fix compile warning

-- 
https://patchwork.kernel.org/patch/9831499/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches