public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
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

      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