From: "Yang, Wenyou" <Wenyou.Yang@Microchip.com>
To: Masahiro Yamada <yamada.masahiro@socionext.com>,
<linux-mtd@lists.infradead.org>
Cc: Cyrille Pitchen <cyrille.pitchen@wedev4u.fr>,
Josh Wu <rainyfeeling@outlook.com>,
Wan ZongShun <mcuos.com@gmail.com>,
<linux-kernel@vger.kernel.org>,
Boris Brezillon <boris.brezillon@free-electrons.com>,
Marek Vasut <marek.vasut@gmail.com>,
Brian Norris <computersforpeace@gmail.com>,
Richard Weinberger <richard@nod.at>,
David Woodhouse <dwmw2@infradead.org>,
Wenyou Yang <wenyou.yang@atmel.com>,
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v2 1/2] mtd: nand: introduce NAND_ROW_ADDR_3 flag
Date: Wed, 13 Sep 2017 10:45:36 +0800 [thread overview]
Message-ID: <b7bc2678-bb43-07e1-4661-1510d8518412@Microchip.com> (raw)
In-Reply-To: <1505268351-31941-2-git-send-email-yamada.masahiro@socionext.com>
On 2017/9/13 10:05, Masahiro Yamada wrote:
> Several drivers check ->chipsize to see if the third row address cycle
> is needed. Instead of embedding magic sizes such as 32MB, 128MB in
> drivers, introduce a new flag NAND_ROW_ADDR_3 for clean-up. Since
> nand_scan_ident() knows well about the device, it can handle this
> properly. The flag is set if the row address bit width is greater
> than 16.
>
> Delete comments such as "One more address cycle for ..." because
> intention is now clear enough from the code.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
For atmel/nand-controller,
Acked-by: Wenyou Yang <wenyou.yang@microchip.com>
> ---
>
> Changes in v2:
> - Fix build error
>
> drivers/mtd/nand/atmel/nand-controller.c | 3 +--
> drivers/mtd/nand/au1550nd.c | 3 +--
> drivers/mtd/nand/diskonchip.c | 3 +--
> drivers/mtd/nand/hisi504_nand.c | 3 +--
> drivers/mtd/nand/mxc_nand.c | 3 +--
> drivers/mtd/nand/nand_base.c | 9 +++++----
> drivers/mtd/nand/nuc900_nand.c | 2 +-
> include/linux/mtd/rawnand.h | 3 +++
> 8 files changed, 14 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/mtd/nand/atmel/nand-controller.c b/drivers/mtd/nand/atmel/nand-controller.c
> index f25eca7..7bc8d20 100644
> --- a/drivers/mtd/nand/atmel/nand-controller.c
> +++ b/drivers/mtd/nand/atmel/nand-controller.c
> @@ -718,8 +718,7 @@ static void atmel_nfc_set_op_addr(struct nand_chip *chip, int page, int column)
> nc->op.addrs[nc->op.naddrs++] = page;
> nc->op.addrs[nc->op.naddrs++] = page >> 8;
>
> - if ((mtd->writesize > 512 && chip->chipsize > SZ_128M) ||
> - (mtd->writesize <= 512 && chip->chipsize > SZ_32M))
> + if (chip->options & NAND_ROW_ADDR_3)
> nc->op.addrs[nc->op.naddrs++] = page >> 16;
> }
> }
> diff --git a/drivers/mtd/nand/au1550nd.c b/drivers/mtd/nand/au1550nd.c
> index 9d4a28f..8ab827e 100644
> --- a/drivers/mtd/nand/au1550nd.c
> +++ b/drivers/mtd/nand/au1550nd.c
> @@ -331,8 +331,7 @@ static void au1550_command(struct mtd_info *mtd, unsigned command, int column, i
>
> ctx->write_byte(mtd, (u8)(page_addr >> 8));
>
> - /* One more address cycle for devices > 32MiB */
> - if (this->chipsize > (32 << 20))
> + if (this->options & NAND_ROW_ADDR_3)
> ctx->write_byte(mtd,
> ((page_addr >> 16) & 0x0f));
> }
> diff --git a/drivers/mtd/nand/diskonchip.c b/drivers/mtd/nand/diskonchip.c
> index c3aa53c..72671dc 100644
> --- a/drivers/mtd/nand/diskonchip.c
> +++ b/drivers/mtd/nand/diskonchip.c
> @@ -705,8 +705,7 @@ static void doc2001plus_command(struct mtd_info *mtd, unsigned command, int colu
> if (page_addr != -1) {
> WriteDOC((unsigned char)(page_addr & 0xff), docptr, Mplus_FlashAddress);
> WriteDOC((unsigned char)((page_addr >> 8) & 0xff), docptr, Mplus_FlashAddress);
> - /* One more address cycle for higher density devices */
> - if (this->chipsize & 0x0c000000) {
> + if (this->options & NAND_ROW_ADDR_3) {
> WriteDOC((unsigned char)((page_addr >> 16) & 0x0f), docptr, Mplus_FlashAddress);
> printk("high density\n");
> }
> diff --git a/drivers/mtd/nand/hisi504_nand.c b/drivers/mtd/nand/hisi504_nand.c
> index d9ee1a7..0897261 100644
> --- a/drivers/mtd/nand/hisi504_nand.c
> +++ b/drivers/mtd/nand/hisi504_nand.c
> @@ -432,8 +432,7 @@ static void set_addr(struct mtd_info *mtd, int column, int page_addr)
> host->addr_value[0] |= (page_addr & 0xffff)
> << (host->addr_cycle * 8);
> host->addr_cycle += 2;
> - /* One more address cycle for devices > 128MiB */
> - if (chip->chipsize > (128 << 20)) {
> + if (chip->options & NAND_ROW_ADDR_3) {
> host->addr_cycle += 1;
> if (host->command == NAND_CMD_ERASE1)
> host->addr_value[0] |= ((page_addr >> 16) & 0xff) << 16;
> diff --git a/drivers/mtd/nand/mxc_nand.c b/drivers/mtd/nand/mxc_nand.c
> index 53e5e03..bacdd04 100644
> --- a/drivers/mtd/nand/mxc_nand.c
> +++ b/drivers/mtd/nand/mxc_nand.c
> @@ -859,8 +859,7 @@ static void mxc_do_addr_cycle(struct mtd_info *mtd, int column, int page_addr)
> host->devtype_data->send_addr(host,
> (page_addr >> 8) & 0xff, true);
> } else {
> - /* One more address cycle for higher density devices */
> - if (mtd->size >= 0x4000000) {
> + if (nand_chip->options & NAND_ROW_ADDR_3) {
> /* paddr_8 - paddr_15 */
> host->devtype_data->send_addr(host,
> (page_addr >> 8) & 0xff,
> diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c
> index bcc8cef1..3bc4404 100644
> --- a/drivers/mtd/nand/nand_base.c
> +++ b/drivers/mtd/nand/nand_base.c
> @@ -727,8 +727,7 @@ static void nand_command(struct mtd_info *mtd, unsigned int command,
> chip->cmd_ctrl(mtd, page_addr, ctrl);
> ctrl &= ~NAND_CTRL_CHANGE;
> chip->cmd_ctrl(mtd, page_addr >> 8, ctrl);
> - /* One more address cycle for devices > 32MiB */
> - if (chip->chipsize > (32 << 20))
> + if (chip->options & NAND_ROW_ADDR_3)
> chip->cmd_ctrl(mtd, page_addr >> 16, ctrl);
> }
> chip->cmd_ctrl(mtd, NAND_CMD_NONE, NAND_NCE | NAND_CTRL_CHANGE);
> @@ -854,8 +853,7 @@ static void nand_command_lp(struct mtd_info *mtd, unsigned int command,
> chip->cmd_ctrl(mtd, page_addr, ctrl);
> chip->cmd_ctrl(mtd, page_addr >> 8,
> NAND_NCE | NAND_ALE);
> - /* One more address cycle for devices > 128MiB */
> - if (chip->chipsize > (128 << 20))
> + if (chip->options & NAND_ROW_ADDR_3)
> chip->cmd_ctrl(mtd, page_addr >> 16,
> NAND_NCE | NAND_ALE);
> }
> @@ -4000,6 +3998,9 @@ static int nand_detect(struct nand_chip *chip, struct nand_flash_dev *type)
> chip->chip_shift += 32 - 1;
> }
>
> + if (chip->chip_shift - chip->page_shift > 16)
> + chip->options |= NAND_ROW_ADDR_3;
> +
> chip->badblockbits = 8;
> chip->erase = single_erase;
>
> diff --git a/drivers/mtd/nand/nuc900_nand.c b/drivers/mtd/nand/nuc900_nand.c
> index 7bb4d2e..af5b32c9 100644
> --- a/drivers/mtd/nand/nuc900_nand.c
> +++ b/drivers/mtd/nand/nuc900_nand.c
> @@ -154,7 +154,7 @@ static void nuc900_nand_command_lp(struct mtd_info *mtd, unsigned int command,
> if (page_addr != -1) {
> write_addr_reg(nand, page_addr);
>
> - if (chip->chipsize > (128 << 20)) {
> + if (chip->options & NAND_ROW_ADDR_3) {
> write_addr_reg(nand, page_addr >> 8);
> write_addr_reg(nand, page_addr >> 16 | ENDADDR);
> } else {
> diff --git a/include/linux/mtd/rawnand.h b/include/linux/mtd/rawnand.h
> index 2b05f42..749bb08 100644
> --- a/include/linux/mtd/rawnand.h
> +++ b/include/linux/mtd/rawnand.h
> @@ -177,6 +177,9 @@ enum nand_ecc_algo {
> */
> #define NAND_NEED_SCRAMBLING 0x00002000
>
> +/* Device needs 3rd row address cycle */
> +#define NAND_ROW_ADDR_3 0x00004000
> +
> /* Options valid for Samsung large page devices */
> #define NAND_SAMSUNG_LP_OPTIONS NAND_CACHEPRG
>
Regards,
Wenyou Yang
next prev parent reply other threads:[~2017-09-13 2:45 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-13 2:05 [PATCH v2 0/2] mtd: nand: introduce NAND_ROW_ADDR_3 flag and improve Denali driver Masahiro Yamada
2017-09-13 2:05 ` [PATCH v2 1/2] mtd: nand: introduce NAND_ROW_ADDR_3 flag Masahiro Yamada
2017-09-13 2:45 ` Yang, Wenyou [this message]
2017-09-13 2:05 ` [PATCH v2 2/2] mtd: nand: denali: support two row address cycle devices Masahiro Yamada
2017-09-22 10:08 ` [PATCH v2 0/2] mtd: nand: introduce NAND_ROW_ADDR_3 flag and improve Denali driver Boris Brezillon
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=b7bc2678-bb43-07e1-4661-1510d8518412@Microchip.com \
--to=wenyou.yang@microchip.com \
--cc=boris.brezillon@free-electrons.com \
--cc=computersforpeace@gmail.com \
--cc=cyrille.pitchen@wedev4u.fr \
--cc=dwmw2@infradead.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=marek.vasut@gmail.com \
--cc=mcuos.com@gmail.com \
--cc=rainyfeeling@outlook.com \
--cc=richard@nod.at \
--cc=wenyou.yang@atmel.com \
--cc=yamada.masahiro@socionext.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox