> + if (hartid == boot_cpu_hartid) > + return 0; > + else > + return -1;
No need for an else after a return statement.
> +}
> +static inline unsigned long cpuid_to_hartid_map(int cpu)
> +{
> +
> + return boot_cpu_hartid;
spurious empty line above.

