From: Sascha Hauer <s.hauer@pengutronix.de>
To: Han Xu <han.xu@nxp.com>
Cc: linux-mtd@lists.infradead.org, miquel.raynal@bootlin.com
Subject: Re: [PATCH 1/2] mtd: rawnand: gpmi: Fix the driver only sense CS0 R/B issue
Date: Mon, 7 Dec 2020 10:50:06 +0100 [thread overview]
Message-ID: <20201207095006.GK3977@pengutronix.de> (raw)
In-Reply-To: <20201205063004.3099-1-han.xu@nxp.com>
On Sat, Dec 05, 2020 at 12:30:03AM -0600, Han Xu wrote:
> set the GPMI CTRL1 GANGED_RDYBUSY bit so dirver can sense the R/B signal
> from all CS.
>
> For the NAND chip MT29F64G08AFAAAWP, only the first chip detected
> without the patch.
>
> [ 3.764118] nand: device found, Manufacturer ID: 0x2c, Chip ID: 0x68
> [ 3.770613] nand: Micron MT29F64G08AFAAAWP
> [ 3.774752] nand: 4096 MiB, SLC, erase size: 1024 KiB, page size: 8192, OOB size: 448
> [ 3.786421] Bad block table found at page 524160, version 0x01
> [ 3.792730] Bad block table found at page 524032, version 0x01
>
> After applying the patch
>
> [ 3.764445] nand: device found, Manufacturer ID: 0x2c, Chip ID: 0x68
> [ 3.770941] nand: Micron MT29F64G08AFAAAWP
> [ 3.775080] nand: 4096 MiB, SLC, erase size: 1024 KiB, page size: 8192, OOB size: 448
> [ 3.784390] nand: 2 chips detected
> [ 3.790900] Bad block table found at page 524160, version 0x01
> [ 3.796776] Bad block table found at page 1048448, version 0x01
>
> Fixes: 3045f8e36963 ("mtd: rawnand: gpmi: move all driver code into single file")
I don't see how 3045f8e36963 changes his behaviour. Are you sure it
worked without this patch?
Sascha
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
next prev parent reply other threads:[~2020-12-07 9:51 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-12-05 6:30 [PATCH 1/2] mtd: rawnand: gpmi: Fix the driver only sense CS0 R/B issue Han Xu
2020-12-05 6:30 ` [PATCH 2/2] mtd: rawnand: gpmi: Fix the random DMA timeout issue Han Xu
2020-12-07 10:41 ` Sascha Hauer
2020-12-07 9:50 ` Sascha Hauer [this message]
2020-12-07 16:14 ` [EXT] Re: [PATCH 1/2] mtd: rawnand: gpmi: Fix the driver only sense CS0 R/B issue Han Xu
2020-12-07 16:43 ` Sascha Hauer
2020-12-08 21:53 ` Miquel Raynal
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=20201207095006.GK3977@pengutronix.de \
--to=s.hauer@pengutronix.de \
--cc=han.xu@nxp.com \
--cc=linux-mtd@lists.infradead.org \
--cc=miquel.raynal@bootlin.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.