On Fri, 2013-08-16 at 10:10 +0800, Huang Shijie wrote:
> +> +What: /sys/class/mtd/mtdX/ecc_step_size> +Date: May 2013> +KernelVersion: 3.10> +Contact: linux-mtd@lists.infradead.org> +Description:> + The size of each ECC step which is used for ECC.> + Note that some devices will have multiple ecc steps within each> + writesize region.
Actually this phrase is a bit confusing because it may be interpreted as
that one write-size may have ECC steps of multiple sizes. Would you
re-phrase, may be?
Otherwise the patch-set looks good, thanks, but I am not merging the
patches to let Brian review.
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>

On Fri, Aug 16, 2013 at 04:45:59PM +0300, Artem Bityutskiy wrote:
> On Fri, 2013-08-16 at 10:10 +0800, Huang Shijie wrote:> > +> > +What: /sys/class/mtd/mtdX/ecc_step_size> > +Date: May 2013> > +KernelVersion: 3.10> > +Contact: linux-mtd@lists.infradead.org> > +Description:> > + The size of each ECC step which is used for ECC.> > + Note that some devices will have multiple ecc steps within each> > + writesize region.> > Actually this phrase is a bit confusing because it may be interpreted as> that one write-size may have ECC steps of multiple sizes. Would you> re-phrase, may be?
What's about the following:
-----------------------------------------------------------------
The size of each ECC step which is used for ECC.
Note that some devices will have multiple ecc steps within each
writesize region, and the ecc steps share the same size.
-----------------------------------------------------------------
thanks
Huang Shijie

Patch

diff --git a/Documentation/ABI/testing/sysfs-class-mtd b/Documentation/ABI/testing/sysfs-class-mtdindex 3105644..da112ab 100644--- a/Documentation/ABI/testing/sysfs-class-mtd+++ b/Documentation/ABI/testing/sysfs-class-mtd@@ -173,3 +173,13 @@ Description:
This is generally applicable only to NAND flash devices with ECC
capability. It is ignored on devices lacking ECC capability;
i.e., devices for which ecc_strength is zero.
++What: /sys/class/mtd/mtdX/ecc_step_size+Date: May 2013+KernelVersion: 3.10+Contact: linux-mtd@lists.infradead.org+Description:+ The size of each ECC step which is used for ECC.+ Note that some devices will have multiple ecc steps within each+ writesize region. See more in the ecc_strength above. This will+ always be a non-negative integer.