From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933108AbeCEHUt (ORCPT ); Mon, 5 Mar 2018 02:20:49 -0500 Received: from mail3-relais-sop.national.inria.fr ([192.134.164.104]:50292 "EHLO mail3-relais-sop.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932679AbeCEHUq (ORCPT ); Mon, 5 Mar 2018 02:20:46 -0500 X-IronPort-AV: E=Sophos;i="5.47,426,1515452400"; d="scan'208";a="256958319" Date: Mon, 5 Mar 2018 08:20:44 +0100 (CET) From: Julia Lawall X-X-Sender: jll@hadrien To: Stefan Agner cc: Boris Brezillon , Miquel Raynal , Richard Weinberger , David Woodhouse , Brian Norris , Marek Vasut , Cyrille Pitchen , linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org, kbuild-all@01.org Subject: [PATCH] mtd: rawnand: vf610_nfc: fix returnvar.cocci warnings Message-ID: User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Fengguang Wu Remove unneeded variable used to store return value. Generated by: scripts/coccinelle/misc/returnvar.cocci CC: Stefan Agner Signed-off-by: Fengguang Wu Signed-off-by: Julia Lawall --- Not a big deal, but the code can be a tiny bit simpler. vf610_nfc.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) --- a/drivers/mtd/nand/raw/vf610_nfc.c +++ b/drivers/mtd/nand/raw/vf610_nfc.c @@ -870,7 +870,6 @@ static int vf610_nfc_write_page(struct m struct vf610_nfc *nfc = mtd_to_nfc(mtd); int trfr_sz = mtd->writesize + mtd->oobsize; u32 row = 0, cmd1 = 0, cmd2 = 0, code = 0; - int ret = 0; cmd2 |= NAND_CMD_SEQIN << CMD_BYTE1_SHIFT; code |= COMMAND_CMD_BYTE1 | COMMAND_CAR_BYTE1 | COMMAND_CAR_BYTE2; @@ -894,7 +893,7 @@ static int vf610_nfc_write_page(struct m vf610_nfc_run(nfc, 0, row, cmd1, cmd2, trfr_sz); vf610_nfc_ecc_mode(nfc, ECC_BYPASS); - return ret; + return 0; } static int vf610_nfc_read_page_raw(struct mtd_info *mtd,