On Tue, Nov 07, 2017 at 01:32:08PM -0800, Azhar Shaikh wrote: > +#ifdef CONFIG_X86 > + if (is_bsw()) > + iounmap(phy->ilb_base_addr); > +#endif This whole thing would be much better if is_bsw was just bool is_bsw(void) { if (!IS_ENABLED(CONFIG_X86)) return false; [..] } Then drop every single one of these #ifdef CONFIG_X86 Jason