[U-Boot] [PATCH v3 7/8] sandbox: Add flags for open() call

Mike Frysinger vapier at gentoo.org
Thu Feb 16 04:25:00 CET 2012


On Wednesday 15 February 2012 17:58:35 Simon Glass wrote:
> On Tue, Jan 24, 2012 at 1:45 PM, Mike Frysinger wrote:
> > On Monday 23 January 2012 01:48:52 Simon Glass wrote:
> >> -/*
> >> - * Operating System Interface
> >> - *
> >> - * This provides access to useful OS routines from the sandbox
> >> architecture - */
> >> +struct sandbox_state;
> > 
> > looks like unrelated stuff got squished into here
> 
> We need to declare the struct. The comment change is because you said
> we should put the purpose of the file at the top, and I thought it
> didn't really warrant a new commit?

i meant that adding the struct forward decl in this commit doesn't make sense.  
it should be part of one of the sandbox state patches.

deleting the comment is fine, but the changelog should mention that this was on 
purpose and not just an accident.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120215/f57f1619/attachment.pgp>


More information about the U-Boot mailing list