Hi Stefano, On Friday, May 31, 2013 10:59:03 AM, Stefano Babic wrote: > 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.
OK. Reviewed-by: Benoît Thébaudeau <benoit.thebaud...@advansee.com> Best regards, Benoît _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
