<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/board/mpl/common, branch v2011.12</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>http://cgit.235523.xyz/u-boot.git/atom/board/mpl/common?h=v2011.12</id>
<link rel='self' href='http://cgit.235523.xyz/u-boot.git/atom/board/mpl/common?h=v2011.12'/>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<updated>2011-12-12T13:23:22Z</updated>
<entry>
<title>board/mpl/common/isa.c: Fix GCC 4.6 build warnings</title>
<updated>2011-12-12T13:23:22Z</updated>
<author>
<name>David Müller (ELSOFT AG)</name>
<email>d.mueller@elsoft.ch</email>
</author>
<published>2011-12-12T01:36:53Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=99f2c76821e4ba84be1f5cc0a9d06582a2d56df6'/>
<id>urn:sha1:99f2c76821e4ba84be1f5cc0a9d06582a2d56df6</id>
<content type='text'>
Fix:
../common/isa.c: In function 'handle_isa_int':
../common/isa.c:385:21: warning: variable 'isr2' set but not used
[-Wunused-but-set-variable]
../common/isa.c:385:16: warning: variable 'isr1' set but not used
[-Wunused-but-set-variable]

Signed-off-by: David Mueller &lt;d.mueller@elsoft.ch&gt;
</content>
</entry>
<entry>
<title>board/mpl/common/flash.c: Fix GCC 4.6 build warning</title>
<updated>2011-12-01T23:16:11Z</updated>
<author>
<name>Wolfgang Denk</name>
<email>wd@denx.de</email>
</author>
<published>2011-11-29T22:17:53Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=ae5f6b3b2fa26eb82b58bd162535e5a80ebed543'/>
<id>urn:sha1:ae5f6b3b2fa26eb82b58bd162535e5a80ebed543</id>
<content type='text'>
Fix:
../common/flash.c: In function 'flash_init':
../common/flash.c:160:16: warning: variable 'size_b1' set but not used
[-Wunused-but-set-variable]

Signed-off-by: Wolfgang Denk &lt;wd@denx.de&gt;
Cc: Denis Peter &lt;d.peter@mpl.ch&gt;
Acked-by: Marek Vasut &lt;marek.vasut@gmail.com&gt;
</content>
</entry>
<entry>
<title>board/mpl/common/flash.c: fix GCC 4.6 build warning</title>
<updated>2011-11-07T21:09:23Z</updated>
<author>
<name>Wolfgang Denk</name>
<email>wd@denx.de</email>
</author>
<published>2011-11-04T15:55:05Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=ee6ed89c5b144f621969a2ef7188b52ec0c2ac9b'/>
<id>urn:sha1:ee6ed89c5b144f621969a2ef7188b52ec0c2ac9b</id>
<content type='text'>
Fix:
../common/flash.c: In function 'flash_erase':
../common/flash.c:603:24: warning: variable 'l_sect' set but not used
[-Wunused-but-set-variable]

Signed-off-by: Wolfgang Denk &lt;wd@denx.de&gt;
Cc: Denis Peter &lt;d.peter@mpl.ch&gt;
</content>
</entry>
<entry>
<title>Convert ISO-8859 files to UTF-8</title>
<updated>2011-08-04T21:34:02Z</updated>
<author>
<name>Albert ARIBAUD</name>
<email>albert.u.boot@aribaud.net</email>
</author>
<published>2011-08-04T16:45:45Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=fa82f871c8dbc9a15e8dc274b3f99dd5fa0da458'/>
<id>urn:sha1:fa82f871c8dbc9a15e8dc274b3f99dd5fa0da458</id>
<content type='text'>
There was a mix of UTF-8 and ISO-8859 files in the U-Boot source
tree, which could cause issues with the patchwork review system.
This commit converts all ISO-8859 files to UTF-8.

Signed-off-by: Albert ARIBAUD &lt;albert.u.boot@aribaud.net&gt;
</content>
</entry>
<entry>
<title>cleanup: Fix typos and misspellings in various files.</title>
<updated>2011-07-28T19:27:36Z</updated>
<author>
<name>Mike Williams</name>
<email>mike@mikebwilliams.com</email>
</author>
<published>2011-07-22T04:01:30Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=1626308797ac4184e73e56d275a1b8da11a62d5b'/>
<id>urn:sha1:1626308797ac4184e73e56d275a1b8da11a62d5b</id>
<content type='text'>
Recieve/Receive
recieve/receive
Interupt/Interrupt
interupt/interrupt
Addres/Address
addres/address

Signed-off-by: Mike Williams &lt;mike@mikebwilliams.com&gt;
</content>
</entry>
<entry>
<title>board/mpl: Remove mpl-specific memory test command</title>
<updated>2010-10-06T20:25:38Z</updated>
<author>
<name>Peter Tyser</name>
<email>ptyser@xes-inc.com</email>
</author>
<published>2010-10-06T04:48:47Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=a7d54346e025d91bb4413c61b5f5ec13ae2a6bfc'/>
<id>urn:sha1:a7d54346e025d91bb4413c61b5f5ec13ae2a6bfc</id>
<content type='text'>
The mpl-specfic memory test is only documented for one board, doesn't
compile cleanly, uses improper coding style, and overlaps functionality
with U-Boot's common 'mtest' command, so lets get rid of it.

