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