[U-Boot] [PATCH v3 5/5] mtd: nand: mxs_nand: add minimal ECC support

[Stefan Agner <stefan@agner.ch>]

On 22.06.2018 10:46, Stefano Babic wrote:
> Hi Stefan,
> 
> On 21/06/2018 18:39, Stefano Babic wrote:
>> Hi Stefan,
>>
>> On 21/06/2018 17:22, Tom Rini wrote:
>>> On Thu, Jun 21, 2018 at 01:46:10PM +0200, Stefan Agner wrote:
>>>
>>>> Tom, Stefano,
>>>>
>>>> Scott Wood seems not to be very active this days in the U-Boot
>>>> community.
>>>>
>>>> This patchsets seem all to be blocked due to that:
>>>> https://patchwork.ozlabs.org/cover/897263/
>>>> https://patchwork.ozlabs.org/cover/901995/
>>>> https://patchwork.ozlabs.org/patch/922907/
>>>>
>>>> Those patches have been around quite a while and I tested them well. Any
>>>> change to get that still in?
>>>
>>> I don't object to those coming in via one of the SoC trees, thanks!
>>>
>>
>> Ok - we have a general agreement, I pick your patches into u-boot-imx.
>>
> 
> I get several broken boards and many warnings when I build with your
> patches. Maybe have I missed some of them ? I have pushed a temporary
> branch (nand-imx) on the server, let me know if I miss something.

Patches look good.

> 
> There are 15 boards broken, errors seem trivial but I do not want to fix
> anything without showing to you. There are mainly two issues, for example:
> 
> drivers/mtd/nand/mxs_nand_spl.c: In function ?mxs_nand_init?:
> drivers/mtd/nand/mxs_nand_spl.c:148:2: error: too many arguments to
> function ?board_nand_init?
>   board_nand_init(&nand_chip);
>   ^~~~~~~~~~~~~~~
> In file included from drivers/mtd/nand/mxs_nand_spl.c:7:0:
> include/nand.h:39:6: note: declared here
>  void board_nand_init(void);
>       ^~~~~~~~~~~~~~~
> make[4]: *** [spl/drivers/mtd/nand/mxs_nand_spl.o] Error 1
> make[3]: *** [spl/drivers/mtd/nand] Error 2
> make[2]: *** [spl/drivers] Error 2
> make[1]: *** [spl/u-boot-spl] Error 2
> make: *** [sub-make] Error 2
> 
> 
> The second one is a missing include, like in mx28evk:
> 
> 
> drivers/mtd/nand/mxs_nand.c: In function ?mxs_nand_calc_ecc_layout_by_info?:
> drivers/mtd/nand/mxs_nand.c:172:7: error: ?SZ_512? undeclared (first use
> in this function)
>   case SZ_512:
>        ^~~~~~
> drivers/mtd/nand/mxs_nand.c:172:7: note: each undeclared identifier is
> reported only once for each function it appears in
> drivers/mtd/nand/mxs_nand.c:175:7: error: ?SZ_1K? undeclared (first use
> in this function)
>   case SZ_1K:
>        ^~~~~
> make[2]: *** [drivers/mtd/nand/mxs_nand.o] Error 1
> make[1]: *** [drivers/mtd/nand] Error 2
> make: *** [sub-make] Error 2
> 
> Can you take a look ?

Hm, I thought I did some testing across all boards, but maybe not with
all patches applied.

Will look into it.

--
Stefan