I had always misunderstood failover. I thought it was the last resort if normal & fallback didn't work. It actually is just a minimal image that chooses whether to run normal or fallback.
This patch makes that clear for the s2895 by separating the files. I like it much better. I'm eventually planning to do the same for all of the AMD Tyan boards. For the patch to apply, you have to svn cp cache_as_ram_auto.c to failover.c
The only problem I had was where to put the definition of post_code. Since it is trivial, I made it a #define. I tried to put it in other places, but it affected the PPC targets and others because we define post_code in other places.
Boot tested.
Signed-off-by: Myles Watson mylesgw@gmail.com
Thanks, Myles
On Thu, Jul 16, 2009 at 8:42 AM, Myles Watsonmylesgw@gmail.com wrote:
I had always misunderstood failover. I thought it was the last resort if normal & fallback didn't work. It actually is just a minimal image that chooses whether to run normal or fallback.
that's why I never liked the name :-)
It's confusing.
Acked-by: Ronald G. Minnich rminich@gmail.com
On Thu, Jul 16, 2009 at 9:46 AM, ron minnich rminnich@gmail.com wrote:
On Thu, Jul 16, 2009 at 8:42 AM, Myles Watsonmylesgw@gmail.com wrote:
I had always misunderstood failover. I thought it was the last resort if normal & fallback didn't work. It actually is just a minimal image that chooses whether to run normal or fallback.
that's why I never liked the name :-) It's confusing.
One of these days we'll have to change the name of cache_as_ram_auto.c and failover.c at the same time.
Acked-by: Ronald G. Minnich rminich@gmail.com
Rev 4427.
Thanks, Myles