[U-Boot] [PATCH v7 1/5] mtd: nand: omap: enable BCH ECC scheme using ELM for generic platform
Tom Rini
trini at ti.com
Tue Oct 8 03:59:32 CEST 2013
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 10/07/2013 09:06 PM, Scott Wood wrote:
> On Sat, 2013-10-05 at 06:11 +0000, Gupta, Pekon wrote:
>> Hi,
>> Please see the replies inline..
>>
>>> From: Scott Wood [mailto:scottwood at freescale.com]
>>>> On Mon, 2013-09-30 at 19:43 +0530, Pekon Gupta wrote:
>>>> +Platform specific options
>>>> +=========================
>>>> +
>>>> + CONFIG_NAND_OMAP_ECCSCHEME
>>>> + On OMAP platforms, this specifies NAND ECC scheme.
>>>> + 1 - HAM1_SW 1-bit Hamming code using software library
>>>> + (for legacy devices only)
>>>> + 2 - HAM1_HW 1-bit Hamming code using GPMC hardware engine
>>>> + (for legacy devices only)
>>>> + 3 - BCH4_SW 4-bit BCH code (unsupported)
>>>> + 4 - BCH4_HW 4-bit BCH code (unsupported)
>>>> + 5 - BCH8_SW 8-bit BCH code with
>>>> + - ecc calculation using GPMC hardware engine,
>>>> + - error detection using software library.
>>>> + - requires CONFIG_BCH to enable software BCH
>>> library
>>>> + (For legacy device which do not have ELM h/w
>>> engine)
>>>> + 6 - BCH8_HW 8-bit BCH code with
>>>> + - ecc calculation using GPMC hardware engine,
>>>> + - error detection using ELM hardware engine.
>>>
>>> You should document the symbols, not the numbers that happen to be
>>> assigned to them.
>>>
>> Sorry din't get you. This is based on your below feedback
>> http://lists.denx.de/pipermail/u-boot/2013-September/162773.html
>>
>> Example: "6 - BCH8_HW" means 8-bit BCH ECC scheme using h/w engine.
>> It is this number is what user needs to specify in include/configs/*.h
>> Any other internal symbol like "OMAP_ECC_BCH8_CODE_SW" should
>> not be exposed to user, user-interface should remain constant.
>
> I disagree. The user should specify OMAP_ECC_BCH8_CODE_SW. It's more
> greppable, and it's more likely that the number will change than that
> the name will.
I agree with Scott, enum (if it's not already one) and use the name so
it's clear what you're picking.
- --
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
iQIcBAEBAgAGBQJSU2cEAAoJENk4IS6UOR1WnOQP/30AInl+d0QVU6QHq+DsGC/q
Wh/yfXbN/EkWVPEvpASS5gftTSd+UjLSzUSWca+TcQxUVdzD64H1u4j8An2cSwJp
vQuX1QZzMWA+g6D+u75BKMs/cIXb6rwX9A5is5R/k5cU4mew+ecLmBeRLcTENfpM
+ROD7XfSJsXYff+dLhpOeVWC6ELzjgmHWajoqpdZNefrMtsrRHOeLd9ccbqYxoNZ
Td56vwMYfk8f+BBRmGlFRdAf+xN+9ssou1CipK6E/H0QHnexRkuPINRdmPD4/p0o
yP+opvOUq8XMYiO76HOGb37tMJvjgziEcZUy4S3Q0gbhRlC9EXz1D0qxs6ti1EFs
9M/Wsl8bsKx4rgdf12hj8iebFxvTdT1lXejJGNk9ThHqsHE0pl+K1cyouZ/GL0NA
sk/UCPe7DL3RIAtkUy+pLT7A3nOKIhqJY1kTsPN24qTF7UVlatWov6C33d42jDzp
YH5CkdP1FRffkRj1in1ex0CiVB/3h3SuI4kA+hceWdx0rGYRT/PRKQA3qSulU1p4
lx3jtzt07fSflPdeX9PxW5k9CqAmXut4rw5USU1OAdpRKANzp1FPXcKXjEVd2uQ1
zkfgUB4MeR6xs6HE6q5xBbaaRhy9zJX415ZG10NtKUXCrVBIoif51EQQmlvehgcy
ufn39bBOmsjkXYXzJI8W
=tQHU
-----END PGP SIGNATURE-----
More information about the U-Boot
mailing list