From: Sam Ravnborg <sam@xxxxxxxxxxxx> Date: Tue, 1 Dec 2009 19:21:16 +0100 > On Tue, Dec 01, 2009 at 03:52:55PM +0100, Kristoffer Glembo wrote: >> LEON does not have an idprom and idprom_init prints >> some bogus info (e.g. incorrect Ethernet address). >> >> Signed-off-by: Kristoffer Glembo <kristoffer@xxxxxxxxxxx> >> --- >> arch/sparc/kernel/setup_32.c | 3 +++ >> 1 files changed, 3 insertions(+), 0 deletions(-) >> >> diff --git a/arch/sparc/kernel/setup_32.c b/arch/sparc/kernel/setup_32.c >> index 9be2af5..1d3cf56 100644 >> --- a/arch/sparc/kernel/setup_32.c >> +++ b/arch/sparc/kernel/setup_32.c >> @@ -271,7 +271,10 @@ void __init setup_arch(char **cmdline_p) >> #endif >> boot_flags_init(*cmdline_p); >> >> +#ifndef CONFIG_SPARC_LEON >> idprom_init(); >> +#endif >> + >> if (ARCH_SUN4C) >> sun4c_probe_vac(); >> load_mmu(); > > I suggest to drop all idprom support in case of > leon and then use stub functions so you do not > need ifdef like above. Also, the idprom provides a default ethernet address for the system. It might be even better to just cons up a fake idprom or similar. -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