Signed-off-by: Peter Tyser &lt;ptyser@xes-inc.com&gt;
CC: d.peter@mpl.ch
CC: d.mueller@elsoft.ch
CC: wd@denx.de
</content>
</entry>
<entry>
<title>ppc4xx: Move ppc4xx headers to powerpc include directory</title>
<updated>2010-09-23T07:02:05Z</updated>
<author>
<name>Stefan Roese</name>
<email>sr@denx.de</email>
</author>
<published>2010-09-09T17:18:00Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=b36df561154bdd0a41bb77e09c5575ca2cf48013'/>
<id>urn:sha1:b36df561154bdd0a41bb77e09c5575ca2cf48013</id>
<content type='text'>
This patch moves some ppc4xx related headers from the common include
directory (include/) to the powerpc specific one
(arch/powerpc/include/asm/). This way to common include directory is not
so cluttered with files.

Signed-off-by: Stefan Roese &lt;sr@denx.de&gt;
</content>
</entry>
<entry>
<title>Rename getenv_r() into getenv_f()</title>
<updated>2010-08-03T22:45:36Z</updated>
<author>
<name>Wolfgang Denk</name>
<email>wd@denx.de</email>
</author>
<published>2010-07-24T19:55:43Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=cdb749778aa3a8f8d2a41dd4ad811ef822aecfe6'/>
<id>urn:sha1:cdb749778aa3a8f8d2a41dd4ad811ef822aecfe6</id>
<content type='text'>
While running from flash, i. e. before relocation, we have only a
limited C runtime environment without writable data segment. In this
phase, some configurations (for example with environment in EEPROM)
must not use the normal getenv(), but a special function.  This
function had been called getenv_r(), with the idea that the "_r"
suffix would mean the same as in the _r_eentrant versions of some of
the C library functions (for example getdate vs. getdate_r, getgrent
vs. getgrent_r, etc.).

Unfortunately this was a misleading name, as in U-Boot the "_r"
generally means "running from RAM", i. e. _after_ relocation.

To avoid confusion, rename into getenv_f() [as "running from flash"]

Signed-off-by: Wolfgang Denk &lt;wd@denx.de&gt;
Acked-by: Detlev Zundel &lt;dzu@denx.de&gt;
</content>
</entry>
<entry>
<title>cmd_usage(): simplify return code handling</title>
<updated>2010-07-24T18:43:57Z</updated>
<author>
<name>Wolfgang Denk</name>
<email>wd@denx.de</email>
</author>
<published>2010-07-16T23:06:04Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=47e26b1bf91ddef69f4a3892815c857db094cef9'/>
<id>urn:sha1:47e26b1bf91ddef69f4a3892815c857db094cef9</id>
<content type='text'>
Lots of code use this construct:

	cmd_usage(cmdtp);
	return 1;

Change cmd_usage() let it return 1 - then we can replace all these
ocurrances by

	return cmd_usage(cmdtp);

This fixes a few places with incorrect return code handling, too.

Signed-off-by: Wolfgang Denk &lt;wd@denx.de&gt;
</content>
</entry>
<entry>
<title>Make sure that argv[] argument pointers are not modified.</title>
<updated>2010-07-04T21:55:42Z</updated>
<author>
<name>Wolfgang Denk</name>
<email>wd@denx.de</email>
</author>
<published>2010-06-28T20:00:46Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=54841ab50c20d6fa6c9cc3eb826989da3a22d934'/>
<id>urn:sha1:54841ab50c20d6fa6c9cc3eb826989da3a22d934</id>
<content type='text'>
The hush shell dynamically allocates (and re-allocates) memory for the
argument strings in the "char *argv[]" argument vector passed to
commands.  Any code that modifies these pointers will cause serious
corruption of the malloc data structures and crash U-Boot, so make
sure the compiler can check that no such modifications are being done
by changing the code into "char * const argv[]".

This modification is the result of debugging a strange crash caused
after adding a new command, which used the following argument
processing code which has been working perfectly fine in all Unix
systems since version 6 - but not so in U-Boot:

int main (int argc, char **argv)
{
	while (--argc &gt; 0 &amp;&amp; **++argv == '-') {
/* ====&gt; */	while (*++*argv) {
			switch (**argv) {
			case 'd':
				debug++;
				break;
			...
			default:
				usage ();
			}
		}
	}
	...
}

The line marked "====&gt;" will corrupt the malloc data structures and
usually cause U-Boot to crash when the next command gets executed by
the shell.  With the modification, the compiler will prevent this with
an
	error: increment of read-only location '*argv'

N.B.: The code above can be trivially rewritten like this:

	while (--argc &gt; 0 &amp;&amp; **++argv == '-') {
		char *arg = *argv;
		while (*++arg) {
			switch (*arg) {
			...

Signed-off-by: Wolfgang Denk &lt;wd@denx.de&gt;
Acked-by: Mike Frysinger &lt;vapier@gentoo.org&gt;
</content>
</entry>
</feed>
