On Friday, December 18, 2015 at 03:04:59 AM, Masahiro Yamada wrote: > Please stop such coding habit as follows: > > #ifdef CONFIG_FOO > # define ENABLE_FOO 1 > #else > # define ENABLE_FOO 0 > #endif
Oh yes, this is horrible, kill this with fire :) > Use IS_ENABLED(CONFIG_FOO), instead. > > > > Masahiro Yamada (11): > image: zap IMAGE_ENABLE_RAMDISK_HIGH > image: zap IMAGE_ENABLE_OF_LIBFDT > image: zap IMAGE_BOOT_GET_CMDLINE > image: zap IMAGE_OF_BOARD_SETUP > image: zap IMAGE_OF_SYSTEM_SETUP > ARM: bootm: BOOTM_ENABLE_SERIAL_TAG > ARM: bootm: BOOTM_ENABLE_CMDLINE_TAG > ARM: bootm: BOOTM_ENABLE_REVISION_TAG > ARM: bootm: BOOTM_ENABLE_MEMORY_TAG > ARM: bootm: BOOTM_ENABLE_INITRD_TAG > ARM: bootm: drop redundant #ifdef conditional > > arch/arc/lib/bootm.c | 2 +- > arch/arm/include/asm/bootm.h | 22 ---------------------- > arch/arm/lib/bootm.c | 16 +++++++--------- > common/image-fdt.c | 6 +++--- > common/image.c | 10 +++++----- > include/image.h | 30 ------------------------------ > 6 files changed, 16 insertions(+), 70 deletions(-) Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
