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

Benoît Thébaudeau benoit.thebaudeau at advansee.com
Fri May 31 18:10:45 CEST 2013


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.thebaudeau at advansee.com>

Best regards,
Benoît


More information about the U-Boot mailing list