From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Sat, 5 Jan 2013 09:03:48 +0100 Subject: [U-Boot] [RFC PATCH 05/44] image: Move timestamp #ifdefs to header file In-Reply-To: <1357350734-13737-6-git-send-email-sjg@chromium.org> References: <1357350734-13737-1-git-send-email-sjg@chromium.org> <1357350734-13737-6-git-send-email-sjg@chromium.org> Message-ID: <201301050903.48755.marex@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Dear Simon Glass, > Rather than repeat the line > #if defined(CONFIG_TIMESTAMP) || defined(CONFIG_CMD_DATE) || \ > defined(USE_HOSTCC) > > everywhere, put this in a header file and #define IMAGE_ENABLE_TIMESTAMP > to either 1 or 0. Then we can use a plain if() in most code and avoid > the #ifdefs. > > The compiler's dead code elimination ensures that the result is the same. > > Signed-off-by: Simon Glass Acked-by: Marek Vasut Best regards, Marek Vasut