All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arseniy Krasnov <AVKrasnov@sberdevices.ru>
To: Liang Yang <liang.yang@amlogic.com>,
	Miquel Raynal <miquel.raynal@bootlin.com>,
	Richard Weinberger <richard@nod.at>,
	Vignesh Raghavendra <vigneshr@ti.com>,
	Neil Armstrong <neil.armstrong@linaro.org>,
	Kevin Hilman <khilman@baylibre.com>,
	Jerome Brunet <jbrunet@baylibre.com>,
	Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Cc: <oxffffaa@gmail.com>, <kernel@sberdevices.ru>,
	Arseniy Krasnov <AVKrasnov@sberdevices.ru>,
	<linux-mtd@lists.infradead.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-amlogic@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>
Subject: [RFC PATCH v5 6/6] mtd: rawnand: meson: remove unneeded bitwise OR with zeroes
Date: Thu, 1 Jun 2023 09:18:49 +0300	[thread overview]
Message-ID: <20230601061850.3907800-7-AVKrasnov@sberdevices.ru> (raw)
In-Reply-To: <20230601061850.3907800-1-AVKrasnov@sberdevices.ru>

Both operations have no effect.

Signed-off-by: Arseniy Krasnov <AVKrasnov@sberdevices.ru>
---
 drivers/mtd/nand/raw/meson_nand.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/mtd/nand/raw/meson_nand.c b/drivers/mtd/nand/raw/meson_nand.c
index 2a91916566f4..e78912bd3b4d 100644
--- a/drivers/mtd/nand/raw/meson_nand.c
+++ b/drivers/mtd/nand/raw/meson_nand.c
@@ -626,12 +626,12 @@ static int meson_nfc_rw_cmd_prepare_and_execute(struct nand_chip *nand,
 	cmd0 = in ? NAND_CMD_READ0 : NAND_CMD_SEQIN;
 	nfc->cmdfifo.rw.cmd0 = cs | NFC_CMD_CLE | cmd0;
 
-	addrs[0] = cs | NFC_CMD_ALE | 0;
+	addrs[0] = cs | NFC_CMD_ALE;
 	if (mtd->writesize <= 512) {
 		cmd_num--;
 		row_start = 1;
 	} else {
-		addrs[1] = cs | NFC_CMD_ALE | 0;
+		addrs[1] = cs | NFC_CMD_ALE;
 		row_start = 2;
 	}
 
-- 
2.35.0


WARNING: multiple messages have this Message-ID (diff)
From: Arseniy Krasnov <AVKrasnov@sberdevices.ru>
To: Liang Yang <liang.yang@amlogic.com>,
	Miquel Raynal <miquel.raynal@bootlin.com>,
	Richard Weinberger <richard@nod.at>,
	Vignesh Raghavendra <vigneshr@ti.com>,
	Neil Armstrong <neil.armstrong@linaro.org>,
	Kevin Hilman <khilman@baylibre.com>,
	Jerome Brunet <jbrunet@baylibre.com>,
	Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Cc: <oxffffaa@gmail.com>, <kernel@sberdevices.ru>,
	Arseniy Krasnov <AVKrasnov@sberdevices.ru>,
	<linux-mtd@lists.infradead.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-amlogic@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>
Subject: [RFC PATCH v5 6/6] mtd: rawnand: meson: remove unneeded bitwise OR with zeroes
Date: Thu, 1 Jun 2023 09:18:49 +0300	[thread overview]
Message-ID: <20230601061850.3907800-7-AVKrasnov@sberdevices.ru> (raw)
In-Reply-To: <20230601061850.3907800-1-AVKrasnov@sberdevices.ru>

Both operations have no effect.

Signed-off-by: Arseniy Krasnov <AVKrasnov@sberdevices.ru>
---
 drivers/mtd/nand/raw/meson_nand.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/mtd/nand/raw/meson_nand.c b/drivers/mtd/nand/raw/meson_nand.c
index 2a91916566f4..e78912bd3b4d 100644
--- a/drivers/mtd/nand/raw/meson_nand.c
+++ b/drivers/mtd/nand/raw/meson_nand.c
@@ -626,12 +626,12 @@ static int meson_nfc_rw_cmd_prepare_and_execute(struct nand_chip *nand,
 	cmd0 = in ? NAND_CMD_READ0 : NAND_CMD_SEQIN;
 	nfc->cmdfifo.rw.cmd0 = cs | NFC_CMD_CLE | cmd0;
 
-	addrs[0] = cs | NFC_CMD_ALE | 0;
+	addrs[0] = cs | NFC_CMD_ALE;
 	if (mtd->writesize <= 512) {
 		cmd_num--;
 		row_start = 1;
 	} else {
-		addrs[1] = cs | NFC_CMD_ALE | 0;
+		addrs[1] = cs | NFC_CMD_ALE;
 		row_start = 2;
 	}
 
-- 
2.35.0


______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

WARNING: multiple messages have this Message-ID (diff)
From: Arseniy Krasnov <AVKrasnov@sberdevices.ru>
To: Liang Yang <liang.yang@amlogic.com>,
	Miquel Raynal <miquel.raynal@bootlin.com>,
	Richard Weinberger <richard@nod.at>,
	Vignesh Raghavendra <vigneshr@ti.com>,
	Neil Armstrong <neil.armstrong@linaro.org>,
	Kevin Hilman <khilman@baylibre.com>,
	Jerome Brunet <jbrunet@baylibre.com>,
	Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Cc: <oxffffaa@gmail.com>, <kernel@sberdevices.ru>,
	Arseniy Krasnov <AVKrasnov@sberdevices.ru>,
	<linux-mtd@lists.infradead.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-amlogic@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>
Subject: [RFC PATCH v5 6/6] mtd: rawnand: meson: remove unneeded bitwise OR with zeroes
Date: Thu, 1 Jun 2023 09:18:49 +0300	[thread overview]
Message-ID: <20230601061850.3907800-7-AVKrasnov@sberdevices.ru> (raw)
In-Reply-To: <20230601061850.3907800-1-AVKrasnov@sberdevices.ru>

Both operations have no effect.

Signed-off-by: Arseniy Krasnov <AVKrasnov@sberdevices.ru>
---
 drivers/mtd/nand/raw/meson_nand.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/mtd/nand/raw/meson_nand.c b/drivers/mtd/nand/raw/meson_nand.c
index 2a91916566f4..e78912bd3b4d 100644
--- a/drivers/mtd/nand/raw/meson_nand.c
+++ b/drivers/mtd/nand/raw/meson_nand.c
@@ -626,12 +626,12 @@ static int meson_nfc_rw_cmd_prepare_and_execute(struct nand_chip *nand,
 	cmd0 = in ? NAND_CMD_READ0 : NAND_CMD_SEQIN;
 	nfc->cmdfifo.rw.cmd0 = cs | NFC_CMD_CLE | cmd0;
 
-	addrs[0] = cs | NFC_CMD_ALE | 0;
+	addrs[0] = cs | NFC_CMD_ALE;
 	if (mtd->writesize <= 512) {
 		cmd_num--;
 		row_start = 1;
 	} else {
-		addrs[1] = cs | NFC_CMD_ALE | 0;
+		addrs[1] = cs | NFC_CMD_ALE;
 		row_start = 2;
 	}
 
-- 
2.35.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: Arseniy Krasnov <AVKrasnov@sberdevices.ru>
To: Liang Yang <liang.yang@amlogic.com>,
	Miquel Raynal <miquel.raynal@bootlin.com>,
	Richard Weinberger <richard@nod.at>,
	Vignesh Raghavendra <vigneshr@ti.com>,
	Neil Armstrong <neil.armstrong@linaro.org>,
	Kevin Hilman <khilman@baylibre.com>,
	Jerome Brunet <jbrunet@baylibre.com>,
	Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Cc: <oxffffaa@gmail.com>, <kernel@sberdevices.ru>,
	Arseniy Krasnov <AVKrasnov@sberdevices.ru>,
	<linux-mtd@lists.infradead.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-amlogic@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>
Subject: [RFC PATCH v5 6/6] mtd: rawnand: meson: remove unneeded bitwise OR with zeroes
Date: Thu, 1 Jun 2023 09:18:49 +0300	[thread overview]
Message-ID: <20230601061850.3907800-7-AVKrasnov@sberdevices.ru> (raw)
In-Reply-To: <20230601061850.3907800-1-AVKrasnov@sberdevices.ru>

Both operations have no effect.

Signed-off-by: Arseniy Krasnov <AVKrasnov@sberdevices.ru>
---
 drivers/mtd/nand/raw/meson_nand.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/mtd/nand/raw/meson_nand.c b/drivers/mtd/nand/raw/meson_nand.c
index 2a91916566f4..e78912bd3b4d 100644
--- a/drivers/mtd/nand/raw/meson_nand.c
+++ b/drivers/mtd/nand/raw/meson_nand.c
@@ -626,12 +626,12 @@ static int meson_nfc_rw_cmd_prepare_and_execute(struct nand_chip *nand,
 	cmd0 = in ? NAND_CMD_READ0 : NAND_CMD_SEQIN;
 	nfc->cmdfifo.rw.cmd0 = cs | NFC_CMD_CLE | cmd0;
 
-	addrs[0] = cs | NFC_CMD_ALE | 0;
+	addrs[0] = cs | NFC_CMD_ALE;
 	if (mtd->writesize <= 512) {
 		cmd_num--;
 		row_start = 1;
 	} else {
-		addrs[1] = cs | NFC_CMD_ALE | 0;
+		addrs[1] = cs | NFC_CMD_ALE;
 		row_start = 2;
 	}
 
-- 
2.35.0


_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic

  parent reply	other threads:[~2023-06-01  6:24 UTC|newest]

Thread overview: 128+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-01  6:18 [RFC PATCH v5 0/6] refactoring, fixes and updates for Meson NAND Arseniy Krasnov
2023-06-01  6:18 ` Arseniy Krasnov
2023-06-01  6:18 ` Arseniy Krasnov
2023-06-01  6:18 ` Arseniy Krasnov
2023-06-01  6:18 ` [RFC PATCH v5 1/6] mtd: rawnand: meson: fix ready/busy command Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  7:51   ` Miquel Raynal
2023-06-01  7:51     ` Miquel Raynal
2023-06-01  7:51     ` Miquel Raynal
2023-06-01  7:51     ` Miquel Raynal
2023-06-01 22:44     ` Arseniy Krasnov
2023-06-01 22:44       ` Arseniy Krasnov
2023-06-01 22:44       ` Arseniy Krasnov
2023-06-01 22:44       ` Arseniy Krasnov
2023-06-05  7:08       ` Miquel Raynal
2023-06-05  7:08         ` Miquel Raynal
2023-06-05  7:08         ` Miquel Raynal
2023-06-05  7:08         ` Miquel Raynal
2023-06-01  6:18 ` [RFC PATCH v5 2/6] mtd: rawnand: meson: wait for command in polling mode Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  7:57   ` Arseniy Krasnov
2023-06-01  7:57     ` Arseniy Krasnov
2023-06-01  7:57     ` Arseniy Krasnov
2023-06-01  7:57     ` Arseniy Krasnov
2023-06-01  8:07   ` Miquel Raynal
2023-06-01  8:07     ` Miquel Raynal
2023-06-01  8:07     ` Miquel Raynal
2023-06-01  8:07     ` Miquel Raynal
2023-06-01 23:09     ` Arseniy Krasnov
2023-06-01 23:09       ` Arseniy Krasnov
2023-06-01 23:09       ` Arseniy Krasnov
2023-06-01 23:09       ` Arseniy Krasnov
2023-06-05  9:05       ` Miquel Raynal
2023-06-05  9:05         ` Miquel Raynal
2023-06-05  9:05         ` Miquel Raynal
2023-06-05  9:05         ` Miquel Raynal
2023-06-05 13:19         ` Liang Yang
2023-06-05 13:19           ` Liang Yang
2023-06-05 13:19           ` Liang Yang
2023-06-05 13:19           ` Liang Yang
2023-06-05 13:30           ` Liang Yang
2023-06-05 13:30             ` Liang Yang
2023-06-05 13:30             ` Liang Yang
2023-06-05 13:30             ` Liang Yang
2023-06-05 16:58             ` Arseniy Krasnov
2023-06-05 16:58               ` Arseniy Krasnov
2023-06-05 16:58               ` Arseniy Krasnov
2023-06-05 16:58               ` Arseniy Krasnov
2023-06-06  7:03               ` Miquel Raynal
2023-06-06  7:03                 ` Miquel Raynal
2023-06-06  7:03                 ` Miquel Raynal
2023-06-06  7:03                 ` Miquel Raynal
2023-06-06  7:40                 ` Arseniy Krasnov
2023-06-06  7:40                   ` Arseniy Krasnov
2023-06-06  7:40                   ` Arseniy Krasnov
2023-06-06  7:40                   ` Arseniy Krasnov
2023-06-06  7:55                   ` Miquel Raynal
2023-06-06  7:55                     ` Miquel Raynal
2023-06-06  7:55                     ` Miquel Raynal
2023-06-06  7:55                     ` Miquel Raynal
2023-06-06 11:49             ` Arseniy Krasnov
2023-06-06 11:49               ` Arseniy Krasnov
2023-06-06 11:49               ` Arseniy Krasnov
2023-06-06 11:49               ` Arseniy Krasnov
2023-06-06 12:11               ` Miquel Raynal
2023-06-06 12:11                 ` Miquel Raynal
2023-06-06 12:11                 ` Miquel Raynal
2023-06-06 12:11                 ` Miquel Raynal
2023-06-06 12:10                 ` Arseniy Krasnov
2023-06-06 12:10                   ` Arseniy Krasnov
2023-06-06 12:10                   ` Arseniy Krasnov
2023-06-06 12:10                   ` Arseniy Krasnov
2023-06-01  6:18 ` [RFC PATCH v5 3/6] mtd: rawnand: meson: only expose unprotected user OOB bytes Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  8:31   ` Miquel Raynal
2023-06-01  8:31     ` Miquel Raynal
2023-06-01  8:31     ` Miquel Raynal
2023-06-01  8:31     ` Miquel Raynal
2023-06-02  8:53     ` Arseniy Krasnov
2023-06-02  8:53       ` Arseniy Krasnov
2023-06-02  8:53       ` Arseniy Krasnov
2023-06-02  8:53       ` Arseniy Krasnov
2023-06-05  9:48       ` Miquel Raynal
2023-06-05  9:48         ` Miquel Raynal
2023-06-05  9:48         ` Miquel Raynal
2023-06-05  9:48         ` Miquel Raynal
2023-06-06  4:42         ` Arseniy Krasnov
2023-06-06  4:42           ` Arseniy Krasnov
2023-06-06  4:42           ` Arseniy Krasnov
2023-06-06  4:42           ` Arseniy Krasnov
2023-06-06  7:11           ` Miquel Raynal
2023-06-06  7:11             ` Miquel Raynal
2023-06-06  7:11             ` Miquel Raynal
2023-06-06  7:11             ` Miquel Raynal
2023-06-06  7:41             ` Arseniy Krasnov
2023-06-06  7:41               ` Arseniy Krasnov
2023-06-06  7:41               ` Arseniy Krasnov
2023-06-06  7:41               ` Arseniy Krasnov
2023-06-01  6:18 ` [RFC PATCH v5 4/6] mtd: rawnand: meson: use macro for OOB area Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  8:34   ` Miquel Raynal
2023-06-01  8:34     ` Miquel Raynal
2023-06-01  8:34     ` Miquel Raynal
2023-06-01  8:34     ` Miquel Raynal
2023-06-01  6:18 ` [RFC PATCH v5 5/6] mtd: rawnand: meson: check buffer length Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  6:18 ` Arseniy Krasnov [this message]
2023-06-01  6:18   ` [RFC PATCH v5 6/6] mtd: rawnand: meson: remove unneeded bitwise OR with zeroes Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  6:18   ` Arseniy Krasnov
2023-06-01  7:50 ` [RFC PATCH v5 0/6] refactoring, fixes and updates for Meson NAND Miquel Raynal
2023-06-01  7:50   ` Miquel Raynal
2023-06-01  7:50   ` Miquel Raynal
2023-06-01  7:50   ` Miquel Raynal
2023-06-01  7:51   ` Arseniy Krasnov
2023-06-01  7:51     ` Arseniy Krasnov
2023-06-01  7:51     ` Arseniy Krasnov
2023-06-01  7:51     ` Arseniy Krasnov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230601061850.3907800-7-AVKrasnov@sberdevices.ru \
    --to=avkrasnov@sberdevices.ru \
    --cc=jbrunet@baylibre.com \
    --cc=kernel@sberdevices.ru \
    --cc=khilman@baylibre.com \
    --cc=liang.yang@amlogic.com \
    --cc=linux-amlogic@lists.infradead.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=martin.blumenstingl@googlemail.com \
    --cc=miquel.raynal@bootlin.com \
    --cc=neil.armstrong@linaro.org \
    --cc=oxffffaa@gmail.com \
    --cc=richard@nod.at \
    --cc=vigneshr@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.