On Mon, Nov 4, 2019 at 5:13 AM Marc Zyngier <maz@xxxxxxxxxx> wrote: > Is there any reason why you're not consistently using only one of > "#if IS_ENABLED(...)" or "#ifdef ...", but instead a mix of both? This is to match the style already used in each file. For example, fs/proc/meminfo.c uses #ifdef for other configs in the same function, and include/linux/mmzone.h uses #if IS_ENABLED(...). Sami