From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefano Babic <sbabic@denx.de>
Date: Fri, 22 Jun 2018 10:46:45 +0200
Subject: [U-Boot] [PATCH v3 5/5] mtd: nand: mxs_nand: add minimal ECC
 support
In-Reply-To: <87bfabb3-7cfa-4890-fd1b-21613626283f@denx.de>
References: <20180411160452.2087-1-stefan@agner.ch>
 <20180411160452.2087-6-stefan@agner.ch>
 <ba513acd-00c6-0945-a24b-9e5a3454e654@denx.de>
 <91682986f9f635d650ebb41c86b0da09@agner.ch>
 <0ab21b8ec4ee2d0711643711ddac0873@agner.ch>
 <814aea3bd0e80aff7422f001b268fee3@agner.ch>
 <20180621152212.GW4609@bill-the-cat.ec.rr.com>
 <87bfabb3-7cfa-4890-fd1b-21613626283f@denx.de>
Message-ID: <cbefd969-afc4-2cfa-761a-08db201ca802@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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.

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 ?

Thanks,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================