-Original Message-
From: Hiremath, Vaibhav
Sent: Tuesday, May 11, 2010 10:30 AM
To: Hiremath, Vaibhav; u-boot@lists.denx.de
Cc: w...@denx.de; t...@bumblecow.com; Paulraj, Sandeep; Premi, Sanjeev
Subject: RE: [PATCH-V3 1/2] omap3: Calculate CS1 size only when SDRC is
initialized for
Hello Maxim,
Maxim Podbereznyy wrote:
I can't find a support for i2c functionality in u-boot 1.3.4. I've found a
please use actual code, 1.3.4 is very old ...
discussion of such patches from Jean-Christophein in March 2009 and still no
code in u-boot.
You mean?:
This patch series adds support for SM502 on ICON
board.
Anatolij Gustschin (3):
video: sm501: add support for SM501 chips on PCI bus
video: sm501.c: add weak default functions
ppc4xx: icon: add support for SM502 chip
board/mosaixtech/icon/icon.c | 69 +
Adds initialization code for SM502 graphics controller
and NL6448BC20-21D LCD panel.
Signed-off-by: Anatolij Gustschin ag...@denx.de
Cc: Stefan Roese s...@denx.de
---
board/mosaixtech/icon/icon.c | 69 ++
include/configs/icon.h | 23
For boards using sm501/sm502 on PCI bus some driver
functions normaly defined in the board code are not
needed and empty. Provide weak default functions for
them and do not enforce board code to define empty
functions.
Signed-off-by: Anatolij Gustschin ag...@denx.de
---
drivers/video/sm501.c |
Signed-off-by: Anatolij Gustschin ag...@denx.de
---
drivers/video/sm501.c | 89 ++---
include/pci_ids.h |1 +
2 files changed, 85 insertions(+), 5 deletions(-)
diff --git a/drivers/video/sm501.c b/drivers/video/sm501.c
index 283d2d9..544e0a0
Dear Maxim Podbereznyy,
please keep the mailing list on cc:
In message aanlktikzgywfuvnktdd97pleuin0wn5mrzuvoqdjv...@mail.gmail.com you
wrote:
I'm sorry I did not mention about I need *AT91* i2c support in u-boot. I
know that a general i2c is supported in u-boot since the ages.
It seems
Dear Wolfgang,
Am 25.05.2010 13:29, schrieb Wolfgang Denk:
In message 4bfb8708.4010...@corscience.de you wrote:
I think the easiest way to solve this is to create another patch
including exactely the changes sent before plus removing {in|out}[bwl]
macros in omap1510.h.
Well ... this
For platforms that implement a hardware watchdog, call its initialization
routine in init_sequence.
This location has been chosen to be the closest to initialization of console as
some watchdog drivers are writing status messages. On the other hand, watchdog
setup should be close to chip startup
Internet Users Reward
For the Last time, we are pleased to notify you of your Windows© E-Award for
2.5, million. Please confirm if you received our previous message with;
E-ticket number: 7219231BB-988,
Category: A,
Draw: 6756
Amount: 2.5, million Dollars
You may establish contact with the
Watchdog resets were experienced during autoboot delay. Petting the watchdog
during abortboot() function solve the issue.
Signed-off-by: Nicolas Ferre nicolas.fe...@atmel.com
---
common/main.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/common/main.c b/common/main.c
inline: a.gif___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Hi,
Thank you for the input, I will resend the patch with Magnus as author.
Daniel
Mike Frysinger wrote:
the subject line is incorrect. you dont credit people via summary/changelogs,
you do it via the tags. so if Magnus wrote this patch, he should be credited
with authorship (so use
Compilation was broken in atstk100x boards (at least) due to
missing file unaligned.h
I hope following patch helps.
Regards,
Xulio Coira.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Signed-off-by: Xulio Coira xul...@gmail.com
---
arch/avr32/include/asm/unaligned.h |6 ++
1 files changed, 6 insertions(+), 0 deletions(-)
create mode 100644 arch/avr32/include/asm/unaligned.h
diff --git a/arch/avr32/include/asm/unaligned.h
b/arch/avr32/include/asm/unaligned.h
new file
From: Magnus Sjalander mag...@gaisler.com
Signed-off-by: Daniel Hellstrom dan...@gaisler.com
---
arch/sparc/include/asm/byteorder.h |1 +
arch/sparc/include/asm/unaligned.h | 10 ++
2 files changed, 11 insertions(+), 0 deletions(-)
create mode 100644
Dear Wolfgang,
Please pull the 17 patches in the master branch of u-boot-sparc
repository. I have rebased the patches and updated the first patch
according to Mike Drysingers comments.
Thanks,
Daniel
The following changes since commit 209c040b86ce7081f25dd547913d86d597e8ac34:
Magnus
Hi,
On Wed, May 05, 2010 at 03:10:20PM +0100, Quotient Remainder wrote:
On Wed, May 5, 2010 at 1:59 PM, Stefano Babic sba...@denx.de wrote:
Quotient Remainder wrote:
Out of interest, how did something like this get away with only causing
an occasional failure?
Well, there are
This trivial patch remove the 64-bit division into print_size code.
Luigi 'Comio' Mantellini (1):
[OLT-M68K] Avoid 64bit division in print_size
lib/display_options.c | 15 ---
1 files changed, 8 insertions(+), 7 deletions(-)
___
Signed-off-by: Luigi 'Comio' Mantellini luigi.mantell...@idf-hit.com
---
lib/display_options.c | 15 ---
1 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/lib/display_options.c b/lib/display_options.c
index 86df05d..eca5415 100644
--- a/lib/display_options.c
+++
Wolfgang Denk wrote:
Thanks. Just to be clear, do you expect fdt_fw_addr always to point to a
FIT-wrapped firmware binary?
Please re-read the IRC log. Kumar explicitly stated he was trying to
avoid making FIT images mandatory, at least for now.
And he proposed a board-specific function
On Wed, May 26, 2010 at 9:01 AM, Luigi 'Comio' Mantellini
luigi.mantell...@idf-hit.com wrote:
Signed-off-by: Luigi 'Comio' Mantellini luigi.mantell...@idf-hit.com
---
You're a little late:
http://git.denx.de/?p=u-boot.git;a=commit;h=f2d76ae4fdde180e120ea2d29d6ef881360b3cba
--
Timur Tabi
Dear Timur Tabi,
In message 4bfd3a39.4090...@freescale.com you wrote:
Please re-read the IRC log. Kumar explicitly stated he was trying to
avoid making FIT images mandatory, at least for now.
And he proposed a board-specific function that would allow this to work, but
you rejected it.
I'm sorry! my tree is out of date (2010/05/15) :)
I'm using the toolchain coldfire-4.4 from Codesourcery (Sourcery G++ Lite
4.4-217)
Today I tried to build the cf547x_8x target and I noticed a div by 0 issue on
the __udivdi3 (n = size / d) ... I checked the operands that were ok, but I
Wolfgang Denk wrote:
And he proposed a board-specific function that would allow this to work, but
you rejected it. So I don't still know how to implement what you want.
Well, in a way that may be image-type dependent, but that is not
board-specific.
Technically, that's true, but in most
On Wed, May 26, 2010 at 11:38:27AM -0500, Timur Tabi wrote:
I believe we should have a board-specific function that figures out how much
extra space is needed, and just returns a single integer that the
boot_relocate_fdt() uses to pad the FDT when it relocates it.
Why don't we just grow the
create_pipe() can give wrong result if an expression is passed as the 'endpoint'
argument -- due to missing parentheses.
Thanks to Martin Mueller for finding the bug and providing the patch.
Signed-off-by: Sergei Shtylyov sshtyl...@ru.mvista.com
---
include/usb.h |2 +-
1 file changed, 1
On Wednesday 26 May 2010 05:49:21 Andreas Bießmann wrote:
Am 25.05.2010 13:29, schrieb Wolfgang Denk:
In message 4bfb8708.4010...@corscience.de you wrote:
I think the easiest way to solve this is to create another patch
including exactely the changes sent before plus removing {in|out}[bwl]
Scott Wood wrote:
On Wed, May 26, 2010 at 11:38:27AM -0500, Timur Tabi wrote:
I believe we should have a board-specific function that figures out how much
extra space is needed, and just returns a single integer that the
boot_relocate_fdt() uses to pad the FDT when it relocates it.
Why
On 05/26/2010 12:56 PM, Timur Tabi wrote:
Scott Wood wrote:
On Wed, May 26, 2010 at 11:38:27AM -0500, Timur Tabi wrote:
I believe we should have a board-specific function that figures out how much
extra space is needed, and just returns a single integer that the
boot_relocate_fdt() uses to
On Wednesday 26 May 2010 07:52:37 Xulio Coira wrote:
Signed-off-by: Xulio Coira xul...@gmail.com
---
arch/avr32/include/asm/unaligned.h |6 ++
1 files changed, 6 insertions(+), 0 deletions(-)
create mode 100644 arch/avr32/include/asm/unaligned.h
diff --git
Wolfgang Denk wrote:
Hm... looks as if you were repeating the same code 3 times. Please make
this a function.
The code isn't really the same. I would need to pass a lot of
parameters to this function: the hose, the devdisr mask, the slot
name, the slot number, the bus number, and so
On Wednesday 26 May 2010 06:34:49 Nicolas Ferre wrote:
--- a/common/main.c
+++ b/common/main.c
@@ -159,6 +159,7 @@ static __inline__ int abortboot(int bootdelay)
* when catch up.
*/
do {
+ WATCHDOG_RESET(); /* Trigger watchdog, if needed */
On 05/26/2010 01:12 PM, Timur Tabi wrote:
Wolfgang Denk wrote:
Hm... looks as if you were repeating the same code 3 times. Please make
this a function.
The code isn't really the same. I would need to pass a lot of
parameters to this function: the hose, the devdisr mask, the slot
name, the
Scott Wood wrote:
Perhaps (most of) this information could be put in a data structure to
which you point?
That doesn't change the amount of information that needs to be passed, it
only makes the prototype have fewer lines.
The point I'm trying to make is that the code in question is not the
Scott Wood wrote:
But you can reasonably allocate significantly more than you'll need
without actually causing the fdt to get that big. The actual cap could
be a board specific magic number (like CONFIG_SYS_MALLOC_LEN), or we
could cap it at something based on the amount of RAM.
We have
Signed-off-by: Asen Dimov di...@ronetix.at
---
drivers/misc/Makefile |1 +
drivers/misc/ds2401.c | 265 +
include/ds2401.h | 36 +++
3 files changed, 302 insertions(+), 0 deletions(-)
create mode 100644 drivers/misc/ds2401.c
I have read section 10 of the manual which describes debugging u-boot
and have further questions.
1) In section 10.4, Tips and Tricks, it says To prevent GDB from
jumping around in the code when trying to single step, i. e. when it
seems as if the code is not executing line by line, you can
On 05/26/2010 01:19 PM, Timur Tabi wrote:
Scott Wood wrote:
Perhaps (most of) this information could be put in a data structure to
which you point?
That doesn't change the amount of information that needs to be passed, it
only makes the prototype have fewer lines.
Which is relevant, given
Dear Timur Tabi,
In message 4bfd4e83.2080...@freescale.com you wrote:
Well, in a way that may be image-type dependent, but that is not
board-specific.
Technically, that's true, but in most cases the function that returns the
address/size of the firmware would exist in board code.
For
Dear Timur Tabi,
In message 4bfd6704.2040...@freescale.com you wrote:
We have something like that already:
#ifndef CONFIG_SYS_FDT_PAD
#define CONFIG_SYS_FDT_PAD 0x3000
#endif
And Wolfgang doesn't like it.
Because nobody can explain where this magic number 0x3000 is coming
from or why
On Wed, May 26, 2010 at 1:31 PM, Asen Dimov di...@ronetix.at wrote:
The one-wire bus is generic, so I think it would make sense to split
the bus access logic and the handling of the DS2401 into parts so that
other people could make use of it.
We did an implementation a long time ago for the
Scott Wood wrote:
Which is relevant, given that you're whipping out a big scary-looking
prototype as a reason to avoid refactoring. :-)
So instead of this:
configure_pci(PCIE1, PCIE1, Slot 1, pcie_ep, num, LAW_TRGT_IF_PCIE_1,
CONFIG_SYS_PCIE1_MEM_PHYS, LAW_SIZE_512M,
On 05/26/2010 02:34 PM, Timur Tabi wrote:
Scott Wood wrote:
Which is relevant, given that you're whipping out a big scary-looking
prototype as a reason to avoid refactoring. :-)
So instead of this:
configure_pci(PCIE1, PCIE1, Slot 1, pcie_ep, num, LAW_TRGT_IF_PCIE_1,
Dear Timur Tabi,
In message 4bfc1736.5030...@freescale.com you wrote:
+ { 0, 333,1,5, 31,3, 0},
+ {334, 400,1,5, 31,3, 0},
+ {401, 549,1,5, 31,3, 0},
+ {550, 680,1,5, 31,5, 0},
+ {681, 850,1,5, 31,5, 0},
+
Dear Timur Tabi,
In message 1274308618-2974-1-git-send-email-ti...@freescale.com you wrote:
Introduce function fdt_get_max_phandle(), which returns the largest value
of all phandles in a device tree. This is useful for allocating a new phandle
property, since all phandles must be unique.
Wolfgang Denk wrote:
There is exactly two entries in that table where it would make any
difference, and if that's really that important to you, adding two
spaces after the TAB would work wonders.
The column headers would be unaligned:
static const board_specific_parameters_t bsp[] = {
/*
*
Dear Mike Frysinger,
In message 201005261350.16464.vap...@gentoo.org you wrote:
i changed the accessors to use in/out macros as i thought that was the
correct
set of accessor functions to use. looking at a set of definitions and
picking
a set because they look like they'll just work
Use readX() / writeX() accessors instead of inX() / outX().
Suggested-by: Mike Frysinger vap...@gentoo.org
Signed-off-by: Wolfgang Denk w...@denx.de
---
drivers/net/dm9000x.c | 12 ++--
1 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/net/dm9000x.c
Wolfgang Denk wrote:
It appears you really haven't bothered checking. We have been using
it on PowerPC since day 1 - for well over 10 years by now. On many,
many systems. Get a clue!
I've looked at get_ram_size(), and I don't think I can use it.
We use phys_addr_t to represent DDR sizes,
Dear Detlev Zundel,
In message 1274364575-9764-2-git-send-email-...@denx.de you wrote:
From: Michael Weiss michael.we...@ifm.com
For CONFIG_SYS_BOOTCOUNT_SINGLEWORD the code had an endianness problem.
Signed-off-by: Michael Weiss michael.we...@ifm.com
Signed-off-by: Detlev Zundel
Dear Kumar Gala,
In message 1274433468-31825-1-git-send-email-ga...@kernel.crashing.org you
wrote:
Match style we use almost everywhere else
Signed-off-by: Kumar Gala ga...@kernel.crashing.org
---
arch/powerpc/cpu/mpc512x/Makefile| 10 +-
board/freescale/common/Makefile
Dear Andreas Bießmann,
In message 1274527041-62757-1-git-send-email-andreas.de...@googlemail.com you
wrote:
Compiling tools subdirectory on Mac OS X 10.6 (Snow Leopard) complains about
wrong syntax in system includes.
In file included from /usr/include/stdio.h:444,
from
Dear Kim Phillips,
In message 20100521153045.5b6f38a8.kim.phill...@freescale.com you wrote:
Wolfgang,
please pull one cpu init fix, and a typo fix*:
The following changes since commit 2f05e394fccf62a4693c6b8323de725f90d1f003:
fsl_diu_fb.c: fix build warnings (2010-05-17 23:34:18
Dear Daniel Hellstrom,
In message 1274194143-8994-1-git-send-email-dan...@gaisler.com you wrote:
Signed-off-by: Daniel Hellstrom dan...@gaisler.com
---
drivers/net/greth.c | 69 +++---
1 files changed, 43 insertions(+), 26 deletions(-)
Sorry for
Dear Daniel Hellstrom,
In message 1274194143-8994-2-git-send-email-dan...@gaisler.com you wrote:
Signed-off-by: Daniel Hellstrom dan...@gaisler.com
---
drivers/net/greth.c | 14 --
1 files changed, 12 insertions(+), 2 deletions(-)
diff --git a/drivers/net/greth.c
On Fri, May 21, 2010 at 4:17 AM, Kumar Gala ga...@kernel.crashing.org wrote:
#ifdef CONFIG_PCIE3
ft_fsl_pci_setup(blob, pci0, pcie3_hose);
+#else
+ ft_fsl_pci_setup(blob, pci0, NULL);
#endif
What is the reason why CONFIG_PCIE3 setups up the pci node called
pci0? Is it
Dear Daniel Hellstrom,
In message 1274194143-8994-3-git-send-email-dan...@gaisler.com you wrote:
Signed-off-by: Daniel Hellstrom dan...@gaisler.com
...
- * (C) Copyright 2007
- * Daniel Hellstrom, Gaisler Research, dan...@gaisler.com
+ * (C) Copyright 2010
+ * Daniel Hellstrom, Aeroflex
Dear Daniel Hellstrom,
In message 1274194143-8994-4-git-send-email-dan...@gaisler.com you wrote:
Signed-off-by: Daniel Hellstrom dan...@gaisler.com
---
arch/sparc/cpu/leon3/Makefile |5 +-
arch/sparc/cpu/leon3/memcfg.c | 276
+
Dear Daniel Hellstrom,
In message 1274194143-8994-5-git-send-email-dan...@gaisler.com you wrote:
Signed-off-by: Daniel Hellstrom dan...@gaisler.com
---
arch/sparc/cpu/leon3/cpu_init.c | 10 +--
arch/sparc/cpu/leon3/interrupts.c |7 +-
arch/sparc/cpu/leon3/memcfg.h |1 -
Dear Daniel Hellstrom,
In message 4bfd1176.6080...@gaisler.com you wrote:
Please pull the 17 patches in the master branch of u-boot-sparc
repository. I have rebased the patches and updated the first patch
according to Mike Drysingers comments.
Sorry, but I won't.
I have found a number of
On Wed, 26 May 2010 23:02:43 0200, Wolfgang Denk wrote:
Dear Daniel Hellstrom,
In message 4bfd1176.6080...@gaisler.com you wrote:
Please pull the 17 patches in the master branch of u-boot-sparc
repository. I have rebased the patches and updated the first patch
according to
Just a codeing-style question, I have seen other writers of assembly
code add an extra space before instruction executed in a delay-slot
(typically after a branch for SPARC) just to make it clear that
instruction is executed as well. I find that quite good and have
adopted that habit too, is
Dear Wolfgang,
Am 26.05.2010 um 22:21 schrieb Wolfgang Denk:
Use readX() / writeX() accessors instead of inX() / outX().
this works for me.
Suggested-by: Mike Frysinger vap...@gentoo.org
Signed-off-by: Wolfgang Denk w...@denx.de
Tested-by: Andreas Bießmann biessm...@corscience.de
regards
On Fri, May 21, 2010 at 4:17 AM, Kumar Gala ga...@kernel.crashing.org wrote:
#ifdef CONFIG_PCIE2
- pcie_configured = is_fsl_pci_cfg(LAW_TRGT_IF_PCIE_2, io_sel);
+ pcie_configured = is_serdes_configured(PCIE2);
if (pcie_configured !(devdisr MPC85xx_DEVDISR_PCIE2)) {
+
Hi Wolfgang,
On 5/26/2010 1:21 PM, Wolfgang Denk wrote:
Use readX() / writeX() accessors instead of inX() / outX().
Suggested-by: Mike Frysingervap...@gentoo.org
Signed-off-by: Wolfgang Denkw...@denx.de
---
drivers/net/dm9000x.c | 12 ++--
1 files changed, 6 insertions(+), 6
Scott Wood wrote:
A few of those things don't belong there -- I think first_free_busno
should be a static variable inside the pci setup function, for example
(does Linux still even need separate bus number spaces on different
hoses?). pcie_ep should just be a local variable. The hose
Dear Ben Warren,
In message 4bfd9623.3070...@gmail.com you wrote:
Looks good to me. Please apply directly.
Done, thanks.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell,
Dear dan...@gaisler.com,
In message 20100526172855.2w3qpyvp3akgc...@webmail.bluegenesis.com you wrote:
Just a codeing-style question, I have seen other writers of assembly
code add an extra space before instruction executed in a delay-slot
(typically after a branch for SPARC) just to make it
On Fri, May 21, 2010 at 1:39 AM, Sudhakar Rajashekhara
sudhakar@ti.com wrote:
Provides initial support for TI OMAP-L138/DA850 SoC devices on
a Logic PD EVM board.
Provides:
Initial boot and configuration.
Support for i2c.
UART support (console).
Signed-off-by: Sudhakar Rajashekhara
On Mon, May 17, 2010 at 05:04:30PM -0400, Ben Gardiner wrote:
diff --git a/common/cmd_dynenv.c b/common/cmd_dynenv.c
new file mode 100644
index 000..5167875
--- /dev/null
+++ b/common/cmd_dynenv.c
@@ -0,0 +1,112 @@
+/*
+ * (C) Copyright 2006-2007 OpenMoko, Inc.
+ * Author: Harald
The following changes since commit 01f03bda5b22e5aeae5f02fd537da97a41485c73:
Wolfgang Denk (1):
Prepare v2010.06-rc1
are available in the git repository at:
git://git.denx.de/u-boot-nand-flash.git master
Andrew Caldwell (1):
Blackfin: nand: drain the write buffer before
Thank you for the thorough review, Scott.
On Wed, May 26, 2010 at 6:58 PM, Scott Wood scottw...@freescale.com wrote:
On Mon, May 17, 2010 at 05:04:30PM -0400, Ben Gardiner wrote:
diff --git a/common/cmd_dynenv.c b/common/cmd_dynenv.c
new file mode 100644
index 000..5167875
--- /dev/null
Wolfgang Denk wrote:
Dear Timur Tabi,
In message 1274308618-2974-1-git-send-email-ti...@freescale.com you wrote:
Introduce function fdt_get_max_phandle(), which returns the largest value
of all phandles in a device tree. This is useful for allocating a new
phandle
property, since all
Hi Mark,
On 5/27/10, Mark Fanara mark.fan...@gmail.com wrote:
...
2) I am using a bdi3000. Is there no way to build u-boot so that it is
directly loaded to RAM by the debugger and does not relocate itself?
#define CONFIG_SKIP_RELOCATE_UBOOT
BRs,
Alan
75 matches
Mail list logo