[U-Boot] [PATCH v4 2/7] arm: vf610: Add Vybrid VF610 CPU support

[Stefano Babic]

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: sbabic at denx.de
=====================================================================