From: Marek Vasut <marex@denx.de>
To: Huang Shijie <b32955@freescale.com>
Cc: "fabio.estevam@freescale.com" <fabio.estevam@freescale.com>,
Hector Palacios <hector.palacios@digi.com>,
linux-mtd@lists.infradead.org, dzu@denx.de
Subject: Re: gpmi-nand driver and jffs2 support
Date: Mon, 30 Sep 2013 00:52:46 +0200 [thread overview]
Message-ID: <201309300052.46883.marex@denx.de> (raw)
In-Reply-To: <5242A805.50804@freescale.com>
Dear Huang Shijie,
> 于 2013年09月24日 20:53, Marek Vasut 写道:
> > Hello Huang,
> >
> >> 于 2013年09月24日 17:50, Marek Vasut 写道:
> >>> Hello Huang,
> >>>
> >>> Can you please explain what is the exact difference between the NAND
> >>> layout when using JFFS2 in FSL 2.6.35 and in mainline with your patches
> >>> ?
> >>
> >> In FSL 2.6.35, there are some free space in the OOB;
> >> while in mainline we use all the free space in the OOB.
> >
> > OK, let me understand this in more detail.
> >
> > How does the NAND page look when written by FSL 2.6.35 and how does the
> > NAND page look when written by current mainline ? Can you please tell in
> > detail?
>
> [1] for FSL 2.6.35, see gpmi_nfc_set_geometry():
> The ecc_strength is set at a fix value, such as:
> "set ecc_strength with 8 for page size 2048";
>
> Why set the ecc_strength with 8? just based on the experiences, not
> based on the
> chip's datasheets.
>
> So we can have some spare space in the OOB, after we setting the
> BCH with the ecc_strength.
>
> [2] for mainline, see common_nfc_set_geometry():
> Please see the diagrams in the set_geometry_by_ecc_info() and
> legacy_set_geometry().
>
>
> If we CAN use the ecc info from the chip's datasheet, just like in
> FSL_2.6.35, we may
> have some spare space in the OOB; else we use all the OOB.
This is the vital information, thank you!
So by applying this [1] and calling the legacy_set_geometry() instead of
set_geometry_by_ecc_info(), I can get the JFFS2 to work. Nice, thanks again!
[1] http://lists.infradead.org/pipermail/linux-mtd/2007-December/020047.html
> >>> Is there any way to mount JFFS2 written with FSL 2.6.35 on current
> >>> mainline? Thus far, neither me nor Hector were able to mount such
> >>> partition with the same result.
> >>
> >> The gpmi driver in FSL 2.6.35 is different from the mainline.
> >
> > I was under the impression the mainline driver is an evolution of the FSL
> > 2.6.35 driver. Is this assumption incorrect?
>
> yes.
>
> >> The two drivers are not compatible to each other.
> >
> > OK, like I said, if we cannot produce a drop-in replacement of old FSL
> > 2.6.35 kernel, this is a problem. People will come and they will be
> > running into this issue. Thus, let us focus on producing a compatible
> > JFFS2 patch.
>
> I do not think it is a problem. The FSL 2.6.35 does not support the JFFS2.
It does for SLC NAND.
Best regards,
Marek Vasut
prev parent reply other threads:[~2013-09-29 22:53 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-30 9:15 gpmi-nand driver and jffs2 support Hector Palacios
2013-08-30 9:55 ` Huang Shijie
2013-08-30 14:24 ` Hector Palacios
2013-08-31 2:53 ` Huang Shijie
2013-08-30 15:23 ` Hector Palacios
2013-08-31 3:34 ` Huang Shijie
2013-08-30 16:31 ` Hector Palacios
2013-08-30 16:37 ` Fabio Estevam
2013-08-30 16:41 ` Hector Palacios
2013-08-31 13:37 ` Huang Shijie
2013-09-02 8:12 ` Hector Palacios
2013-09-02 8:24 ` Huang Shijie
2013-09-02 8:42 ` Hector Palacios
2013-09-02 8:50 ` Huang Shijie
2013-09-02 10:10 ` Hector Palacios
2013-09-02 10:23 ` Huang Shijie
2013-09-02 11:32 ` Marek Vasut
2013-09-03 2:06 ` Huang Shijie
2013-09-03 11:53 ` Marek Vasut
2013-09-04 2:26 ` Huang Shijie
2013-09-04 14:00 ` Marek Vasut
2013-09-05 2:41 ` Huang Shijie
2013-09-04 14:38 ` Marek Vasut
2013-09-04 15:46 ` Hector Palacios
2013-09-05 6:01 ` Huang Shijie
2013-09-15 14:18 ` Marek Vasut
2013-09-16 2:35 ` Huang Shijie
2013-09-19 16:07 ` Marek Vasut
2013-09-19 16:13 ` Hector Palacios
2013-09-19 16:14 ` Hector Palacios
2013-09-19 16:16 ` Marek Vasut
2013-09-19 17:20 ` Marek Vasut
2013-09-15 14:12 ` Marek Vasut
2013-08-31 17:51 ` Huang Shijie
2013-09-24 9:50 ` Marek Vasut
2013-09-24 10:46 ` Huang Shijie
2013-09-24 12:53 ` Marek Vasut
2013-09-25 9:08 ` Huang Shijie
2013-09-29 22:52 ` Marek Vasut [this message]
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=201309300052.46883.marex@denx.de \
--to=marex@denx.de \
--cc=b32955@freescale.com \
--cc=dzu@denx.de \
--cc=fabio.estevam@freescale.com \
--cc=hector.palacios@digi.com \
--cc=linux-mtd@lists.infradead.org \
/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