From patchwork Sat Aug 5 17:41:37 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Manivannan Sadhasivam X-Patchwork-Id: 131520 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:c44e:0:b0:3f2:4152:657d with SMTP id w14csp587928vqr; Sat, 5 Aug 2023 11:36:15 -0700 (PDT) X-Google-Smtp-Source: AGHT+IFVq8HU3Pm8I9YlmXasJzSythwYXJZ82Q5VkHbuOuCn15xO221TNKXWqU5mr7HHAqrX2lIz X-Received: by 2002:a17:907:168a:b0:99b:f42d:b3f6 with SMTP id hc10-20020a170907168a00b0099bf42db3f6mr3659313ejc.32.1691260575638; Sat, 05 Aug 2023 11:36:15 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1691260575; cv=none; d=google.com; s=arc-20160816; b=FP5QgMzr7xJcM2dhsbBXvrrj0MJswU3DDHnkcqoXhfBrfmjGoZsLCuNgxDueiTzDfQ jPYXmuQHV6BBd/TbPHZBOQzNu5TISB75RmxwTMFw7wDD5f6pTYz/g4dzh2KXukwsUycs zRxBZ6AASPwSCssMnW9RFmKqlz63jTvfg0mMe4LkHBDjiLU0Dq+nEqKoD629jHkPqC8P vL4mG7Z69TwzYCiqnFKJF6tquXPbr8wWj1v5XlxkbDMLR4YTRmKHjvOI1ZaoSDpySljD yf0R/OAFYCC4gWBXQaa3Z1wXdvO2l+EBzgaojlAMAxURDKOX/yxumeUc5WnLsBU8/u88 17Lw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=LzHoBg1aTtFdNgIFHIxm61nutJbPmolyU88qKYX8avE=; fh=C3rMIGKHWoyxqoAraqqJMAgcBCOzfE2wKLTb5YQG2FY=; b=I1b1n22+US+B8PCNujnfDxEq8a//fUUQ6EjNHEyO5owU/NtNdehhB7bAlSt5v9tkbL c0/EZteTYPJqXWZIwtpXGxS0yuOlttJEw4dE1UWl1q3n/wJBHjsaIT2GP+t7voWfCBW+ bLdqgCXSB/y5uACJOiq88neHjMvgv9zFCiMgoeo93c6oP4BEvT3TVP4QIPAVZmxXnEyB YurVqSxwtH4sHeB6gdtf+FPgKXsEktCoRWcmS67cXFXsCXiIZASq85tOyIucbWKIBsmQ t2ylQ0gE0KZBrSZ09STqInwIfpgiQP7p4/WQtjRj13+alVD3ajGa13J4tAya6gplQ54r BgRA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=e1bZg3WP; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id fi13-20020a170906da0d00b0099bd0a03abesi3514296ejb.931.2023.08.05.11.35.52; Sat, 05 Aug 2023 11:36:15 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=e1bZg3WP; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230123AbjHERmN (ORCPT + 99 others); Sat, 5 Aug 2023 13:42:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56498 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230100AbjHERmK (ORCPT ); Sat, 5 Aug 2023 13:42:10 -0400 Received: from mail-pf1-x429.google.com (mail-pf1-x429.google.com [IPv6:2607:f8b0:4864:20::429]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 98852106 for ; Sat, 5 Aug 2023 10:42:07 -0700 (PDT) Received: by mail-pf1-x429.google.com with SMTP id d2e1a72fcca58-686be3cbea0so2837331b3a.0 for ; Sat, 05 Aug 2023 10:42:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1691257327; x=1691862127; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=LzHoBg1aTtFdNgIFHIxm61nutJbPmolyU88qKYX8avE=; b=e1bZg3WPmbUxYy4tVUd2YabTZ8JXTzEPZRnA0bqC3L7uQXcn6U887EK3lAeNlDZyNC vQjEbD9X1fJLxuU5Zv/7ik4UhOHvvdSysx4b4cCOLfdSo9+IbFEQx/QDw9Zair9WhJyJ 0pDNjZAmedfhhDhiC+hcnm6hLq+kf2Ma3ecnjHM/R6tfqcRftOpsw1bXYOrQVsOLJjas J4p85ACOLMtCOVfV3nW/3jAGe11uzBmQzbdlzmGs/O+TIpgy8SLk3TpWPKtYyvnk5v4d hkHTWzB5dWlbQWhRalHEGZghsjx5CPq/FKL3q4R5PtlvenwD61jnGfrZk9y6vn4kTYIG YIFw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691257327; x=1691862127; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=LzHoBg1aTtFdNgIFHIxm61nutJbPmolyU88qKYX8avE=; b=IUpmAr2/ssv4DqMkrty4wXD7qXJN44iimQVyryKmnPbUL3EVTcWF/KX2pHlW4boycw Jln4S9s5YJ3nN79/l+mPgZQqWYk4oZh++L3mEkssjgA67ArMgxdt6hc3AAJid50Y7qHk OuQg7cC8SyKzKRwAtENRFV3+LVAUFZAYfzH/E1XFC2+J8kd+B9iZqN2IDVw1xKkaYnGM +qAFARhUtxkLYRlBr599+cTvRSPWEI5wqqTqXwyBVxZWAR0P3Kau7g6xjnEW/Fr9gsQd t1KsJSzT6/IbwhcMXxPELNq6KxIM9hWBFhP/NjXucb1e/Hj+u4rbEnRl5CdCROKZuNzH oQ+Q== X-Gm-Message-State: AOJu0YybdgzlIdM+z4Nh5LxHJaJeOGQGMSHloj3unDioyPzXKgHO4Hqo bebPCX7h+qyuEy+yTgLd5p76 X-Received: by 2002:a05:6a20:7fa3:b0:12f:d350:8a12 with SMTP id d35-20020a056a207fa300b0012fd3508a12mr4024851pzj.21.1691257326933; Sat, 05 Aug 2023 10:42:06 -0700 (PDT) Received: from localhost.localdomain ([103.28.246.21]) by smtp.gmail.com with ESMTPSA id i14-20020aa78b4e000000b006874a6850e9sm3363160pfd.215.2023.08.05.10.42.04 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 05 Aug 2023 10:42:06 -0700 (PDT) From: Manivannan Sadhasivam To: miquel.raynal@bootlin.com, vigneshr@ti.com Cc: richard@nod.at, linux-mtd@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, quic_mdalam@quicinc.com, quic_srichara@quicinc.com, dan.carpenter@linaro.org, Manivannan Sadhasivam Subject: [PATCH 01/10] mtd: rawnand: qcom: Remove superfluous initialization of "ret" Date: Sat, 5 Aug 2023 23:11:37 +0530 Message-Id: <20230805174146.57006-2-manivannan.sadhasivam@linaro.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> References: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> MIME-Version: 1.0 X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,RCVD_IN_SBL, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1773415249063517802 X-GMAIL-MSGID: 1773415249063517802 In all the cases, "ret" variable is assigned a value before returning it. So there is no need to explicitly initialize it with 0. Signed-off-by: Manivannan Sadhasivam --- drivers/mtd/nand/raw/qcom_nandc.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c index b485d8517fce..b6751fb17587 100644 --- a/drivers/mtd/nand/raw/qcom_nandc.c +++ b/drivers/mtd/nand/raw/qcom_nandc.c @@ -1554,7 +1554,7 @@ check_for_erased_page(struct qcom_nand_host *host, u8 *data_buf, struct mtd_info *mtd = nand_to_mtd(chip); struct nand_ecc_ctrl *ecc = &chip->ecc; u8 *cw_data_buf, *cw_oob_buf; - int cw, data_size, oob_size, ret = 0; + int cw, data_size, oob_size, ret; if (!data_buf) data_buf = nand_get_data_buf(chip); @@ -2684,7 +2684,7 @@ static int qcom_read_status_exec(struct nand_chip *chip, const struct nand_op_instr *instr = NULL; unsigned int op_id = 0; unsigned int len = 0; - int ret = 0, num_cw, i; + int ret, num_cw, i; u32 flash_status; host->status = NAND_STATUS_READY | NAND_STATUS_WP; @@ -2747,7 +2747,7 @@ static int qcom_read_id_type_exec(struct nand_chip *chip, const struct nand_subo const struct nand_op_instr *instr = NULL; unsigned int op_id = 0; unsigned int len = 0; - int ret = 0; + int ret; qcom_parse_instructions(chip, subop, &q_op); @@ -2795,7 +2795,7 @@ static int qcom_misc_cmd_type_exec(struct nand_chip *chip, const struct nand_sub struct qcom_nand_controller *nandc = get_qcom_nand_controller(chip); struct qcom_nand_host *host = to_qcom_nand_host(chip); struct qcom_op q_op = {}; - int ret = 0; + int ret; qcom_parse_instructions(chip, subop, &q_op); @@ -2841,7 +2841,7 @@ static int qcom_param_page_type_exec(struct nand_chip *chip, const struct nand_ const struct nand_op_instr *instr = NULL; unsigned int op_id = 0; unsigned int len = 0; - int ret = 0; + int ret; qcom_parse_instructions(chip, subop, &q_op); @@ -2935,7 +2935,7 @@ static int qcom_erase_cmd_type_exec(struct nand_chip *chip, const struct nand_su struct qcom_nand_host *host = to_qcom_nand_host(chip); struct qcom_nand_controller *nandc = get_qcom_nand_controller(chip); struct qcom_op q_op = {}; - int ret = 0; + int ret; qcom_parse_instructions(chip, subop, &q_op); From patchwork Sat Aug 5 17:41:38 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Manivannan Sadhasivam X-Patchwork-Id: 131521 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:c44e:0:b0:3f2:4152:657d with SMTP id w14csp589747vqr; Sat, 5 Aug 2023 11:42:22 -0700 (PDT) X-Google-Smtp-Source: AGHT+IEp1gr6Mz5OmNmE75WhjEl5Jj4k+Lm8MDUixOLxZ/PfnQniERdHX5aSXhXcZfPxE3RsAK9I X-Received: by 2002:a17:907:2723:b0:99c:1ed1:65b2 with SMTP id d3-20020a170907272300b0099c1ed165b2mr3886329ejl.33.1691260942351; Sat, 05 Aug 2023 11:42:22 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1691260942; cv=none; d=google.com; s=arc-20160816; b=R775sErRMsYWTye19H8w/eidwnnwgejTtZKF2NHPPPEd313hjicqoJNlhr5jpkkv7n D7BgSJxcAwcsfQA2zTa+Z42JJll/S2DNLDN6O0OqnGvUHrPCEto6D+a+7tndJHSU+oS2 oNb4G3zsFjyuqvRh7vqEelhmUyQ1Q2Lz3Ikco+x51ysV0+8/OfrhxV8s/O0AjK0cCyIN nFCV3NPOck3waavxsG7SPmGuFIFT7gcVRepps9FdbT3pYtOFnhrSotbljHlJD5vAY/bJ Nmxe3xsMTDp+0nQuCxFgLJce8yCN9tOAueP4CJLdy8GIODC5wMj2Y7oVXjOes50QYSfG 479A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=CXEH8Wa3ojD+esCQhV22he8Pc5JwMdbmIGFKAtqq3lU=; fh=C3rMIGKHWoyxqoAraqqJMAgcBCOzfE2wKLTb5YQG2FY=; b=F+0Klh4n9qGfSZuNzZYLCHGvKeIxxsw9SX0TcKvW6cNxI2Ag6gVoQNBcCgdMYi6BM8 odmsp85HLTf3GTH52fQFmmWji4ZNtCLfUun3IvBvohZWmB61sf2hjNOONxTjFZcJ8Rbi I9Fx1XHSki/NHfLXHk+06QQoOgSC5KGJY9YT00fII1PzYwte0UMSctYOy6SGrIe0w1GZ kKzYFzKJ8w8Zf5SfftkStFzYhnhT6/ZnTV1k11y3IQQBpZM87MWIj0zeSJUjFahQAbF1 wO9NaWI8U6HfLsRvjnkXsVvijKi2uv+wn6iBw46Gj41L7UrhG1h0D2rLNac6ethS8dZ4 qKog== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=i9tbqf5T; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id o17-20020a170906775100b0099ba5ef93d6si3588957ejn.891.2023.08.05.11.41.56; Sat, 05 Aug 2023 11:42:22 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=i9tbqf5T; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230134AbjHERmP (ORCPT + 99 others); Sat, 5 Aug 2023 13:42:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56520 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230113AbjHERmM (ORCPT ); Sat, 5 Aug 2023 13:42:12 -0400 Received: from mail-pf1-x42f.google.com (mail-pf1-x42f.google.com [IPv6:2607:f8b0:4864:20::42f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A6DA0194 for ; Sat, 5 Aug 2023 10:42:10 -0700 (PDT) Received: by mail-pf1-x42f.google.com with SMTP id d2e1a72fcca58-68706b39c4cso2161382b3a.2 for ; Sat, 05 Aug 2023 10:42:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1691257330; x=1691862130; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=CXEH8Wa3ojD+esCQhV22he8Pc5JwMdbmIGFKAtqq3lU=; b=i9tbqf5TzsxdR8yFifphKZWnrPl1Ckv74LxFEHOMpMMvv5kdJzVDtoDtExAZ7t0N9t KzG638x7INF+Mac7aTRK4k7wN2umrVA+od63KF/I/lzeYFvljlGw64ocISCIS3r2OxqU nZAjyQ2VCY15S2Va+1eFSMG3rK0RbgmLaFC6VJLxa9rODoISrce5p32Kr4G1cn85p6Qi 8twOkwW0nP5ucMoTQ59lLnVlpkxHcIHdLd3FseL8GOWelgkX3m7jrUKqdPWMMkrakER+ vlUgPoF/zCoAbcvoiUdWHiNlMnFtN33kkQarBJtlyWG/av+eu8MN5H6qne6JGZb0ElWK F0Vw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691257330; x=1691862130; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=CXEH8Wa3ojD+esCQhV22he8Pc5JwMdbmIGFKAtqq3lU=; b=E7uSLVRttr67lGDX9wD5m+sv7uRZCzVungwRQTTfg5EQqOsNGF2dFQzptOhK9yTdth Gunp+IwUKNBoluaHtp/v3YTV4FEzMOegUTZqG6ufIngA2Yp44Xg4Vp9UepFCVKL3ifv8 bL4ZUxNp/N0dBSdOI1zywdCLKZnVJ3tFCps4vFkrrOrnGgiXqscUB/V8KDrTbF3uO+th sJ4LRVaIBqwU/xS1/buHTFq9db6pl+A9IJmYpTRGlZOyksCpqvd1C0BERNr/OIrGtbyJ WzwSOmG1eoDAa65FB7fkEmOOPSd27v7nE3gZIEknoNoNZBiU27Op50fXDnq8q1vUQVNd WzGQ== X-Gm-Message-State: AOJu0YxBg8wVkYuGw22Znpg/oEZQYhO86xppdlpNE6pWnTGLXhXOtxvU XZl7niPnNyDsO2aBJOvz+Atc X-Received: by 2002:a05:6a00:1501:b0:675:ef91:7922 with SMTP id q1-20020a056a00150100b00675ef917922mr4571732pfu.4.1691257330155; Sat, 05 Aug 2023 10:42:10 -0700 (PDT) Received: from localhost.localdomain ([103.28.246.21]) by smtp.gmail.com with ESMTPSA id i14-20020aa78b4e000000b006874a6850e9sm3363160pfd.215.2023.08.05.10.42.07 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 05 Aug 2023 10:42:09 -0700 (PDT) From: Manivannan Sadhasivam To: miquel.raynal@bootlin.com, vigneshr@ti.com Cc: richard@nod.at, linux-mtd@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, quic_mdalam@quicinc.com, quic_srichara@quicinc.com, dan.carpenter@linaro.org, Manivannan Sadhasivam Subject: [PATCH 02/10] mtd: rawnand: qcom: Rename variables in qcom_op_cmd_mapping() Date: Sat, 5 Aug 2023 23:11:38 +0530 Message-Id: <20230805174146.57006-3-manivannan.sadhasivam@linaro.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> References: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> MIME-Version: 1.0 X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,RCVD_IN_SBL, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1773415633774138552 X-GMAIL-MSGID: 1773415633774138552 qcom_op_cmd_mapping() function accepts opcode and returns the corresponding command register. So let's rename the local variables and parameters to reflect the same. Reported-by: Miquel Raynal Closes: https://lore.kernel.org/all/20230804190750.3367a044@xps-13/ Signed-off-by: Manivannan Sadhasivam --- drivers/mtd/nand/raw/qcom_nandc.c | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c index b6751fb17587..be428b80a39b 100644 --- a/drivers/mtd/nand/raw/qcom_nandc.c +++ b/drivers/mtd/nand/raw/qcom_nandc.c @@ -2555,39 +2555,39 @@ static int qcom_nand_attach_chip(struct nand_chip *chip) return 0; } -static int qcom_op_cmd_mapping(struct qcom_nand_controller *nandc, u8 cmd, +static int qcom_op_cmd_mapping(struct qcom_nand_controller *nandc, u8 opcode, struct qcom_op *q_op) { - int ret; + int cmd; - switch (cmd) { + switch (opcode) { case NAND_CMD_RESET: - ret = OP_RESET_DEVICE; + cmd = OP_RESET_DEVICE; break; case NAND_CMD_READID: - ret = OP_FETCH_ID; + cmd = OP_FETCH_ID; break; case NAND_CMD_PARAM: if (nandc->props->qpic_v2) - ret = OP_PAGE_READ_ONFI_READ; + cmd = OP_PAGE_READ_ONFI_READ; else - ret = OP_PAGE_READ; + cmd = OP_PAGE_READ; break; case NAND_CMD_ERASE1: case NAND_CMD_ERASE2: - ret = OP_BLOCK_ERASE; + cmd = OP_BLOCK_ERASE; break; case NAND_CMD_STATUS: - ret = OP_CHECK_STATUS; + cmd = OP_CHECK_STATUS; break; case NAND_CMD_PAGEPROG: - ret = OP_PROGRAM_PAGE; + cmd = OP_PROGRAM_PAGE; q_op->flag = OP_PROGRAM_PAGE; nandc->exec_opwrite = true; break; } - return ret; + return cmd; } /* NAND framework ->exec_op() hooks and related helpers */ From patchwork Sat Aug 5 17:41:39 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Manivannan Sadhasivam X-Patchwork-Id: 131517 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:c44e:0:b0:3f2:4152:657d with SMTP id w14csp577272vqr; Sat, 5 Aug 2023 11:07:28 -0700 (PDT) X-Google-Smtp-Source: AGHT+IEPTj5Zs5SX0cbr+A9Wj4Sxu1ENAxsLRiS2rzb6l1GCyeDOmhbrG+qejLVsEbBS67r2dhKC X-Received: by 2002:a17:907:75c9:b0:994:577:f9dd with SMTP id jl9-20020a17090775c900b009940577f9ddmr3922318ejc.9.1691258848566; Sat, 05 Aug 2023 11:07:28 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1691258848; cv=none; d=google.com; s=arc-20160816; b=kbMvsQa1AymIMUpGD50dLBt7Ty8QEuXN6iZKlyO0DRs5c/6v3XaBYRcZOOs6zdyXMC Rw1+MsYK4C1PxPKUltuSKOVWN+FnTJS3z/zkFYq/VkzF1Q5GoZ7jpw85/6tV88QfqLcG BR3//MBe8lMucAtXXeimICiBOhPaWrekXInWZ6od22pYSam4A1KP2mkSVvnoeXBwkh2m NbveG0MPoj2bEWRGm6dd27f+4qrleCqAzbpbzqb6WpI9e62Y0Va1N9jkULSkrDM0R+X/ U3dLcMJZGI1u7M+XEhueyA1TxPdDS15ljuPEgW3czpS+wbG2HGR3a7XpY1vhQ4qyVt0I V4jA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=uWgVvz/rSzQAaNBR+0GRt5Tgri/Am0g/0Utn1T92Y2Y=; fh=S7ZgxTf+7nlg1BFiMufKdujgJVzcI6Uo8vtRGhDMHEQ=; b=esLqneZERJssskML/KRne6KZbot/JhoHoXXdTxFBaCoxCXrOPEBHrSfwkQGJ3vUUJC 08U2wINAkawQsgmE39eQbJLHSaXJl5re99t+iIfMC+A+n6s7dm0votyh3vohAYmXpeCc bIH5JjSN4ZmzYNALOQ2Q0G1BCyhar/F4lQmzLWq9WY5CXjJs4bfXuC+P21b23w9NiTDi ZBpLgWtelKLOqBzAyFVAiP345edUAcbUiaj+iWK56FTii2/5EEGgJtpyJxotMfXVW5Rv o+h757YukHdK1Q9Hu3ivrQajyMSvwOwPbWIvgcTdSG0+qpvujPWh31VqViqSukeHyIvV 8QJw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=spmuW220; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id n1-20020a1709062bc100b0099351214a8esi3246252ejg.645.2023.08.05.11.07.05; Sat, 05 Aug 2023 11:07:28 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=spmuW220; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230140AbjHERmX (ORCPT + 99 others); Sat, 5 Aug 2023 13:42:23 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56684 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230129AbjHERmT (ORCPT ); Sat, 5 Aug 2023 13:42:19 -0400 Received: from mail-pf1-x42a.google.com (mail-pf1-x42a.google.com [IPv6:2607:f8b0:4864:20::42a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9E1F5103 for ; Sat, 5 Aug 2023 10:42:14 -0700 (PDT) Received: by mail-pf1-x42a.google.com with SMTP id d2e1a72fcca58-686efb9ee3cso3005452b3a.3 for ; Sat, 05 Aug 2023 10:42:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1691257333; x=1691862133; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=uWgVvz/rSzQAaNBR+0GRt5Tgri/Am0g/0Utn1T92Y2Y=; b=spmuW2205c4AMxvmTZqta+KfpbmGiMADfX552zTgjqqhuoU5jB50Rvj6YA5huHCmxS Clj8Tc/0sYIjn8v3eOqz9XQ/ICo1AOxtgTjNwJn285MAh7sCg0B6/9qmkvNTKmqY31KM kF1ntOt634CLv/Qdwqiiwb2WbWP5MocBnkT7Hj96LUqUuSGE8g9FdfBT64hp5UemWICs uwVemBDY2CR9JB7S/JBkewlSzcg2alQG4DzmdXAuZQpoTdUqiYiHwIo2cw9fbXvyOaip vDDwiawt5foQ3dgQ2Och6bf1DoT/UDLHoj87uHzsYv/h3mQQcFlvkFrYqiB/E/U1Bb+m +0Pw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691257333; x=1691862133; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=uWgVvz/rSzQAaNBR+0GRt5Tgri/Am0g/0Utn1T92Y2Y=; b=D+CQyq2sjXT3HX/VpTVIIaUBSu+H1Ep3MbDBZ63ctodWb1l9EtBEyOSNCasFhQhDvg wrmEbwIvFeZVjMkL6B5kn4kK8WuVk8b7NxAN5Y/LuZKRRBsQa4OmIjQXYhD/sFemG2z2 CKmfDbIHP4zEJl3NodY3uOTDeGMsw6dnOti7fzPveN2K7CagTxnZp/4+uvqsPRe68zXH a0B9nLCcFxqb77QVBsQVdeFnjmeELNs3e45q3id/eDd+qsjQAP73IMza8e4FowwAfjNS R6l1KuRXIDoxpjoh5U5tBZZiUZuZHycBGcpgV02M8ZVTtCI5NgxEUIP/J32H/xRiloh3 14xg== X-Gm-Message-State: AOJu0YwweCn5M9pLnWoA64nzJN0zB4OWt17F3bTXV5/yo6zXTkwDkBe0 cL116M/zJ4/o4NSqWKdej9Hb X-Received: by 2002:a05:6a20:1456:b0:13f:5234:24ce with SMTP id a22-20020a056a20145600b0013f523424cemr6875817pzi.28.1691257333498; Sat, 05 Aug 2023 10:42:13 -0700 (PDT) Received: from localhost.localdomain ([103.28.246.21]) by smtp.gmail.com with ESMTPSA id i14-20020aa78b4e000000b006874a6850e9sm3363160pfd.215.2023.08.05.10.42.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 05 Aug 2023 10:42:13 -0700 (PDT) From: Manivannan Sadhasivam To: miquel.raynal@bootlin.com, vigneshr@ti.com Cc: richard@nod.at, linux-mtd@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, quic_mdalam@quicinc.com, quic_srichara@quicinc.com, dan.carpenter@linaro.org, Manivannan Sadhasivam , kernel test robot Subject: [PATCH 03/10] mtd: rawnand: qcom: Handle unsupported opcode in qcom_op_cmd_mapping() Date: Sat, 5 Aug 2023 23:11:39 +0530 Message-Id: <20230805174146.57006-4-manivannan.sadhasivam@linaro.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> References: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> MIME-Version: 1.0 X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,RCVD_IN_SBL, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1773413438511015468 X-GMAIL-MSGID: 1773413438511015468 Handle the scenario where the caller has passed an unsupported opcode to qcom_op_cmd_mapping(). In that case, log the error and return the -EOPNOTSUPP errono. Also, let's propagate this error code all the way up. This also fixes the following smatch warning: drivers/mtd/nand/raw/qcom_nandc.c:2941 qcom_op_cmd_mapping() error: uninitialized symbol 'ret'. Reported-by: kernel test robot Reported-by: Dan Carpenter Closes: https://lore.kernel.org/r/202308032022.SnXkKyFs-lkp@intel.com/ Signed-off-by: Manivannan Sadhasivam --- drivers/mtd/nand/raw/qcom_nandc.c | 35 ++++++++++++++++++++++++------- 1 file changed, 27 insertions(+), 8 deletions(-) diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c index be428b80a39b..4957e9253878 100644 --- a/drivers/mtd/nand/raw/qcom_nandc.c +++ b/drivers/mtd/nand/raw/qcom_nandc.c @@ -2585,20 +2585,23 @@ static int qcom_op_cmd_mapping(struct qcom_nand_controller *nandc, u8 opcode, q_op->flag = OP_PROGRAM_PAGE; nandc->exec_opwrite = true; break; + default: + dev_err(nandc->dev, "Opcode not supported: %u\n", opcode); + return -EOPNOTSUPP; } return cmd; } /* NAND framework ->exec_op() hooks and related helpers */ -static void qcom_parse_instructions(struct nand_chip *chip, +static int qcom_parse_instructions(struct nand_chip *chip, const struct nand_subop *subop, struct qcom_op *q_op) { struct qcom_nand_controller *nandc = get_qcom_nand_controller(chip); const struct nand_op_instr *instr = NULL; unsigned int op_id; - int i; + int i, ret; for (op_id = 0; op_id < subop->ninstrs; op_id++) { unsigned int offset, naddrs; @@ -2608,7 +2611,11 @@ static void qcom_parse_instructions(struct nand_chip *chip, switch (instr->type) { case NAND_OP_CMD_INSTR: - q_op->cmd_reg = qcom_op_cmd_mapping(nandc, instr->ctx.cmd.opcode, q_op); + ret = qcom_op_cmd_mapping(nandc, instr->ctx.cmd.opcode, q_op); + if (ret < 0) + return ret; + + q_op->cmd_reg = ret; q_op->rdy_delay_ns = instr->delay_ns; break; @@ -2641,6 +2648,8 @@ static void qcom_parse_instructions(struct nand_chip *chip, break; } } + + return 0; } static void qcom_delay_ns(unsigned int ns) @@ -2689,7 +2698,9 @@ static int qcom_read_status_exec(struct nand_chip *chip, host->status = NAND_STATUS_READY | NAND_STATUS_WP; - qcom_parse_instructions(chip, subop, &q_op); + ret = qcom_parse_instructions(chip, subop, &q_op); + if (ret) + return ret; num_cw = nandc->exec_opwrite ? ecc->steps : 1; nandc->exec_opwrite = false; @@ -2749,7 +2760,9 @@ static int qcom_read_id_type_exec(struct nand_chip *chip, const struct nand_subo unsigned int len = 0; int ret; - qcom_parse_instructions(chip, subop, &q_op); + ret = qcom_parse_instructions(chip, subop, &q_op); + if (ret) + return ret; nandc->buf_count = 0; nandc->buf_start = 0; @@ -2797,7 +2810,9 @@ static int qcom_misc_cmd_type_exec(struct nand_chip *chip, const struct nand_sub struct qcom_op q_op = {}; int ret; - qcom_parse_instructions(chip, subop, &q_op); + ret = qcom_parse_instructions(chip, subop, &q_op); + if (ret) + return ret; if (q_op.flag == OP_PROGRAM_PAGE) goto wait_rdy; @@ -2843,7 +2858,9 @@ static int qcom_param_page_type_exec(struct nand_chip *chip, const struct nand_ unsigned int len = 0; int ret; - qcom_parse_instructions(chip, subop, &q_op); + ret = qcom_parse_instructions(chip, subop, &q_op); + if (ret) + return ret; q_op.cmd_reg |= PAGE_ACC | LAST_PAGE; @@ -2937,7 +2954,9 @@ static int qcom_erase_cmd_type_exec(struct nand_chip *chip, const struct nand_su struct qcom_op q_op = {}; int ret; - qcom_parse_instructions(chip, subop, &q_op); + ret = qcom_parse_instructions(chip, subop, &q_op); + if (ret) + return ret; q_op.cmd_reg |= PAGE_ACC | LAST_PAGE; From patchwork Sat Aug 5 17:41:40 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Manivannan Sadhasivam X-Patchwork-Id: 131524 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:c44e:0:b0:3f2:4152:657d with SMTP id w14csp592735vqr; Sat, 5 Aug 2023 11:52:55 -0700 (PDT) X-Google-Smtp-Source: AGHT+IG3H+1cE5IpfACj18Vm4PTcWEGLsq9Xhf9qeBaxUoCRw0u6qddrNSPcDzkM3jJbI5BpIYr2 X-Received: by 2002:a05:6a20:6a1c:b0:13e:1712:4c01 with SMTP id p28-20020a056a206a1c00b0013e17124c01mr5126374pzk.42.1691261574938; Sat, 05 Aug 2023 11:52:54 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1691261574; cv=none; d=google.com; s=arc-20160816; b=0J3NPtb4SkaCPRSkMlHcC8prgwb0bbxbn9yzCwtBryysT9e8x2yetrbhLjFhpp/jja Uo7JQFHV9dGOUtIhHTPURDn1SFRiNn1Iu8JvaRCznGqQd7nICPWRCdX1LaxwFHZKZcx/ TbF1JaVeqJLad/ECn5z110Fwv7djBpjFt4PtBX+Nuqchdv2x/bHpx644end3bTI9g5Gc ZIXfddpI6wUhukcESz5GZJYPI2DJIIxRzXstcAjb3kP6YMme6I1/zWiMPAj11Uwefm+C IPd0YxuXJvTJ6GFepHcGaNYaz+q8Lxp8jTJ3igUvwTt2gTdksn1nCVRH39OeNQevMOdf gqNA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=fw1sNygr+6EOLB0Bp4iKWYmwaK5BTcQtXrfXFuq/ooI=; fh=S7ZgxTf+7nlg1BFiMufKdujgJVzcI6Uo8vtRGhDMHEQ=; b=fnyqZIFOiaTkzmWOcH9oAHMmKoGxIPL5WLlj5zNEESDI3i6TlToEMcN3QVK/GFcLLW TUDyMMoZqcPSxQuqEQJj/MjBIjtHzxIfw9GYdY4FCmIDOKwIM2kC1/rfev+Ney8tnRwq 7YUMKp8oyl2Y85JYmjXXvvhDF8dZ8CVQbMmapt89n71unYQsCru6av+VHlEAadume0Gu F4/eHVJPjVJ2cEvokM9Lp/uJH/6C321SGKzf8yP1YkqgYwPoohIEr/17tnRQm4gGN6kN 7+8SY992yEt2gC3rO0702Yy+L90pXruiueN5jMUCHfOISWm4o7KEh9NL2fHGs6WW28HI BHWA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=RvaRNKwq; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id d7-20020a634f07000000b0055c07307e5bsi3523992pgb.64.2023.08.05.11.52.37; Sat, 05 Aug 2023 11:52:54 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=RvaRNKwq; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230133AbjHERm1 (ORCPT + 99 others); Sat, 5 Aug 2023 13:42:27 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56812 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230136AbjHERmZ (ORCPT ); Sat, 5 Aug 2023 13:42:25 -0400 Received: from mail-pf1-x431.google.com (mail-pf1-x431.google.com [IPv6:2607:f8b0:4864:20::431]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1D2C2272A for ; Sat, 5 Aug 2023 10:42:17 -0700 (PDT) Received: by mail-pf1-x431.google.com with SMTP id d2e1a72fcca58-686b9964ae2so2213886b3a.3 for ; Sat, 05 Aug 2023 10:42:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1691257337; x=1691862137; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=fw1sNygr+6EOLB0Bp4iKWYmwaK5BTcQtXrfXFuq/ooI=; b=RvaRNKwqniSRcehrR0lce9LhJjPEMUynlSdYrqa0l0Stp5DYnvJBYyg5XWgF5xF+09 NCRQTj+LLwGgcnNuaeSKibgV1TC6oesV1KVl0/ZMKD9g1UaDJX8pvnIi5IAkjf05Vm4p RukbhjCgP8zj6IZYmiZYQ5TLsQ2W1Gm4VN5hVWKOOPU9FdzgBQziLCrPmRhQU5gZNtOu 6qpshnoZBT+0w5A/Cw7JgP0RJ15MUojMgQqYgP4r3nCpBscB7FNj/6xz8iH5z8LCgqrF AbRHBjHs7FVep9gQsJUCVe/6LdP6LXna8hY+lV/UN7jeQ12TtZyzczJls80inwqLaRI3 fNlQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691257337; x=1691862137; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=fw1sNygr+6EOLB0Bp4iKWYmwaK5BTcQtXrfXFuq/ooI=; b=RXO3k/KHQajFRa+mN/KbfWmPXBGVIC6nqzv3tuQE4M7ZpOpNS+Xgje/vhOwHyUUwEI 8vzcapFp2qrzXhUqJCTwAITtIAwllfVmt+giNxDf7h83kCEwYD8RYVxC5hdbLWD++KOz UdNXFSfVyEMyP9zW1ovUS2+AEoI4q7tTB91iMy843KXB8kM5UIic8rZeDA6az+KgrrPD p9DY6QKp80uwg6guJ4v871L2BgmmDMs+lf1FL7Wl3ThWcubmI44AyP3lSFfNrNpfp/Lb Ri2Zixzi0z335Fboc1uiVvrTeCyI85FZefQ4oGAEzxsab2PK7pcgJkpt3ptENi15ZAH1 ygsw== X-Gm-Message-State: AOJu0YwQWznhA387JZnuMMrC9djQYOStXGyRZEOP+4kw21NGfka+nm5I wYQQv2jTLQuhFUxUIqq/pmj1 X-Received: by 2002:a05:6a00:2d13:b0:67b:f249:35e3 with SMTP id fa19-20020a056a002d1300b0067bf24935e3mr4741963pfb.26.1691257336916; Sat, 05 Aug 2023 10:42:16 -0700 (PDT) Received: from localhost.localdomain ([103.28.246.21]) by smtp.gmail.com with ESMTPSA id i14-20020aa78b4e000000b006874a6850e9sm3363160pfd.215.2023.08.05.10.42.13 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 05 Aug 2023 10:42:16 -0700 (PDT) From: Manivannan Sadhasivam To: miquel.raynal@bootlin.com, vigneshr@ti.com Cc: richard@nod.at, linux-mtd@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, quic_mdalam@quicinc.com, quic_srichara@quicinc.com, dan.carpenter@linaro.org, Manivannan Sadhasivam , kernel test robot Subject: [PATCH 04/10] mtd: rawnand: qcom: Fix the opcode check in qcom_check_op() Date: Sat, 5 Aug 2023 23:11:40 +0530 Message-Id: <20230805174146.57006-5-manivannan.sadhasivam@linaro.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> References: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> MIME-Version: 1.0 X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,RCVD_IN_SBL, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1773416296999312077 X-GMAIL-MSGID: 1773416296999312077 qcom_check_op() function checks for the invalid opcode for the instruction types. Currently, it just returns -ENOTSUPP for all opcodes of NAND_OP_CMD_INSTR type due to the use of "||" operator instead of "&&". Fix it! This also fixes the following smatch warning: drivers/mtd/nand/raw/qcom_nandc.c:3036 qcom_check_op() warn: was && intended here instead of ||? Reported-by: kernel test robot Reported-by: Dan Carpenter Closes: https://lore.kernel.org/r/202308032022.SnXkKyFs-lkp@intel.com/ Fixes: 89550beb098e ("mtd: rawnand: qcom: Implement exec_op()") Signed-off-by: Manivannan Sadhasivam --- drivers/mtd/nand/raw/qcom_nandc.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c index 4957e9253878..29fcdac932f1 100644 --- a/drivers/mtd/nand/raw/qcom_nandc.c +++ b/drivers/mtd/nand/raw/qcom_nandc.c @@ -3033,12 +3033,12 @@ static int qcom_check_op(struct nand_chip *chip, switch (instr->type) { case NAND_OP_CMD_INSTR: - if (instr->ctx.cmd.opcode != NAND_CMD_RESET || - instr->ctx.cmd.opcode != NAND_CMD_READID || - instr->ctx.cmd.opcode != NAND_CMD_PARAM || - instr->ctx.cmd.opcode != NAND_CMD_ERASE1 || - instr->ctx.cmd.opcode != NAND_CMD_ERASE2 || - instr->ctx.cmd.opcode != NAND_CMD_STATUS || + if (instr->ctx.cmd.opcode != NAND_CMD_RESET && + instr->ctx.cmd.opcode != NAND_CMD_READID && + instr->ctx.cmd.opcode != NAND_CMD_PARAM && + instr->ctx.cmd.opcode != NAND_CMD_ERASE1 && + instr->ctx.cmd.opcode != NAND_CMD_ERASE2 && + instr->ctx.cmd.opcode != NAND_CMD_STATUS && instr->ctx.cmd.opcode != NAND_CMD_PAGEPROG) return -ENOTSUPP; break; From patchwork Sat Aug 5 17:41:41 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Manivannan Sadhasivam X-Patchwork-Id: 131529 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:c44e:0:b0:3f2:4152:657d with SMTP id w14csp607790vqr; Sat, 5 Aug 2023 12:37:41 -0700 (PDT) X-Google-Smtp-Source: AGHT+IHRZEXvynkAGvLWG83mTIApM1MQgN3T+jwnXBFow+NoqLqEOEtfK6J79o3D1vhmWej6s7iL X-Received: by 2002:a05:6a21:3b49:b0:140:2805:6cc8 with SMTP id zy9-20020a056a213b4900b0014028056cc8mr2478964pzb.27.1691264261144; Sat, 05 Aug 2023 12:37:41 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1691264261; cv=none; d=google.com; s=arc-20160816; b=o8sT/Hi+OGfLEp4PLARruNKPhIscsLCNGEIrLBb42XzyLEV3CJSsuRp4Z0eGnXuVqH oETN2KLOJH5UQ08EmKZya+XXmMYS14N9V52HVjHNhF31wk4OZA5x/VwqZ0OZvXpREx9a /x17nenylc6ivc+sD1wfJFOvNF7dviuw8lyddh5jtoUSl8T8SGURDiEwVhtx6+vvxJAq lTEpA9YDe8OMxbqtBF6geA7CHv5p3ewYtPCZoxVaanqvPp7RUEJb0VFU2Yy+xmnJ3eDs mxw5q3loF5ZWKdjZGOercZBmylE24SK8IWcGD8GgEmBJViEBFsKu9FIYu0gNuT376XPh +bFg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=6DdQ17i05e7SXOOOl5eag5513m9DyQQ031lLX7RYOeM=; fh=C3rMIGKHWoyxqoAraqqJMAgcBCOzfE2wKLTb5YQG2FY=; b=eQuRGLa5UQnZmEqgHS98vrKk39D6Q4RWLxNDBOJeB599o6T/fYbnIp0M1W+3POvaGv 9R9UynmlPnMvT6mh4FKeYpbSltUg+0QZP5/Cc22uK8xIZ0/PbAbBjlMcL8xA9k92bKJW jfaIrhdDhu18bYWngNWuOxglq2FBG+9CpamVA2rLB/wfGHptlqUSH14yoJ+i0kh8ybHt NVa94fd+pZn945nFR/iAUbg1cOTERFkqWUpxZI3R2GWMFFfZu79SOd9e+dRn9DcYqMjU 9H28oY4ulerpUIVpu7sPNfki+gZVWc9iEQXHqbD3FsLgy1pqsi+LTjny4RoKuH8ljfy3 /Bbg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=z4YGgwR2; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id a23-20020a62d417000000b00666c9148d03si970140pfh.6.2023.08.05.12.37.22; Sat, 05 Aug 2023 12:37:41 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=z4YGgwR2; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230163AbjHERmf (ORCPT + 99 others); Sat, 5 Aug 2023 13:42:35 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56884 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230160AbjHERmb (ORCPT ); Sat, 5 Aug 2023 13:42:31 -0400 Received: from mail-pf1-x435.google.com (mail-pf1-x435.google.com [IPv6:2607:f8b0:4864:20::435]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 227DFE76 for ; Sat, 5 Aug 2023 10:42:21 -0700 (PDT) Received: by mail-pf1-x435.google.com with SMTP id d2e1a72fcca58-68783004143so2214795b3a.2 for ; Sat, 05 Aug 2023 10:42:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1691257340; x=1691862140; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=6DdQ17i05e7SXOOOl5eag5513m9DyQQ031lLX7RYOeM=; b=z4YGgwR2elh/iFIEMIUuNAj75llc2sReLBB/7yRb9iY10hRhPLGSIKMAOsbKqiruEJ e85zr/jUwZeceCfIFHWnSjwCUynVarvL9gIntTCBoIFNHDNG3jBkfvS5kuXKOkkRLlYc lQX4Zignzpz7MQw2vQs/TjRvy4buMPaSBMCEDzA/iNe60kq3H4nwf36F00sUo4Ia0Moy hNl0DKN8mImdD6VcaBgs8v6ftvL2fz6IkM7C2zySYqRW/cjpzZRqndVbN+QNqgBMKIWu CwKRaJCs6xZEaauXEJoVLEKvQN9IFredPVAFJgHzLGaaZAhLfvyyxaizTUbNx6B0qbCw 6Gng== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691257340; x=1691862140; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=6DdQ17i05e7SXOOOl5eag5513m9DyQQ031lLX7RYOeM=; b=CPRsanpZsHFJYA/3Nfa6CmyAT+FzWJ03ovseX6JAToplzOLd2Pt215ra+8nyqptM5P sGoAopUKoqk81/efvPkFuFvxOTQbsDoMqjhGZZ0Jc5aaRQhEv78pLq8oKBP5//gVRSYj PGWkRMqXizeP/apmMuCyYUvPB5idX36F/vNj+/IwQaeqI6D64IqzJy6/OKhuyiZX0N8s sE5WOT4+yGC5T3KnACFsUZD1+m76she88KzqSbGG5Lszas08vNtq3LeDpG1wTg4En0Pi ghN16XZ5CgEG1nZ1zEeJz1RyZW8Zbk6n7zOjYkTZOvVrktUT/9v/EL8UscSvZStIljuj 6zDg== X-Gm-Message-State: AOJu0Yyfryvi3/CmXNQzRJoASRBEACudKUQj/r7dqwE+TLRasCudbv/Q V4oPaTfn4wJhRLFEg6xZKz2t X-Received: by 2002:a05:6a00:2d91:b0:687:29ce:2fa6 with SMTP id fb17-20020a056a002d9100b0068729ce2fa6mr4911550pfb.7.1691257340058; Sat, 05 Aug 2023 10:42:20 -0700 (PDT) Received: from localhost.localdomain ([103.28.246.21]) by smtp.gmail.com with ESMTPSA id i14-20020aa78b4e000000b006874a6850e9sm3363160pfd.215.2023.08.05.10.42.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 05 Aug 2023 10:42:19 -0700 (PDT) From: Manivannan Sadhasivam To: miquel.raynal@bootlin.com, vigneshr@ti.com Cc: richard@nod.at, linux-mtd@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, quic_mdalam@quicinc.com, quic_srichara@quicinc.com, dan.carpenter@linaro.org, Manivannan Sadhasivam Subject: [PATCH 05/10] mtd: rawnand: qcom: Use EOPNOTSUPP instead of ENOTSUPP Date: Sat, 5 Aug 2023 23:11:41 +0530 Message-Id: <20230805174146.57006-6-manivannan.sadhasivam@linaro.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> References: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> MIME-Version: 1.0 X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,RCVD_IN_SBL, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1773419114057756265 X-GMAIL-MSGID: 1773419114057756265 Checkpatch complains over the usage of ENOTSUPP for new patches as below: WARNING: ENOTSUPP is not a SUSV4 error code, prefer EOPNOTSUPP So let's fix the error code which is already present in qcom_check_op(). Signed-off-by: Manivannan Sadhasivam --- drivers/mtd/nand/raw/qcom_nandc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c index 29fcdac932f1..a7a9421ef003 100644 --- a/drivers/mtd/nand/raw/qcom_nandc.c +++ b/drivers/mtd/nand/raw/qcom_nandc.c @@ -3040,7 +3040,7 @@ static int qcom_check_op(struct nand_chip *chip, instr->ctx.cmd.opcode != NAND_CMD_ERASE2 && instr->ctx.cmd.opcode != NAND_CMD_STATUS && instr->ctx.cmd.opcode != NAND_CMD_PAGEPROG) - return -ENOTSUPP; + return -EOPNOTSUPP; break; default: break; From patchwork Sat Aug 5 17:41:42 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Manivannan Sadhasivam X-Patchwork-Id: 131530 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:c44e:0:b0:3f2:4152:657d with SMTP id w14csp608319vqr; Sat, 5 Aug 2023 12:39:35 -0700 (PDT) X-Google-Smtp-Source: AGHT+IGgvTmfIDdSBDdwLnyjEfdm73dRsWdqrFTqmZMhWxmxFb/fvDW2D+oQrAmY/yqVuovHkcV6 X-Received: by 2002:a17:90a:d348:b0:267:f1d7:ed68 with SMTP id i8-20020a17090ad34800b00267f1d7ed68mr3902014pjx.14.1691264374837; Sat, 05 Aug 2023 12:39:34 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1691264374; cv=none; d=google.com; s=arc-20160816; b=v3IektgpknVkYASQrf5SrQbYGlZD5dkIE5u8j/KJtjoC/S4WjG+2bHpCoM+NLKD2Dk O9EKD8EGPGzl31Wzi8qZ8CDsKV4Z8l99zQe8pbTsRTdLe0NLe9rK123nWh62d1ZfKQuS oNwhwnZGg6Wl+jZZK4F8zpH4OiWC1+rWPBCmqNqxGC86aNXosxz1BmSeoe2DzNTC7rH/ bGtiK01Hl+h9KxdageBEjmSAohI2mzoK7TeRzxvkeia21oPUL0+CEvKUJrcvTBaksYzB ADm21thjPYjt9jzDV23/zE1cwToo6wgjbyNMcs6jKqgt3Y76aBhdeDiyWnQexyQ2bkIK vP7Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=1Sk0pPWqBxPd0LZRhqUZVkkGQlVge4WAhNj/aqE/s4Y=; fh=C3rMIGKHWoyxqoAraqqJMAgcBCOzfE2wKLTb5YQG2FY=; b=xGbNxqd+1v1QxR/mxysdGTAQ0YDfBS00Y6WLc/upV2cYf+b9rV8ZGkFLMgUKGCmUHG NnQj7rAR2KqOyAPNSVNTqF/rtp8oiQ+48ouNOv1WI4E+zUmBdY4OCs03l8+xSoNzSKSf B8sxTjvdroZaCU0zVafxzBoAzg3SJKxD+qKGbV4QLnD2u7RM5Wb2uoSpRTnUA1IoA1MB 9NTcUP4vZiqG2ZvSlxIBy/kyMsC4xMulkZin+9EWRfgIeMsVJUIEre4QQCGQEmnJYZcK DW8/+Vxb5f+nBilFwTAAF/rJuGDNzp59LSoqKav6wXu77fvFSy3K5YvK5uI/WlOXifkW WR2Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=r3e0kB4h; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id p20-20020a170902e35400b001b9ed01eb1dsi3352015plc.361.2023.08.05.12.39.21; Sat, 05 Aug 2023 12:39:34 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=r3e0kB4h; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229943AbjHERmp (ORCPT + 99 others); Sat, 5 Aug 2023 13:42:45 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57228 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229970AbjHERmn (ORCPT ); Sat, 5 Aug 2023 13:42:43 -0400 Received: from mail-pf1-x429.google.com (mail-pf1-x429.google.com [IPv6:2607:f8b0:4864:20::429]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4852D1BD0 for ; Sat, 5 Aug 2023 10:42:24 -0700 (PDT) Received: by mail-pf1-x429.google.com with SMTP id d2e1a72fcca58-686f94328a4so2123371b3a.0 for ; Sat, 05 Aug 2023 10:42:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1691257343; x=1691862143; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=1Sk0pPWqBxPd0LZRhqUZVkkGQlVge4WAhNj/aqE/s4Y=; b=r3e0kB4hxBfTtxIB0onMjnTolvvCfOWZ6jFqGifW4WvCZljrocy7UTblzz5SgbZSMV Tsg6Grb5tNsecdh7a3xvki0xfGCzsjstt2awTgo5qChvIy5eZZHhzs3p9ax0WcWC3Tei tgcyE9SjI8JNLSsZz7XZn+YlxNV9B7RczmluWvy6OA5QTxU+XnstbRxTykip1aunBR+w wvO3V5k/mzf7dNLhvllWENRiaYAON41uRSDRlsd3pI4/QUAx8NVDMFoRJUe9M331xiDK WRxJ80B9Pix7WXIMMX/Eh7wTIMNZMUz6ESVIzmjKRIGxflRDWv8dy9e9WDLjaLBsQmj/ CLow== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691257343; x=1691862143; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=1Sk0pPWqBxPd0LZRhqUZVkkGQlVge4WAhNj/aqE/s4Y=; b=Y6Lk7JQwTLbOQPZPnQosL+IU1XwanD4eIZL+r2YUzTVC6uTNL9iVbYJbZQ+CGljuj0 oxoSMYrYMIEQmsj2/5z0N1minOjKvT6rmrdVZ85OqudTK3G30ZChJZBxQaL8gif7K088 F1nvYM/nLjN3GYIrsJoBBQ5qpj7omtXoNbZh0Bdt/xyCoEbPBReQUvdBZTvttkv/L/Ja x/pSKHzPWRkflngODK0QJy2nzqeP/xm6TXZUTb7VA/bpirJcOJCr2FOvFD7c0phUU52G jh700uEkEpRrImMQRMdOCeDGt1PXDeW4PUCgROPGKBU6hyo5A4/nHgtzDK15O3c7QGTv PGFg== X-Gm-Message-State: AOJu0YxHVmlNooJFOBgUTc/SdtTa4M3qYBTAsg9u9ZO9LPR+xt05NMW6 Vohun/NSMTibN7m/a9X4liEB X-Received: by 2002:a05:6a20:12c2:b0:12e:caac:f263 with SMTP id v2-20020a056a2012c200b0012ecaacf263mr3443189pzg.20.1691257343130; Sat, 05 Aug 2023 10:42:23 -0700 (PDT) Received: from localhost.localdomain ([103.28.246.21]) by smtp.gmail.com with ESMTPSA id i14-20020aa78b4e000000b006874a6850e9sm3363160pfd.215.2023.08.05.10.42.20 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 05 Aug 2023 10:42:22 -0700 (PDT) From: Manivannan Sadhasivam To: miquel.raynal@bootlin.com, vigneshr@ti.com Cc: richard@nod.at, linux-mtd@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, quic_mdalam@quicinc.com, quic_srichara@quicinc.com, dan.carpenter@linaro.org, Manivannan Sadhasivam Subject: [PATCH 06/10] mtd: rawnand: qcom: Wrap qcom_nand_exec_op() to 80 columns Date: Sat, 5 Aug 2023 23:11:42 +0530 Message-Id: <20230805174146.57006-7-manivannan.sadhasivam@linaro.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> References: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> MIME-Version: 1.0 X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,RCVD_IN_SBL, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1773419233246567469 X-GMAIL-MSGID: 1773419233246567469 Both the function arguments and the definition could be wrapped to 80 columns to save line space. Signed-off-by: Manivannan Sadhasivam --- drivers/mtd/nand/raw/qcom_nandc.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c index a7a9421ef003..4f38579ae03e 100644 --- a/drivers/mtd/nand/raw/qcom_nandc.c +++ b/drivers/mtd/nand/raw/qcom_nandc.c @@ -3051,14 +3051,12 @@ static int qcom_check_op(struct nand_chip *chip, } static int qcom_nand_exec_op(struct nand_chip *chip, - const struct nand_operation *op, - bool check_only) + const struct nand_operation *op, bool check_only) { if (check_only) return qcom_check_op(chip, op); - return nand_op_parser_exec_op(chip, &qcom_op_parser, - op, check_only); + return nand_op_parser_exec_op(chip, &qcom_op_parser, op, check_only); } static const struct nand_controller_ops qcom_nandc_ops = { From patchwork Sat Aug 5 17:41:43 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Manivannan Sadhasivam X-Patchwork-Id: 131518 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:c44e:0:b0:3f2:4152:657d with SMTP id w14csp581544vqr; Sat, 5 Aug 2023 11:17:21 -0700 (PDT) X-Google-Smtp-Source: AGHT+IFWSlOI/T3Sy4aiY4qXvb1zG+3VGaGWFg5tjQOxQmfkNaUn/lQpBOqEV/k2R+lVnf5LCaCR X-Received: by 2002:a05:6a20:a109:b0:13a:43e8:3fb8 with SMTP id q9-20020a056a20a10900b0013a43e83fb8mr6395353pzk.54.1691259441376; Sat, 05 Aug 2023 11:17:21 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1691259441; cv=none; d=google.com; s=arc-20160816; b=RqIgz1HV+cAvg2dmm2j4iUacMxSfGfHDQk6Ttut8uZ5gUaYnq0+pDa24twnwkaU/Qk 6v9ke0DCm1YvCHkUEP5Zv9Gb5kw60ckGDd6Ugu1XEyGU1PSLxyxH4MeXtvIgFicQhjFf SmBPZypaDWALPrssqXD4LqRc+n4jHB86CLEt3G0cNUFO1bcSlKHfdlRx+DtnSFfSlC0E +P89YXkqis8aAY97+eWhCmtxbobtP/PfRnTvOcuO3gTGjf69hTWm8Qy3hafdAzq27zK+ 4t05l/OQo0gImTzyoN9bShYpRCwmgaN1qqxDQM98VksKlC1ewGMm/fwl0Y2i6kbu/jPd r3+w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=VYFveuajqMNbJCnKR8oO9CBtWwxD9/VCo5pORmPNdlI=; fh=C3rMIGKHWoyxqoAraqqJMAgcBCOzfE2wKLTb5YQG2FY=; b=gH6oL4gC147T1OLq0TVgMPvDApepnZFLTO4sVjxTFLtfxZbveVJr6NRnnkR2B09H1u tQAbPE9S2j46NDSGeMRaRwgR9EbO1p3C3h4KAzL8gKZXGOeKp4fQMPh7UnPTrAL+fN1L CfVwAqzJqLqvUW1vH5ELTWO30IzwG//2RPypbUcRDGgaVqZs7Xj3vUulraWHRo5vTLc3 ddzvx7GKEPGEdBvW0GO2ZWN45QDo1ElWaIdL5W8BtYB+1zCAMuDz4HWp7kb3Q48Syu+q qgmtUexCwf/kfQlTv48iHayw/DYAHUVQGNszRM1D5zNnzImZoW/EeyGurgmTxZd0wglr ql4g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=Z0rsGlk9; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id x35-20020a631723000000b00563856f6d3bsi3314925pgl.55.2023.08.05.11.17.07; Sat, 05 Aug 2023 11:17:21 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=Z0rsGlk9; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229464AbjHERnA (ORCPT + 99 others); Sat, 5 Aug 2023 13:43:00 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57484 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229602AbjHERm5 (ORCPT ); Sat, 5 Aug 2023 13:42:57 -0400 Received: from mail-pf1-x430.google.com (mail-pf1-x430.google.com [IPv6:2607:f8b0:4864:20::430]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BC3143581 for ; Sat, 5 Aug 2023 10:42:31 -0700 (PDT) Received: by mail-pf1-x430.google.com with SMTP id d2e1a72fcca58-6877eb31261so2324182b3a.1 for ; Sat, 05 Aug 2023 10:42:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1691257346; x=1691862146; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=VYFveuajqMNbJCnKR8oO9CBtWwxD9/VCo5pORmPNdlI=; b=Z0rsGlk9CrnX6KVTupzoH16ngQguKUiMLp4IcGcF5qF83R9Sae4ptKVIGeRPYzFddo A4PQYtAEU4Y1mEqmjEEu0i/lUPVSw6Hvl9wDZDVjwoO1Qc7fpEFlUI5zN7gbRDwKtIzO 7UeXU2v9CDUIuXdP6WfMQaV9Adham0dZOHzjBQqr7/Gua9r2byQzkTJE8EA9+Pb7JYt1 uKm4tNjU6zcLt2Wfxh9nJHHoAy4Djy1gVJWbiOpl0CpDYvpBAoJlUlDdjzNmQOFq5xYI XPDX9hZw+LB/CkeVpTF6VWFSb/MUm+S80lupIp9hW+IaRYFGCasX8hi+rCvVffLNUhEo lHwA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691257346; x=1691862146; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=VYFveuajqMNbJCnKR8oO9CBtWwxD9/VCo5pORmPNdlI=; b=OcNjoBHJe8qkjbwPAdD0+XzGxYJfMplY94Flzcm3Ai3qObqDS9a5x8MAqAFQNm8Jz6 qIo0vihPuAuW4ojow9UKghUCzV6SEdmp55nJCcD6mvY/Rk5mkHf96tLU314f4CONNcDZ /UgrPTLYlFgLQ6XeJgFu0bXLRDEjdAAadL4mwLKGwN0mgO5bZLDjO/uS+uZzz79uX0Ve LSamptmX8Bb2ADYBTHcNEW+g/K6PNDtOdORkyrC61CiteW0MDi5k0VZnlj5C6k3m2b38 eIZC68EzkdeGq+2VixFXEbVB+MFtwCOWXAx8hoDa/DN1NOU0IjfnrBVtTaOPyG82E9TD jwbg== X-Gm-Message-State: AOJu0YzV0CPFEz+btsV84E+p59BAgsn0l7Z97jRNow2hM/j5Twu2eOfm MPtPNznY5w92vqwRDmpaOnEE X-Received: by 2002:a05:6a00:1a44:b0:687:404f:4d5a with SMTP id h4-20020a056a001a4400b00687404f4d5amr5179650pfv.13.1691257346243; Sat, 05 Aug 2023 10:42:26 -0700 (PDT) Received: from localhost.localdomain ([103.28.246.21]) by smtp.gmail.com with ESMTPSA id i14-20020aa78b4e000000b006874a6850e9sm3363160pfd.215.2023.08.05.10.42.23 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 05 Aug 2023 10:42:25 -0700 (PDT) From: Manivannan Sadhasivam To: miquel.raynal@bootlin.com, vigneshr@ti.com Cc: richard@nod.at, linux-mtd@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, quic_mdalam@quicinc.com, quic_srichara@quicinc.com, dan.carpenter@linaro.org, Manivannan Sadhasivam Subject: [PATCH 07/10] mtd: rawnand: qcom: Unmap sg_list and free desc within submic_descs() Date: Sat, 5 Aug 2023 23:11:43 +0530 Message-Id: <20230805174146.57006-8-manivannan.sadhasivam@linaro.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> References: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> MIME-Version: 1.0 X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,RCVD_IN_SBL, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1773414059862563270 X-GMAIL-MSGID: 1773414059862563270 There are two types of dma descriptors being used in this driver allocated by, prepare_bam_async_desc() and prep_adm_dma_desc() helper functions. These functions map and prepare the descriptors to be used for dma transfers. And all the descriptors are submitted inside the submit_descs() function. Once the transfer completion happens, those descriptors should be unmapped and freed as a part of cleanup. Currently, free_descs() function is doing the said cleanup of descriptors. But the callers of submit_descs() are required to call free_descs() in both the success and error cases. Since there are no other transactions need to be done after submit_descs(), let's just move the contents of free_descs() inside submit_descs() itself. This makes sure that the cleanup is handled within the submit_descs() thereby offloading the cleanup part from callers. While at it, let's also rename the return variable from "r" to "ret". Signed-off-by: Manivannan Sadhasivam --- drivers/mtd/nand/raw/qcom_nandc.c | 65 ++++++++++--------------------- 1 file changed, 20 insertions(+), 45 deletions(-) diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c index 4f38579ae03e..1dbe17dcdb84 100644 --- a/drivers/mtd/nand/raw/qcom_nandc.c +++ b/drivers/mtd/nand/raw/qcom_nandc.c @@ -1306,30 +1306,30 @@ static void config_nand_cw_write(struct nand_chip *chip) /* helpers to submit/free our list of dma descriptors */ static int submit_descs(struct qcom_nand_controller *nandc) { - struct desc_info *desc; + struct desc_info *desc, *n; dma_cookie_t cookie = 0; struct bam_transaction *bam_txn = nandc->bam_txn; - int r; + int ret = 0; if (nandc->props->is_bam) { if (bam_txn->rx_sgl_pos > bam_txn->rx_sgl_start) { - r = prepare_bam_async_desc(nandc, nandc->rx_chan, 0); - if (r) - return r; + ret = prepare_bam_async_desc(nandc, nandc->rx_chan, 0); + if (ret) + goto err_unmap_free_desc; } if (bam_txn->tx_sgl_pos > bam_txn->tx_sgl_start) { - r = prepare_bam_async_desc(nandc, nandc->tx_chan, + ret = prepare_bam_async_desc(nandc, nandc->tx_chan, DMA_PREP_INTERRUPT); - if (r) - return r; + if (ret) + goto err_unmap_free_desc; } if (bam_txn->cmd_sgl_pos > bam_txn->cmd_sgl_start) { - r = prepare_bam_async_desc(nandc, nandc->cmd_chan, + ret = prepare_bam_async_desc(nandc, nandc->cmd_chan, DMA_PREP_CMD); - if (r) - return r; + if (ret) + goto err_unmap_free_desc; } } @@ -1351,19 +1351,17 @@ static int submit_descs(struct qcom_nand_controller *nandc) if (!wait_for_completion_timeout(&bam_txn->txn_done, QPIC_NAND_COMPLETION_TIMEOUT)) - return -ETIMEDOUT; + ret = -ETIMEDOUT; } else { if (dma_sync_wait(nandc->chan, cookie) != DMA_COMPLETE) - return -ETIMEDOUT; + ret = -ETIMEDOUT; } - return 0; -} - -static void free_descs(struct qcom_nand_controller *nandc) -{ - struct desc_info *desc, *n; - +err_unmap_free_desc: + /* + * Unmap the dma sg_list and free the desc allocated by both + * prepare_bam_async_desc() and prep_adm_dma_desc() functions. + */ list_for_each_entry_safe(desc, n, &nandc->desc_list, node) { list_del(&desc->node); @@ -1376,6 +1374,8 @@ static void free_descs(struct qcom_nand_controller *nandc) kfree(desc); } + + return ret; } /* reset the register read buffer for next NAND operation */ @@ -1521,7 +1521,6 @@ qcom_nandc_read_cw_raw(struct mtd_info *mtd, struct nand_chip *chip, read_data_dma(nandc, reg_off, oob_buf + oob_size1, oob_size2, 0); ret = submit_descs(nandc); - free_descs(nandc); if (ret) { dev_err(nandc->dev, "failure to read raw cw %d\n", cw); return ret; @@ -1775,8 +1774,6 @@ static int read_page_ecc(struct qcom_nand_host *host, u8 *data_buf, } ret = submit_descs(nandc); - free_descs(nandc); - if (ret) { dev_err(nandc->dev, "failure to read page/oob\n"); return ret; @@ -1815,8 +1812,6 @@ static int copy_last_cw(struct qcom_nand_host *host, int page) if (ret) dev_err(nandc->dev, "failed to copy last codeword\n"); - free_descs(nandc); - return ret; } @@ -2024,8 +2019,6 @@ static int qcom_nandc_write_page(struct nand_chip *chip, const u8 *buf, if (ret) dev_err(nandc->dev, "failure to write page\n"); - free_descs(nandc); - if (!ret) ret = nand_prog_page_end_op(chip); @@ -2100,8 +2093,6 @@ static int qcom_nandc_write_page_raw(struct nand_chip *chip, if (ret) dev_err(nandc->dev, "failure to write raw page\n"); - free_descs(nandc); - if (!ret) ret = nand_prog_page_end_op(chip); @@ -2149,9 +2140,6 @@ static int qcom_nandc_write_oob(struct nand_chip *chip, int page) config_nand_cw_write(chip); ret = submit_descs(nandc); - - free_descs(nandc); - if (ret) { dev_err(nandc->dev, "failure to write oob\n"); return -EIO; @@ -2228,9 +2216,6 @@ static int qcom_nandc_block_markbad(struct nand_chip *chip, loff_t ofs) config_nand_cw_write(chip); ret = submit_descs(nandc); - - free_descs(nandc); - if (ret) { dev_err(nandc->dev, "failure to update BBM\n"); return -EIO; @@ -2722,10 +2707,8 @@ static int qcom_read_status_exec(struct nand_chip *chip, ret = submit_descs(nandc); if (ret) { dev_err(nandc->dev, "failure in submitting status descriptor\n"); - free_descs(nandc); goto err_out; } - free_descs(nandc); nandc_read_buffer_sync(nandc, true); @@ -2787,10 +2770,8 @@ static int qcom_read_id_type_exec(struct nand_chip *chip, const struct nand_subo ret = submit_descs(nandc); if (ret) { dev_err(nandc->dev, "failure in submitting read id descriptor\n"); - free_descs(nandc); goto err_out; } - free_descs(nandc); instr = q_op.data_instr; op_id = q_op.data_instr_idx; @@ -2835,10 +2816,8 @@ static int qcom_misc_cmd_type_exec(struct nand_chip *chip, const struct nand_sub ret = submit_descs(nandc); if (ret) { dev_err(nandc->dev, "failure in submitting misc descriptor\n"); - free_descs(nandc); goto err_out; } - free_descs(nandc); wait_rdy: qcom_delay_ns(q_op.rdy_delay_ns); @@ -2932,10 +2911,8 @@ static int qcom_param_page_type_exec(struct nand_chip *chip, const struct nand_ ret = submit_descs(nandc); if (ret) { dev_err(nandc->dev, "failure in submitting param page descriptor\n"); - free_descs(nandc); goto err_out; } - free_descs(nandc); ret = qcom_wait_rdy_poll(chip, q_op.rdy_timeout_ms); if (ret) @@ -2981,10 +2958,8 @@ static int qcom_erase_cmd_type_exec(struct nand_chip *chip, const struct nand_su ret = submit_descs(nandc); if (ret) { dev_err(nandc->dev, "failure in submitting erase descriptor\n"); - free_descs(nandc); goto err_out; } - free_descs(nandc); ret = qcom_wait_rdy_poll(chip, q_op.rdy_timeout_ms); if (ret) From patchwork Sat Aug 5 17:41:44 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Manivannan Sadhasivam X-Patchwork-Id: 131528 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:c44e:0:b0:3f2:4152:657d with SMTP id w14csp603924vqr; Sat, 5 Aug 2023 12:25:18 -0700 (PDT) X-Google-Smtp-Source: AGHT+IG9cfpx0ZJLSKL+Oy4a8YJm0c+tgWDy4W6EHZbtNiCn3UpWhQxP/jo4DPqW0G/ZmGsfNN15 X-Received: by 2002:a17:903:428c:b0:1b8:5bcc:2ffc with SMTP id ju12-20020a170903428c00b001b85bcc2ffcmr3954998plb.45.1691263517952; Sat, 05 Aug 2023 12:25:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1691263517; cv=none; d=google.com; s=arc-20160816; b=tXyX3p/8M3aqw2BR1RLq1yVcHYJgExoRvRlbDuetB32qHCnMKKDl1QC3h491BjCpYC j1/LdUComnyjdJKWeqISRcNA/Ugqm1pi4bpjICfZtuh0UfClsMa+e47dtsZ/Bctn+yWS YcQAJ/4P10erFri0/eAGVqXAWWXZxy2ee2WnGqseF9mSJSygIwwUPYp+a9oIbA21PgdR cbXZvQrxNsJfzXLsaI/ZFBQPd3F+GP5lKq3WzmKTabSMvvd7SNywJMpT9c2dZEsdWpRd QMGurZyDxOSrLlkC3mxAlxkfEFenNyjc71KgUSfruD6Pk1vNdaeQ/fiQW/CumVhuw2tC z7Mw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=VzjUVbmqujkXx5jszw63KmEj0mqPrkYgNLPBEumwUM4=; fh=C3rMIGKHWoyxqoAraqqJMAgcBCOzfE2wKLTb5YQG2FY=; b=XbAn1LwvaRbf2/fYiuEC1op0E6bYJhaW9631H0PhLoHeCxXHVs/LrZCAWjQ0x0AKWk X8RryeIibQe7npefrdiRxiVQx7JzXeMwJ3sEt9MXAwFGrD+Ik/+758LhfD48Hn8VMTYm yxarLryUe3RHXF5j+Wk4PFkxsGE381BjdJkc0fNZqnCV/LBax7mClsHdwlL/xH80ni6+ 9lm5GmeysYewWuGyZWHI5XoPy0TVzzz034ciKtn68IATtJAKJgOW9c1ZBjgAmpKlBjlh 3xKXWkQ2V82oK33eA1W3TTuBheSjqX9sgVajTgMW0k+c+tdjpVqJNJ0D1mmwXGiFdNdR +Jpw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=UjtG8HGD; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id y16-20020a170902b49000b001b0044f186fsi3396439plr.41.2023.08.05.12.25.04; Sat, 05 Aug 2023 12:25:17 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=UjtG8HGD; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230182AbjHERnM (ORCPT + 99 others); Sat, 5 Aug 2023 13:43:12 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57600 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230180AbjHERnJ (ORCPT ); Sat, 5 Aug 2023 13:43:09 -0400 Received: from mail-pf1-x429.google.com (mail-pf1-x429.google.com [IPv6:2607:f8b0:4864:20::429]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 46D634495 for ; Sat, 5 Aug 2023 10:42:41 -0700 (PDT) Received: by mail-pf1-x429.google.com with SMTP id d2e1a72fcca58-68783004143so2214839b3a.2 for ; Sat, 05 Aug 2023 10:42:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1691257349; x=1691862149; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=VzjUVbmqujkXx5jszw63KmEj0mqPrkYgNLPBEumwUM4=; b=UjtG8HGD0HXhbYE6KOMjYvoRrgHtcqD2ZE8722LK6PTedcUT6bAAuZT3T5lZa/0BFq dpRdRzNQV2xMdM2b+6b+GcCfiDZ6u5eMcDb9c1Zuc/Ng3RH3SLGtds/VgBpuZ4HfHyxt KHokfIMsu0H0rfXzPOoDDFVbVxs+V+ke7GyWVrUygP8tu3B+HsJyM6cVo2gxbMgP3Wli ypdb3XWpikZowDgZgaAy5c/UImiho4fNo3gUdTiqDDJwF1a67JSsrBJ+dM6M40sxaZv0 otPqs4LjwZvCsIHBKI59i3308tE/Q2IfLhO0XhkJSFtZ1npPdl466eg2XEpsx0lh9jJl gP5w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691257349; x=1691862149; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=VzjUVbmqujkXx5jszw63KmEj0mqPrkYgNLPBEumwUM4=; b=dT1HkOQmtlkJdbsDpq9brk5Nq3F4smr5dZZl1gd7muspdL0SjEkMTLOVMo5mU4gsMU 0EOYUdVB2KGlJyvzkIMJDL5dvXMBp9efr5wPF3va5dCwIZR7oX+GGQPhPlfiUDC8vx4Q fvxbT+1ZsuofIQ9rwPMy5TWEu8D7cnflOcNLOskJIQk5ZmY2plZnOM0gBy64tKoJcK1l F4IFsYlqWxzEH4uRUbJ7efbSh3uHq9zXdtEmTOH66RKGiZSx9lA+pmfG9BpOD3VBajOt +0vH4c71Uy+2fvJFTAJ79ht9pVsu+C9N8GV1FzXcNHodntlJHQedk4n/TJhbyw6X4ciF uAvg== X-Gm-Message-State: AOJu0YxBEzCnlyXoHYO6aJuKE06pM8F8SOic2Anc5xgi2dv3DUqpzCIv eY1xOfu2fGl3JPnXK8Cvxod2 X-Received: by 2002:a05:6a00:114d:b0:687:3bc2:58de with SMTP id b13-20020a056a00114d00b006873bc258demr4086067pfm.6.1691257349344; Sat, 05 Aug 2023 10:42:29 -0700 (PDT) Received: from localhost.localdomain ([103.28.246.21]) by smtp.gmail.com with ESMTPSA id i14-20020aa78b4e000000b006874a6850e9sm3363160pfd.215.2023.08.05.10.42.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 05 Aug 2023 10:42:29 -0700 (PDT) From: Manivannan Sadhasivam To: miquel.raynal@bootlin.com, vigneshr@ti.com Cc: richard@nod.at, linux-mtd@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, quic_mdalam@quicinc.com, quic_srichara@quicinc.com, dan.carpenter@linaro.org, Manivannan Sadhasivam Subject: [PATCH 08/10] mtd: rawnand: qcom: Simplify the call to nand_prog_page_end_op() Date: Sat, 5 Aug 2023 23:11:44 +0530 Message-Id: <20230805174146.57006-9-manivannan.sadhasivam@linaro.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> References: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> MIME-Version: 1.0 X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,RCVD_IN_SBL, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1773418334775411569 X-GMAIL-MSGID: 1773418334775411569 Now that the dma desc cleanup is moved inside submit_descs(), let's simplify the call to nand_prog_page_end_op() inside qcom_nandc_write_page() and qcom_nandc_write_page_raw() to match other functions. Signed-off-by: Manivannan Sadhasivam --- drivers/mtd/nand/raw/qcom_nandc.c | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c index 1dbe17dcdb84..6b81781aa3ad 100644 --- a/drivers/mtd/nand/raw/qcom_nandc.c +++ b/drivers/mtd/nand/raw/qcom_nandc.c @@ -2016,13 +2016,12 @@ static int qcom_nandc_write_page(struct nand_chip *chip, const u8 *buf, } ret = submit_descs(nandc); - if (ret) + if (ret) { dev_err(nandc->dev, "failure to write page\n"); + return ret; + } - if (!ret) - ret = nand_prog_page_end_op(chip); - - return ret; + return nand_prog_page_end_op(chip); } /* implements ecc->write_page_raw() */ @@ -2090,13 +2089,12 @@ static int qcom_nandc_write_page_raw(struct nand_chip *chip, } ret = submit_descs(nandc); - if (ret) + if (ret) { dev_err(nandc->dev, "failure to write raw page\n"); + return ret; + } - if (!ret) - ret = nand_prog_page_end_op(chip); - - return ret; + return nand_prog_page_end_op(chip); } /* From patchwork Sat Aug 5 17:41:45 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Manivannan Sadhasivam X-Patchwork-Id: 131526 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:c44e:0:b0:3f2:4152:657d with SMTP id w14csp594635vqr; Sat, 5 Aug 2023 11:59:50 -0700 (PDT) X-Google-Smtp-Source: AGHT+IFm1hr//3R3gCNrkym2CrtPkHnxzB2rS/WqrNqrvqhBtDCcPv/hxByKObVcXq2u8ASlJP/o X-Received: by 2002:a05:6a20:550a:b0:131:5995:340 with SMTP id ko10-20020a056a20550a00b0013159950340mr4187542pzb.20.1691261990596; Sat, 05 Aug 2023 11:59:50 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1691261990; cv=none; d=google.com; s=arc-20160816; b=Smz8+ok7VxV1Fd/SDlWhV0JQTV7Idr2LR8nJX0Pi9AQ+9Q61VB1KVPOdXC8ov9hEmA pIKdlG2Epn5O4CF5KcB9OPAYBGn6h235hNfyhrokMAULKHqjF2MEuzrna5r8FmW8PrkR hhGGyLUyg3zzsJVKtXOyqbMHwZhjA1h7/fRfp2FQyA7KUAG+qXLqaqO9ttJLZ9Feh6TA 4DwUz/UEKN8xiA9RzQNTjcF6/u3QEaDdOBx4EM/gtw5rekHqwNf428B7iarWLosDvTJW FrlCqOAdiyx+Muf21vEdGA66VJAe2lWzaT3Fix0CqFBOI91jgKEKmezB3fpSbJLSKigU GwXw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=QJ7ZrjoLTJ5efHJXOrPgGJNKRMjipI8+wvO528txH4s=; fh=C3rMIGKHWoyxqoAraqqJMAgcBCOzfE2wKLTb5YQG2FY=; b=CnXLFBRchBZVf2fuV96+o5InTRxzUa7Ly0Nu0Y0NcBx4PEvhbDnKBI5HQBXBZOJrXV uT0+y1N0IbSCqTVhd6qgbD2taQ0wZ5X8GcwKGRTEee5F3vvmP76lDvxzh02nU6z4U7Gn NiwVvIxKrW5hedNzhSPpKR74m78Zj343O9M9gMeuwyIo+DTKzwMfUgNbGZgfAYeknJVS Glhsa+Lte7UjI86YrD9YPl5ZOEbNAIxDriFKukRK4hsoOncPmKlbFMc+dRt05LD7pN3A fvZWBmziSkS2qDK2+JNI5uZULqVFkJntDjXhEFEjjCbO+YtKoZoP6gxDZjb5NOjRhm2Z 8H7g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=B7+DzBY2; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id k63-20020a638442000000b0055c7d9edbc5si3483670pgd.174.2023.08.05.11.59.37; Sat, 05 Aug 2023 11:59:50 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=B7+DzBY2; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230191AbjHERnU (ORCPT + 99 others); Sat, 5 Aug 2023 13:43:20 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57630 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230185AbjHERnN (ORCPT ); Sat, 5 Aug 2023 13:43:13 -0400 Received: from mail-pf1-x436.google.com (mail-pf1-x436.google.com [IPv6:2607:f8b0:4864:20::436]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6B30A3C3A for ; Sat, 5 Aug 2023 10:42:48 -0700 (PDT) Received: by mail-pf1-x436.google.com with SMTP id d2e1a72fcca58-686bc261111so2241861b3a.3 for ; Sat, 05 Aug 2023 10:42:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1691257352; x=1691862152; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=QJ7ZrjoLTJ5efHJXOrPgGJNKRMjipI8+wvO528txH4s=; b=B7+DzBY2BMwN0TLLydmymYkwIsWR3JxW+iSLL/V0XXHiIcE3NOr18Mx0HwbSsKBhWO iSMn8u1sw5GfUeTPmKYFpey/hPwsfNH6CjWl5ihSCT5no007WqjK5pReXK+gZlb0WAY0 KWbAiyuDbbu8HgDPCO7iP2Xtsrb4JM+LVllxFOk18UAS3QLkqjL99MFkl5burHWLq8Ir dd9XoQiNvoT4dchGITvTV03HB+KUPoGQOEGX1HBN8XR6LVkZ2PfQZpG5jXn0Bh5ciSIk BBuCkn1sWiwt9otxwf0K5C3sJ9ddgPDJt6mLgXI8STYQ1qbePkvSExEqvAz2MaeMIxs1 Mrkw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691257352; x=1691862152; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=QJ7ZrjoLTJ5efHJXOrPgGJNKRMjipI8+wvO528txH4s=; b=im2bgO0+/krq1+aEDmfoREY9lfx/sprBY7wW7ty9HAVtpxSpae9jXv1CL+OHn5j95o Y/ijo2khvongqqqjnQsteU9rDFWSuBR9yAGlrigq4jOQZk6j64LYX5GB7v9Yf3p73Awt XqxPD83z254dHFkR/X8dAclFzzJ2Gprj/kSX07RJvcCcXa6onSaJxH/j/MC1oHJnReMF ORzzZ80zljNFgn66c2HGs3kUEZZVHqnhj0oxppvi2e90EkXXWeZAhjpNyuvkNnXzE1r1 ITiVsi3RBcvy4lCCmYuKaDv4Cmnnx+xFiNmRTOOfzOAjhcxKXh+paFqIGcCCNlJ7GpfN s9qw== X-Gm-Message-State: AOJu0YyzQW8lgmAbKsBB9jwTXc8jTg/vI884pL8njKBXDnLGdARmP5b5 1yEAEkWaQ9vvVW/++VEeWK2R X-Received: by 2002:a05:6a00:814:b0:687:78fc:e961 with SMTP id m20-20020a056a00081400b0068778fce961mr5558898pfk.22.1691257352461; Sat, 05 Aug 2023 10:42:32 -0700 (PDT) Received: from localhost.localdomain ([103.28.246.21]) by smtp.gmail.com with ESMTPSA id i14-20020aa78b4e000000b006874a6850e9sm3363160pfd.215.2023.08.05.10.42.29 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 05 Aug 2023 10:42:32 -0700 (PDT) From: Manivannan Sadhasivam To: miquel.raynal@bootlin.com, vigneshr@ti.com Cc: richard@nod.at, linux-mtd@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, quic_mdalam@quicinc.com, quic_srichara@quicinc.com, dan.carpenter@linaro.org, Manivannan Sadhasivam Subject: [PATCH 09/10] mtd: rawnand: qcom: Do not override the error no of submit_descs() Date: Sat, 5 Aug 2023 23:11:45 +0530 Message-Id: <20230805174146.57006-10-manivannan.sadhasivam@linaro.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> References: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> MIME-Version: 1.0 X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,RCVD_IN_SBL, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1773416733273373169 X-GMAIL-MSGID: 1773416733273373169 Just use the error no returned by submit_descs() instead of overriding it with -EIO. Signed-off-by: Manivannan Sadhasivam --- drivers/mtd/nand/raw/qcom_nandc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c index 6b81781aa3ad..0fbc6d1a558c 100644 --- a/drivers/mtd/nand/raw/qcom_nandc.c +++ b/drivers/mtd/nand/raw/qcom_nandc.c @@ -2140,7 +2140,7 @@ static int qcom_nandc_write_oob(struct nand_chip *chip, int page) ret = submit_descs(nandc); if (ret) { dev_err(nandc->dev, "failure to write oob\n"); - return -EIO; + return ret; } return nand_prog_page_end_op(chip); @@ -2216,7 +2216,7 @@ static int qcom_nandc_block_markbad(struct nand_chip *chip, loff_t ofs) ret = submit_descs(nandc); if (ret) { dev_err(nandc->dev, "failure to update BBM\n"); - return -EIO; + return ret; } return nand_prog_page_end_op(chip); From patchwork Sat Aug 5 17:41:46 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Manivannan Sadhasivam X-Patchwork-Id: 131525 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:c44e:0:b0:3f2:4152:657d with SMTP id w14csp593996vqr; Sat, 5 Aug 2023 11:57:28 -0700 (PDT) X-Google-Smtp-Source: AGHT+IFihtpom/kt3u0WoK5TkoARwyFH1K1+jU1kLYO2FjkH3tzAz0jqQPc+sbQ7fU85cNvaParE X-Received: by 2002:a05:6a00:21d5:b0:687:7d09:c0f with SMTP id t21-20020a056a0021d500b006877d090c0fmr6481046pfj.12.1691261848164; Sat, 05 Aug 2023 11:57:28 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1691261848; cv=none; d=google.com; s=arc-20160816; b=EvyKReVH8zNrbjaKYS14EMz9z3BYeA9TbEa8WEeJKDeojbna+wyos3Mk5tF7jRanrz lPICulI4EV2cStwbqsgF0yL1/NqqqyMtPy6sJafpl7A0COxefPaG5Pb1j7Re67TXDppx qhzXBYSfxZoOI61e9FYX5nXsetWTJwxvhcm5Ay/iB470qYqZ1eSJvNR1KvaHbcxdfyWW 6We84+s2GsRKroTKQSDiFJBfBmQqeas634sZYdQ+WwK5ge7MW0DLoxq+w+NqZZQmw+s5 nSOzQ6mRJMTLLLl7rqUUl47TRWDD4NeRNDYv7epKhtsWUyX/SL5nvFlCceEcD525zsnH BbTA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=HN0f2LOKeZ6eEoWkB3wmreNy3AFR6iUd1H9kbvp/ehA=; fh=C3rMIGKHWoyxqoAraqqJMAgcBCOzfE2wKLTb5YQG2FY=; b=AcSI9t5kWOUV0hosx61dHKl/wEIzr07/edSK+yLGTrR7wlgO0Xb4jtWuYc8oIjSV7P IX/T+5yY7Al9CKfz0KejgwGj4MAo0XrC/ytpUB2ad02eqTMRFY/b2Fr59aeBeD09TPAL ec5BvdrLPzruTJxk9c4Bkelz6LvXhAKfacVGnGENcnPMXLpouX35jqJMapigDNxFLVC8 XeSa82JuHQZ7cdG/k0kM8GMS0+lPX2z42PYw1titFIVy1zVGFMfkTqystnKwtzjj06tl x+hb7aEspAhs4j57xkK04oC2Fhba7uCk0N1vyym8eE2NTnWJdDJe3nXkfeTi5ThcV5DE /0VA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=kAh3rzHl; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id y25-20020a056a00181900b006872b7b1b69si3494507pfa.353.2023.08.05.11.57.15; Sat, 05 Aug 2023 11:57:28 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=kAh3rzHl; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229657AbjHERn3 (ORCPT + 99 others); Sat, 5 Aug 2023 13:43:29 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57822 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229580AbjHERnW (ORCPT ); Sat, 5 Aug 2023 13:43:22 -0400 Received: from mail-pg1-x52f.google.com (mail-pg1-x52f.google.com [IPv6:2607:f8b0:4864:20::52f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C4917420F for ; Sat, 5 Aug 2023 10:42:56 -0700 (PDT) Received: by mail-pg1-x52f.google.com with SMTP id 41be03b00d2f7-563e860df0fso2208259a12.2 for ; Sat, 05 Aug 2023 10:42:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1691257355; x=1691862155; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=HN0f2LOKeZ6eEoWkB3wmreNy3AFR6iUd1H9kbvp/ehA=; b=kAh3rzHl75lVGO7PZrVPAypy0Oexu0VKTje94TlCGbSduQ+Man2JdW9EbqXb7oNuJD Uc3fItb3E9yDaFi0dayjguINssLDunrCXo2we89n4zjze6JekLxTQhV2RiiZZ+gDv7lX /c0EDF4xL8NMKtp69j9pc0+k17hR3sHwo3tzmrcK4/yyJHT7B669scqmRJYcsENqdLrv 0ecYm0IuWiMAFg3OZFHOn+GLL+ovcplRgWCNezFF2UiiKENlKru/EskP01YIqKX2lLXt nbrTaCNxB5UZnoGRuDcowQQQZ9NKJyqpMoFbO/B8sWhj7cu4zeeNdHMISM4JObt+soxW MSPA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691257355; x=1691862155; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=HN0f2LOKeZ6eEoWkB3wmreNy3AFR6iUd1H9kbvp/ehA=; b=GAYBGfA5u7qknrm9Xgfdam9mqeIA+raWIc0jIqx5sevOYb4oDp1vC7zWDIjQ8Pttv9 NQhrhFyXG7qBYvkxu5gWPe8G+QHl/emyWF3ACTd/rrrvHPcshyxxgnNmusRrtkYD27Dk pQB3kGUKHDoGJD7UgF8dLspUj596fdm4sIfpACTnwekxOUUyNo9lH3363nmf1kqQpH3Y b0YCKQbqYAgk98vbL8BFN6nrevRF1nqewNFJqrLV0AD3fJkpmQl2TNazfw3+pcrUBmAl sa2qIbtwEoVy64ikgQoKPe/rgTi5dKbhaQocuH27VP05tbW3gjQThZi9lQFc2Ip+SF18 nfgg== X-Gm-Message-State: AOJu0YxoewNOqmd2uLYE9ft5N8QLP+6RRDZr/Tk/FBd3SJ1UElAArDWb n3q+wI02ltqMPvsJmxP8jrIY X-Received: by 2002:a05:6a21:789a:b0:133:7276:324b with SMTP id bf26-20020a056a21789a00b001337276324bmr6489967pzc.23.1691257355606; Sat, 05 Aug 2023 10:42:35 -0700 (PDT) Received: from localhost.localdomain ([103.28.246.21]) by smtp.gmail.com with ESMTPSA id i14-20020aa78b4e000000b006874a6850e9sm3363160pfd.215.2023.08.05.10.42.32 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 05 Aug 2023 10:42:35 -0700 (PDT) From: Manivannan Sadhasivam To: miquel.raynal@bootlin.com, vigneshr@ti.com Cc: richard@nod.at, linux-mtd@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, quic_mdalam@quicinc.com, quic_srichara@quicinc.com, dan.carpenter@linaro.org, Manivannan Sadhasivam Subject: [PATCH 10/10] mtd: rawnand: qcom: Sort includes alphabetically Date: Sat, 5 Aug 2023 23:11:46 +0530 Message-Id: <20230805174146.57006-11-manivannan.sadhasivam@linaro.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> References: <20230805174146.57006-1-manivannan.sadhasivam@linaro.org> MIME-Version: 1.0 X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,RCVD_IN_SBL, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1773416583646207459 X-GMAIL-MSGID: 1773416583646207459 Sort includes in alphabetical order. Signed-off-by: Manivannan Sadhasivam --- drivers/mtd/nand/raw/qcom_nandc.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c index 0fbc6d1a558c..d4ba0d04c970 100644 --- a/drivers/mtd/nand/raw/qcom_nandc.c +++ b/drivers/mtd/nand/raw/qcom_nandc.c @@ -2,19 +2,19 @@ /* * Copyright (c) 2016, The Linux Foundation. All rights reserved. */ -#include -#include -#include #include -#include -#include +#include +#include #include +#include +#include +#include #include -#include #include +#include #include -#include -#include +#include +#include /* NANDc reg offsets */ #define NAND_FLASH_CMD 0x00