Re: [patch] x86, mm: Clean up initmem_init

[David Rientjes]

On Thu, 3 Mar 2011, Yinghai Lu wrote:

> he want
> 
> > +#ifdef CONFIG_ACPI_NUMA
> >> > +		ret = numa_init(x86_acpi_numa_init);
> >> > +		if (!ret)
> >> > +			return;
> >> > +#endif
> >> > +#ifdef CONFIG_AMD_NUMA
> >> > +		ret = numa_init(amd_numa_init);
> >> > +		if (!ret)
> >> > +			return;
> >> > +#endif
> 
> to be replaced by:
> 
> > +#ifdef CONFIG_ACPI_NUMA
> >> > +		if (!numa_init(x86_acpi_numa_init))
> >> > +			return;
> >> > +#endif
> >> > +#ifdef CONFIG_AMD_NUMA
> >> > +		if (!numa_init(amd_numa_init))
> >> > +			return;
> >> > +#endif

It's a matter of style and I think it's up to Ingo what he'd prefer to 
see.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/