Dear Fernando, In message <CAK3Qp=uwwr62hg50lxdq+44z5wetxvnxzzvdcqkn_tptsdj...@mail.gmail.com> you wrote: > > I'm rebasing our internal code over v2017.11 and I found an unexpected > change in drivers/mtd/nand/nand_base.c:
In such cases it is always a good idea to run "git blame" to find out where the code is coming from... > /* See nand_bch_init() for details. */ > ecc->bytes = 0; > ecc->priv = nand_bch_init(mtd); > ----- > > This means that ecc->bytes = 0 is overwritting the board driver > supplied value. Actually, back in v2014.03, that specific line > (ecc->bytes = 0) did not exit. I also checked today in v2018.07 and > master, and this potential bug is still there. git blame says this comes from commit ceee07b65875bb01bef55cba06940ef7afc1afba Author: Scott Wood <[email protected]> Date: Mon May 30 13:57:58 2016 -0500 mtd: nand: Sync with Linux v4.6 Updates the NAND code to match Linux v4.6. The previous sync was from Linux v4.1 in commit d3963721d93fafa. Note that none of the individual NAND drivers tracked Linux closely enough to be synced themselves, other than manually applying a few cross-tree changes. Signed-off-by: Scott Wood <[email protected]> Tested-by: Heiko Schocher <[email protected]> Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [email protected] That's their goal, remember, a goal that's really contrary to that of the programmer or administrator. We just want to get our jobs done. $Bill just wants to become $$Bill. These aren't even marginallly congruent. -- Tom Christiansen in <[email protected]> _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
