Re: CVS: blob/src/blob memsetup-sa1110.S,1.9,1.10

[Abraham vd M. <ab...@2d...>]

Hi Erik!

> > Update of /cvsroot/blob/blob/src/blob
> > In directory usw-pr-cvs1:/tmp/cvs-serv2855/src/blob
> >=20
> > Modified Files:
> > 	memsetup-sa1110.S=20
> > Log Message:
> > Changed the SA-1110 memory setup routines so that stuff related to exte=
rnal
> > memory and memory in banks 2/3 is optional. Simply defining values such=
 as
> > MDCAS21_VALUE to junk doesn't work.
>=20
> Then set them to something useful like the *documented* reset value.
> Making it optional creates an #ifdef jungle that I tried to avoid.
> Please back out this change.

That means everyone has to know what the _documented_ reset values is :P I
can clean up the ifdef mess, but I still think it should reprogram memory
settings that is undefined.

--=20

Regards
 Abraham

Weinberg's Principle:
	An expert is a person who avoids the small errors while
	sweeping on to the grand fallacy.

__________________________________________________________
 Abraham vd Merwe - 2d3D, Inc.

 Device Driver Development, Outsourcing, Embedded Systems

  Cell: +27 82 565 4451         Snailmail:
   Tel: +27 21 761 7549            Block C, Antree Park
   Fax: +27 21 761 7648            Doncaster Road
 Email: ab...@2d...         Kenilworth, 7700
  Http: http://www.2d3d.com        South Africa