<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
<title></title>
</head>
<body bgcolor="#ffffff" text="#000000">
Am 07.06.2009 15:10, schrieb Carl-Daniel Hailfinger:
<blockquote cite="mid:4A2BBC3B.3060408@gmx.net" type="cite">
<pre wrap="">Hi Patrick,
On 06.06.2009 15:14, Patrick Georgi wrote:
</pre>
<blockquote type="cite">
<pre wrap="">"failover"-infrastructure to CBFS.
2. Set ROM_IMAGE_SIZE so that
FAILOVER_SIZE + ROM_IMAGE_SIZE = XIP_ROM_SIZE
to avoid problems with XIP.
</pre>
</blockquote>
<pre wrap=""><!---->
Actually, this is a big problem for me. Which XIP_ROM_SIZE setting is
honored? The one in src/mainboard/$VENDOR/$BOARD/Config.lb or the one in
src/mainboard/$VENDOR/$BOARD/Options.lb or the one in
targets/$VENDOR/$BOARD/Config.lb or targets/$VENDOR/$BOARD/Config-abuild.lb?
</pre>
</blockquote>
Whatever is effective ;-) (helpful, I know)<br>
<br>
In my opinion targets/*/*/Config*lb shouldn't define any XIP_* values
except for failover (and maybe not even that)<br>
And in src/mainboard, it should also only be in one of the files, to
not confuse users. (again imho)<br>
<blockquote cite="mid:4A2BBC3B.3060408@gmx.net" type="cite">
<pre wrap="">I have a script which does all conversions except ROM_IMAGE_SIZE. Once
my question above is answered, I should be able to fix my script and
post it.
What bothers me is that my script changed more in the iwill/dk8_htx case
than your patch.
</pre>
</blockquote>
I wrote the instructions from looking at the patches and not the other
way around, so the "canonical" solution might be different. But there's
more than one valid way to do it. As long as it builds, it's a step in
the right direction, in my opinion. There will be more changes to make
the rom layout more space efficient etc, that should simplify and
disambiguate things.<br>
<br>
I'm on vacation the next week and thus offline.<br>
<br>
<br>
Regards,<br>
Patrick<br>
</body>
</html>