On 4/1/19 11:04 AM, Eugeniy Paltsev wrote: >> + if (IS_ENABLED(CONFIG_HIGHMEM) || is_pae40_enabled()) > As for today PAE40 couldn't be enabled without HIGHMEM for ARC, so probably we can > simplify this to > > ----------------------------->8---------------------------------- > if (IS_ENABLED(CONFIG_HIGHMEM)) > ioc_enable = 0; That is true, but mentioning both sort of documents the requirements. Also these checks are compile time only so don't add any extra code whatsoever. -Vineet _______________________________________________ linux-snps-arc mailing list linux-snps-arc@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/linux-snps-arc