All of lore.kernel.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: 58+ 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:10                             ` [U-Boot] " Hector Palacios
2013-09-02 10:23                             ` Huang Shijie
2013-09-02 10:23                               ` [U-Boot] " Huang Shijie
2013-09-02 11:32                               ` Marek Vasut
2013-09-02 11:32                                 ` [U-Boot] " Marek Vasut
2013-09-03  2:06                                 ` Huang Shijie
2013-09-03  2:06                                   ` [U-Boot] " Huang Shijie
2013-09-03 11:53                                   ` Marek Vasut
2013-09-03 11:53                                     ` [U-Boot] " Marek Vasut
2013-09-04  2:26                                     ` Huang Shijie
2013-09-04  2:26                                       ` [U-Boot] " Huang Shijie
2013-09-04 14:00                                       ` Marek Vasut
2013-09-04 14:00                                         ` [U-Boot] " Marek Vasut
2013-09-05  2:41                                         ` Huang Shijie
2013-09-05  2:41                                           ` [U-Boot] " Huang Shijie
2013-09-04 14:38                                           ` Marek Vasut
2013-09-04 14:38                                             ` [U-Boot] " Marek Vasut
2013-09-04 15:46                                             ` Hector Palacios
2013-09-04 15:46                                               ` [U-Boot] " Hector Palacios
2013-09-05  6:01                                               ` Huang Shijie
2013-09-05  6:01                                                 ` [U-Boot] " Huang Shijie
2013-09-15 14:18                                                 ` Marek Vasut
2013-09-15 14:18                                                   ` [U-Boot] " Marek Vasut
2013-09-16  2:35                                                   ` Huang Shijie
2013-09-16  2:35                                                     ` [U-Boot] " Huang Shijie
2013-09-19 16:07                                                     ` Marek Vasut
2013-09-19 16:07                                                       ` [U-Boot] " Marek Vasut
2013-09-19 16:13                                                       ` Hector Palacios
2013-09-19 16:13                                                         ` [U-Boot] " Hector Palacios
2013-09-19 16:14                                                       ` Hector Palacios
2013-09-19 16:14                                                         ` [U-Boot] " Hector Palacios
2013-09-19 16:16                                                         ` Marek Vasut
2013-09-19 16:16                                                           ` [U-Boot] " Marek Vasut
2013-09-19 17:20                                                         ` Marek Vasut
2013-09-19 17:20                                                           ` [U-Boot] " Marek Vasut
2013-09-15 14:12                                               ` Marek Vasut
2013-09-15 14:12                                                 ` [U-Boot] " 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 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.