[U-Boot] [RFC PATCH v2 01/15] bootstage: Create an initial header for boot progress integers

[Mike Frysinger]

On Sunday 08 January 2012 12:22:54 Simon Glass wrote:
> On Sun, Jan 8, 2012 at 12:26 AM, Mike Frysinger wrote:
> > On Saturday 10 December 2011 16:07:53 Simon Glass wrote:
> >> +enum bootstage_id {
> >> +     BOOTSTAGE_ID_RUN_OS     = 15,   /* Exiting U-Boot, entering OS */
> >> +};
> > 
> > what relevance does this # have ?  since it gets passed to Linux, it
> > becomes part of the ABI and cannot be changed, so that should be
> > mentioned in the comment.
> 
> The number is currently open-coded in U-Boot but this series turns all
> these occurrences into a global enum.

yes, but the enum's have a numeric value :)

> Actually I don't plan that the
> number be passed to Linux, or at least that Linux can rely on it being
> anything in particular.

ok, as long as the end goal has the # not being part of the ABI, that's good
-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/20120108/f1a80eee/attachment.pgp>