From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752024AbdINLIM (ORCPT ); Thu, 14 Sep 2017 07:08:12 -0400 Received: from mout.kundenserver.de ([217.72.192.74]:55858 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751875AbdINLIK (ORCPT ); Thu, 14 Sep 2017 07:08:10 -0400 From: Arnd Bergmann To: Cyrille Pitchen , Marek Vasut Cc: Arnd Bergmann , David Woodhouse , Brian Norris , Boris Brezillon , Richard Weinberger , Maxime Coquelin , Alexandre Torgue , Ludovic Barre , linux-mtd@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: [PATCH] mtd: spi-nor: stm32-quadspi: avoid unintialized return code Date: Thu, 14 Sep 2017 13:06:59 +0200 Message-Id: <20170914110709.3591691-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Provags-ID: V03:K0:E8JqzB6AzgYnMvatpFTscg0rtb0LaZ1CTbyabfEhWsygsvbUk1s QPti6WnPk6Imy6vRXizNsfpgxVl3C816jKSQ1lNkvsRqEkXU8KGII559ugcFMRF3CZ9uZ/2 E86g7r980kJwj0nK7ILSfhwgsw0+XasADv6zUXVKr1Z4VjyZZACEOqiErTQL3ZVb3SVyKgk 6y+Qs6U90nOVsvdArir4A== X-UI-Out-Filterresults: notjunk:1;V01:K0:xp3DOooirCo=:4jNWEtnyEMQQhPLOug1VLW 5J9ITGFZE2GBVQO6BnZoLxjKHhTIrxR+daZ34qpUDU4bC4/Dmu3TJqUfuAgyllwesag2Vonao iafoRhV+girQ8qUt6iZ905A63TIkrL+Ra3dvFeOuG69jB6ogHhzDeEt7ZAAWHqCQbSUMiUoQt BLVuLx8eBitM/6+S53Fom3qwn3YiPUjuEHXQ8yoj6WjT88gqeb45hmO7JjChcWjJfk2V6Rgfh ZBIDqJkn3MT3yozCflHWcFyvF3JxwMiCMDFPil++0Bzll6q2yhLawUBXPvKIJQ6jGrsn+Scza Pse3ik7pSK8hcqhbd3qgZO8bKI2sVVei0jxz/HtR/nANhQA0PR+TQgQ0PVmakWC/tk47muU7Q Xx6OXVT3QNByI0F+Jj1sUM/9JbI0pjngT3q+pVeddYtAro+Oe7dF2BMV8yDEyKCMGQhWh75e7 CVyA2Bgg77Axw9aW5MRisWT758iUiNyDxBVnCmxpU/YY8x4gnjxgvEpeWPw6JG1Nt2udWfOCC lCOoW8aOiwyG4BrGXJAzve2RnKHTIiPb0LiF10/waE4JvF3jkf7qKghsW+SJf1YpN+YYs4jsw 6IMgb27fpRqYk6Aedz9Go9jk/Pvd3geqWoXIqLqWqTkkRHlwEqbDt+B2+ZeeXdj663gncnZdN Ku7BoPEvKoRiaZ/kj5fCdMYsAbS3cUe9AyIM+dWUm0FJhsa6AjO+ffmuY7c7E5gbq3hsKYS7P BorTRAaP+N9gvJpa9ZGpV7FtRBpVs6tUcXLwVbd8KdY+/YlGjt9la1ychso= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org If we send zero-length data to stm32_qspi_tx_poll() on older compiler versions such as gcc-4.6, we get warned that the return code is uninitialized: drivers/mtd/spi-nor/stm32-quadspi.c:248:2: error: ‘ret’ may be used uninitialized in this function [-Werror=uninitialized] On newer compiler versions, the return code is always zero in this case, as the local variable gets optimized away and is assumed to be zero after the loop completes without error. This changes the function to instead return -EINVAL if it ever gets called with a zero length buffer. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82203 Signed-off-by: Arnd Bergmann --- drivers/mtd/spi-nor/stm32-quadspi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mtd/spi-nor/stm32-quadspi.c b/drivers/mtd/spi-nor/stm32-quadspi.c index 86c0931543c5..711cfe7aa4bf 100644 --- a/drivers/mtd/spi-nor/stm32-quadspi.c +++ b/drivers/mtd/spi-nor/stm32-quadspi.c @@ -227,7 +227,7 @@ static int stm32_qspi_tx_poll(struct stm32_qspi *qspi, void (*tx_fifo)(u8 *, void __iomem *); u32 len = cmd->len, sr; u8 *buf = cmd->buf; - int ret; + int ret = -EINVAL; if (cmd->qspimode == CCR_FMODE_INDW) tx_fifo = stm32_qspi_write_fifo; -- 2.9.0