Sorry , it is attached. -- bryan Geert Uytterhoeven wrote: > On Mon, 21 Jan 2002, Bryan Chua wrote: > >>I recall a bunch of disussion about changing arch/mips/setup.c to >>simplify adding vendor-specific platform code in setup_arch, but to date >>nothing has come of it. So while this is a dramatic oversimplification >>of the various proposals, how about this for now -- >> >>just a vendor-defined function "platform_setup (void)" and it is up to >>the vendor to figure out what to do from there. >> >>-- bryan >> >> >>Index: arch/mips/kernel/setup.c >>=================================================================== >>RCS file: /cvs/linux/arch/mips/kernel/setup.c,v >>retrieving revision 1.96.2.3 >>diff -u -r1.96.2.3 setup.c >>--- arch/mips/kernel/setup.c 2001/12/26 23:27:02 1.96.2.3 >>+++ arch/mips/kernel/setup.c 2002/01/21 22:55:35 >>@@ -666,6 +666,7 @@ >> void it8172_setup(void); >> void swarm_setup(void); >> void hp_setup(void); >>+ >>void platform_setup (void); >> >> unsigned long bootmap_size; >> unsigned long start_pfn, max_pfn, first_usable_pfn; >>@@ -793,7 +794,8 @@ >> break; >> #endif >> default: >>- >> panic("Unsupported architecture"); >>+ >> platform_setup (); >>+ >> > > At first I thought: he's adding code after a call to panic(), but it turns out > your mailer screwed your patch... > > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like that. > -- Linus Torvalds > >
Index: arch/mips/kernel/setup.c =================================================================== RCS file: /cvs/linux/arch/mips/kernel/setup.c,v retrieving revision 1.96.2.3 diff -u -r1.96.2.3 setup.c --- arch/mips/kernel/setup.c 2001/12/26 23:27:02 1.96.2.3 +++ arch/mips/kernel/setup.c 2002/01/22 20:52:05 @@ -666,6 +666,7 @@ void it8172_setup(void); void swarm_setup(void); void hp_setup(void); + void platform_setup (void); unsigned long bootmap_size; unsigned long start_pfn, max_pfn, first_usable_pfn; @@ -793,7 +794,8 @@ break; #endif default: - panic("Unsupported architecture"); + platform_setup (); + break; } strncpy(command_line, arcs_cmdline, sizeof command_line);