[U-Boot-Users] [rfc] warning about overlapping regions when booting with bootm
we semi-frequently get users who try to boot an image on top of itself and
when when things crash, dont realize why. i put together this quick little
warning, but i'm guessing that Wolfgang's answer is "don't bloat the code for
stupid people" ...
--- common/cmd_bootm.c
+++ common/cmd_bootm.c
@@ -1434,6 +1439,16 @@ int gunzip(void *dst, int dstlen, unsign
z_stream s;
int r, i, flags;
+ /* If memory regions overlap, whine about it. We cannot check
+* the dstlen value as it is usually set to the max possible value
+* (CFG_BOOTM_LEN) which can be huge (64M). Instead, we'll assume
+* the decompression size will be at least as big as the compressed
+* size so that we can at least check part of the destination.
+*/
+ if ((dst <= (void *)src && (void *)src < dst + /*dstlen*/ *lenp) ||
+ (dst <= (void *)(src + *lenp) && (void *)(src + *lenp) < dst +
/*dstlen*/ *lenp))
+ puts ("\n Warning: src and dst regions overlap ... ");
+
/* skip header */
i = 10;
flags = src[3];
-mike
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] [PATCH] easylogo: clean up some more and add -r (rgb) support
Michael Hennerich added support for outputting an image in RGB format rather
than forcing YUYV all the time. This makes obvious sense if the display you
have takes RGB input rather than YUYV.
Rather than hack in support for options, I've converted it to use getopt and
cleaned up the argument parsing in the process.
Signed-off-by: Michael Hennerich <[EMAIL PROTECTED]>
Signed-off-by: Mike Frysinger <[EMAIL PROTECTED]>
---
tools/easylogo/easylogo.c | 125 +---
1 files changed, 71 insertions(+), 54 deletions(-)
diff --git a/tools/easylogo/easylogo.c b/tools/easylogo/easylogo.c
index 080bea9..c49a0e3 100644
--- a/tools/easylogo/easylogo.c
+++ b/tools/easylogo/easylogo.c
@@ -7,6 +7,8 @@
** This is still under construction!
*/
+#include
+#include
#include
#include
#include
@@ -216,15 +218,10 @@ int image_load_tga (image_t * image, char *filename)
return 0;
}
-int image_free (image_t * image)
+void image_free (image_t * image)
{
- if (image->data != NULL)
- free (image->data);
-
- if (image->palette != NULL)
- free (image->palette);
-
- return 0;
+ free (image->data);
+ free (image->palette);
}
int image_rgb_to_yuyv (image_t * rgb_image, image_t * yuyv_image)
@@ -353,59 +350,76 @@ int image_save_header (image_t * image, char *filename,
char *varname)
#define DEF_FILELEN256
+static void usage (int exit_status)
+{
+ puts (
+ "EasyLogo 1.0 (C) 2000 by Paolo Scaffardi\n"
+ "\n"
+ "Syntax:easylogo [options] inputfile [outputvar
[outputfile]]\n"
+ "\n"
+ "Options:\n"
+ " -r Output RGB instead of YUYV\n"
+ " -h Help output\n"
+ "\n"
+ "Where: 'inputfile' is the TGA image to load\n"
+ " 'outputvar' is the variable name to create\n"
+ " 'outputfile' is the output header file (default is
'inputfile.h')"
+ );
+ exit (exit_status);
+}
+
int main (int argc, char *argv[])
{
+ int c;
+ bool use_rgb = false;
char inputfile[DEF_FILELEN],
outputfile[DEF_FILELEN], varname[DEF_FILELEN];
image_t rgb_logo, yuyv_logo;
- switch (argc) {
- case 2:
- case 3:
- case 4:
- strcpy (inputfile, argv[1]);
-
- if (argc > 2)
- strcpy (varname, argv[2]);
- else {
- char *dot = strchr (inputfile, '.');
- int pos = dot - inputfile;
-
- if (dot) {
- strncpy (varname, inputfile, pos);
- varname[pos] = 0;
- }
- }
-
- if (argc > 3)
- strcpy (outputfile, argv[3]);
- else {
- char *dot = strchr (varname, '.');
- int pos = dot - varname;
-
- if (dot) {
- char app[DEF_FILELEN];
-
- strncpy (app, varname, pos);
- app[pos] = 0;
- sprintf (outputfile, "%s.h", app);
- }
+ while ((c = getopt(argc, argv, "hr")) > 0) {
+ switch (c) {
+ case 'h':
+ usage (0);
+ break;
+ case 'r':
+ use_rgb = true;
+ puts ("Using 24-bit RGB Output Fromat");
+ break;
+ default:
+ usage (1);
+ break;
}
- break;
-
- default:
- printf ("EasyLogo 1.0 (C) 2000 by Paolo Scaffardi\n\n");
+ }
- printf("Syntax: easylogo inputfile [outputvar {outputfile}]
\n");
- printf("\n");
- printf("Where: 'inputfile' is the TGA image to load\n");
- printf("'outputvar' is the variable name to
create\n");
- printf("'outputfile'is the output header file
(default is 'inputfile.h')\n");
+ c = argc - optind;
+ if (c > 4 || c < 1)
+ usage (1);
+
+ strcpy (inputfile, argv[optind]);
+
+ if (c > 1)
+ strcpy (varname, argv[optind + 1]);
+ else {
+ /* transform "input.tga" to just "input" */
+ char *dot;
+ strcpy (varname, inputfile);
+ dot = strchr (varname, '.');
+ if (dot)
+ *dot = '\0';
+ }
- return -1;
+ if (c > 2)
+ strcpy (outputfile, argv[optind + 2]);
+ else {
+ /* just append ".h" to input file name */
+ strcpy (outputfile, inp
[U-Boot-Users] [PATCH] add target for $(LDSCRIPT)
If the $(LDSCRIPT) does not exist (normally it's board/$(BOARD)/u-boot.lds), then change into the board directory and try and create it. This allows you to generate the linker script on the fly based upon board defines (like the Blackfin boards do). There should be no regressions due to this change as the normal case is to already have a u-boot.lds file. If that's the case, then there's nothing to generate, and so make will always exit. The fix here is that if the linker script does not exist, the implicit rules take over and attempt to guess how to generate the file. Signed-off-by: Mike Frysinger <[EMAIL PROTECTED]> --- Makefile |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/Makefile b/Makefile index a731ee8..7446bc7 100644 --- a/Makefile +++ b/Makefile @@ -326,6 +326,9 @@ $(LIBS):depend $(obj)include/autoconf.mk $(SUBDIRS):depend $(obj)include/autoconf.mk $(MAKE) -C $@ all +$(LDSCRIPT): depend $(obj)include/autoconf.mk + $(MAKE) -C $(dir $@) $(notdir $@) + $(NAND_SPL): $(VERSION_FILE) $(obj)include/autoconf.mk $(MAKE) -C nand_spl/board/$(BOARDDIR) all -- 1.5.4 - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] [ppc4xx] Please pull git://www.denx.de/git/u-boot-ppc4xx.git
The following changes since commit 5db6138565ad4da190f94e0bc1d89407d58a2ab2: Wolfgang Denk (1): Merge branch 'master' of git://www.denx.de/git/u-boot-arm are available in the git repository at: git://www.denx.de/git/u-boot-ppc4xx.git master Larry Johnson (1): ppc4xx: Beautify configuration files for Sequoia and Korat boards Mike Nuss (1): PPC440EPx: Optionally enable second I2C bus Niklaus Giger (3): ppc4xx: PPC405GPr fix missing register definitions ppc4xx: HCU4/5. Cleanups ppc4xx: HCU4/5. Cleanup configs board/amcc/sequoia/sequoia.c |5 +- board/netstal/common/fixed_sdram.c |2 +- board/netstal/common/hcu_flash.c | 14 -- board/netstal/common/nm.h | 11 ++- board/netstal/common/nm_bsp.c |4 +- board/netstal/hcu4/hcu4.c | 80 ++--- board/netstal/hcu5/hcu5.c |8 +- board/netstal/hcu5/sdram.c | 32 +++-- include/configs/hcu4.h | 26 ++--- include/configs/hcu5.h | 47 +++ include/configs/korat.h| 234 ++- include/configs/sequoia.h | 238 ++-- include/ppc405.h |8 ++ 13 files changed, 346 insertions(+), 363 deletions(-) - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH 4/4: resubmit] ppc4xx: HCU4/5. Cleanup configs
On Tuesday 05 February 2008, Niklaus Giger wrote: > - hcu4.h: Removed define of CONFIG_PPC405GPr > - Corrected phy addresses > - Fix boot variables > - Respect line length of 80 chars > > Signed-off-by: Niklaus Giger <[EMAIL PROTECTED]> Applied. Thanks. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] = - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH 3/4:resubmit] ppc4xx: HCU4/5. Cleanups
On Tuesday 05 February 2008, Niklaus Giger wrote: > - Fix some coding style violations. > - Use in/out_u16/32 where appropriate. > - Use register names from ppc405.h. > - Fix trace useage for Lauterbach. > - Remove obsolete generation HCU2. > - Renamed fixed_hcu4_sdram to init_ppc405_sdram. > > Signed-off-by: Niklaus Giger <[EMAIL PROTECTED]> Applied. Thanks. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] = - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH 1/4: resubmit] ppc4xx: PPC405GPr fix missing register definitions
On Tuesday 05 February 2008, Niklaus Giger wrote: > Signed-off-by: Niklaus Giger <[EMAIL PROTECTED]> Applied. Thanks. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] = - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] ppc4xx: Beautify configuration files for Sequoia and Korat boards
On Saturday 19 January 2008, Larry Johnson wrote: > Signed-off-by: Larry Johnson <[EMAIL PROTECTED]> Applied. Thanks. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] = - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] PPC440EPx: Optionally enable second I2C bus
On Wednesday 06 February 2008, Mike Nuss wrote: > The option CONFIG_I2C_MULTI_BUS does not have any effect on Sequoia, the > PPC440EPx reference platform, because IIC1 is never enabled. Add Sequoia > board code to turn on IIC1 if CONFIG_I2C_MULTI_BUS is selected. > > Signed-off-by: Mike Nuss <[EMAIL PROTECTED]> > Cc: Stefan Roese <[EMAIL PROTECTED]> Applied. Thanks. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] = - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] PPC440EPx: Reconfigure PLL for 667MHz processor
Hi Mike,
sorry for the late response.
On Tuesday 05 February 2008, Mike Nuss wrote:
> On PPC440EPx without a bootstrap I2C EEPROM, the PLL can be reconfigured
> after startup in order to change the speed of the clocks. This patch adds
> the option CONFIG_667MHZ. If set, it will set the clocks to run at full
> speed on a 667MHz PPC440EPx without the need for an external EEPROM.
I think it makes sense to move these PLL checking/reconfiguration stuff into a
separate function. Perhaps like this:
In you board config file:
#define CFG_PLL_RECONFIG667 /* comment please */
And in cpu_init.c:
#ifndef CFG_PLL_RECONFIG
#define CFG_PLL_RECONFIG0
#endif
void reconfigure_pll(u32 new_cpu_freq)
{
#if defined(CONFIG_440EPX)
if (new_cpu_freq == 667) {
...
your code here...
...
}
#endif
}
void
cpu_init_f (void)
{
#if defined(CONFIG_WATCHDOG)
unsigned long val;
#endif
reconfigure_pll(CFG_PLL_RECONFIG);
...
I think this is clearer. What do you think? Please clean up and resend.
Thanks.
Best regards,
Stefan
=
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED]
=
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] Enabling UART0 and UART1
On Friday 15 February 2008, Sean wrote: > I have enabled both UART0 and UART 1 using the CONFIG_SERIAL_MULTI in my > board specific header file. The board is based on PPC 440GX processor. I > used the serial1_puts and serial1_getc to access the UART1 but without > access. Does anybody have used this functions before? Yes, this was tested successfully on multiple 4xx platforms. I don't remember testing it on a 440GX though. Could be that the pin multiplexing is not initialized correctly. Please recheck that the GPIO configuration and the PFC registers are setup correctly for UART1. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] = - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] [PATCH] make define2mk.sed work on FreeBSD
In the thread "[1.3.2-rc1] MPC8548CDS/MPC8555CDS configs fails to link", the define2mk.sed script was identified as the source of the link failure on FreeBSD. The problem is that sed(1) does not always support the '+' operator. It isn't on FreeBSD. The attach patch implements the equivalent, using the '*' operator instead and should work everywhere. -- Marcel Moolenaar [EMAIL PROTECTED] u-boot.patch Description: Binary data - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] smc9111_eeprom committed by mistake?
On Friday 15 February 2008, Shinya Kuribayashi wrote: > [EMAIL PROTECTED]:~/devel/u-boot.git$ make mrproper > [EMAIL PROTECTED]:~/devel/u-boot.git$ > [EMAIL PROTECTED]:~/devel/u-boot.git$ git status > # On branch master > # Changed but not updated: > # (use "git add/rm ..." to update what will be committed) > # > # deleted:examples/smc9_eeprom > # > # Untracked files: > # (use "git add ..." to include in what will be committed) > # > # mips_bios.bin > # mipsel_bios.bin > no changes added to commit (use "git add" and/or "git commit -a") > [EMAIL PROTECTED]:~/devel/u-boot.git$ > > Where does this smc9_eeprom come from? I don't know since when. > A quick search picked a commit below: > > commit 32a9f5f2160a034ea87ea651b233ef7c635e55cf > Author: Mike Frysinger <[EMAIL PROTECTED]> > Date: Mon Feb 4 19:26:54 2008 -0500 > > make smc9_eeprom managment simpler by depending on the board > configurati > > Signed-off-by: Mike Frysinger <[EMAIL PROTECTED]> i had noticed that as well, but i thought i backed that out before pushing the repo for Woflgang to pull. it should not be in the repository. -mike signature.asc Description: This is a digitally signed message part. - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] smc9111_eeprom committed by mistake?
[EMAIL PROTECTED]:~/devel/u-boot.git$ make mrproper [EMAIL PROTECTED]:~/devel/u-boot.git$ [EMAIL PROTECTED]:~/devel/u-boot.git$ git status # On branch master # Changed but not updated: # (use "git add/rm ..." to update what will be committed) # # deleted:examples/smc9_eeprom # # Untracked files: # (use "git add ..." to include in what will be committed) # # mips_bios.bin # mipsel_bios.bin no changes added to commit (use "git add" and/or "git commit -a") [EMAIL PROTECTED]:~/devel/u-boot.git$ Where does this smc9_eeprom come from? I don't know since when. A quick search picked a commit below: commit 32a9f5f2160a034ea87ea651b233ef7c635e55cf Author: Mike Frysinger <[EMAIL PROTECTED]> Date: Mon Feb 4 19:26:54 2008 -0500 make smc9_eeprom managment simpler by depending on the board configurati Signed-off-by: Mike Frysinger <[EMAIL PROTECTED]> - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [1.3.2-rc1] MPC8548CDS/MPC8555CDS configs fails to link
On Feb 15, 2008, at 3:34 PM, Wolfgang Denk wrote:
>> Build machine: FreeBSD-6.1
>>
>> apg-bbuild09% gmake distclean
>> find: -lname: unknown option
>> find: -lname: unknown option
>> find: -lname: unknown option
> ^
>
> And you should pay attention to error messages like that. After such
> errors, all other problems may be just subsequent faults.
Those errors are harmless, I've been seeing those for a long
time. The problem is with sed(1). The following patch makes
things work for me:
apg-bbuild09% diff -u ~/U-Boot/u-boot-1.3.2/tools/scripts/
define2mk.sed.orig ~/U-Boot/u-boot-1.3.2/tools/scripts/define2mk.sed
--- /homes/marcelm/U-Boot/u-boot-1.3.2/tools/scripts/
define2mk.sed.orig Thu Dec 6 01:21:19 2007
+++ /homes/marcelm/U-Boot/u-boot-1.3.2/tools/scripts/define2mk.sed Fri
Feb 15 16:43:56 2008
@@ -7,11 +7,11 @@
#
# Only process values prefixed with #define CONFIG_
-/^#define CONFIG_[A-Za-z0-9_]\+/ {
+/^#define CONFIG_[A-Za-z0-9_]+/ {
# Strip the #define prefix
s/#define *//;
# Change to form CONFIG_*=VALUE
- s/ \+/=/;
+ s/ +/=/;
# Drop trailing spaces
s/ *$//;
# drop quotes around string values
Escaping the '+' makes it a literal character and that's
not what's meant here...
...or am I missing something obvious?
--
Marcel Moolenaar
[EMAIL PROTECTED]
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] Pull request:[ARM]
In message <[EMAIL PROTECTED]> you wrote: > Wolfgang > > Please pull from the merge branch at git://denx.de/git/u-boot-arm.git master > > --- > The following changes since commit 45da195cf675d0b4054aeeabf7b9c0daf798841d: > Wolfgang Denk (1): > Merge branch 'master' of > git+ssh://10.10.0.7/home/wd/git/u-boot/master > > are available in the git repository at: > > git://www.denx.de/git/u-boot-arm.git master > > Andreas Engel (1): > ARM: cleanup duplicated exception handlingcode > > Haavard Skinnemoen (1): > Move AT91RM9200DK board support under board/atmel > > Peter Pearse (6): > Update board NetStar > Merge branch '070524_netstar' of git://linux-arm.org/u-boot-armdev > Merge branch '080116_at91cap9' of git://linux-arm.org/u-boot-armdev > Merge branch '080131_artila' of git://linux-arm.org/u-boot-armdev > Merge branch '080202_at91rm9200dk' of > git://linux-arm.org/u-boot-armdev > Merge branch '080208_dupint' of git://linux-arm.org/u-boot-armdev > > Stelian Pop (9): > Fix arm926ejs compile when SKIP_LOWLEVEL_INIT is on > Improve DataFlash CS definition. > AT91CAP9 support : build integration > AT91CAP9 support : include/ files > AT91CAP9 support : cpu/ files > AT91CAP9 support : board/ files > AT91CAP9 support : MACB changes > AT91CAP9 support : move board files to Atmel vendor directory. > AT91CAP9 support > > Timo Tuunainen (1): > Support for Artila M-501 starter kit Done, thanks lot! Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] [War] is instinctive. But the instinct can be fought. We're human beings with the blood of a million savage years on our hands! But we can stop it. We can admit that we're killers ... but we're not going to kill today. That's all it takes! Knowing that we're not going to kill today! -- Kirk, "A Taste of Armageddon", stardate 3193.0 - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [1.3.2-rc1] MPC8548CDS/MPC8555CDS configs fails to link
Marcel, in message <[EMAIL PROTECTED]> you wrote: > > I've just pulled the 1.3.2-rc1 snapshot from GIT and I run > into a link failure right out of the box (ok, after changing > the Makefile to the right cross-compiler :-) There is no need to modify the Makefile. All you have to do is set and export the CROSS_COMPILE environment variable as needed. > Build machine: FreeBSD-6.1 > > apg-bbuild09% gmake distclean > find: -lname: unknown option > find: -lname: unknown option > find: -lname: unknown option ^ And you should pay attention to error messages like that. After such errors, all other problems may be just subsequent faults. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] How many QA engineers does it take to screw in a lightbulb? 3: 1 to screw it in and 2 to say "I told you so" when it doesn't work. - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] Wipe out assembler warnings while compiling x86 biosemu
In message <[EMAIL PROTECTED]> you wrote: > This patch tries to get rid of some assembler warnings about > changed .got2 section type while compiling x86 bios emulator > code. > > Signed-off-by: Anatolij Gustschin <[EMAIL PROTECTED]> > --- > drivers/bios_emulator/biosemu.c|4 ++-- > drivers/bios_emulator/include/x86emu.h | 10 ++ > drivers/bios_emulator/x86emu/ops.c | 14 +++--- > drivers/bios_emulator/x86emu/ops2.c|2 +- > 4 files changed, 20 insertions(+), 10 deletions(-) Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] Schshschshchsch. -- The Gorn, "Arena", stardate 3046.2 - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [1.3.2-rc1] MPC8548CDS/MPC8555CDS configs fails to link
On Feb 15, 2008, at 3:18 PM, Kumar Gala wrote: >> I think this also means that config.mk is different. Do >> you have CONFIG_CMD_FLASH config.mk? > > include/autoconf.mk has CONFIG_CMD_FLASH=y Mine is empty (size = 0). Thanks, that's all I need to know! -- Marcel Moolenaar [EMAIL PROTECTED] - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [1.3.2-rc1] MPC8548CDS/MPC8555CDS configs fails to link
On Feb 15, 2008, at 5:14 PM, Marcel Moolenaar wrote: > > On Feb 15, 2008, at 2:59 PM, Kumar Gala wrote: > >> >> On Feb 15, 2008, at 4:15 PM, Marcel Moolenaar wrote: >> >>> I've just pulled the 1.3.2-rc1 snapshot from GIT and I run >>> into a link failure right out of the box (ok, after changing >>> the Makefile to the right cross-compiler :-) >>> >>> Build machine: FreeBSD-6.1 >>> >> I just did a fresh clone and it worked fine for me. (Building both >> MPC8548CDS and MPC8555CDS) > > Interesting. That means that cmd_flash.o is built for > you. Can you verify that you have cmd_flash.c in > common/.depend and cmd_flash.o in common? I do have cmd_flash.o in common/.depend. and common/cmd_flash.o exists (its about 20k) > > I think this also means that config.mk is different. Do > you have CONFIG_CMD_FLASH config.mk? include/autoconf.mk has CONFIG_CMD_FLASH=y > BTW: Thanks for the feedback. It's probably a tool > issue then. - k - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [1.3.2-rc1] MPC8548CDS/MPC8555CDS configs fails to link
On Feb 15, 2008, at 2:59 PM, Kumar Gala wrote: > > On Feb 15, 2008, at 4:15 PM, Marcel Moolenaar wrote: > >> I've just pulled the 1.3.2-rc1 snapshot from GIT and I run >> into a link failure right out of the box (ok, after changing >> the Makefile to the right cross-compiler :-) >> >> Build machine: FreeBSD-6.1 >> > I just did a fresh clone and it worked fine for me. (Building both > MPC8548CDS and MPC8555CDS) Interesting. That means that cmd_flash.o is built for you. Can you verify that you have cmd_flash.c in common/.depend and cmd_flash.o in common? I think this also means that config.mk is different. Do you have CONFIG_CMD_FLASH config.mk? BTW: Thanks for the feedback. It's probably a tool issue then. -- Marcel Moolenaar [EMAIL PROTECTED] - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [1.3.2-rc1] MPC8548CDS/MPC8555CDS configs fails to link
On Feb 15, 2008, at 4:15 PM, Marcel Moolenaar wrote: > All, > > I've just pulled the 1.3.2-rc1 snapshot from GIT and I run > into a link failure right out of the box (ok, after changing > the Makefile to the right cross-compiler :-) > > Build machine: FreeBSD-6.1 > > apg-bbuild09% gmake distclean > find: -lname: unknown option > find: -lname: unknown option > find: -lname: unknown option > apg-bbuild09% gmake MPC8548CDS_config > Configuring for MPC8548CDS board... > apg-bbuild09% gmake > for dir in tools examples api_examples ; do gmake -C $dir _depend ; > done > gmake[1]: Entering directory `/.amd/uranus1/vol/homes/homes1/marcelm/ > U- > Boot/u-boot-1.3.2/tools' > ... > gmake[1]: Entering directory `/.amd/uranus1/vol/homes/homes1/marcelm/ > U- > Boot/u-boot-1.3.2/api' > /volume/fwtools/gnusense/340_2004a-magnesium.1/bin/powerpc-juniper- > eabi-ar crv libapi.a > gmake[1]: Leaving directory `/.amd/uranus1/vol/homes/homes1/marcelm/U- > Boot/u-boot-1.3.2/api' > UNDEF_SYM=`/volume/fwtools/gnusense/340_2004a-magnesium.1/bin/powerpc- > juniper-eabi-objdump -x lib_generic/libgeneric.a board/freescale/ > common/libfreescale.a board/freescale/mpc8548cds/libmpc8548cds.a cpu/ > mpc85xx/libmpc85xx.a lib_ppc/libppc.a fs/cramfs/libcramfs.a fs/fat/ > libfat.a fs/fdos/libfdos.a fs/jffs2/libjffs2.a fs/reiserfs/ > libreiserfs.a fs/ext2/libext2fs.a net/libnet.a disk/libdisk.a drivers/ > bios_emulator/libatibiosemu.a drivers/block/libblock.a drivers/dma/ > libdma.a drivers/hwmon/libhwmon.a drivers/i2c/libi2c.a drivers/input/ > libinput.a drivers/misc/libmisc.a drivers/mtd/libmtd.a drivers/mtd/ > nand/libnand.a drivers/mtd/nand_legacy/libnand_legacy.a drivers/mtd/ > onenand/libonenand.a drivers/net/libnet.a drivers/net/sk98lin/ > libsk98lin.a drivers/pci/libpci.a drivers/pcmcia/libpcmcia.a drivers/ > spi/libspi.a drivers/qe/qe.a drivers/rtc/librtc.a drivers/serial/ > libserial.a drivers/usb/libusb.a drivers/video/libvideo.a post/ > libpost.a post/drivers/libpostdrivers.a post/lib_ppc/libpostppc.a > post/ > lib_ppc/fpu/libpostppcfpu.a common/libcommon.a libfdt/libfdt.a api/ > libapi.a |sed -n -e 's/.*\(__u_boot_cmd_.*\)/-u\1/p'|sort|uniq`;\ > cd /.amd/uranus1/vol/homes/homes1/marcelm/U-Boot/u-boot-1.3.2 > && / > volume/fwtools/gnusense/340_2004a-magnesium.1/bin/powerpc-juniper- > eabi- > ld -Bstatic -T /.amd/uranus1/vol/homes/homes1/marcelm/U-Boot/u- > boot-1.3.2/board/freescale/mpc8548cds/u-boot.lds -n -Ttext 0xfff8 > $UNDEF_SYM cpu/mpc85xx/start.o cpu/mpc85xx/resetvec.o \ > --start-group lib_generic/libgeneric.a > board/freescale/common/ > libfreescale.a board/freescale/mpc8548cds/libmpc8548cds.a cpu/mpc85xx/ > libmpc85xx.a lib_ppc/libppc.a fs/cramfs/libcramfs.a fs/fat/libfat.a > fs/ > fdos/libfdos.a fs/jffs2/libjffs2.a fs/reiserfs/libreiserfs.a fs/ext2/ > libext2fs.a net/libnet.a disk/libdisk.a drivers/bios_emulator/ > libatibiosemu.a drivers/block/libblock.a drivers/dma/libdma.a drivers/ > hwmon/libhwmon.a drivers/i2c/libi2c.a drivers/input/libinput.a > drivers/ > misc/libmisc.a drivers/mtd/libmtd.a drivers/mtd/nand/libnand.a > drivers/ > mtd/nand_legacy/libnand_legacy.a drivers/mtd/onenand/libonenand.a > drivers/net/libnet.a drivers/net/sk98lin/libsk98lin.a drivers/pci/ > libpci.a drivers/pcmcia/libpcmcia.a drivers/spi/libspi.a drivers/qe/ > qe.a drivers/rtc/librtc.a drivers/serial/libserial.a drivers/usb/ > libusb.a drivers/video/libvideo.a post/libpost.a post/drivers/ > libpostdrivers.a post/lib_ppc/libpostppc.a post/lib_ppc/fpu/ > libpostppcfpu.a common/libcommon.a libfdt/libfdt.a api/libapi.a --end- > group -L /.amd/svl-eng001-cf2/vol/tools/bt/fwtools-fbsd6.1-x86/ > gnusense/340_2004a-magnesium.1/bin/../lib/gcc/powerpc-juniper-eabi/ > 3.4.0/nof -lgcc \ > -Map u-boot.map -o u-boot > /volume/fwtools/gnusense/340_2004a-magnesium.1/bin/powerpc-juniper- > eabi-ld: skipping incompatible /usr/lib/libgcc.a when searching for - > lgcc > cpu/mpc85xx/cpu_init.o(.text+0xa4): In function `cpu_init_early_f': > /.amd/uranus1/vol/homes/homes1/marcelm/U-Boot/u-boot-1.3.2/cpu/ > mpc85xx/ > cpu_init.c:150: undefined reference to `init_laws' > cpu/mpc85xx/cpu_init.o(.text+0x1ac): In function `cpu_init_r': > /.amd/uranus1/vol/homes/homes1/marcelm/U-Boot/u-boot-1.3.2/cpu/ > mpc85xx/ > cpu_init.c:258: undefined reference to `disable_law' > common/libcommon.a(cmd_bootm.o)(.text+0x950): In function > `do_bootm_linux': > /.amd/uranus1/vol/homes/homes1/marcelm/U-Boot/u-boot-1.3.2/common/ > cmd_bootm.c:748: undefined reference to `fdt_check_header' > common/libcommon.a(cmd_bootm.o)(.text+0xacc):/.amd/uranus1/vol/homes/ > homes1/marcelm/U-Boot/u-boot-1.3.2/common/cmd_bootm.c:799: undefined > reference to `fdt_check_header' > common/libcommon.a(cmd_bootm.o)(.text+0xbc4):/.amd/uranus1/vol/homes/ > homes1/marcelm/U-Boot/u-boot-1.3.2/common/cmd_bootm.c:849: undefined > reference to `fdt_check_header' > common/libcommon.a(cmd_bootm.o)(.text+0xcfc):/.amd/uranus1/vol
[U-Boot-Users] [1.3.2-rc1] MPC8548CDS/MPC8555CDS configs fails to link
All, I've just pulled the 1.3.2-rc1 snapshot from GIT and I run into a link failure right out of the box (ok, after changing the Makefile to the right cross-compiler :-) Build machine: FreeBSD-6.1 apg-bbuild09% gmake distclean find: -lname: unknown option find: -lname: unknown option find: -lname: unknown option apg-bbuild09% gmake MPC8548CDS_config Configuring for MPC8548CDS board... apg-bbuild09% gmake for dir in tools examples api_examples ; do gmake -C $dir _depend ; done gmake[1]: Entering directory `/.amd/uranus1/vol/homes/homes1/marcelm/U- Boot/u-boot-1.3.2/tools' ... gmake[1]: Entering directory `/.amd/uranus1/vol/homes/homes1/marcelm/U- Boot/u-boot-1.3.2/api' /volume/fwtools/gnusense/340_2004a-magnesium.1/bin/powerpc-juniper- eabi-ar crv libapi.a gmake[1]: Leaving directory `/.amd/uranus1/vol/homes/homes1/marcelm/U- Boot/u-boot-1.3.2/api' UNDEF_SYM=`/volume/fwtools/gnusense/340_2004a-magnesium.1/bin/powerpc- juniper-eabi-objdump -x lib_generic/libgeneric.a board/freescale/ common/libfreescale.a board/freescale/mpc8548cds/libmpc8548cds.a cpu/ mpc85xx/libmpc85xx.a lib_ppc/libppc.a fs/cramfs/libcramfs.a fs/fat/ libfat.a fs/fdos/libfdos.a fs/jffs2/libjffs2.a fs/reiserfs/ libreiserfs.a fs/ext2/libext2fs.a net/libnet.a disk/libdisk.a drivers/ bios_emulator/libatibiosemu.a drivers/block/libblock.a drivers/dma/ libdma.a drivers/hwmon/libhwmon.a drivers/i2c/libi2c.a drivers/input/ libinput.a drivers/misc/libmisc.a drivers/mtd/libmtd.a drivers/mtd/ nand/libnand.a drivers/mtd/nand_legacy/libnand_legacy.a drivers/mtd/ onenand/libonenand.a drivers/net/libnet.a drivers/net/sk98lin/ libsk98lin.a drivers/pci/libpci.a drivers/pcmcia/libpcmcia.a drivers/ spi/libspi.a drivers/qe/qe.a drivers/rtc/librtc.a drivers/serial/ libserial.a drivers/usb/libusb.a drivers/video/libvideo.a post/ libpost.a post/drivers/libpostdrivers.a post/lib_ppc/libpostppc.a post/ lib_ppc/fpu/libpostppcfpu.a common/libcommon.a libfdt/libfdt.a api/ libapi.a |sed -n -e 's/.*\(__u_boot_cmd_.*\)/-u\1/p'|sort|uniq`;\ cd /.amd/uranus1/vol/homes/homes1/marcelm/U-Boot/u-boot-1.3.2 && / volume/fwtools/gnusense/340_2004a-magnesium.1/bin/powerpc-juniper-eabi- ld -Bstatic -T /.amd/uranus1/vol/homes/homes1/marcelm/U-Boot/u- boot-1.3.2/board/freescale/mpc8548cds/u-boot.lds -n -Ttext 0xfff8 $UNDEF_SYM cpu/mpc85xx/start.o cpu/mpc85xx/resetvec.o \ --start-group lib_generic/libgeneric.a board/freescale/common/ libfreescale.a board/freescale/mpc8548cds/libmpc8548cds.a cpu/mpc85xx/ libmpc85xx.a lib_ppc/libppc.a fs/cramfs/libcramfs.a fs/fat/libfat.a fs/ fdos/libfdos.a fs/jffs2/libjffs2.a fs/reiserfs/libreiserfs.a fs/ext2/ libext2fs.a net/libnet.a disk/libdisk.a drivers/bios_emulator/ libatibiosemu.a drivers/block/libblock.a drivers/dma/libdma.a drivers/ hwmon/libhwmon.a drivers/i2c/libi2c.a drivers/input/libinput.a drivers/ misc/libmisc.a drivers/mtd/libmtd.a drivers/mtd/nand/libnand.a drivers/ mtd/nand_legacy/libnand_legacy.a drivers/mtd/onenand/libonenand.a drivers/net/libnet.a drivers/net/sk98lin/libsk98lin.a drivers/pci/ libpci.a drivers/pcmcia/libpcmcia.a drivers/spi/libspi.a drivers/qe/ qe.a drivers/rtc/librtc.a drivers/serial/libserial.a drivers/usb/ libusb.a drivers/video/libvideo.a post/libpost.a post/drivers/ libpostdrivers.a post/lib_ppc/libpostppc.a post/lib_ppc/fpu/ libpostppcfpu.a common/libcommon.a libfdt/libfdt.a api/libapi.a --end- group -L /.amd/svl-eng001-cf2/vol/tools/bt/fwtools-fbsd6.1-x86/ gnusense/340_2004a-magnesium.1/bin/../lib/gcc/powerpc-juniper-eabi/ 3.4.0/nof -lgcc \ -Map u-boot.map -o u-boot /volume/fwtools/gnusense/340_2004a-magnesium.1/bin/powerpc-juniper- eabi-ld: skipping incompatible /usr/lib/libgcc.a when searching for - lgcc cpu/mpc85xx/cpu_init.o(.text+0xa4): In function `cpu_init_early_f': /.amd/uranus1/vol/homes/homes1/marcelm/U-Boot/u-boot-1.3.2/cpu/mpc85xx/ cpu_init.c:150: undefined reference to `init_laws' cpu/mpc85xx/cpu_init.o(.text+0x1ac): In function `cpu_init_r': /.amd/uranus1/vol/homes/homes1/marcelm/U-Boot/u-boot-1.3.2/cpu/mpc85xx/ cpu_init.c:258: undefined reference to `disable_law' common/libcommon.a(cmd_bootm.o)(.text+0x950): In function `do_bootm_linux': /.amd/uranus1/vol/homes/homes1/marcelm/U-Boot/u-boot-1.3.2/common/ cmd_bootm.c:748: undefined reference to `fdt_check_header' common/libcommon.a(cmd_bootm.o)(.text+0xacc):/.amd/uranus1/vol/homes/ homes1/marcelm/U-Boot/u-boot-1.3.2/common/cmd_bootm.c:799: undefined reference to `fdt_check_header' common/libcommon.a(cmd_bootm.o)(.text+0xbc4):/.amd/uranus1/vol/homes/ homes1/marcelm/U-Boot/u-boot-1.3.2/common/cmd_bootm.c:849: undefined reference to `fdt_check_header' common/libcommon.a(cmd_bootm.o)(.text+0xcfc):/.amd/uranus1/vol/homes/ homes1/marcelm/U-Boot/u-boot-1.3.2/common/cmd_bootm.c:965: undefined reference to `fdt_open_into' common/libcommon.a(cmd_bootm.o)(.text+0xd44):/.amd/uranus1/vol/homes/ homes1/marcelm/U-Boot/u-
[U-Boot-Users] [PATCH v3] Add sub-commands to fdt
>From e76f97d16c5932d19d698e42c6376d6bbdca2ecf Mon Sep 17 00:00:00 2001
From: Kumar Gala <[EMAIL PROTECTED]>
Date: Fri, 15 Feb 2008 03:34:36 -0600
Subject: [PATCH] Add sub-commands to fdt
fdt header - Display header info
fdt bootcpu - Set boot cpuid
fdt memory - Add/Update memory node
fdt rsvmem print- Show current mem reserves
fdt rsvmem add - Add a mem reserve
fdt rsvmem delete- Delete a mem reserves
Signed-off-by: Kumar Gala <[EMAIL PROTECTED]>
---
renamed memrsv to rsvmem to be unique from memory
common/cmd_fdt.c | 112 +-
1 files changed, 111 insertions(+), 1 deletions(-)
diff --git a/common/cmd_fdt.c b/common/cmd_fdt.c
index 9cd22ee..c31560b 100644
--- a/common/cmd_fdt.c
+++ b/common/cmd_fdt.c
@@ -260,7 +260,7 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char
*argv[])
/
* Remove a property/node
/
- } else if (argv[1][0] == 'r') {
+ } else if ((argv[1][0] == 'r') && (argv[1][1] == 'm')) {
int nodeoffset;/* node offset from libfdt */
int err;
@@ -296,6 +296,110 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char
*argv[])
return err;
}
}
+
+ /
+* Display header info
+/
+ } else if (argv[1][0] == 'h') {
+ u32 version = fdt_version(fdt);
+ printf("magic:\t\t\t0x%x\n", fdt_magic(fdt));
+ printf("totalsize:\t\t0x%x (%d)\n", fdt_totalsize(fdt),
fdt_totalsize(fdt));
+ printf("off_dt_struct:\t\t0x%x\n", fdt_off_dt_struct(fdt));
+ printf("off_dt_strings:\t\t0x%x\n", fdt_off_dt_strings(fdt));
+ printf("off_mem_rsvmap:\t\t0x%x\n", fdt_off_mem_rsvmap(fdt));
+ printf("version:\t\t%d\n", version);
+ printf("last_comp_version:\t%d\n", fdt_last_comp_version(fdt));
+ if (version >= 2)
+ printf("boot_cpuid_phys:\t0x%x\n",
+ fdt_boot_cpuid_phys(fdt));
+ if (version >= 3)
+ printf("size_dt_strings:\t0x%x\n",
+ fdt_size_dt_strings(fdt));
+ if (version >= 17)
+ printf("size_dt_struct:\t\t0x%x\n",
+ fdt_size_dt_struct(fdt));
+ printf("number mem_rsv:\t\t0x%x\n", fdt_num_mem_rsv(fdt));
+ printf("\n");
+
+ /
+* Set boot cpu id
+/
+ } else if ((argv[1][0] == 'b') && (argv[1][1] == 'o')) {
+ unsigned long tmp = simple_strtoul(argv[2], NULL, 16);
+ fdt_set_boot_cpuid_phys(fdt, tmp);
+
+ /
+* memory command
+/
+ } else if ((argv[1][0] == 'm') && (argv[1][1] == 'e')) {
+ uint64_t addr, size;
+ int err;
+#ifdef CFG_64BIT_STRTOUL
+ addr = simple_strtoull(argv[2], NULL, 16);
+ size = simple_strtoull(argv[3], NULL, 16);
+#else
+ addr = simple_strtoul(argv[2], NULL, 16);
+ size = simple_strtoul(argv[3], NULL, 16);
+#endif
+ err = fdt_fixup_memory(fdt, addr, size);
+ if (err < 0)
+ return err;
+
+ /
+* mem reserve commands
+/
+ } else if ((argv[1][0] == 'r') && (argv[1][1] == 's')) {
+ if (argv[2][0] == 'p') {
+ uint64_t addr, size;
+ int total = fdt_num_mem_rsv(fdt);
+ int j, err;
+ printf("index\t\t start\t\tsize\n");
+ printf("---"
+ "-\n");
+ for (j = 0; j < total; j++) {
+ err = fdt_get_mem_rsv(fdt, j, &addr, &size);
+ if (err < 0) {
+ printf("libfdt fdt_get_mem_rsv():
%s\n",
+ fdt_strerror(err));
+ re
Re: [U-Boot-Users] [PATCH] Add sub-commands to fdt
On Feb 15, 2008, at 3:44 PM, Jerry Van Baren wrote: > Kumar Gala wrote: >> fdt header - Display header info >> fdt bootcpu - Set boot cpuid >> fdt memory - Add/Update memory node >> fdt memrsv print- Show current mem reserves >> fdt memrsv add - Add a mem reserve >> fdt memrsv delete- Delete a mem reserves >> Signed-off-by: Kumar Gala <[EMAIL PROTECTED]> > > Hi Kumar, > > I was thinking about this a bit more. How about if we change the > "memrsv" subcommand to "rsvmem" then we can abbreviate the commands > more: > fdt mo[ve] > fdt me[mory] > fdt rm<- would no longer be distinctive on 'r', NBD > fdt rs[vmem] > and save ourselves some typing. works for me. I'll make these changes. I don't plan on dealing with the header info you asked about (beyond reporting the number of memrsv). I've given you the framework, I'll leave it to you to figure out how to compute the values you want :) I also have some questions about how we can "dynamically" grow the device tree so we don't have to pad the .dtb's - k - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] Add sub-commands to fdt
Kumar Gala wrote: > fdt header - Display header info > fdt bootcpu - Set boot cpuid > fdt memory - Add/Update memory node > fdt memrsv print- Show current mem reserves > fdt memrsv add - Add a mem reserve > fdt memrsv delete- Delete a mem reserves > > Signed-off-by: Kumar Gala <[EMAIL PROTECTED]> Hi Kumar, I was thinking about this a bit more. How about if we change the "memrsv" subcommand to "rsvmem" then we can abbreviate the commands more: fdt mo[ve] fdt me[mory] fdt rm<- would no longer be distinctive on 'r', NBD fdt rs[vmem] and save ourselves some typing. What do you think? gvb - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] [PATCH] ppc: Allow boards to specify how much memory they can map
For historical reasons we limited the stack to 256M because some boards could only map that much via BATS. However newer boards are capable of mapping more memory (for example 85xx is capble of doing up to 2G). Signed-off-by: Kumar Gala <[EMAIL PROTECTED]> --- lib_ppc/board.c |7 ++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/lib_ppc/board.c b/lib_ppc/board.c index 45d1328..8c9f6e6 100644 --- a/lib_ppc/board.c +++ b/lib_ppc/board.c @@ -419,12 +419,17 @@ void board_init_f (ulong bootflag) */ len = (ulong)&_end - CFG_MONITOR_BASE; +#ifndef CONFIG_MAX_MEM_MAPPED +#define CONFIG_MAX_MEM_MAPPED (256 << 20) +#endif + #ifndefCONFIG_VERY_BIG_RAM addr = CFG_SDRAM_BASE + gd->ram_size; #else /* only allow stack below 256M */ addr = CFG_SDRAM_BASE + - (gd->ram_size > 256 << 20) ? 256 << 20 : gd->ram_size; + (gd->ram_size > CONFIG_MAX_MEM_MAPPED) ? + CONFIG_MAX_MEM_MAPPED : gd->ram_size; #endif #ifdef CONFIG_LOGBUFFER -- 1.5.3.8 - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] Fix return value of mtest when CFG_ALT_MEMTEST set
On Fri, 15 Feb 2008, Jon Loeliger wrote: > Guennadi Liakhovetski wrote: > > On Thu, 14 Feb 2008, Wolfgang Denk wrote: > > > > > In message <[EMAIL PROTECTED]> you wrote: > > > > Fix a missing return statement from a non-void function. > > > > > > > > Signed-off-by: Guennadi Liakhovetski <[EMAIL PROTECTED]> > > > Applied, thanks. > > > > > > Ummm... I had to apply this manually: > > > > > > error: patch failed: common/cmd_mem.c:695 > > > error: common/cmd_mem.c: patch does not apply > > > fatal: sha1 information is lacking or useless (common/cmd_mem.c). > > > Repository lacks necessary blobs to fall back on 3-way merge. > > > Cannot fall back to three-way merge. > > > Patch failed at 0001. > > > > > > How old is your source tree? > > > > I produced the patch against 1.2.0, but before that I've verified, that the > > file, or at least the affected function hasn't changed, so, thought it would > > be ok. Sorry. But what does the error message actually mean? Is it just > > because I referenced some "way too old" commit? > > > > No. "Way too old" is not the issue with the error message. > > First off, the patch didn't apply directly to the file. > That's our key that the patch isn't up-to-date with the > current tree, as that file has changed significantly enough > that your changes are no longer applicable. Ok, I think, I know the reason. Wolfgang, can it be, that you first tried to apply this patch, and only then my other patch [PATCH v2] Fix wrong memory limit calculation in memory-test which I sent 5 days earlier? Then there would be a conflict yes. Otherwise this specific function hasn't change since then. Probably, I should have specified a dependency on my previous patch, sorry. Thanks Guennadi --- Guennadi Liakhovetski - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] Delete all env vars except read onlys
On 17:40 Fri 15 Feb , Wolfgang Denk wrote: > Uff > > After a longish discussion with Detlev we came up with a new > proposal, which hopefully is acceptable to all. We agreed that a > command name like "clearenv" or "scrubenv" or similar (which doesn't > require any arguments) might be too dangerous for the unexperienced > user why randomly types commands "just to see what happens". For > example, even if you type "erase" it will not do any damage because > arguments are missing - but "erase all" will blow away most of your > flash content without asking. > > So the idea is to call the new command "environment" (or short "env"); > usage would be then: > > => env clear > > to clear the envrionment (except read-only variables); thsi also > allows for pretty useful additional functionality like this: > > => env default > > to reset the environment to the (compiled in) default settings. > > One might even consider something like > > => env clear all > > to blow away the whole env, including the read-only variables. > > > Would that be acceptable to everybody? > > Ack-By : Jean-Christophe PLAGNIOL-VILLARD <[EMAIL PROTECTED]> Best Regards, J. - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] Fix return value of mtest when CFG_ALT_MEMTEST set
Guennadi Liakhovetski wrote: > On Thu, 14 Feb 2008, Wolfgang Denk wrote: > >> In message <[EMAIL PROTECTED]> you wrote: >>> Fix a missing return statement from a non-void function. >>> >>> Signed-off-by: Guennadi Liakhovetski <[EMAIL PROTECTED]> >> Applied, thanks. >> >> Ummm... I had to apply this manually: >> >> error: patch failed: common/cmd_mem.c:695 >> error: common/cmd_mem.c: patch does not apply >> fatal: sha1 information is lacking or useless (common/cmd_mem.c). >> Repository lacks necessary blobs to fall back on 3-way merge. >> Cannot fall back to three-way merge. >> Patch failed at 0001. >> >> How old is your source tree? > > I produced the patch against 1.2.0, but before that I've verified, that > the file, or at least the affected function hasn't changed, so, thought it > would be ok. Sorry. But what does the error message actually mean? Is it > just because I referenced some "way too old" commit? > No. "Way too old" is not the issue with the error message. First off, the patch didn't apply directly to the file. That's our key that the patch isn't up-to-date with the current tree, as that file has changed significantly enough that your changes are no longer applicable. In that situation, git tries to do a clever trick by backing off to a common ancestor where the patch was originally created. That is the first SHA1 in the diff header, assuming the patch was generated by git. If it can find this commit, git knows that the patch will apply at that point as that is what your presumably started with for your patch creation. Git will apply the patch there, and try to follow the changes forward in an attempt to bring the changes up to date itself. However, in this case, the SHA1 was only in your repository, and not a common commit that was also in the public repository. Thus, git couldn't fall back on the three-way merge trick, and ultimately was not able to apply your patch. The remedy is to rebase your patch to a current repository and resubmit it! :-) Chance are it will require some conflict resolution. jdl - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] Fix return value of mtest when CFG_ALT_MEMTEST set
On Thu, 14 Feb 2008, Wolfgang Denk wrote: > In message <[EMAIL PROTECTED]> you wrote: > > Fix a missing return statement from a non-void function. > > > > Signed-off-by: Guennadi Liakhovetski <[EMAIL PROTECTED]> > > Applied, thanks. > > Ummm... I had to apply this manually: > > error: patch failed: common/cmd_mem.c:695 > error: common/cmd_mem.c: patch does not apply > fatal: sha1 information is lacking or useless (common/cmd_mem.c). > Repository lacks necessary blobs to fall back on 3-way merge. > Cannot fall back to three-way merge. > Patch failed at 0001. > > How old is your source tree? I produced the patch against 1.2.0, but before that I've verified, that the file, or at least the affected function hasn't changed, so, thought it would be ok. Sorry. But what does the error message actually mean? Is it just because I referenced some "way too old" commit? Thanks Guennadi --- Guennadi Liakhovetski - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] [PATCH] Wipe out assembler warnings while compiling x86 biosemu
This patch tries to get rid of some assembler warnings about
changed .got2 section type while compiling x86 bios emulator
code.
Signed-off-by: Anatolij Gustschin <[EMAIL PROTECTED]>
---
drivers/bios_emulator/biosemu.c|4 ++--
drivers/bios_emulator/include/x86emu.h | 10 ++
drivers/bios_emulator/x86emu/ops.c | 14 +++---
drivers/bios_emulator/x86emu/ops2.c|2 +-
4 files changed, 20 insertions(+), 10 deletions(-)
diff --git a/drivers/bios_emulator/biosemu.c b/drivers/bios_emulator/biosemu.c
index 75ceb45..decdb79 100644
--- a/drivers/bios_emulator/biosemu.c
+++ b/drivers/bios_emulator/biosemu.c
@@ -53,7 +53,7 @@
#include "biosemui.h"
BE_sysEnv _BE_env = {{0}};
-static X86EMU_memFuncs _BE_mem __attribute__((section(".got2"))) = {
+static X86EMU_memFuncs _BE_mem __attribute__((section(GOT2_TYPE))) = {
BE_rdb,
BE_rdw,
BE_rdl,
@@ -62,7 +62,7 @@ static X86EMU_memFuncs _BE_mem
__attribute__((section(".got2"))) = {
BE_wrl,
};
-static X86EMU_pioFuncs _BE_pio __attribute__((section(".got2"))) = {
+static X86EMU_pioFuncs _BE_pio __attribute__((section(GOT2_TYPE))) = {
BE_inb,
BE_inw,
BE_inl,
diff --git a/drivers/bios_emulator/include/x86emu.h
b/drivers/bios_emulator/include/x86emu.h
index 6004beb..a70a768 100644
--- a/drivers/bios_emulator/include/x86emu.h
+++ b/drivers/bios_emulator/include/x86emu.h
@@ -53,6 +53,16 @@ typedef u16 X86EMU_pioAddr;
/*-- Macros and type definitions --*/
+#if defined (CONFIG_ARM)
+#define GAS_LINE_COMMENT "@"
+#elif defined(CONFIG_MIPS) || defined(CONFIG_PPC)
+#define GAS_LINE_COMMENT "#"
+#elif defined (CONFIG_SH)
+#define GAS_LINE_COMMENT "!"
+#endif
+
+#define GOT2_TYPE ".got2,\"aw\"\t"GAS_LINE_COMMENT
+
#pragma pack(1)
/
diff --git a/drivers/bios_emulator/x86emu/ops.c
b/drivers/bios_emulator/x86emu/ops.c
index a77bd9b..10f2757 100644
--- a/drivers/bios_emulator/x86emu/ops.c
+++ b/drivers/bios_emulator/x86emu/ops.c
@@ -91,7 +91,7 @@ static char *x86emu_GenOpName[8] = {
#endif
/* used by several opcodes */
-static u8 (*genop_byte_operation[])(u8 d, u8 s) __attribute__
((section(".got2"))) =
+static u8 (*genop_byte_operation[])(u8 d, u8 s) __attribute__
((section(GOT2_TYPE))) =
{
add_byte, /* 00 */
or_byte, /* 01 */
@@ -103,7 +103,7 @@ static u8 (*genop_byte_operation[])(u8 d, u8 s)
__attribute__ ((section(".got2")
cmp_byte, /* 07 */
};
-static u16 (*genop_word_operation[])(u16 d, u16 s) __attribute__
((section(".got2"))) =
+static u16 (*genop_word_operation[])(u16 d, u16 s) __attribute__
((section(GOT2_TYPE))) =
{
add_word, /*00 */
or_word, /*01 */
@@ -115,7 +115,7 @@ static u16 (*genop_word_operation[])(u16 d, u16 s)
__attribute__ ((section(".got
cmp_word, /*07 */
};
-static u32 (*genop_long_operation[])(u32 d, u32 s) __attribute__
((section(".got2"))) =
+static u32 (*genop_long_operation[])(u32 d, u32 s) __attribute__
((section(GOT2_TYPE))) =
{
add_long, /*00 */
or_long, /*01 */
@@ -128,7 +128,7 @@ static u32 (*genop_long_operation[])(u32 d, u32 s)
__attribute__ ((section(".got
};
/* used by opcodes 80, c0, d0, and d2. */
-static u8(*opcD0_byte_operation[])(u8 d, u8 s) __attribute__
((section(".got2"))) =
+static u8(*opcD0_byte_operation[])(u8 d, u8 s) __attribute__
((section(GOT2_TYPE))) =
{
rol_byte,
ror_byte,
@@ -141,7 +141,7 @@ static u8(*opcD0_byte_operation[])(u8 d, u8 s)
__attribute__ ((section(".got2"))
};
/* used by opcodes c1, d1, and d3. */
-static u16(*opcD1_word_operation[])(u16 s, u8 d) __attribute__
((section(".got2"))) =
+static u16(*opcD1_word_operation[])(u16 s, u8 d) __attribute__
((section(GOT2_TYPE))) =
{
rol_word,
ror_word,
@@ -154,7 +154,7 @@ static u16(*opcD1_word_operation[])(u16 s, u8 d)
__attribute__ ((section(".got2"
};
/* used by opcodes c1, d1, and d3. */
-static u32 (*opcD1_long_operation[])(u32 s, u8 d) __attribute__
((section(".got2"))) =
+static u32 (*opcD1_long_operation[])(u32 s, u8 d) __attribute__
((section(GOT2_TYPE))) =
{
rol_long,
ror_long,
@@ -5147,7 +5147,7 @@ void x86emuOp_opcFF_word_RM(u8 X86EMU_UNUSED(op1))
/***
* Single byte operation code table:
**/
-void (*x86emu_optab[256])(u8) __attribute__ ((section(".got2"))) =
+void (*x86emu_optab[256])(u8) __attribute__ ((section(GOT2_TYPE))) =
{
/* 0x00 */ x86emuOp_genop_byte_RM_R,
/* 0x01 */ x86emuOp_genop_word_RM_R,
diff --git a/drivers/bios_emulator/x86emu/ops2.c
b/drivers/bios_emulator/x86emu/ops2.c
index d6a210c..d90d366 100644
--- a/drivers/bios_emulator/x86emu/ops2.c
+++ b/drivers/bios_e
Re: [U-Boot-Users] [PPC] PLEASE READ - was: [PATCH] Fix linker scripts: add NOLOAD atribute to .bss/.sbss sections
> -Original Message- > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] > On Behalf Of Wolfgang Denk > Sent: den 14 februari 2008 23:19 > To: Scott Wood > Cc: u-boot-users@lists.sourceforge.net; Stefan Roese > Subject: Re: [U-Boot-Users] [PPC] PLEASE READ - was: [PATCH] Fix linker > scripts: add NOLOAD atribute > to .bss/.sbss sections > > In message <[EMAIL PROTECTED]> you wrote: > > > > > Hm... R2 is documented to be the TOC pointer? > > > > The ABI document says, "Register r2 is reserved for system use and > > should not be changed by application code." > > OK, tried this, and seems to work. Well, let's throw it out and see > what else it breaks ;-) > > Patch posted - my own tests look good, so I will apply it ;-) Works here too, limited testing though Jocke - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] U-boot Bus Fault Error while accessing NOR flash
Nikhil Gautam wrote:
> Hi Everyone,
>
> I am try to use the u-boot to program the NOR flash on my proprietary
> board based on AMCC sequoia board. I can boot from 1Gb NAND flash ok
> but everytime I try access the last 4K of NOR flash, U-boot gives me
> following error and reboots:-
>
> {{{
> U-Boot$ md 0xF000
> f000:Bus Fault @ 0x, fixup 0x
> Machine check in kernel mode.
> Caused by (from msr): regs 0fe8fca8 Unknown values in msr
> NIP: XER: 2000 LR: 0FF6412C REGS: 0fe8fca8 TRAP: 0200 DAR:
> 041F
> MSR: EE: 0 PR: 0 FP: 0 ME: 0 IR/DR: 00
>
> GPR00: 0FF6412C 0FE8FD98 7FFF EF600300 0001 0001 C838AE89
> GPR08: 0FF5145C 01FCA055 03F940AB 0FE8FB50 041F 0FFACD00 0EED
> GPR16: 0FE8FDA0 0040 0FE8FDA0 0FE8FDA0
> GPR24: 0100 0010 0004 F000 0FE8FF40 0FFADB58 0FE8FDA0
> Call backtrace:
> machine check
> }}}
>
> I get the same error if I press "ENTER" after u-boot gives me the
> prompt. If I type some command its ok and then its fine thereafter.
>
> I know the NOR flash is ok because I used BDI to program it and also I
> can access this address when I boot from NOR flash.
>
> Any Clue?
>
> Thanks In Advance!
>
> Nikhil Gautam
Hi Nikhil,
If I understand you correctly, this sounds like improperly initialized
variables in your build. This is very likely a (build | link |
initialization | relocation) issue (pick any combination).
The "md 0xF000" command is using the default value for the number of
objects, which comes from a static variable dp_last_length that is
initialized to 0x40 (see common/cmd_mem.c):
uintdp_last_length = 0x40;
If dp_last_length is not initialized properly to 0x40 on start up,
unexpected things are going to happen. I don't see an obvious path to
an illegal access to location 0x, but it could be.
In addition, if you hit with no command the first time your
board boots, the command parser tries to run the last command you
executed. Of course, there *is no* "last command", which ties into my
theory of uninitialized static/global variables.
I would suspect your link control isn't linking the static variable
initialization in the right section or your (compiler's) initialization
of the static variables isn't working as expected.
HTH,
gvb
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] kernel bss size from a uImage?
Does anyone have any bright ideas on how to know what the BSS size of the kernel will be from a uImage. It seems we just hope and pray that u-boot doesn't load things (ramdisk, bd_t, device tree, etc). Clearly the new uImage work should encompass this information, but I'm looking for ideas on how to possibly handle this with the existing format. - k - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] U-boot Bus Fault Error while accessing NOR flash
Hi Everyone,
I am try to use the u-boot to program the NOR flash on my proprietary
board based on AMCC sequoia board. I can boot from 1Gb NAND flash ok
but everytime I try access the last 4K of NOR flash, U-boot gives me
following error and reboots:-
{{{
U-Boot$ md 0xF000
f000:Bus Fault @ 0x, fixup 0x
Machine check in kernel mode.
Caused by (from msr): regs 0fe8fca8 Unknown values in msr
NIP: XER: 2000 LR: 0FF6412C REGS: 0fe8fca8 TRAP: 0200 DAR:
041F
MSR: EE: 0 PR: 0 FP: 0 ME: 0 IR/DR: 00
GPR00: 0FF6412C 0FE8FD98 7FFF EF600300 0001 0001 C838AE89
GPR08: 0FF5145C 01FCA055 03F940AB 0FE8FB50 041F 0FFACD00 0EED
GPR16: 0FE8FDA0 0040 0FE8FDA0 0FE8FDA0
GPR24: 0100 0010 0004 F000 0FE8FF40 0FFADB58 0FE8FDA0
Call backtrace:
machine check
}}}
I get the same error if I press "ENTER" after u-boot gives me the
prompt. If I type some command its ok and then its fine thereafter.
I know the NOR flash is ok because I used BDI to program it and also I
can access this address when I boot from NOR flash.
Any Clue?
Thanks In Advance!
Nikhil Gautam
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] problems with CRC in fw_printenv and fw_setenv utility and use
techie_vasu <[EMAIL PROTECTED]> writes: > Can you please tell me how to build the fw_setenv and fw_printenv > using Please make sure to read tools/env/README from *top of tree* U-Boot sources. > the mtd-user.h file? I am new to Linux. Please help. I don't see mtd/ in > proc/ directory. What kernel version? Looks like you have not compiled in mtd support. Best regards Markus Klotzbuecher -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] Delete all env vars except read onlys
Wolfgang Denk <[EMAIL PROTECTED]> writes: > After a longish discussion with Detlev we came up with a new > proposal, which hopefully is acceptable to all. We agreed that a > command name like "clearenv" or "scrubenv" or similar (which doesn't > require any arguments) might be too dangerous for the unexperienced > user why randomly types commands "just to see what happens". For > example, even if you type "erase" it will not do any damage because > arguments are missing - but "erase all" will blow away most of your > flash content without asking. [...] > Would that be acceptable to everybody? Sounds good! Best regards Markus Klotzbücher -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] Delete all env vars except read onlys
Uff After a longish discussion with Detlev we came up with a new proposal, which hopefully is acceptable to all. We agreed that a command name like "clearenv" or "scrubenv" or similar (which doesn't require any arguments) might be too dangerous for the unexperienced user why randomly types commands "just to see what happens". For example, even if you type "erase" it will not do any damage because arguments are missing - but "erase all" will blow away most of your flash content without asking. So the idea is to call the new command "environment" (or short "env"); usage would be then: => env clear to clear the envrionment (except read-only variables); thsi also allows for pretty useful additional functionality like this: => env default to reset the environment to the (compiled in) default settings. One might even consider something like => env clear all to blow away the whole env, including the read-only variables. Would that be acceptable to everybody? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] "There is such a fine line between genius and stupidity." - David St. Hubbins, "Spinal Tap" - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] problems with CRC in fw_printenv and fw_setenv utility and use
Hello, Can you please tell me how to build the fw_setenv and fw_printenv using the mtd-user.h file? I am new to Linux. Please help. I don't see mtd/ in proc/ directory. Thanks u-boot user wrote: > > Hello all, > > I have built the binary fw_printenv by using mtd-user.h from mtd-utils on > the internet as it fw_printenv won't compile it without using that file. > So I have used mtd-user.h and mtd-abi.h. The contents of my configuration > file are > # MTD device name Device offset Env. size Flash sector size > /dev/mtd1 0x 0x4000 0x4000 > > When I run fw_printenv it complained with error Warning : Bad CRC using > default environment in env_init function in fw_env.c source code. To gain > a better understanding I print the value crc1 calculated using > crc32(0,environment.data,ENV_SIZE). The value it prints is CRC > computed:FE641197 > If I print the environment.crc value it is D0CA8A0E. As the checksum > values are different crc1_ok is not set and I get the error. Any clue as > to why these values would be different? > > When I use the fw_setenv utility to set an environment variable it gives > the following errors > Writing CRC value to flash: D0C78567 > Unlocking flash... > Other dev values are: 0 > Current dev value: 0 > Done > Erasing old environment... > MTD erase error on /dev/mtd1: Invalid argument > Error: can't write fw_env to flash > > I want to be able to set bootvars and print bootvars from user space Linux > code. Linux has support for mtd-utils we tested the mtd-utils interface. > > I look forward to your replies. I appreciate your help. > > Thanks and Regards > -- View this message in context: http://www.nabble.com/problems-with-CRC-in-fw_printenv-and-fw_setenv-utility-and-use-tp14767876p15502930.html Sent from the Uboot - Users mailing list archive at Nabble.com. - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] Sandeep added you as a friend on WAYN
Hi u-boot-users <@@MEMBER_NAME> added you as a friend on WAYN (Where Are You Now?). To confirm that you are friends and join the worlds' largest travel and lifestyle community, click on the following link: http://www.wayn.com/waynfx.html?wci=link&id=1445&mks=<@@MEMBERS_KEYS>&mx=<@@MX>&cx=<@@CX>&cx_token=<@@CX_TOKEN> WAYN has changed for the better and is now free to chat, share, search and join. You also have 3 unread messages Regards, WAYN This invitation was sent to <@@CONTACT_EMAIL> on behalf of <@@MEMBER_NAME> If you do not wish to receive invitations from <@@MEMBER_NAME>, click here (http://www.wayn.com/waynfx.html?wci=link&id=1446&mkeys=<@@MEMBERS_KEYS>&hash=<@@BLOCK_HASH>) If you do not wish to receive invitations from any members, click here (http://www.wayn.com/waynfx.html?wci=link&id=1447&code=<@@CONTACT_EMAIL_HASH>&cm=<@@CAMPAIGN_KEY>) - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] Delete all env vars except read onlys
In message <[EMAIL PROTECTED]> you wrote: > > As I hinted at in my other mail, git is my prime example of living up to > the spirit of unix - and still many commands there, e.g. git-push need a > '-f' to prevent clobbering costly changes inadvertently. The important point is the "inadvertently". It is OK if you need an overwrite flag for commands that in the normal course of action are non-destructive, but in special situations are. However, commands which are designed to cause "destructive" actions don't ask: * rm does not ask by default. Not even for "rm -r *" * mkfs does not ask normally - it will happily kill the whole partition's content * cp or mv will happily overwrite target files (assuming sufficient permissions) etc. The "clearenv" command has a well-defined purpose: clear the environment. Similar like "erase" will clear the flash. Or "cp" or "mw" will overwrite even vital parts of the RAM. No questions being asked. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] Bus error -- please leave by the rear door. - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] Enabling UART0 and UART1
Hi All, I have enabled both UART0 and UART 1 using the CONFIG_SERIAL_MULTI in my board specific header file. The board is based on PPC 440GX processor. I used the serial1_puts and serial1_getc to access the UART1 but without access. Does anybody have used this functions before? Thanks, -- Sean - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] Add 'imload' command
Hi Grant, > Ugh, please don't continue down that path. Dataflash is a serial > flash technology, but the driver pretends that it is memory mapped. > It is not a good abstraction that I really think needs to be removed. > I don't think it is a good idea to add that mis-feature into new > commands. Seconded. That abstraction always irritated me a lot and made the code only harder to read. Cheers Detlev -- Markov does it in chains. -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] problems with CRC in fw_printenv and fw_setenv utility and use
Hello, Can you please tell me how to build the fw_setenv and fw_printenv using the mtd-user.h file? I am new to Linux. Please help. I don't see mtd/ in proc/ directory. Thanks u-boot user wrote: > > Hello all, > > I have built the binary fw_printenv by using mtd-user.h from mtd-utils on > the internet as it fw_printenv won't compile it without using that file. > So I have used mtd-user.h and mtd-abi.h. The contents of my configuration > file are > # MTD device name Device offset Env. size Flash sector size > /dev/mtd1 0x 0x4000 0x4000 > > When I run fw_printenv it complained with error Warning : Bad CRC using > default environment in env_init function in fw_env.c source code. To gain > a better understanding I print the value crc1 calculated using > crc32(0,environment.data,ENV_SIZE). The value it prints is CRC > computed:FE641197 > If I print the environment.crc value it is D0CA8A0E. As the checksum > values are different crc1_ok is not set and I get the error. Any clue as > to why these values would be different? > > When I use the fw_setenv utility to set an environment variable it gives > the following errors > Writing CRC value to flash: D0C78567 > Unlocking flash... > Other dev values are: 0 > Current dev value: 0 > Done > Erasing old environment... > MTD erase error on /dev/mtd1: Invalid argument > Error: can't write fw_env to flash > > I want to be able to set bootvars and print bootvars from user space Linux > code. Linux has support for mtd-utils we tested the mtd-utils interface. > > I look forward to your replies. I appreciate your help. > > Thanks and Regards > -- View this message in context: http://www.nabble.com/problems-with-CRC-in-fw_printenv-and-fw_setenv-utility-and-use-tp14767876p15502924.html Sent from the Uboot - Users mailing list archive at Nabble.com. - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] Query about U-Boot NAND ENV section
In message <[EMAIL PROTECTED]> you wrote: > > I tried to dig into this issue, and found that, U-Boot for NAND Flash is > designed the same way. The function "nand_init" gets invoked after > "env_init" and "serial_init", and sue to this U-Boot always uses the > default baudrate. Thjis is a dilemma - U-Boot tries to support porting and debugging by enabling the serial console very, very early, typically long before relocation to RAM. In this state, we have only a verly limited environment - small stack, no writable data segment, no BSS, etc. This is not sufficient for the (pretty complicated) NAND flash driver. So U-Boot cannot read the baudrate setting from the environment in NAND... Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] Remember, an int is not always 16 bits. I'm not sure, but if the 80386 is one step closer to Intel's slugfest with the CPU curve that is aymptotically approaching a real machine, perhaps an int has been implemented as 32 bits by some Unix vendors...? - Derek Terveer - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] Delete all env vars except read onlys
Hi, > In message <[EMAIL PROTECTED]> you wrote: >> >> > Oops??? "rm" never asks unless you ask it to ask. >> >> Not true. It will ask if you don't have write permission to the file, >> even if you are able to delete because you have write permission to the >> directory. > > Does it? Indeed. Must be a GNUism. I bet this was't there in Unix v6 > when I learned it ;-) > > > You are right, but this is still a special case where another, > explicit protection is being overwritten. * mkfs.ext2 on a file, needs -F package managers with unfulfilled * dependencies always ask and need a "--force-xxx" to go ahead whatsoever. * bzip2, bunzip2, bzcat, gzip, gzcat, gunzip -f to overwrite files * git-checkout-index needs -f to overwrite files * git-push needs -f to push non-linear head * git-fetch needs -f in same situtation * rsync -force to force deletion of dirs even if not empty I'll stop here. It seems the unix commandline tilted to a behaviour where interactive commands tend to ask in dangerous situations but scripts can explicitely override it - which is definitely what I prefer. Cheers Detlev -- Woman who seek to be equal with men lack ambition -- Timothy Leary -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] new-image allocation mechanism
Guys, Look at the code in the branch I was wondering what your feeling was about changing things around a bit with regards to allocation mechanisms. My thinking is that when we call do_bootm_linux() what ever the end of the kernel is we should start allocating from there towards the end of memory rather than from the "stack" down towards 0. I think this would greatly simplify things for us. - k - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] Delete all env vars except read onlys
Hi Wolfgang, > In message <[EMAIL PROTECTED]> you wrote: >> >> I would still prefer the version with the '--force' parameter. Just my 2 >> cents. > > I really don't want to have this, for two reasons: 1) U-Boot doesn't > do "Do you really want to" confirmations, because these don't fit into > the Unix design philisophy; 2) U-Boot doesn't do GNU style long > command line switches (for several reasons, memory footprint being one > of them). As I hinted at in my other mail, git is my prime example of living up to the spirit of unix - and still many commands there, e.g. git-push need a '-f' to prevent clobbering costly changes inadvertently. In my opinion your argument 1) is not true as it stands. Unix wants its users to be as productive as possible and typing two extra characters when ommitting them can cause hours of work is definitely in this spirit. Exaggerating your argument I would ask why you do not work only as root on your machine. In U-Boot we do not have this "user" level of protection so IMHO it makes sense to have at least a bit of it. It is true however that we don't have any kind of option processing, so probably '-force' wasn't a good choice to begin with. What about only using "all" as a "conformative" option? Cheers Detlev -- Perfection (in design) is achieved not when there is nothing more to add, but rather when there is nothing more to take away. -- Antoine de Saint-Exupery -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] [PATCH 3/3] [new uImage] ppc: Re-order ramdisk/fdt handling sequence
Do the fdt before the ramdisk allows us to grow the fdt w/o concern
however it does me we have to go in and fixup the initrd info since
we dont know where it will be.
Signed-off-by: Kumar Gala <[EMAIL PROTECTED]>
---
Patch against u-boot-testing new-image branch.
lib_ppc/bootm.c | 38 ++
1 files changed, 34 insertions(+), 4 deletions(-)
diff --git a/lib_ppc/bootm.c b/lib_ppc/bootm.c
index 5158ccc..927842c 100644
--- a/lib_ppc/bootm.c
+++ b/lib_ppc/bootm.c
@@ -121,9 +121,6 @@ do_bootm_linux(cmd_tbl_t *cmdtp, int flag,
rd_len = rd_data_end - rd_data_start;
- alloc_current = ramdisk_high (alloc_current, rd_data_start, rd_len,
- sp_limit, get_sp (), &initrd_start, &initrd_end);
-
#if defined(CONFIG_OF_LIBFDT)
/* find flattened device tree */
alloc_current = get_fdt (alloc_current,
@@ -134,7 +131,8 @@ do_bootm_linux(cmd_tbl_t *cmdtp, int flag,
* if the user wants it (the logic is in the subroutines).
*/
if (of_flat_tree) {
- if (fdt_chosen(of_flat_tree, initrd_start, initrd_end, 0) < 0) {
+ /* pass in dummy initrd info, we'll fix up later */
+ if (fdt_chosen(of_flat_tree, rd_data_start, rd_data_end, 0) <
0) {
fdt_error ("/chosen node create failed");
do_reset (cmdtp, flag, argc, argv);
}
@@ -157,6 +155,38 @@ do_bootm_linux(cmd_tbl_t *cmdtp, int flag,
}
#endif /* CONFIG_OF_LIBFDT */
+ alloc_current = ramdisk_high (alloc_current, rd_data_start, rd_len,
+ sp_limit, get_sp (), &initrd_start, &initrd_end);
+
+#if defined(CONFIG_OF_LIBFDT)
+ /* fixup the initrd now that we know where it should be */
+ if ((of_flat_tree) && (initrd_start && initrd_end)) {
+ uint64_t addr, size;
+ int total = fdt_num_mem_rsv(of_flat_tree);
+ int j, err;
+
+ /* Look for the dummy entry and delete it */
+ for (j = 0; j < total; j++) {
+ err = fdt_get_mem_rsv(of_flat_tree, j, &addr, &size);
+ if (addr == rd_data_start) {
+ fdt_del_mem_rsv(of_flat_tree, j);
+ break;
+ }
+ }
+
+ err = fdt_add_mem_rsv(of_flat_tree, initrd_start,
+ initrd_end - initrd_start + 1);
+ if (err < 0) {
+ printf("fdt_chosen: %s\n", fdt_strerror(err));
+ return err;
+ }
+
+ do_fixup_by_path_u32(of_flat_tree, "/chosen",
+ "linux,initrd-start", initrd_start, 0);
+ do_fixup_by_path_u32(of_flat_tree, "/chosen",
+ "linux,initrd-end", initrd_end, 0);
+ }
+#endif
debug ("## Transferring control to Linux (at address %08lx) ...\n",
(ulong)kernel);
--
1.5.3.8
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] [PATCH 2/3] [new uImage] ppc: Determine if we are booting an OF style
If we are bootin OF style than we can skip setting up some things
that are used for the old boot method.
Signed-off-by: Kumar Gala <[EMAIL PROTECTED]>
---
patch against u-boot-testing new-image branch
lib_ppc/bootm.c | 25 -
1 files changed, 20 insertions(+), 5 deletions(-)
diff --git a/lib_ppc/bootm.c b/lib_ppc/bootm.c
index 1f1be69..5158ccc 100644
--- a/lib_ppc/bootm.c
+++ b/lib_ppc/bootm.c
@@ -71,8 +71,21 @@ do_bootm_linux(cmd_tbl_t *cmdtp, int flag,
bd_t*kbd;
void(*kernel)(bd_t *, ulong, ulong, ulong, ulong);
+ int has_of = 0;
+
#if defined(CONFIG_OF_LIBFDT)
char*of_flat_tree;
+
+ /* determine if we are booting w/of */
+ if (argc > 3)
+ has_of = 1;
+ if (image_check_type (hdr, IH_TYPE_MULTI)) {
+ ulong fdt_data, fdt_len;
+ image_multi_getimg (hdr, 2, &fdt_data, &fdt_len);
+
+ if (fdt_len)
+ has_of = 1;
+ }
#endif
/*
@@ -90,12 +103,14 @@ do_bootm_linux(cmd_tbl_t *cmdtp, int flag,
alloc_current = sp_limit = get_boot_sp_limit(sp);
debug ("=> set upper limit to 0x%08lx\n", sp_limit);
- /* allocate space and init command line */
- alloc_current = get_boot_cmdline (alloc_current, &cmd_start, &cmd_end);
+ if (!has_of) {
+ /* allocate space and init command line */
+ alloc_current = get_boot_cmdline (alloc_current, &cmd_start,
&cmd_end);
- /* allocate space for kernel copy of board info */
- alloc_current = get_boot_kbd (alloc_current, &kbd);
- set_clocks_in_mhz(kbd);
+ /* allocate space for kernel copy of board info */
+ alloc_current = get_boot_kbd (alloc_current, &kbd);
+ set_clocks_in_mhz(kbd);
+ }
/* find kernel */
kernel = (void (*)(bd_t *, ulong, ulong, ulong, ulong))image_get_ep
(hdr);
--
1.5.3.8
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] [PATCH 1/3] [new uImage] Don't pass kdb to ramdisk_high since we may not have one
We don't actually need the kdb param as we are just using it to get
bd->bi_memsize which we can get from gd->bd->bi_memsize. Also, if we
boot via OF we might not actually fill out a kdb.
Signed-off-by: Kumar Gala <[EMAIL PROTECTED]>
---
Patch against new-image u-boot-testing branch
common/image.c |9 -
include/image.h |2 +-
lib_m68k/bootm.c |3 +--
lib_ppc/bootm.c |3 +--
4 files changed, 7 insertions(+), 10 deletions(-)
diff --git a/common/image.c b/common/image.c
index 39e5f23..46cecef 100644
--- a/common/image.c
+++ b/common/image.c
@@ -495,7 +495,6 @@ void get_ramdisk (cmd_tbl_t *cmdtp, int flag, int argc,
char *argv[],
* ramdisk_high - relocate init ramdisk
* @rd_data: ramdisk data start address
* @rd_len: ramdisk data length
- * @kbd: kernel board info copy (within BOOTMAPSZ boundary)
* @sp_limit: stack pointer limit (including BOOTMAPSZ)
* @sp: current stack pointer
* @initrd_start: pointer to a ulong variable, will hold final init ramdisk
@@ -513,7 +512,7 @@ void get_ramdisk (cmd_tbl_t *cmdtp, int flag, int argc,
char *argv[],
* - returns new allc_current, next free address below BOOTMAPSZ
*/
ulong ramdisk_high (ulong alloc_current, ulong rd_data, ulong rd_len,
- bd_t *kbd, ulong sp_limit, ulong sp,
+ ulong sp_limit, ulong sp,
ulong *initrd_start, ulong *initrd_end)
{
char*s;
@@ -535,9 +534,9 @@ ulong ramdisk_high (ulong alloc_current, ulong rd_data,
ulong rd_len,
#ifdef CONFIG_LOGBUFFER
/* Prevent initrd from overwriting logbuffer */
- if (initrd_high < (kbd->bi_memsize - LOGBUFF_LEN - LOGBUFF_OVERHEAD))
- initrd_high = kbd->bi_memsize - LOGBUFF_LEN - LOGBUFF_OVERHEAD;
- debug ("## Logbuffer at 0x%08lx ", kbd->bi_memsize - LOGBUFF_LEN);
+ if (initrd_high < (gd->bd->bi_memsize - LOGBUFF_LEN - LOGBUFF_OVERHEAD))
+ initrd_high = gd->bd->bi_memsize - LOGBUFF_LEN - LOGBUFF_OVERHEAD;
+ debug ("## Logbuffer at 0x%08lx ", gd->bd->bi_memsize - LOGBUFF_LEN);
#endif
debug ("## initrd_high = 0x%08lx, copy_to_ram = %d\n",
initrd_high, initrd_copy_to_ram);
diff --git a/include/image.h b/include/image.h
index dbbbee9..a67489e 100644
--- a/include/image.h
+++ b/include/image.h
@@ -344,7 +344,7 @@ void get_ramdisk (cmd_tbl_t *cmdtp, int flag, int argc,
char *argv[],
#if defined(CONFIG_PPC) || defined(CONFIG_M68K)
ulong ramdisk_high (ulong alloc_current, ulong rd_data, ulong rd_len,
- bd_t *kbd, ulong sp_limit, ulong sp,
+ ulong sp_limit, ulong sp,
ulong *initrd_start, ulong *initrd_end);
ulong get_boot_sp_limit (ulong sp);
diff --git a/lib_m68k/bootm.c b/lib_m68k/bootm.c
index ac04da0..da76844 100644
--- a/lib_m68k/bootm.c
+++ b/lib_m68k/bootm.c
@@ -88,8 +88,7 @@ void do_bootm_linux(cmd_tbl_t * cmdtp, int flag,
rd_len = rd_data_end - rd_data_start;
alloc_current = ramdisk_high (alloc_current, rd_data_start, rd_len,
- kbd, sp_limit, get_sp (),
- &initrd_start, &initrd_end);
+ sp_limit, get_sp (), &initrd_start, &initrd_end);
debug("## Transferring control to Linux (at address %08lx) ...\n",
(ulong) kernel);
diff --git a/lib_ppc/bootm.c b/lib_ppc/bootm.c
index 69ec459..1f1be69 100644
--- a/lib_ppc/bootm.c
+++ b/lib_ppc/bootm.c
@@ -107,8 +107,7 @@ do_bootm_linux(cmd_tbl_t *cmdtp, int flag,
rd_len = rd_data_end - rd_data_start;
alloc_current = ramdisk_high (alloc_current, rd_data_start, rd_len,
- kbd, sp_limit, get_sp (),
- &initrd_start, &initrd_end);
+ sp_limit, get_sp (), &initrd_start, &initrd_end);
#if defined(CONFIG_OF_LIBFDT)
/* find flattened device tree */
--
1.5.3.8
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] Das U-Boot support for Atmel AT91SAM926x-EK (or other boards with an AT91SAM926x processor)?
Are there plans to add AT91SAM926x-EK board support to the main U-Boot repository or an appropriate custodian repository? If so, what is the anticipated time frame of this support? -- In the official U-Boot 1.3.1 release, I wasn't able to locate any boards with an AT91SAM926x processor. Are any supported? --- Comment --- I'm interested in EBI NOR booting of AT91SAM926x processors as opposed to SPI NOR (Dataflash) booting of these processors which is the usual case. If necessary, I'll use u-boot 1.2.0 as patched by Atmel to run on AT91SAM926x, but I'd prefer to use the official U-Boot code. Sincerely, Ken Fuchs - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] [PATCH v2] Add sub-commands to fdt
fdt header - Display header info
fdt bootcpu - Set boot cpuid
fdt memory - Add/Update memory node
fdt memrsv print- Show current mem reserves
fdt memrsv add - Add a mem reserve
fdt memrsv delete- Delete a mem reserves
Signed-off-by: Kumar Gala <[EMAIL PROTECTED]>
---
fix help white space
be more explicit in arg checking for "memory" vs "memrsv"
common/cmd_fdt.c | 112 ++
1 files changed, 112 insertions(+), 0 deletions(-)
diff --git a/common/cmd_fdt.c b/common/cmd_fdt.c
index 9cd22ee..fb5d663 100644
--- a/common/cmd_fdt.c
+++ b/common/cmd_fdt.c
@@ -296,6 +296,112 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char
*argv[])
return err;
}
}
+
+ /
+* Display header info
+/
+ } else if (argv[1][0] == 'h') {
+ u32 version = fdt_version(fdt);
+ printf("magic:\t\t\t0x%x\n", fdt_magic(fdt));
+ printf("totalsize:\t\t0x%x (%d)\n", fdt_totalsize(fdt),
fdt_totalsize(fdt));
+ printf("off_dt_struct:\t\t0x%x\n", fdt_off_dt_struct(fdt));
+ printf("off_dt_strings:\t\t0x%x\n", fdt_off_dt_strings(fdt));
+ printf("off_mem_rsvmap:\t\t0x%x\n", fdt_off_mem_rsvmap(fdt));
+ printf("version:\t\t%d\n", version);
+ printf("last_comp_version:\t%d\n", fdt_last_comp_version(fdt));
+ if (version >= 2)
+ printf("boot_cpuid_phys:\t0x%x\n",
+ fdt_boot_cpuid_phys(fdt));
+ if (version >= 3)
+ printf("size_dt_strings:\t0x%x\n",
+ fdt_size_dt_strings(fdt));
+ if (version >= 17)
+ printf("size_dt_struct:\t\t0x%x\n",
+ fdt_size_dt_struct(fdt));
+ printf("number mem_rsv:\t\t0x%x\n", fdt_num_mem_rsv(fdt));
+ printf("\n");
+
+ /
+* Set boot cpu id
+/
+ } else if ((argv[1][0] == 'b') && (argv[1][1] == 'o')) {
+ unsigned long tmp = simple_strtoul(argv[2], NULL, 16);
+ fdt_set_boot_cpuid_phys(fdt, tmp);
+
+ /
+* memory command
+/
+ } else if ((argv[1][0] == 'm') && (argv[1][1] == 'e') &&
+ (argv[1][2] == 'm') && (argv[1][3] == 'o')) {
+ uint64_t addr, size;
+ int err;
+#ifdef CFG_64BIT_STRTOUL
+ addr = simple_strtoull(argv[2], NULL, 16);
+ size = simple_strtoull(argv[3], NULL, 16);
+#else
+ addr = simple_strtoul(argv[2], NULL, 16);
+ size = simple_strtoul(argv[3], NULL, 16);
+#endif
+ err = fdt_fixup_memory(fdt, addr, size);
+ if (err < 0)
+ return err;
+
+ /
+* mem reserve commands
+/
+ } else if ((argv[1][0] == 'm') && (argv[1][1] == 'e') &&
+ (argv[1][2] == 'm') && (argv[1][3] == 'r')) {
+ if (argv[2][0] == 'p') {
+ uint64_t addr, size;
+ int total = fdt_num_mem_rsv(fdt);
+ int j, err;
+ printf("index\t\t start\t\tsize\n");
+ printf("---"
+ "-\n");
+ for (j = 0; j < total; j++) {
+ err = fdt_get_mem_rsv(fdt, j, &addr, &size);
+ if (err < 0) {
+ printf("libfdt fdt_get_mem_rsv():
%s\n",
+ fdt_strerror(err));
+ return err;
+ }
+ printf("%x\t%08x%08x\t%08x%08x\n", j,
+ (u32)(addr >> 32),
+ (u32)(addr & 0x),
+ (u32)(size >> 32),
+ (u32)(size & 0x));
+ }
+ } else if (argv[2][0] == 'a') {
+ uint64_t addr, size;
+
Re: [U-Boot-Users] [PATCH] Add sub-commands to fdt
On Feb 15, 2008, at 8:09 AM, Jerry Van Baren wrote:
> Kumar Gala wrote:
>> fdt header - Display header info
>> fdt bootcpu - Set boot cpuid
>> fdt memory - Add/Update memory node
>> fdt memrsv print- Show current mem reserves
>> fdt memrsv add - Add a mem reserve
>> fdt memrsv delete- Delete a mem reserves
>> Signed-off-by: Kumar Gala <[EMAIL PROTECTED]>
>
> Cool, procrastination pays off again! :-)
>
>> ---
>> common/cmd_fdt.c | 111
>> ++
>> 1 files changed, 111 insertions(+), 0 deletions(-)
>> diff --git a/common/cmd_fdt.c b/common/cmd_fdt.c
>> index 9cd22ee..d0f8c27 100644
>> --- a/common/cmd_fdt.c
>> +++ b/common/cmd_fdt.c
>> @@ -296,6 +296,111 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int
>> argc, char *argv[])
>> return err;
>> }
>> }
>> +
>> +/
>>
>> + * Display header info
>> +
>> /
>> +} else if (argv[1][0] == 'h') {
>> +u32 version = fdt_version(fdt);
>> +printf("magic:\t\t\t0x%x\n", fdt_magic(fdt));
>> +printf("totalsize:\t\t0x%x (%d)\n", fdt_totalsize(fdt),
>> fdt_totalsize(fdt));
>> +printf("off_dt_struct:\t\t0x%x\n", fdt_off_dt_struct(fdt));
>> +printf("off_dt_strings:\t\t0x%x\n", fdt_off_dt_strings(fdt));
>> +printf("off_mem_rsvmap:\t\t0x%x\n", fdt_off_mem_rsvmap(fdt));
>> +printf("version:\t\t%d\n", version);
>> +printf("last_comp_version:\t%d\n", fdt_last_comp_version(fdt));
>> +if (version >= 2)
>> +printf("boot_cpuid_phys:\t0x%x\n",
>> +fdt_boot_cpuid_phys(fdt));
>> +if (version >= 3)
>> +printf("size_dt_strings:\t0x%x\n",
>> +fdt_size_dt_strings(fdt));
>> +if (version >= 17)
>> +printf("size_dt_struct:\t\t0x%x\n",
>> +fdt_size_dt_struct(fdt));
>> +printf("\n");
>
> Very nice.
> * I think we should add the size of the reserved map.
done
> * It would be very useful to know how much space is available in the
> reserved map
not sure how we do this.
> * It would be very useful to know how much space is available in the
> dt_struct and dt_space space (I believe the "spare" in the blob can
> be used for both the dt_struct and the dt_strings, but I'm not sure,
> don't have time to look it up right now).
How about I leave that as an exercise for the reader :)
>> +/
>>
>> + * Set boot cpu id
>> +
>> /
>> +} else if ((argv[1][0] == 'b') && (argv[1][1] == 'o')) {
>> +unsigned long tmp = simple_strtoul(argv[2], NULL, 16);
>> +fdt_set_boot_cpuid_phys(fdt, tmp);
>> +
>> +/
>>
>> + * memory command
>> +
>> /
>> +} else if ((argv[1][0] == 'm') && (argv[1][1] == 'e') &&
>> + (argv[1][3] == 'o')) {
>
> Did we miss (argv[1][2] == 'm'), is the term checking (argv[1][3] ==
> 'o') wrong, or are we trying to accommodate dyslecix people? I can
> picture our fearless leader typing "fdt meromy", assuming he types
> that many characters. :-D
I was trying to be slightly smart, I can add the check for argv[1][2]
== 'm'
>
>
>> +uint64_t addr, size;
>> +int err;
>> +#ifdef CFG_64BIT_STRTOUL
>> +addr = simple_strtoull(argv[2], NULL, 16);
>> +size = simple_strtoull(argv[3], NULL, 16);
>> +#else
>> +addr = simple_strtoul(argv[2], NULL, 16);
>> +size = simple_strtoul(argv[3], NULL, 16);
>> +#endif
>> +err = fdt_fixup_memory(fdt, addr, size);
>> +if (err < 0)
>> +return err;
>> +
>> +/
>>
>> + * mem reserve commands
>> +
>> /
>> +} else if ((argv[1][0] == 'm') && (argv[1][1] == 'e') &&
>> + (argv[1][3] == 'r')) {
>
> Hmm, missing (argv[1][2] == 'm') again. Once is an accident, twice
> is by design. I take it you are skipping the argv[1][2] check since
> it isn't a significant character - code savings. I can accept that,
> although it really hurts my humor to have a logical explanation (pun
> intended ;-).
ditto.
>
>
>> +if (argv[2][0] == 'p') {
>> +
Re: [U-Boot-Users] [PATCH] Add sub-commands to fdt
Kumar Gala wrote:
> fdt header - Display header info
> fdt bootcpu - Set boot cpuid
> fdt memory - Add/Update memory node
> fdt memrsv print- Show current mem reserves
> fdt memrsv add - Add a mem reserve
> fdt memrsv delete- Delete a mem reserves
>
> Signed-off-by: Kumar Gala <[EMAIL PROTECTED]>
Cool, procrastination pays off again! :-)
> ---
> common/cmd_fdt.c | 111
> ++
> 1 files changed, 111 insertions(+), 0 deletions(-)
>
> diff --git a/common/cmd_fdt.c b/common/cmd_fdt.c
> index 9cd22ee..d0f8c27 100644
> --- a/common/cmd_fdt.c
> +++ b/common/cmd_fdt.c
> @@ -296,6 +296,111 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char
> *argv[])
> return err;
> }
> }
> +
> + /
> + * Display header info
> + /
> + } else if (argv[1][0] == 'h') {
> + u32 version = fdt_version(fdt);
> + printf("magic:\t\t\t0x%x\n", fdt_magic(fdt));
> + printf("totalsize:\t\t0x%x (%d)\n", fdt_totalsize(fdt),
> fdt_totalsize(fdt));
> + printf("off_dt_struct:\t\t0x%x\n", fdt_off_dt_struct(fdt));
> + printf("off_dt_strings:\t\t0x%x\n", fdt_off_dt_strings(fdt));
> + printf("off_mem_rsvmap:\t\t0x%x\n", fdt_off_mem_rsvmap(fdt));
> + printf("version:\t\t%d\n", version);
> + printf("last_comp_version:\t%d\n", fdt_last_comp_version(fdt));
> + if (version >= 2)
> + printf("boot_cpuid_phys:\t0x%x\n",
> + fdt_boot_cpuid_phys(fdt));
> + if (version >= 3)
> + printf("size_dt_strings:\t0x%x\n",
> + fdt_size_dt_strings(fdt));
> + if (version >= 17)
> + printf("size_dt_struct:\t\t0x%x\n",
> + fdt_size_dt_struct(fdt));
> + printf("\n");
Very nice.
* I think we should add the size of the reserved map.
* It would be very useful to know how much space is available in the
reserved map
* It would be very useful to know how much space is available in the
dt_struct and dt_space space (I believe the "spare" in the blob can be
used for both the dt_struct and the dt_strings, but I'm not sure, don't
have time to look it up right now).
> + /
> + * Set boot cpu id
> + /
> + } else if ((argv[1][0] == 'b') && (argv[1][1] == 'o')) {
> + unsigned long tmp = simple_strtoul(argv[2], NULL, 16);
> + fdt_set_boot_cpuid_phys(fdt, tmp);
> +
> + /
> + * memory command
> + /
> + } else if ((argv[1][0] == 'm') && (argv[1][1] == 'e') &&
> +(argv[1][3] == 'o')) {
Did we miss (argv[1][2] == 'm'), is the term checking (argv[1][3] ==
'o') wrong, or are we trying to accommodate dyslecix people? I can
picture our fearless leader typing "fdt meromy", assuming he types that
many characters. :-D
> + uint64_t addr, size;
> + int err;
> +#ifdef CFG_64BIT_STRTOUL
> + addr = simple_strtoull(argv[2], NULL, 16);
> + size = simple_strtoull(argv[3], NULL, 16);
> +#else
> + addr = simple_strtoul(argv[2], NULL, 16);
> + size = simple_strtoul(argv[3], NULL, 16);
> +#endif
> + err = fdt_fixup_memory(fdt, addr, size);
> + if (err < 0)
> + return err;
> +
> + /
> + * mem reserve commands
> + /
> + } else if ((argv[1][0] == 'm') && (argv[1][1] == 'e') &&
> +(argv[1][3] == 'r')) {
Hmm, missing (argv[1][2] == 'm') again. Once is an accident, twice is
by design. I take it you are skipping the argv[1][2] check since it
isn't a significant character - code savings. I can accept that,
although it really hurts my humor to have a logical explanation (pun
intended ;-).
> + if (argv[2][0] == 'p') {
> + uint64_t addr, size;
> + int total = fdt_num_mem_rsv(fdt);
> + int j, err;
> + printf("index\t\t start\t\tsize\n");
> + printf("---"
> + "-\n");
> + for (j = 0; j < tota
[U-Boot-Users] Query about U-Boot NAND ENV section
Hi, I am using U-Boot 1.2.0 release of U-Boot and working on ARM1176 platform. I have ported U-Boot for my platform, which boots from NAND Flash, save ENV variable onto NAND Flash. Almost everything works fine for me till now. But today I came across one issue - When I set baudrate to different value from U-Boot prompt, and do a "saveenv", and after that if I reboot the system it should boot with new baudrate set. Ideally it should take new baudrate, but it always configures default baudrate. I tried to dig into this issue, and found that, U-Boot for NAND Flash is designed the same way. The function "nand_init" gets invoked after "env_init" and "serial_init", and sue to this U-Boot always uses the default baudrate. Do I understand it correctly? Is there any way to resolve this issue? Thanks, Vaibhav - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] New growth and girth guaranteed.
When SMALL is a dirty word... --- avast! Antivirus: Outbound message clean. Virus Database (VPS): 080214-0, 02/14/2008 Tested on: 2/15/2008 1:41:13 PM avast! - copyright (c) 1988-2008 ALWIL Software. http://www.avast.com - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] Pull request:[ARM]
Wolfgang
Please pull from the merge branch at git://denx.de/git/u-boot-arm.git master
---
The following changes since commit 45da195cf675d0b4054aeeabf7b9c0daf798841d:
Wolfgang Denk (1):
Merge branch 'master' of
git+ssh://10.10.0.7/home/wd/git/u-boot/master
are available in the git repository at:
git://www.denx.de/git/u-boot-arm.git master
Andreas Engel (1):
ARM: cleanup duplicated exception handlingcode
Haavard Skinnemoen (1):
Move AT91RM9200DK board support under board/atmel
Peter Pearse (6):
Update board NetStar
Merge branch '070524_netstar' of git://linux-arm.org/u-boot-armdev
Merge branch '080116_at91cap9' of git://linux-arm.org/u-boot-armdev
Merge branch '080131_artila' of git://linux-arm.org/u-boot-armdev
Merge branch '080202_at91rm9200dk' of
git://linux-arm.org/u-boot-armdev
Merge branch '080208_dupint' of git://linux-arm.org/u-boot-armdev
Stelian Pop (9):
Fix arm926ejs compile when SKIP_LOWLEVEL_INIT is on
Improve DataFlash CS definition.
AT91CAP9 support : build integration
AT91CAP9 support : include/ files
AT91CAP9 support : cpu/ files
AT91CAP9 support : board/ files
AT91CAP9 support : MACB changes
AT91CAP9 support : move board files to Atmel vendor directory.
AT91CAP9 support
Timo Tuunainen (1):
Support for Artila M-501 starter kit
CREDITS| 20 +
MAKEALL|2 +
Makefile | 18 +-
board/{at91rm9200dk => atmel/at91cap9adk}/Makefile |8 +-
board/atmel/at91cap9adk/at91cap9adk.c | 283 +++
board/atmel/at91cap9adk/config.mk |1 +
board/atmel/at91cap9adk/led.c | 80 +++
board/atmel/at91cap9adk/nand.c | 71 +++
.../{at91rm9200dk => atmel/at91cap9adk}/u-boot.lds | 10 +-
board/{ => atmel}/at91rm9200dk/Makefile|0
board/{ => atmel}/at91rm9200dk/at91rm9200dk.c |0
board/{ => atmel}/at91rm9200dk/config.mk |0
board/{ => atmel}/at91rm9200dk/flash.c |0
board/{ => atmel}/at91rm9200dk/led.c |0
board/{ => atmel}/at91rm9200dk/mux.c |0
board/{ => atmel}/at91rm9200dk/u-boot.lds |0
board/{at91rm9200dk => m501sk}/Makefile| 22 +-
board/{at91rm9200dk => m501sk}/config.mk |0
board/m501sk/eeprom.c | 102
board/m501sk/m501sk.c | 194
board/m501sk/m501sk.h | 167 +++
board/m501sk/memsetup.S| 200
board/{at91rm9200dk => m501sk}/u-boot.lds |8 +-
board/netstar/nand.c | 13 +-
common/env_dataflash.c | 17 +-
cpu/arm1136/interrupts.c | 134 -
cpu/arm720t/interrupts.c | 132 +-
cpu/arm920t/interrupts.c | 135 +-
cpu/arm925t/interrupts.c | 135 -
.../arm926ejs/at91cap9}/Makefile | 24 +-
cpu/arm926ejs/at91cap9/config.mk |2 +
.../u-boot.lds => cpu/arm926ejs/at91cap9/ether.c | 46 +--
.../arm926ejs/at91cap9/lowlevel_init.S | 50 +--
cpu/arm926ejs/at91cap9/spi.c | 119 +
cpu/arm926ejs/at91cap9/timer.c | 149 ++
.../u-boot.lds => cpu/arm926ejs/at91cap9/usb.c | 57 +--
cpu/arm926ejs/interrupts.c | 136 +-
cpu/arm926ejs/start.S |8 +-
cpu/arm946es/interrupts.c | 134 -
cpu/arm_intcm/Makefile |2 +-
cpu/ixp/interrupts.c | 132 +-
cpu/lh7a40x/interrupts.c | 135 -
cpu/pxa/interrupts.c | 117 -
cpu/s3c44b0/interrupts.c | 104
cpu/sa1100/interrupts.c| 137 -
drivers/mtd/dataflash.c| 36 +-
drivers/net/macb.c |8 +
drivers/net/macb.h |6 +
include/asm-arm/arch-at91cap9/AT91CAP9.h | 518
.../asm-arm/arch-at91cap9/clk.h| 50 +--
include/asm-arm/arch-at91cap9/hardware.h | 38 ++
.../asm-arm/arch-at91cap9/memory-map.h | 47 +--
include/asm-arm/dma-mapping.h | 50 ++
include/configs/at91cap9adk.h | 212
include/configs/m501sk.h | 197
include/configs/netstar.h | 86 ++--
lib_arm/Makefile
Re: [U-Boot-Users] [PATCH] Extend ATI Radeon driver to support more video modes
Hello, Jean-Christophe PLAGNIOL-VILLARD wrote: > On 10:24 Fri 15 Feb , Rodolfo Giometti wrote: >> On Thu, Feb 14, 2008 at 01:39:14AM +0100, Wolfgang Denk wrote: >>> In message <[EMAIL PROTECTED]> you wrote: >> +#if 1 /* @ 60 Hz */ >> +mode->crtc_h_total_disp = 0x009f00d2; >> +mode->crtc_h_sync_strt_wid = 0x000e0528; >> +mode->crtc_v_total_disp = 0x03ff0429; >> +mode->crtc_v_sync_strt_wid = 0x00030400; >> +mode->crtc_pitch = 0x00a000a0; >> +mode->ppll_div_3 = 0x00010060; >> +#else /* @ 75 Hz */ >> +mode->crtc_h_total_disp = 0x009f00d2; >> +mode->crtc_h_sync_strt_wid = 0x00120508; >> +mode->crtc_v_total_disp = 0x03ff0429; >> +mode->crtc_v_sync_strt_wid = 0x00030400; >> +mode->crtc_pitch = 0x00a000a0; >> +mode->ppll_div_3 = 0x00010078; >> +#endif > Mmm... better adding a configuration define here or just remove the > unused code. Ok, I'm going to drop the unused code then. >>> No, please keep it as reference in case anybody needs this later. >> Uh? =:-o Why did you change your mind? ;) >> >> | Delivery-date: Sun, 01 Apr 2007 15:18:41 +0200 >> | From: Wolfgang Denk <[EMAIL PROTECTED]> >> | To: Rodolfo Giometti <[EMAIL PROTECTED]> >> | cc: Markus Klotzb?cher <[EMAIL PROTECTED]> >> | Subject: Re: [PATCH] ISP116x HCD (Host Controller Driver) >> | >> | [snip] >> | >> | Please remove code in "#if 0 ... #endif" blocks. >> | >> | [snip] >> | >> | Best regards, >> | >> | Wolfgang Denk >> >> However I suggested also to add a configuration define, something >> like: >> >> #if defined(CFG_ATI_RADEON_60HZ) >> ... >> #elif defined(CFG_ATI_RADEON_75HZ) >> ... >> #else >> #error "..." >> #endif >> > Please use CONFIG and not CFG too allow this parameter in Kconfig new and already applied patch uses CONFIG_RADEON_VREFRESH_75HZ and another case (@ 60HZ) is just default case. Best Regards, Anatolij -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] Extend ATI Radeon driver to support more video modes
On 10:24 Fri 15 Feb , Rodolfo Giometti wrote: > On Thu, Feb 14, 2008 at 01:39:14AM +0100, Wolfgang Denk wrote: > > In message <[EMAIL PROTECTED]> you wrote: > > > > > > >> +#if 1 /* @ 60 Hz */ > > > >> + mode->crtc_h_total_disp = 0x009f00d2; > > > >> + mode->crtc_h_sync_strt_wid = 0x000e0528; > > > >> + mode->crtc_v_total_disp = 0x03ff0429; > > > >> + mode->crtc_v_sync_strt_wid = 0x00030400; > > > >> + mode->crtc_pitch = 0x00a000a0; > > > >> + mode->ppll_div_3 = 0x00010060; > > > >> +#else /* @ 75 Hz */ > > > >> + mode->crtc_h_total_disp = 0x009f00d2; > > > >> + mode->crtc_h_sync_strt_wid = 0x00120508; > > > >> + mode->crtc_v_total_disp = 0x03ff0429; > > > >> + mode->crtc_v_sync_strt_wid = 0x00030400; > > > >> + mode->crtc_pitch = 0x00a000a0; > > > >> + mode->ppll_div_3 = 0x00010078; > > > >> +#endif > > > > > > > > Mmm... better adding a configuration define here or just remove the > > > > unused code. > > > > > > Ok, I'm going to drop the unused code then. > > > > No, please keep it as reference in case anybody needs this later. > > Uh? =:-o Why did you change your mind? ;) > > | Delivery-date: Sun, 01 Apr 2007 15:18:41 +0200 > | From: Wolfgang Denk <[EMAIL PROTECTED]> > | To: Rodolfo Giometti <[EMAIL PROTECTED]> > | cc: Markus Klotzb?cher <[EMAIL PROTECTED]> > | Subject: Re: [PATCH] ISP116x HCD (Host Controller Driver) > | > | [snip] > | > | Please remove code in "#if 0 ... #endif" blocks. > | > | [snip] > | > | Best regards, > | > | Wolfgang Denk > > However I suggested also to add a configuration define, something > like: > > #if defined(CFG_ATI_RADEON_60HZ) > ... > #elif defined(CFG_ATI_RADEON_75HZ) > ... > #else > #error "..." > #endif > Please use CONFIG and not CFG too allow this parameter in Kconfig Best Regards, J. - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
[U-Boot-Users] [PATCH] Add sub-commands to fdt
fdt header - Display header info
fdt bootcpu - Set boot cpuid
fdt memory - Add/Update memory node
fdt memrsv print- Show current mem reserves
fdt memrsv add - Add a mem reserve
fdt memrsv delete- Delete a mem reserves
Signed-off-by: Kumar Gala <[EMAIL PROTECTED]>
---
common/cmd_fdt.c | 111 ++
1 files changed, 111 insertions(+), 0 deletions(-)
diff --git a/common/cmd_fdt.c b/common/cmd_fdt.c
index 9cd22ee..d0f8c27 100644
--- a/common/cmd_fdt.c
+++ b/common/cmd_fdt.c
@@ -296,6 +296,111 @@ int do_fdt (cmd_tbl_t * cmdtp, int flag, int argc, char
*argv[])
return err;
}
}
+
+ /
+* Display header info
+/
+ } else if (argv[1][0] == 'h') {
+ u32 version = fdt_version(fdt);
+ printf("magic:\t\t\t0x%x\n", fdt_magic(fdt));
+ printf("totalsize:\t\t0x%x (%d)\n", fdt_totalsize(fdt),
fdt_totalsize(fdt));
+ printf("off_dt_struct:\t\t0x%x\n", fdt_off_dt_struct(fdt));
+ printf("off_dt_strings:\t\t0x%x\n", fdt_off_dt_strings(fdt));
+ printf("off_mem_rsvmap:\t\t0x%x\n", fdt_off_mem_rsvmap(fdt));
+ printf("version:\t\t%d\n", version);
+ printf("last_comp_version:\t%d\n", fdt_last_comp_version(fdt));
+ if (version >= 2)
+ printf("boot_cpuid_phys:\t0x%x\n",
+ fdt_boot_cpuid_phys(fdt));
+ if (version >= 3)
+ printf("size_dt_strings:\t0x%x\n",
+ fdt_size_dt_strings(fdt));
+ if (version >= 17)
+ printf("size_dt_struct:\t\t0x%x\n",
+ fdt_size_dt_struct(fdt));
+ printf("\n");
+
+ /
+* Set boot cpu id
+/
+ } else if ((argv[1][0] == 'b') && (argv[1][1] == 'o')) {
+ unsigned long tmp = simple_strtoul(argv[2], NULL, 16);
+ fdt_set_boot_cpuid_phys(fdt, tmp);
+
+ /
+* memory command
+/
+ } else if ((argv[1][0] == 'm') && (argv[1][1] == 'e') &&
+ (argv[1][3] == 'o')) {
+ uint64_t addr, size;
+ int err;
+#ifdef CFG_64BIT_STRTOUL
+ addr = simple_strtoull(argv[2], NULL, 16);
+ size = simple_strtoull(argv[3], NULL, 16);
+#else
+ addr = simple_strtoul(argv[2], NULL, 16);
+ size = simple_strtoul(argv[3], NULL, 16);
+#endif
+ err = fdt_fixup_memory(fdt, addr, size);
+ if (err < 0)
+ return err;
+
+ /
+* mem reserve commands
+/
+ } else if ((argv[1][0] == 'm') && (argv[1][1] == 'e') &&
+ (argv[1][3] == 'r')) {
+ if (argv[2][0] == 'p') {
+ uint64_t addr, size;
+ int total = fdt_num_mem_rsv(fdt);
+ int j, err;
+ printf("index\t\t start\t\tsize\n");
+ printf("---"
+ "-\n");
+ for (j = 0; j < total; j++) {
+ err = fdt_get_mem_rsv(fdt, j, &addr, &size);
+ if (err < 0) {
+ printf("libfdt fdt_get_mem_rsv():
%s\n",
+ fdt_strerror(err));
+ return err;
+ }
+ printf("%x\t%08x%08x\t%08x%08x\n", j,
+ (u32)(addr >> 32),
+ (u32)(addr & 0x),
+ (u32)(size >> 32),
+ (u32)(size & 0x));
+ }
+ } else if (argv[2][0] == 'a') {
+ uint64_t addr, size;
+ int err;
+#ifdef CFG_64BIT_STRTOUL
+ addr = simple_strtoull(argv[3], NULL, 16);
+ size = simple_strtoull(argv[4], NULL, 16);
+#else
+
Re: [U-Boot-Users] [PATCH] Add call command on PPC
On Friday 15 February 2008, Haavard Skinnemoen wrote: > Mike Frysinger <[EMAIL PROTECTED]> wrote: > > x86 does 0 while Blackfin does 3. > > x86 can do more (at least 3) if you use the regparm attribute. But you > obviously need to keep both sides in sync. i'm aware of all the neat tricks you can do with gcc, but we were talking about the ABI standard (which the gcc tricks violate) -mike signature.asc Description: This is a digitally signed message part. - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] Extend ATI Radeon driver to support more video modes
On Thu, Feb 14, 2008 at 01:39:14AM +0100, Wolfgang Denk wrote: > In message <[EMAIL PROTECTED]> you wrote: > > > > >> +#if 1 /* @ 60 Hz */ > > >> +mode->crtc_h_total_disp = 0x009f00d2; > > >> +mode->crtc_h_sync_strt_wid = 0x000e0528; > > >> +mode->crtc_v_total_disp = 0x03ff0429; > > >> +mode->crtc_v_sync_strt_wid = 0x00030400; > > >> +mode->crtc_pitch = 0x00a000a0; > > >> +mode->ppll_div_3 = 0x00010060; > > >> +#else /* @ 75 Hz */ > > >> +mode->crtc_h_total_disp = 0x009f00d2; > > >> +mode->crtc_h_sync_strt_wid = 0x00120508; > > >> +mode->crtc_v_total_disp = 0x03ff0429; > > >> +mode->crtc_v_sync_strt_wid = 0x00030400; > > >> +mode->crtc_pitch = 0x00a000a0; > > >> +mode->ppll_div_3 = 0x00010078; > > >> +#endif > > > > > > Mmm... better adding a configuration define here or just remove the > > > unused code. > > > > Ok, I'm going to drop the unused code then. > > No, please keep it as reference in case anybody needs this later. Uh? =:-o Why did you change your mind? ;) | Delivery-date: Sun, 01 Apr 2007 15:18:41 +0200 | From: Wolfgang Denk <[EMAIL PROTECTED]> | To: Rodolfo Giometti <[EMAIL PROTECTED]> | cc: Markus Klotzb?cher <[EMAIL PROTECTED]> | Subject: Re: [PATCH] ISP116x HCD (Host Controller Driver) | | [snip] | | Please remove code in "#if 0 ... #endif" blocks. | | [snip] | | Best regards, | | Wolfgang Denk However I suggested also to add a configuration define, something like: #if defined(CFG_ATI_RADEON_60HZ) ... #elif defined(CFG_ATI_RADEON_75HZ) ... #else #error "..." #endif This allow keeping the code in a better style... or we can rewrite it as follow: /* @ 60 Hz */ mode->crtc_h_total_disp = 0x009f00d2; mode->crtc_h_sync_strt_wid = 0x000e0528; mode->crtc_v_total_disp = 0x03ff0429; mode->crtc_v_sync_strt_wid = 0x00030400; mode->crtc_pitch = 0x00a000a0; mode->ppll_div_3 = 0x00010060; /* @ 75 Hz * * Still untested. Just use it as reference for future improvemets. * * mode->crtc_h_total_disp = 0x009f00d2; * mode->crtc_h_sync_strt_wid = 0x00120508; * mode->crtc_v_total_disp = 0x03ff0429; * mode->crtc_v_sync_strt_wid = 0x00030400; * mode->crtc_pitch = 0x00a000a0; * mode->ppll_div_3 = 0x00010078; */ Ciao, Rodolfo -- GNU/Linux Solutions e-mail:[EMAIL PROTECTED] Linux Device Driver [EMAIL PROTECTED] Embedded Systems[EMAIL PROTECTED] UNIX programming phone: +39 349 2432127 - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] UNCACHED_SDRAM macro issue
Shinya Kuribayashi wrote: >> I think that on my application the UNCACHED_SDRAM should map the address >> on KSEG1 (how it is now) but this simply doesn't work. Instead, using >> the PHYSADDR(a) macro... the kernel is able to start. >> >> I suspect that there are issues on cache management. Can be? > > IMHO it's not related to cache. How do you set ERL and EXL bits? Please try to clear them at the STATUS register initialization like: reset: /* STATUS register */ mfc0k0, CP0_STATUS - li k1, ~ST0_IE + li k1, ~(ST0_ERL | ST0_EXL | ST0_IE) and k0, k1 mtc0k0, CP0_STATUS ERL and EXL disable exceptions. Due to this spec, we are in danger of overlooking something critical. If this change brings in new exception(s), please fix the causes of them first. Hope this helps. Shinya - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] Add call command on PPC
On Fri, 15 Feb 2008 02:29:39 -0500 Mike Frysinger <[EMAIL PROTECTED]> wrote: > x86 does 0 while Blackfin does 3. x86 can do more (at least 3) if you use the regparm attribute. But you obviously need to keep both sides in sync. Haavard - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] TQM834x: clean up configuration
[EMAIL PROTECTED] wrote on : > Get board name consistent with Linux and elsewhere; > get rid of local network definitions etc. > > Signed-off-by: Wolfgang Denk <[EMAIL PROTECTED]> > --- Acked-by: Martin Krause tqs.de> I've not tested the patch actually, but the changes are looking reasonable. Best Regards, Martin Krause -- TQ-Systems GmbH Muehlstrasse 2, Gut Delling, D-82229 Seefeld Amtsgericht Muenchen, HRB 105 018, UST-IdNr. DE 811 607 913 Geschaeftsfuehrer: Dipl.-Ing. (FH) Detlef Schneider, Dipl.-Ing. (FH) Ruediger Stahl http://www.tq-group.com - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] Add call command on PPC
In message <[EMAIL PROTECTED]> you wrote: > > > I guess the real question is whether it's acceptable to pass arguments > > on the stack to such subprograms. If the answer is yes, why not simply > > use varargs? > > for my purposes the answer is no. Then you cannot use C code. Or you rely on the ABI, and then you can use "go" as well. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] Generally speaking, there are other ways to accomplish whatever it is that you think you need ... - Doug Gwyn - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] Add call command on PPC
In message <[EMAIL PROTECTED]> you wrote: > The call command tries to mimic a function call in that the 'arg's to > the command are passed in registers according to the PPC ABI. > > The prototype that call mimics is some variation of the following > depending on how many arguments are passed to the command: > > void func(void) > void func(unsigned long r3) > void func(unsigned long r3, unsigned long r4) > ... > void func(unsigned long r3, unsigned long r4, ... unsigned long r10) > > The maximum number of 'arg's is 8. There are no arguments passed on > a stack, no floating point or vector arguments. Sorry, but this looks still broken to me. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] A Puritan is someone who is deathly afraid that someone, somewhere, is having fun. - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH v2] Add call command on PPC
In message <[EMAIL PROTECTED]> you wrote: > The call command tries to mimic a function call in that the 'arg's to > the command are passed in registers according to the PPC ABI. > > The prototype that call mimics is some variation of the following > depending on how many arguments are passed to the command: > > void func(void) > void func(unsigned long r3) > void func(unsigned long r3, unsigned long r4) > ... > void func(unsigned long r3, unsigned long r4, ... unsigned long r10) > > The maximum number of 'arg's is 8. There are no arguments passed on > a stack, no floating point or vector arguments. This really makes zero sense to me. > + img = (int (*)(ulong, ulong, ulong, ulong, > + ulong, ulong, ulong, ulong)) addr; > + > + for (i = 2; i < argc; i++) > + r[i-2] = simple_strtoul(argv[i], NULL, 16); > + > + return (*img)(r[0], r[1], r[2], r[3], r[4], r[5], r[6], r[7]); This looks not clean. Either this is a function with exactly 8 arguments, or it isn't. And where is the code that makes sure that ``the 'arg's to the command are passed in registers'' ? I see no difference between this call and what "go" does. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] "More software projects have gone awry for lack of calendar time than for all other causes combined." - Fred Brooks, Jr., _The Mythical Man Month_ - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] Add call command on PPC
In message <[EMAIL PROTECTED]> you wrote: > > The main reason for me doing this was to provide a "generic" mechanism > that mimic how we boot kernels on PPC. May I ask why exactly this is needed at all? Kernels should be booted using the bootm command - if somethingis missing there, it should be added / fixed in this contexzt instead of adding another highly specialized command. And *if* we add this, then please let's make it general enough so we can use it on all architectures without adding a lot of #ifdef mess. > The other issue w/providing this on all arch's is that the number of > params passed via regs may vary and I don't know what is for arm, > avr32, blackfin, i386, m68k, microblaze, mips, nios, nios2, and sh. > and each variation will require a different prototype. No, just change the design, then. Make it a varargs function and pass the number of registers as one argument. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] Death. Destruction. Disease. Horror. That's what war is all about. That's what makes it a thing to be avoided. -- Kirk, "A Taste of Armageddon", stardate 3193.0 - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] Add call command on PPC
On Feb 15, 2008, at 1:29 AM, Mike Frysinger wrote: > On Friday 15 February 2008, Haavard Skinnemoen wrote: >> Mike Frysinger <[EMAIL PROTECTED]> wrote: if someone can tell me what the number of args are for each arch we can see about providing this everywhere. >>> >>> are you sure you actually need to worry about such things ? i'm >>> pretty >>> sure any relevant architecture says that the first X arguments go >>> via >>> registers and the rest go on the stack. let the compiler worry >>> about the >>> exact value for X. >> >> I guess the real question is whether it's acceptable to pass >> arguments >> on the stack to such subprograms. If the answer is yes, why not >> simply >> use varargs? > > if the goal is to keep things in registers, "call" and its > description are way > too vague. the description makes me think this is meant to be a > "go" for > functions. As per Wolfgang's desires the help is terse, the commit message was suppose to convey more details. But I can see where the confusion comes from. I'm open to better wording and will try to think of some myself. >> If the answer is no: avr32 can pass up to 5 arguments through >> registers. > > x86 does 0 while Blackfin does 3. 3 arch's down, 6 to go. - k - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
Re: [U-Boot-Users] [PATCH] Add call command on PPC
On Feb 15, 2008, at 1:21 AM, Haavard Skinnemoen wrote: > On Fri, 15 Feb 2008 02:08:23 -0500 > Mike Frysinger <[EMAIL PROTECTED]> wrote: > >>> if someone can tell me what the number of args are for each arch we >>> can see about providing this everywhere. >> >> are you sure you actually need to worry about such things ? i'm >> pretty sure >> any relevant architecture says that the first X arguments go via >> registers >> and the rest go on the stack. let the compiler worry about the >> exact value >> for X. > > I guess the real question is whether it's acceptable to pass arguments > on the stack to such subprograms. If the answer is yes, why not simply > use varargs? for my purposes the answer is no. > > If the answer is no: avr32 can pass up to 5 arguments through > registers. thanks - k - This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/ ___ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
