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
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
