-----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:scottw...@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-----
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to