On Sunday 18 December 2011 13:54:21 Mike Frysinger wrote:
> On Saturday 17 December 2011 12:03:04 Pali Rohár wrote:
> > + s = getenv ("atagaddr");
>
> no space before the "("
>
> > #if defined (CONFIG_SETUP_MEMORY_TAGS) || \
> >
> > defined (CONFIG_CMDLINE_TAG) || \
> > defined (CONFIG_INITRD_TAG) || \
> > defined (CONFIG_SERIAL_TAG) || \
> > defined (CONFIG_REVISION_TAG)
> >
> > ...
> > +#if defined (CONFIG_SETUP_MEMORY_TAGS) || \
> > + defined (CONFIG_CMDLINE_TAG) || \
> > + defined (CONFIG_INITRD_TAG) || \
> > + defined (CONFIG_SERIAL_TAG) || \
> > + defined (CONFIG_REVISION_TAG)
>
> rather than duplicating the same list in multiple places, why not setup a
> local define in this file and then use that everywhere else.
> -mikeI will add local define CONFIG_ANY_TAG to bootm.c +#if defined (CONFIG_SETUP_MEMORY_TAGS) || \ + defined (CONFIG_CMDLINE_TAG) || \ + defined (CONFIG_INITRD_TAG) || \ + defined (CONFIG_SERIAL_TAG) || \ + defined (CONFIG_REVISION_TAG) +#define CONFIG_ANY_TAG +#endif -- Pali Rohár [email protected]
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
