Hi Wilson,

Wilson Callan wrote:
Hi Greg,

1. The linker script changes don't conform to the modern way of setting
    the base and size of kernel RAM
2. Why the special case for vectors 240->255?
    (I am unlikely to merge any special cases in that code)

1. I'm not sure what you mean by this. config.linux-2.6.x looks the same as M5235EVB regarding CONFIG_RAMBASE and SIZE. Are you talking about config.c?

No, this change:

diff -Naur uClinux-dist/linux-2.6.x/arch/m68knommu/kernel/vmlinux.lds.S uClinux-dist-bsp/linux-2.6.x/arch/m68knommu/kernel/vmlinux.lds.S --- uClinux-dist/linux-2.6.x/arch/m68knommu/kernel/vmlinux.lds.S 2007-01-02 22:18:03.000000000 -0500 +++ uClinux-dist-bsp/linux-2.6.x/arch/m68knommu/kernel/vmlinux.lds.S 2007-07-12 16:26:58.000000000 -0400
@@ -9,6 +9,14 @@

 #include <asm-generic/vmlinux.lds.h>

+/*
+ * The Savant boards start the load at 400 to make use of lower memory
+ */
+#if defined(CONFIG_SAVANT)
+#define        RAM_START       0x400
+#define        RAM_LENGTH      0x3e0000
+#endif
+
 #if defined(CONFIG_RAMKERNEL)
 #define        RAM_START       CONFIG_KERNELBASE
#define RAM_LENGTH (CONFIG_RAMBASE + CONFIG_RAMSIZE - CONFIG_KERNELBASE)


Given you have the CONFIG_KERNELBASE, etc, set correctly (and it
does look like they are) there is no need to special case in here.


2. I dont blame you for not wanting that code. That grew out of the fact that i changed to boot image.bin at 0x400, but then changed to needing the u-boot mkimage header before the image start. there were already boot roms out there so i couldnt move the start up to something logical. I want to get away from the that hack ASAP so i'll resubmit the BSP without that after i understand what is needed to fix issue #1.

OK, sounds good.

Regards
Greg



------------------------------------------------------------------------
Greg Ungerer  --  Chief Software Dude       EMAIL:     [EMAIL PROTECTED]
Secure Computing Corporation                PHONE:       +61 7 3435 2888
825 Stanley St,                             FAX:         +61 7 3891 3630
Woolloongabba, QLD, 4102, Australia         WEB: http://www.SnapGear.com
_______________________________________________
uClinux-dev mailing list
[email protected]
http://mailman.uclinux.org/mailman/listinfo/uclinux-dev
This message was resent by [email protected]
To unsubscribe see:
http://mailman.uclinux.org/mailman/options/uclinux-dev

Reply via email to