On 29/05/2013 16:46, Benoît Thébaudeau wrote: > Hi Alison, > Hi Benoit,
>>> I have just noticed that this was actually in 6/7. Why are you putting >>> this into a separate patch? >> [Alison Wang] Because patch 2/7 is about VF610 CPU support, and >> doc/README.vf610 is also a document about >> VF610 SoC. Doc/README.mxc_ocotp is the document about a driver (IP OCOTP), so >> this driver document should be >> separated from CPU patch 2/7. > > I don't think so: It's part of what comes with the addition of the VF610 > platform, so 6/7 could be merged into 2/7. But it does not really matter. It's > probably also fine if you keep what you did. > > Stefano, any opinion? Strictly speaking, you're right. But as you mentioned, it does not really matter. Without reposting, I can myself squash 6/7 into 2/7 by merging the patchset. Best regards, Stefano -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: [email protected] ===================================================================== _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
