[ECOS] [PATCHES] Update to pcmb memory sizing and two tiny fixes

Ian Campbell icampbell@arcom.co.uk
Fri Oct 19 06:29:00 GMT 2001

Hi all,

Sometime ago I sent memsize-fixes.patch to the list which allows the
choice at compile time of hardcoding the memory regions or probing the
BIOS. Fortunately this patch doesn't seem to have been applied yet as it
contains a glaring error. At some point I renamed the CDL option, but
neglected to also rename the #ifdef in the c file. I have attached
memsize-fixes-2.patch which is corrected. (It also changes
CYGNUM_HAL_RTC_PERIOD to use default_value rather than calculated, which
allows the value to be overriden in the board specific HAL)

In the process of getting the FLASH to work on our board I noticed that
the macro HAL_[ID]CACHE_IS_ENABLED was not defined for i386 targets
(preventing the flash stuff from building). The attached patch
'cache-is-enabled.patch' implements this in the spirit of the other i386
CACHE macros (ie does nothing).

Finally, I tried to use the tcp-echo test in the net package, the target
part was unable to setup it's network via DHCP, I think because the main
thread and load threads were of higher priority then the networking
thread, so the DHCP replies were not received. tcp-echo-test.patch
delays the starting of the load threads until after the network is
initialised and starts the main thread with a lower priority, bumping it
up after the network has initialised.


Ian Campbell
Design Engineer

Arcom Control Systems Ltd,
Clifton Road,
Cambridge CB1 7EA
United Kingdom

Tel: +44 (0)1223 411200 ext. 3204
E-Mail: icampbell@arcom.co.uk
Web: http://www.arcomcontrols.com

The message in this transmission is sent in confidence for the attention of the addressee only and should not be disclosed to any other party. Unauthorised recipients are requested to preserve this confidentiality. Please advise the sender if the addressee is not resident at the receiving end.

This message has been checked for all viruses by MessageLabs Virus Control Centre. 

Company registered in England No. 1608562.
Registered Office: Unit 8, Clifton Road, Cambridge, CB1 7EA, United Kingdom, Tel: 01223 411200.
Index: packages/hal/i386/pcmb/current/ChangeLog
RCS file: /cvs/ecos/ecos/packages/hal/i386/pcmb/current/ChangeLog,v
retrieving revision 1.8
diff -u -r1.8 ChangeLog
--- packages/hal/i386/pcmb/current/ChangeLog	2001/10/19 07:02:34	1.8
+++ packages/hal/i386/pcmb/current/ChangeLog	2001/10/19 12:43:56
@@ -1,3 +1,12 @@
+2001-10-19  Ian Campbell  <icampbell@arcom.co.uk>
+	* cdl/hal_i386_pcmb.cdl: Add cdl_options for
+	CYGPKG_HAL_I386_PCMB_MEMSIZE to allow configuration of how memory
+	is detected. Change CYGNUM_HAL_RTC_PERIOD to use default_value
+	rather than calculated (so it can be overridden in the platform).
+	* src/pcmb_misc.c: Implement CYGPKG_HAL_I386_PCMB_MEMSIZE
+	configuration options.
 2001-10-16  Mark Salter  <msalter@redhat.com>
 	* src/pcmb_serial.c (cyg_hal_plf_serial_init): Use CDL to determine
Index: packages/hal/i386/pcmb/current/cdl/hal_i386_pcmb.cdl
RCS file: /cvs/ecos/ecos/packages/hal/i386/pcmb/current/cdl/hal_i386_pcmb.cdl,v
retrieving revision 1.3
diff -u -r1.3 hal_i386_pcmb.cdl
--- packages/hal/i386/pcmb/current/cdl/hal_i386_pcmb.cdl	2001/09/12 00:59:23	1.3
+++ packages/hal/i386/pcmb/current/cdl/hal_i386_pcmb.cdl	2001/10/19 12:43:56
@@ -83,11 +83,36 @@
         cdl_option CYGNUM_HAL_RTC_PERIOD {
             display       "Real-time clock period"
             flavor        data
-           calculated     11932
+	    default_value 11932
+    cdl_component CYGPKG_HAL_I386_PCMB_MEMSIZE {
+	display       "How to discover the size of available RAM."
+	flavor        data
+	legal_values  {"BIOS" "HARDCODE"}
+	default_value {"BIOS"}
+	description   "It is possible for the HAL to discover the 
+	               size of RAM In several ways. Currently this
+                       can be done by querying the BIOS or by 
+                       hardcoding the values into the executable."
+	    display       "Amount of Base RAM available."
+	    flavor        data
+	    default_value 0x000F0000
+	    active_if     { CYGPKG_HAL_I386_PCMB_MEMSIZE == "HARDCODE" }
+	}
+	    display       "Amount of Extended RAM available."
+	    flavor        data
+	    default_value 0x00100000
+	    active_if     { CYGPKG_HAL_I386_PCMB_MEMSIZE == "HARDCODE" }
+	}
+    }
     cdl_interface CYGINT_HAL_I386_PCMB_SCREEN_SUPPORT {
 	display       "Enable PC screen support"
 	compile       pcmb_screen.c
Index: packages/hal/i386/pcmb/current/src/pcmb_misc.c
RCS file: /cvs/ecos/ecos/packages/hal/i386/pcmb/current/src/pcmb_misc.c,v
retrieving revision 1.3
diff -u -r1.3 pcmb_misc.c
--- packages/hal/i386/pcmb/current/src/pcmb_misc.c	2001/09/12 00:59:23	1.3
+++ packages/hal/i386/pcmb/current/src/pcmb_misc.c	2001/10/19 12:43:56
@@ -70,6 +70,12 @@
 void hal_pcmb_init(void)
+    cyg_hal_pcmb_memsize_base = CYGPKG_HAL_I386_PCMB_MEMSIZE_BASE;
+    cyg_hal_pcmb_memsize_extended = CYGPKG_HAL_I386_PCMB_MEMSIZE_EXTENDED;
     cyg_uint8 lo,hi;
     HAL_READ_CMOS( 0x15, lo );
@@ -89,7 +95,7 @@
     cyg_hal_pcmb_memsize_extended = ((hi<<8)+lo)*1024;

More information about the Ecos-discuss mailing list