[ECOS] MPC860, MSR initialization

[Jonathan Larmour]

Radu Cernea wrote:
> Hi,
>  
> I think there is a small problem with the definition CYG_MSR_COMMON in the file 
> 
> Â…\hal\powerpc\arch\current\src\vectors.S 
> 
> The FP bit should not be asserted for MPC860. See MPC860UM, chapter 5.1.2.3.1.

True. Below patch checked in.

Jifl

Index: ChangeLog
===================================================================
RCS file: /cvs/ecos/ecos/packages/hal/powerpc/mpc8xx/current/ChangeLog,v
retrieving revision 1.23
diff -u -5 -p -r1.23 ChangeLog
--- ChangeLog	20 Jan 2003 11:13:07 -0000	1.23
+++ ChangeLog	5 Mar 2003 01:56:52 -0000
@@ -1,5 +1,10 @@
+2003-03-05  Jonathan Larmour  <jifl@eCosCentric.com>
+
+	* include/variant.inc: Don't let FP be set in MSR in common HAL.
+	Reported by Radu Cernea <rc@unusus.com>
+
  2003-01-20  Gary Thomas  <gary@mlbassoc.com>

  	* src/variant.S: Be careful about using QUICC functions unless that
  	package is present (some MPC8xx platforms don't use it).

Index: include/variant.inc
===================================================================
RCS file: 
/cvs/ecos/ecos/packages/hal/powerpc/mpc8xx/current/include/variant.inc,v
retrieving revision 1.5
diff -u -5 -p -r1.5 variant.inc
--- include/variant.inc	23 May 2002 23:04:28 -0000	1.5
+++ include/variant.inc	5 Mar 2003 01:56:52 -0000
@@ -57,10 +57,13 @@
  ##
 
##=============================================================================

  #include <pkgconf/hal.h>
  	
+#undef CYG_MSR_COMMON
+#define CYG_MSR_COMMON (MSR_ME | MSR_RI)
+
  #include <cyg/hal/arch.inc>

 
##-----------------------------------------------------------------------------
  ## MPC8xx defined vectors


-- 
eCosCentric    http://www.eCosCentric.com/    The eCos and RedBoot experts
--[ "You can complain because roses have thorns, or you ]--
--[  can rejoice because thorns have roses." -Lincoln   ]-- Opinions==mine