From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Wed, 17 May 2017 17:24:02 -0400 Subject: [U-Boot] [PATCH 25/38] arm: Add explicit include of In-Reply-To: <20170517142311.29484-26-sjg@chromium.org> References: <20170517142311.29484-1-sjg@chromium.org> <20170517142311.29484-26-sjg@chromium.org> Message-ID: <20170517212402.GD4631@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, May 17, 2017 at 08:22:58AM -0600, Simon Glass wrote: > Rather than relying on common.h to provide this include, which is going > away at some point, include it explicitly in each file. > > Signed-off-by: Simon Glass [snip] > diff --git a/arch/arm/lib/spl.c b/arch/arm/lib/spl.c > index 8ff2c5065d..3ea494f5e4 100644 > --- a/arch/arm/lib/spl.c > +++ b/arch/arm/lib/spl.c > @@ -12,6 +12,9 @@ > #include > #include > #include > +#ifdef CONFIG_MACH_TYPE > +#include > +#endif We don't need to #ifdef this part here, we can safely always include this header. You can repost just a v2 of this one with: Reviewed-by: Tom Rini Thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: