Re: [PATCH v2 2/8] RISC-V: Move cpuid to hartid mapping to SMP.

[Atish Patra]

On 1/15/19 5:52 AM, Christoph Hellwig wrote:
> > +	if (hartid == boot_cpu_hartid)
> > +		return 0;
> > +	else
> > +		return -1;
>
> No need for an else after a return statement.

Correct. Will update it.

> > +}
> > +static inline unsigned long cpuid_to_hartid_map(int cpu)
> > +{
> > +
> > +	return boot_cpu_hartid;
>
> spurious empty line above.
Sorry for that. Will fix it.

Regards,
Atish