Re: arch/mips/setup.c

[Bryan Chua <chua@ayrnetworks.com>]

[-- Attachment #1: Type: text/plain, Size: 1702 bytes --]

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
> 
> 



[-- Attachment #2: setup.c --]
[-- Type: text/plain, Size: 730 bytes --]

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);