On 10/23/2016 10:55 PM, [email protected] wrote: > +static uint64_t aarch64_midr_val; > +uint64_t get_aarch64_cpu_id(void) > +{ > +#ifdef CONFIG_LINUX > + aarch64_midr_val = qemu_read_aarch64_midr_el1(); > + aarch64_midr_val &= CPU_MODEL_MASK; > + > + return aarch64_midr_val; > +#else > + return 0; > +#endif > +} > + > +bool is_thunderx_pass2_cpu(void) > +{ > + return aarch64_midr_val == MIDR_THUNDERX_PASS2; > +}
Any particular reason why you want to keep midr_val and MIDR_THUNDERX private to this file? Seems like it would be cheaper to export those in the header. r~
