Re: [U-Boot] [PATCH v3] net: ll_temac: Add LL TEMAC driver to u-boot

2011-09-05 Thread Michal Simek
Dear Wolfgang Denk, > In message <4e60da47.4070...@monstr.eu> you wrote: That driver is not definitely for all ppc systems. That IP is available just for Xilinx FPGA where can be possible to use it with Microblaze and xilinx ppc440 (maybe ppc405). That DCR handling, whic

[U-Boot] [PATCH] drivers/video/Makefile: include object files only once

2011-09-05 Thread Wolfgang Denk
Use "$(sort $(COBJS-y))" to prevent multiple inclusion of the same object files. Also sort driver list. Signed-off-by: Wolfgang Denk --- drivers/video/Makefile |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/video/Makefile b/drivers/video/Makefile index db8

Re: [U-Boot] [PATCH] video: Fix build error with global inclusion of videomodes

2011-09-05 Thread Wolfgang Denk
Dear Kumar Gala, In message <1315176514-4164-1-git-send-email-ga...@kernel.crashing.org> you wrote: > The following commit: > > commit de701d118380de12a48c8a4fb345e9f909329432 > Author: Syed Mohammed Khasim > Date: Tue Apr 19 14:00:34 2011 -0500 > >OMAP3: Add DSS driver for OMAP3 > > Ad

Re: [U-Boot] [PATCH] linkstation: fix warning: "CONFIG_IDENT_STRING" redefined

2011-09-05 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1312108513-29786-1-git-send-email...@denx.de> you wrote: > Commit 09c2e90 "unify version_string" defines a default value for > CONFIG_IDENT_STRING in version.h, so any private settings musty be > done before including this file. Move the include for version.h after

[U-Boot] [PATCH] MPC8xx: fix build problem for ETX094 board

2011-09-05 Thread Wolfgang Denk
Commit 58c583b "net: Check network device driver name" increased the code size and broke building for the ETX094 board. Adjust the linker script to make it build again. Signed-off-by: Wolfgang Denk --- board/etx094/u-boot.lds |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --gi

Re: [U-Boot] [PATCH] drivers/video/Makefile: include object files only once

2011-09-05 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1315207224-21401-1-git-send-email...@denx.de> you wrote: > Use "$(sort $(COBJS-y))" to prevent multiple inclusion of the same > object files. > > Also sort driver list. > > Signed-off-by: Wolfgang Denk > --- > drivers/video/Makefile |6 +++--- > 1 files chan

Re: [U-Boot] [PATCH v3] net: ll_temac: Add LL TEMAC driver to u-boot

2011-09-05 Thread Wolfgang Denk
Dear Michal Simek, In message <4e647722.9080...@monstr.eu> you wrote: > > > I can only offer you a solution that seems acceptable to me. > > seems? Well, yes. As mentioned, I'm not happy with the propsal myself, but as long as I cannot come up with any better one it would be unjust to demand i

Re: [U-Boot] [PATCH 3/5] da830: add support for NAND boot mode

2011-09-05 Thread Netagunte, Nagabhushana
I am just curious, since this patch adds NAND boot mode for the First time in denx git, how this affects you? Are you storing ENV variables in NAND during some other boot mode? In fact we are Adding other boot mode in this patch set. How come you are using Denx tree to flash bios on your da830 boar

Re: [U-Boot] [PATCH 1/1 V2] add support for mcf5307 cpu

2011-09-05 Thread Jin Zhengxiong-R64188
> -Original Message- > From: angelo [mailto:angel...@gmail.com] > Sent: Friday, September 02, 2011 7:41 PM > To: Jin Zhengxiong-R64188 > Cc: U-Boot@lists.denx.de > Subject: [U-Boot] [PATCH 1/1 V2] add support for mcf5307 cpu > > Hi Jason and all, > > i am really interested to bring mcf530

Re: [U-Boot] [PATCH 3/5] da830: add support for NAND boot mode

2011-09-05 Thread Nick Thompson
On 05/09/11 09:26, Netagunte, Nagabhushana wrote: > I am just curious, since this patch adds NAND boot mode for the > First time in denx git, how this affects you? Are you storing > ENV variables in NAND during some other boot mode? In fact we are > Adding other boot mode in this patch set. How com

Re: [U-Boot] [PATCH] MPC8xx: fix build problem for ETX094 board

2011-09-05 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1315209092-14430-1-git-send-email...@denx.de> you wrote: > Commit 58c583b "net: Check network device driver name" increased the > code size and broke building for the ETX094 board. > > Adjust the linker script to make it build again. > > Signed-off-by: Wolfgang De

Re: [U-Boot] [PATCH 3/5] da830: add support for NAND boot mode

2011-09-05 Thread Wolfgang Denk
Dear "Netagunte, Nagabhushana", In message you wrote: > I am just curious, since this patch adds NAND boot mode for the > First time in denx git, how this affects you? Are you storing > ENV variables in NAND during some other boot mode? In fact we are > Adding other boot mode in this patch set.

Re: [U-Boot] [PATCH v2] FPGA: constify Lattice FPGA code

2011-09-05 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1312547354-20051-1-git-send-email...@denx.de> you wrote: > The Lattice code was missed by commit e6a857d "fpga: constify to fix > build warning" resulting in such warnings: > > fpga.c: In function 'fpga_load': > fpga.c:238: warning: passing argument 2 of 'lattice_l

[U-Boot] [PATCH] PPC4xx: ALPR: constify FPGA code

2011-09-05 Thread Wolfgang Denk
The ALPR custom FPGA code was missed by commit e6a857d "fpga: constify to fix build warning" resulting in such warnings: fpga.c:226: warning: initialization from incompatible pointer type Signed-off-by: Wolfgang Denk Cc: Stefan Roese --- board/prodrive/alpr/fpga.c |2 +- 1 files changed, 1

Re: [U-Boot] [PATCH] PPC4xx: ALPR: constify FPGA code

2011-09-05 Thread Stefan Roese
Hi Wolfgang, On Monday 05 September 2011 11:43:29 Wolfgang Denk wrote: > The ALPR custom FPGA code was missed by commit e6a857d "fpga: > constify to fix build warning" resulting in such warnings: > > fpga.c:226: warning: initialization from incompatible pointer type > > Signed-off-by: Wolfgang D

[U-Boot] [PATCH v2] da850evm: Move LPSC configuration to board_early_init_f()

2011-09-05 Thread Christian Riesch
Since commit f1d2b313c9eb6808d30c16a9eb5251240452a56c the serial port of the da850evm is accessed before the UART2 peripheral of the SoC is powered on in the function board_init() in board/davinci/da8xxevm/da850evm.c. When u-boot is used in conjunction with the UBL (user boot loader, see doc/READM

Re: [U-Boot] [PATCH] PPC4xx: ALPR: constify FPGA code

2011-09-05 Thread Sergei Shtylyov
Hello. On 05-09-2011 13:43, Wolfgang Denk wrote: > The ALPR custom FPGA code was missed by commit e6a857d "fpga: > constify to fix build warning" resulting in such warnings: > fpga.c:226: warning: initialization from incompatible pointer type > Signed-off-by: Wolfgang Denk > Cc: Stefan Roese >

Re: [U-Boot] [PATCH] PPC4xx: ALPR: constify FPGA code

2011-09-05 Thread Wolfgang Denk
Dear Stefan Roese, In message <201109051150.46853...@denx.de> you wrote: > > Acked-by: Stefan Roese Thanks. > Wolfgang, please feel free to pull this directly if you prefer. Will do, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zund

[U-Boot] [PATCH] omap3: beagle: Fix build warning

2011-09-05 Thread Sanjeev Premi
This patch fixes the warning dure to recent changes to the board configuration: cmd_i2c.o cmd_i2c.c -c cmd_i2c.c:109:1: warning: missing braces around initializer cmd_i2c.c:109:1: warning: (near initialization for 'i2c_no_probes[0]') Signed-off-by: Sanjeev Premi Cc: Jason Kridner --- Patch was

Re: [U-Boot] [PATCH] PPC4xx: ALPR: constify FPGA code

2011-09-05 Thread Wolfgang Denk
Dear Sergei Shtylyov, In message <4e64a372.8030...@mvista.com> you wrote: > > > -int fpga_write_fn (void *buf, size_t len, int flush, int cookie) > > +int fpga_write_fn (const void *buf, size_t len, int flush, int cookie) > > I guess this causes checkpatch.pl to complain due to space before

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

2011-09-05 Thread Stefano Babic
On 09/02/2011 04:02 PM, Fabio Estevam wrote: > Avoid the usage of extern in C file as pointed out by checkpatch. > > Signed-off-by: Fabio Estevam Hi Fabio, > diff --git a/arch/arm/include/asm/arch-mx25/imx-regs.h > b/arch/arm/include/asm/arch-mx25/imx-regs.h > index 9e30f7c..7e34050 100644 > -

[U-Boot] [PATCH] [COSMETIC] board/prodrive/alpr/fpga.c: Coding style cleanup

2011-09-05 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk --- No functional changes, only coding style cleanup. board/prodrive/alpr/fpga.c | 68 ++- 1 files changed, 35 insertions(+), 33 deletions(-) diff --git a/board/prodrive/alpr/fpga.c b/board/prodrive/alpr/fpga.c index 3666a9a

[U-Boot] mx35pdk vs. mx35_3stack

2011-09-05 Thread Teemu Keskinarkaus
Hi, We have mx35 based hardware where I've been using U-boot from mx35_3stack dev-board so far. It's version 2009.08. Due upgrading the compiler the U-boot didn't compile correctly anymore so I thought I upgrade it to latest available which is 2011.06. That doesn't have the mx35_3stack target a

Re: [U-Boot] [PATCH] arm: Add Prep subcommand support to bootm

2011-09-05 Thread Andreas Bießmann
Dear Simon, Am Mo 29 Aug 2011 18:08:13 CEST, Simon Schwarz schrieb: > Adds prep subcommand to bootm implementation of ARM. When bootm is called with > the subcommand prep the function stops right after ATAGS creation and before > announce_and_cleanup. > > This is used in savebp command savebp? I

[U-Boot] omap3: gpio: build warnings and errors

2011-09-05 Thread Premi, Sanjeev
Sandeep, While trying to build all omap3 boards before I could submit the patches corresponding to RFC for using generic gpio API, I came across few warnings and errors. I have already submitted a patch fixing a warning short while ago: http://marc.info/?l=u-boot&m=131521839308281&w=2 However, I

Re: [U-Boot] mx35pdk vs. mx35_3stack

2011-09-05 Thread Stefano Babic
On 09/05/2011 12:44 PM, Teemu Keskinarkaus wrote: > Hi, > Hi Teemu, > We have mx35 based hardware where I've been using U-boot from > mx35_3stack dev-board so far. It's version 2009.08. Due upgrading the > compiler the U-boot didn't compile correctly anymore so I thought I > upgrade it to latest

[U-Boot] [PATCH] UBIFS: Change ubifsload to set the filesize variable

2011-09-05 Thread Bastian Ruppert
This is the same behaviour like tftp or fatload command. Signed-off-by: Bastian Ruppert CC: kmp...@infradead.org --- fs/ubifs/ubifs.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/fs/ubifs/ubifs.c b/fs/ubifs/ubifs.c index 5a5c739..2b9d2d2 100644 --- a/fs/ubifs/ubifs

Re: [U-Boot] [PATCH] UBIFS: Change ubifsload to set the filesize variable

2011-09-05 Thread Wolfgang Denk
Dear Bastian Ruppert, In message <1315220629-9192-1-git-send-email-bastian.rupp...@sewerin.de> you wrote: > This is the same behaviour like tftp or fatload command. > @@ -719,6 +720,9 @@ int ubifs_load(char *filename, u32 addr, u32 size) > printf("Loading file '%s' to addr 0x%08x with size

[U-Boot] [PATCH v2] [COSMETIC] board/prodrive/alpr/fpga.c: Coding style cleanup

2011-09-05 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk Cc: Stefan Roese --- v2: fix compile issues; must use "do {} while (0)" in macros definitions if we want to omit the braces in if...else statements. Should have done this anyway. board/prodrive/alpr/fpga.c | 76 ++-

Re: [U-Boot] [PATCH] omap3: beagle: Fix build warning

2011-09-05 Thread Albert ARIBAUD
Hi Sanjeev, Le 05/09/2011 12:25, Sanjeev Premi a écrit : > This patch fixes the warning dure to recent changes to the board > configuration: > cmd_i2c.o cmd_i2c.c -c > cmd_i2c.c:109:1: warning: missing braces around initializer > cmd_i2c.c:109:1: warning: (near initialization for 'i2c_no_probes[0]

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

2011-09-05 Thread Fabio Estevam
On Mon, Sep 5, 2011 at 7:43 AM, Stefano Babic wrote: ... > What about to move the prototype ? For MX5/MX35 there is a sys_proto.h > (as it is done for other SOCs), in this case, well, it seems too much to > add a file for a single line. However, we could move it in clock.h and > creating a sys_pro

[U-Boot] Version 2 of 'Moving mx3fb to CONFIG_VIDEO'

2011-09-05 Thread Helmut Raiger
For details of version 2, please consult the patch history of the individual patches. -- Scanned by MailScanner. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 1/2] mx31: make HSP clock for mx3fb driver available

2011-09-05 Thread Helmut Raiger
This additionally updates mx31/generic.c by - replacing __REG() macro accesses with readl() and writel() - providing macros for PDR0 and PLL bit accesses It also fixes a warning about the prototype of imx_get_uartclk(void) Signed-off-by: Helmut Raiger --- V2: uses macros and readl(), writel(), se

[U-Boot] [PATCH 2/2] Moving mx3fb.c to CONFIG_VIDEO

2011-09-05 Thread Helmut Raiger
mx3fb.c was based on CONFIG_LCD and is moved by this patch to CONFIG_VIDEO, which has greater freedom in selecting videomodes even at runtime. This renders the accumulating list of display defines (CONFIG_DISPLAY_VBEST..., CONFIG_DISPLAY_C057...) obsolete as these may be setup through env variable

Re: [U-Boot] [PATCH] omap3: beagle: Fix build warning

2011-09-05 Thread Premi, Sanjeev
> -Original Message- > From: Albert ARIBAUD [mailto:albert.u.b...@aribaud.net] > Sent: Monday, September 05, 2011 5:06 PM > To: Premi, Sanjeev > Cc: u-boot@lists.denx.de; Heiko Schocher; Wolfgang Denk > Subject: Re: [U-Boot] [PATCH] omap3: beagle: Fix build warning > > Hi Sanjeev, > > Le

[U-Boot] My Will

2011-09-05 Thread ms wendy jones
-- I am Mrs Wendy Jones I have willed my oil and real estates investment to you, I would like you to contact my attorney and he will guide you on what to do (barristerrodneyle...@gmail.com) or call him on +234 816-138-0615 Thanks Mrs Wendy Jones ___

Re: [U-Boot] [PATCH v2] [COSMETIC] board/prodrive/alpr/fpga.c: Coding style cleanup

2011-09-05 Thread Stefan Roese
On Monday 05 September 2011 13:34:39 Wolfgang Denk wrote: > Signed-off-by: Wolfgang Denk > Cc: Stefan Roese Acked-by: Stefan Roese Again, please apply directly if you prefer. Thanks. Best regards, Stefan -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Mu

[U-Boot] [PATCH] doc/README.scrapyard: Update commit IDs or board removals

2011-09-05 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk --- doc/README.scrapyard | 50 +- 1 files changed, 25 insertions(+), 25 deletions(-) diff --git a/doc/README.scrapyard b/doc/README.scrapyard index 8da6db6..e7e1151 100644 --- a/doc/README.scrapyard +++ b/doc/README.s

[U-Boot] [PATCH] i2c_no_probes: Fix initialization warnings

2011-09-05 Thread Sanjeev Premi
This patch fixes the initialization warnings similar to the one below: cmd_i2c.o cmd_i2c.c -c cmd_i2c.c:109:1: warning: missing braces around initializer cmd_i2c.c:109:1: warning: (near initialization for 'i2c_no_probes[0]') Signed-off-by: Sanjeev Premi Cc: Albert ARIBAUD --- The problem was o

[U-Boot] [PATCH 05/10] ARM: remove broken "gcplus" board.

2011-09-05 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk Cc: Albert ARIBAUD Cc: George G. Davis --- MAINTAINERS |4 - board/gcplus/Makefile| 53 - board/gcplus/config.mk | 13 -- board/gcplus/flash.c | 438 -- board/gcplus/gcplus.c

[U-Boot] [PATCH 01/10] ARM: remove broken "SMN42" board.

2011-09-05 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk Cc: Albert ARIBAUD --- MAKEALL |1 - board/siemens/SMN42/Makefile| 51 board/siemens/SMN42/config.mk | 30 --- board/siemens/SMN42/flash.c | 475 --- board/siemens/SMN42/l

[U-Boot] [PATCH 09/10] ARM: remove broken "modnet50" board.

2011-09-05 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk Cc: Albert ARIBAUD Cc: Thomas Elste --- CREDITS|5 - MAINTAINERS|4 - MAKEALL|9 - board/modnet50/Makefile| 51 board/modnet50/config.mk | 29 --- board/modnet50/f

[U-Boot] [PATCH 03/10] ARM: remove broken "ep7312" board.

2011-09-05 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk Cc: Albert ARIBAUD Cc: Marius Gröger --- MAINTAINERS |1 - MAKEALL |1 - arch/arm/cpu/arm720t/cpu.c |6 +- arch/arm/cpu/arm720t/interrupts.c|4 +- arc

[U-Boot] [PATCH 00/10] ARM board removal, next round

2011-09-05 Thread Wolfgang Denk
So here we go - current master still (or again) has a number of broken, non-compiling ARM boards. Scrap them. Signed-off-by: Wolfgang Denk Cc: Albert ARIBAUD Cc: Alex Züpke Cc: Curt Brune Cc: George G. Davis Cc: Marius Gröger Cc: Rolf Offermanns Cc: Thomas Elste Wolfgang Denk (10): ARM

[U-Boot] [PATCH 02/10] ARM: remove broken "dnp1110" board.

2011-09-05 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk Cc: Albert ARIBAUD Cc: Alex Züpke --- MAINTAINERS |1 - board/dnp1110/Makefile| 51 - board/dnp1110/config.mk | 17 -- board/dnp1110/dnp1110.c | 67 --- board/dnp1110/flash.c | 422

[U-Boot] [PATCH 08/10] ARM: remove broken "lpc2292sodimm" board.

2011-09-05 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk Cc: Albert ARIBAUD --- MAKEALL |1 - board/lpc2292sodimm/Makefile| 51 --- board/lpc2292sodimm/config.mk | 30 board/lpc2292sodimm/flash.c | 252 --- board/lpc2292sodi

[U-Boot] [PATCH 04/10] ARM: remove broken "evb4510" board.

2011-09-05 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk Cc: Albert ARIBAUD Cc: Curt Brune --- CREDITS |1 - MAKEALL |1 - board/evb4510/Makefile| 51 board/evb4510/config.mk | 27 -- board/evb4510/evb4510.c | 64 - board/evb4510/flash.

[U-Boot] [PATCH 07/10] ARM: remove broken "lart" board.

2011-09-05 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk Cc: Albert ARIBAUD Cc: Alex Züpke --- MAINTAINERS|4 - board/lart/Makefile| 51 - board/lart/config.mk | 23 -- board/lart/flash.c | 476 board/lart/flashasm.S | 17

[U-Boot] [PATCH 06/10] ARM: remove broken "impa7" board.

2011-09-05 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk Cc: Albert ARIBAUD Cc: Marius Gröger --- MAINTAINERS |4 - MAKEALL |1 - arch/arm/cpu/arm720t/cpu.c | 29 +-- arch/arm/cpu/arm720t/interrupts.c| 14 +- ar

[U-Boot] [PATCH 10/10] ARM: remove broken "shannon" board.

2011-09-05 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk Cc: Albert ARIBAUD Cc: Rolf Offermanns --- MAINTAINERS |4 - board/shannon/Makefile| 51 - board/shannon/config.mk | 23 -- board/shannon/flash.c | 475 - board/shannon/infe

[U-Boot] Aviso Urgente!

2011-09-05 Thread Security Alert
Estimado usuario de la cuenta de correo Web, Aviso Urgente! Este mensaje es el mensaje de correo web del centro a todos los propietarios de las cuentas de correo web. Debido a la constante de velocidad de spam actualmente estamos realizando tareas de mantenimiento

[U-Boot] [PATCH v2] UBIFS: Change ubifsload to set the filesize variable

2011-09-05 Thread Bastian Ruppert
This is the same behaviour like tftp or fatload command. Signed-off-by: Bastian Ruppert CC: kmp...@infradead.org --- fs/ubifs/ubifs.c |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/fs/ubifs/ubifs.c b/fs/ubifs/ubifs.c index 5a5c739..75a2edc 100644 --- a/fs/ubifs/ubi

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

2011-09-05 Thread Marek Vasut
On Monday, September 05, 2011 01:37:17 PM Fabio Estevam wrote: > On Mon, Sep 5, 2011 at 7:43 AM, Stefano Babic wrote: > ... > > > What about to move the prototype ? For MX5/MX35 there is a sys_proto.h > > (as it is done for other SOCs), in this case, well, it seems too much to > > add a file for

Re: [U-Boot] [PATCH 1/2] mx31: make HSP clock for mx3fb driver available

2011-09-05 Thread Marek Vasut
On Monday, September 05, 2011 01:47:06 PM Helmut Raiger wrote: > This additionally updates mx31/generic.c by > - replacing __REG() macro accesses with readl() and writel() > - providing macros for PDR0 and PLL bit accesses > It also fixes a warning about the prototype of imx_get_uartclk(void) > >

Re: [U-Boot] [PATCH] UBIFS: Change ubifsload to set the filesize variable

2011-09-05 Thread Marek Vasut
On Monday, September 05, 2011 01:03:49 PM Bastian Ruppert wrote: > This is the same behaviour like tftp or fatload command. > > Signed-off-by: Bastian Ruppert > CC: kmp...@infradead.org > --- > fs/ubifs/ubifs.c |4 > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/fs/

Re: [U-Boot] [PATCH 2/2] MX27: Update to autogenerated asm-offsets.h

2011-09-05 Thread Wolfgang Denk
Dear Stefano Babic, In message <1310045827-14561-2-git-send-email-sba...@denx.de> you wrote: > On i.MX27, the asm-offsets.h file is not yet generated as it should be. > > Signed-off-by: Stefano Babic > CC: Matthias Weisser This commit breaks out-of-tree building of all i.MX27 boards: MAKEALL_

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

2011-09-05 Thread Fabio Estevam
On Mon, Sep 5, 2011 at 10:05 AM, Marek Vasut wrote: ... >> extern void mx25_fec_init_pins(void); >> extern void imx_get_mac_from_fuse(unsigned char *mac); >> extern void mx25_uart1_init_pins(void); > > Ok, this might be a stupid one, but ... > > why use externs in header files ? This will make ch

[U-Boot] [PATCH 1/2] coldfire: Fix compilation for most coldfire boards

2011-09-05 Thread Stany MARCEL
Many different patches to fix compilation of coldfire boards. Compilation could now be done in a different directory from the sources one. Simple board config switched from Makefile to boards.cfg EP2500 broken board removed from boards.cfg. Link files corrected to remove zlib.o and multiple defined

[U-Boot] [PATCH 2/2] coldfire: Fix compilation with CONFIG_SYS_DRAMSZ1 defined

2011-09-05 Thread Stany MARCEL
A temp variable was used but not declared, with CONFIG_SYS_DRAMSZ1 defined. This variable is now declared in the functione when needed. Signed-off-by: Stany MARCEL --- board/freescale/m548xevb/m548xevb.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/board/freescale/m

Re: [U-Boot] [PATCH 2/2] MX27: Update to autogenerated asm-offsets.h

2011-09-05 Thread Stefano Babic
On 09/05/2011 03:13 PM, Wolfgang Denk wrote: > Dear Stefano Babic, > > In message <1310045827-14561-2-git-send-email-sba...@denx.de> you wrote: >> On i.MX27, the asm-offsets.h file is not yet generated as it should be. >> >> Signed-off-by: Stefano Babic >> CC: Matthias Weisser > > This commit b

Re: [U-Boot] [PATCH] omap3: beagle: Fix build warning

2011-09-05 Thread Heiko Schocher
Hello Sanjeev, Premi, Sanjeev wrote: >> -Original Message- >> From: Albert ARIBAUD [mailto:albert.u.b...@aribaud.net] >> Sent: Monday, September 05, 2011 5:06 PM >> To: Premi, Sanjeev >> Cc: u-boot@lists.denx.de; Heiko Schocher; Wolfgang Denk >> Subject: Re: [U-Boot] [PATCH] omap3: beagle

Re: [U-Boot] [PATCH 2/2] MX27: Update to autogenerated asm-offsets.h

2011-09-05 Thread Wolfgang Denk
Dear Stefano Babic, In message <4e64cee5.8050...@denx.de> you wrote: > > > This commit breaks out-of-tree building of all i.MX27 boards: > > It was not exactly this one, but yes, out-of-tree building was broken > adding the asm-offsets.h. Well, that's what git bisect says... > I sent a patch to

Re: [U-Boot] [PATCH] i2c_no_probes: Fix initialization warnings

2011-09-05 Thread Heiko Schocher
Hello Sanjeev, Sanjeev Premi wrote: > This patch fixes the initialization warnings similar to > the one below: > cmd_i2c.o cmd_i2c.c -c > cmd_i2c.c:109:1: warning: missing braces around initializer > cmd_i2c.c:109:1: warning: (near initialization for 'i2c_no_probes[0]') > > Signed-off-by: Sanjeev

Re: [U-Boot] [PATCH V3] Makefile : fix generation of cpu related asm-offsets.h

2011-09-05 Thread Wolfgang Denk
Dear Stefano Babic, In message <1314726584-17203-1-git-send-email-sba...@denx.de> you wrote: > commit 0edf8b5b2fa0d210ebc4d6da0fd1aceeb7e44e47 breaks > building on a different directory with the O= parameter. > The patch wil fix this issue, generating always asm-offsets.h before > the other target

Re: [U-Boot] [PATCH v2] [COSMETIC] board/prodrive/alpr/fpga.c: Coding style cleanup

2011-09-05 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1315222479-23960-1-git-send-email...@denx.de> you wrote: > Signed-off-by: Wolfgang Denk > Cc: Stefan Roese > --- > v2: fix compile issues; must use "do {} while (0)" in macros > definitions if we want to omit the braces in if...else statements. > Should ha

Re: [U-Boot] [PATCH V3] Makefile : fix generation of cpu related asm-offsets.h

2011-09-05 Thread Wolfgang Denk
Dear Stefano, In message <20110905134419.d820f158d...@gemini.denx.de> I wrote: > > This one is not that much better: Maybe it helps debugging - here is an observation: > sed: can't read /work/wd/tmp-arm/arch/arm/cpu/arm1136/omap24xx/asm-offsets.s: > No such file or directory > sed: can't read

[U-Boot] NULL base_name for hwaddr env lookup

2011-09-05 Thread stany MARCEL
in net/eth.c there is this function call with test: if (eth_write_hwaddr(dev, NULL, eth_number)) If I understand correctly this test will always be true as NULL is used. I think it should be "eth" instead of NULL. Am I wrong ? If not I can submit a patch. Regards, Stany

[U-Boot] [PATCH v4 2/3] MX25: tx25: Avoid the usage of extern in C file

2011-09-05 Thread Fabio Estevam
Avoid the usage of extern in C file as pointed out by checkpatch. Signed-off-by: Fabio Estevam --- Changes since v3: - Place mx25_uart1_init_pins inside sys_proto.h Changes since v2: - Place the extern in the MX25 imx-regs.h instead of inside the MX27 imx-regs.h Changes since v1: - No changes ar

[U-Boot] [PATCH v4 1/3] mx25: Introduce sys_proto.h

2011-09-05 Thread Fabio Estevam
Create a sys_proto.h for MX25 as done on other MX25 processors. Signed-off-by: Fabio Estevam --- Changes since v3: - No changes. This patch was introduced in v4 arch/arm/cpu/arm926ejs/mx25/generic.c |1 + arch/arm/include/asm/arch-mx25/imx-regs.h |4 arch/arm/include/asm/arch

[U-Boot] [PATCH v4 3/3] MX25: Add initial support for MX25PDK

2011-09-05 Thread Fabio Estevam
Add the initial support for MX25PDK booting from SD card via internal boot. Signed-off-by: Fabio Estevam --- Changes since v3: - No changes Changes since v2: - No changes Changes since v1: - Keep mx25pdk entry to boards.cfg sorted - Removed unused defines in mx25pdk.h - Do not use the whole RAM s

Re: [U-Boot] [PATCH] sf: winbond: Add support for the Winbond W25X40

2011-09-05 Thread Wolfgang Denk
Dear James Le Cuirot, In message <1311014692-1420-1-git-send-email-ch...@aura-online.co.uk> you wrote: > The Winbond W25X40 is now being used in the IP02 (and possibly IP04). > Tested and working on the actual device. > > Signed-off-by: James Le Cuirot > --- > drivers/mtd/spi/winbond.c |8 +

Re: [U-Boot] [PATCH] Correct call to eth_write_hwaddr()

2011-09-05 Thread Wolfgang Denk
Dear Simon Glass, In message <1314719460-27858-1-git-send-email-...@chromium.org> you wrote: > This fixes "Warning: failed to set MAC address" on platforms which rely on > an 'ethaddr' environment variable to set the MAC address. > > This bug was introduced by this commit: > > 7616e785 Add Ether

Re: [U-Boot] [PATCH] arm: Add Prep subcommand support to bootm

2011-09-05 Thread Simon Schwarz
Dear Andreas, On 09/05/2011 12:58 PM, Andreas Bießmann wrote: > Dear Simon, > > Am Mo 29 Aug 2011 18:08:13 CEST, Simon Schwarz schrieb: >> Adds prep subcommand to bootm implementation of ARM. When bootm is called >> with >> the subcommand prep the function stops right after ATAGS creation and bef

Re: [U-Boot] [PATCH] Flush cache after the OS image is loaded into the memory.

2011-09-05 Thread Wolfgang Denk
Dear Kumar Gala, In message <1314794723-25547-1-git-send-email-ga...@kernel.crashing.org> you wrote: > From: Diana CRACIUN > > Since we are loading an executable image into memory we need flush it > out of the cache to possible maintain coherence on CPUs with split > instruction and data caches

Re: [U-Boot] [PATCH 2/2] powerpc/85xx: standardize display of address map size (32-bit vs. 36-bit)

2011-09-05 Thread Wolfgang Denk
Dear Timur Tabi, In message <4e6002b4.90...@freescale.com> you wrote: > > > What sort of "confusion" do you have? I see two situations: in > > 99.99% of all cases U-Boot is just a means to boot an OS, and nobody > > cares a bit about the actual U-Boot output, as long as the OS is > > runnign aft

Re: [U-Boot] [PATCH 2/2] powerpc/85xx: standardize display of address map size (32-bit vs. 36-bit)

2011-09-05 Thread Wolfgang Denk
Dear Kumar Gala, In message <6a841e78-5d44-4190-9899-1976234ef...@freescale.com> you wrote: > > The FSL PPC board ports are not the typical use case. I view our board > ports having a few different purposes: > * Examples for people building their own boards Especially of this you have addition

Re: [U-Boot] [PATCH 2/2] powerpc/85xx: standardize display of address map size (32-bit vs. 36-bit)

2011-09-05 Thread Wolfgang Denk
Dear Timur Tabi, In message <4e60dcec.9090...@freescale.com> you wrote: > Kumar Gala wrote: > > Both. I'm think for your patch we'd add some general config option for > > extra print info. > > So you want to see this instead: > > /* > * Display whether this is a 32-bit build or a 36-bit build

Re: [U-Boot] NULL base_name for hwaddr env lookup

2011-09-05 Thread Stany MARCEL
On Mon, Sep 5, 2011 at 3:53 PM, stany MARCEL wrote: > in net/eth.c there is this function call with test: > >  if (eth_write_hwaddr(dev, NULL, eth_number)) > > If I understand correctly this test will always be true as NULL is used. > > I think it should be "eth" instead of NULL. > > Am I wrong ?

Re: [U-Boot] [PATCH] UBIFS: Change ubifsload to set the filesize variable

2011-09-05 Thread Wolfgang Denk
Dear Marek Vasut, In message <201109051512.48148.marek.va...@gmail.com> you wrote: > > > + setenv("filesize", buf); > > maybe you want to check set_local_var() too ? Yes, we should clean up all these auto-generated environment variables one day. But what you propose here is not a good approac

Re: [U-Boot] [PATCH V3] Makefile : fix generation of cpu related asm-offsets.h

2011-09-05 Thread Stefano Babic
On 09/05/2011 03:52 PM, Wolfgang Denk wrote: > Dear Stefano, > Hi Wolfgang, > In message <20110905134419.d820f158d...@gemini.denx.de> I wrote: >> >> This one is not that much better: > > Maybe it helps debugging - here is an observation: > >> sed: can't read >> /work/wd/tmp-arm/arch/arm/cpu/ar

[U-Boot] [PATCH V4] Makefile : fix generation of cpu related asm-offsets.h

2011-09-05 Thread Stefano Babic
commit 0edf8b5b2fa0d210ebc4d6da0fd1aceeb7e44e47 breaks building on a different directory with the O= parameter. The patch wil fix this issue, generating always asm-offsets.h before the other targets. Signed-off-by: Stefano Babic CC: Matthias Weisser CC: Wolfgang Denk --- Note: V2 breaks *all*

Re: [U-Boot] NULL base_name for hwaddr env lookup

2011-09-05 Thread Heiko Schocher
Hello MARCEL, stany MARCEL wrote: > in net/eth.c there is this function call with test: > > if (eth_write_hwaddr(dev, NULL, eth_number)) > > If I understand correctly this test will always be true as NULL is used. > > I think it should be "eth" instead of NULL. > > Am I wrong ? No. > If not

Re: [U-Boot] UBI problems on current u-bo

2011-09-05 Thread Stefan Roese
Hi Holger, On Friday 02 September 2011 15:32:40 Holger Brunck wrote: > I address this question to you because one of your commits is connected to > this problem, but other hints from other readers are also welcome ;-) . I'll try to look into this later this week. BTW: Is this problem reproducibl

Re: [U-Boot] [PATCH] omap3: beagle: Fix build warning

2011-09-05 Thread Premi, Sanjeev
> -Original Message- > From: Heiko Schocher [mailto:h...@denx.de] > Sent: Monday, September 05, 2011 7:01 PM > To: Premi, Sanjeev > Cc: Albert ARIBAUD; u-boot@lists.denx.de; Wolfgang Denk > Subject: Re: [U-Boot] [PATCH] omap3: beagle: Fix build warning > > Hello Sanjeev, > > Premi, Sanje

[U-Boot] [PATCH V2] arm: Add Prep subcommand support to bootm

2011-09-05 Thread Simon Schwarz
Adds prep subcommand to bootm implementation of ARM. When bootm is called with the subcommand prep the function stops right after ATAGS creation and before announce_and_cleanup. This is used in command "cmd_spl export" Signed-off-by: Simon Schwarz --- V2 changes: nothing V3 changes: nothing c

Re: [U-Boot] [PATCH] i2c_no_probes: Fix initialization warnings

2011-09-05 Thread Premi, Sanjeev
> -Original Message- > From: Heiko Schocher [mailto:h...@denx.de] > Sent: Monday, September 05, 2011 7:13 PM > To: Premi, Sanjeev > Cc: u-boot@lists.denx.de; Albert ARIBAUD > Subject: Re: [U-Boot] [PATCH] i2c_no_probes: Fix > initialization warnings > > Hello Sanjeev, [snip]...[snip] >

Re: [U-Boot] [PATCH] UBIFS: Change ubifsload to set the filesize variable

2011-09-05 Thread Marek Vasut
On Monday, September 05, 2011 04:27:28 PM Wolfgang Denk wrote: > Dear Marek Vasut, > > In message <201109051512.48148.marek.va...@gmail.com> you wrote: > > > + setenv("filesize", buf); > > > > maybe you want to check set_local_var() too ? > > Yes, we should clean up all these auto-generated envi

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

2011-09-05 Thread Marek Vasut
On Monday, September 05, 2011 03:15:44 PM Fabio Estevam wrote: > On Mon, Sep 5, 2011 at 10:05 AM, Marek Vasut wrote: > ... > > >> extern void mx25_fec_init_pins(void); > >> extern void imx_get_mac_from_fuse(unsigned char *mac); > >> extern void mx25_uart1_init_pins(void); > > > > Ok, this might

Re: [U-Boot] [PATCH] arm: Add Prep subcommand support to bootm

2011-09-05 Thread Andreas Bießmann
Dear Simon, Am 05.09.2011 16:06, schrieb Simon Schwarz: > Dear Andreas, > > On 09/05/2011 12:58 PM, Andreas Bießmann wrote: >> Dear Simon, >> >> Am Mo 29 Aug 2011 18:08:13 CEST, Simon Schwarz schrieb: >> >>> +} >>> + >>> +/* Main Entry point for arm bootm implementation >>> + * >>> + * Modeled

[U-Boot] [PATCH] OMAP3 Beagle: Minor config cleanup

2011-09-05 Thread s-paulraj
From: Sandeep Paulraj This patch removes a hardcoded MAC address Signed-off-by: Sandeep Paulraj --- include/configs/omap3_beagle.h |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/include/configs/omap3_beagle.h b/include/configs/omap3_beagle.h index 18c6deb..57d4bb3 10

[U-Boot] [PATCH] da8xxevm: Fix warning: unused variable 'val'

2011-09-05 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk Cc: Albert ARIBAUD Cc: Sandeep Paulraj Cc: Sudhakar Rajashekhara --- board/davinci/da8xxevm/da850evm.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/board/davinci/da8xxevm/da850evm.c b/board/davinci/da8xxevm/da850evm.c index 2f950e7..

Re: [U-Boot] omap3: gpio: build warnings and errors

2011-09-05 Thread Paulraj, Sandeep
> > However, I couldn't find the definition of omap_get_gpio_dataout() > used in board/ti/beagle/led.c. > > The change was introduced in: > b8bc8973: led: Remove state-saving of led for... > > After much search, I did find a patch that contains the defintion > of omap_get_gpio_dataout() subm

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

2011-09-05 Thread Fabio Estevam
On Mon, Sep 5, 2011 at 12:05 PM, Marek Vasut wrote: > On Monday, September 05, 2011 03:15:44 PM Fabio Estevam wrote: >> On Mon, Sep 5, 2011 at 10:05 AM, Marek Vasut wrote: >> ... >> >> >> extern void mx25_fec_init_pins(void); >> >> extern void imx_get_mac_from_fuse(unsigned char *mac); >> >> exte

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

2011-09-05 Thread Marek Vasut
On Monday, September 05, 2011 05:38:26 PM Fabio Estevam wrote: > On Mon, Sep 5, 2011 at 12:05 PM, Marek Vasut wrote: > > On Monday, September 05, 2011 03:15:44 PM Fabio Estevam wrote: > >> On Mon, Sep 5, 2011 at 10:05 AM, Marek Vasut > >> wrote: ... > >> > >> >> extern void mx25_fec_init_pins(vo

Re: [U-Boot] UBI problems on current u-bo

2011-09-05 Thread Holger Brunck
Hi Stefan, On 09/05/2011 04:37 PM, Stefan Roese wrote: >> I address this question to you because one of your commits is connected to >> this problem, but other hints from other readers are also welcome ;-) . > > I'll try to look into this later this week. > > BTW: Is this problem reproducible on

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

2011-09-05 Thread Fabio Estevam
On Mon, Sep 5, 2011 at 12:40 PM, Marek Vasut wrote: ... >> ./scripts/checkpatch.pl -F u-boot/board/karo/tx25/tx25.c >> >> WARNING: externs should be avoided in .c files >> #144: FILE: home/fabio/denx/u-boot/board/karo/tx25/tx25.c:144: >> +     extern void mx25_uart1_init_pins(void); > > But

[U-Boot] [PATCH] omap24xx: fix 'reset_timer_masked' declaration error

2011-09-05 Thread Wolfgang Denk
Commit 17659d7 "Timer: Remove reset_timer_masked()" introduced a static declaration for reset_timer_masked() which causes build errors: timer.c:45: error: static declaration of 'reset_timer_masked' follows non-static declaration include/asm/u-boot-arm.h:70: error: previous declaration of 'reset_

Re: [U-Boot] UBI problems on current u-boot

2011-09-05 Thread Holger Brunck
Hi Stefan, sorry for the previous mail, but I hit the send button too fast ;-) On 09/05/2011 04:37 PM, Stefan Roese wrote: > > On Friday 02 September 2011 15:32:40 Holger Brunck wrote: >> I address this question to you because one of your commits is connected to >> this problem, but other hints f

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

2011-09-05 Thread Wolfgang Denk
Dear Fabio Estevam, In message you wrote: > > > But you're using extern in _header_ (.h) file ... so ... why ? > > > > My patch removes the extern from the C file and put it on a header > file with other extern's. What Marek probably wants to tell yet (in a bit a complicated way) is that you s

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

2011-09-05 Thread Marek Vasut
On Monday, September 05, 2011 06:02:21 PM Wolfgang Denk wrote: > Dear Fabio Estevam, > > In message you wrote: > > > But you're using extern in _header_ (.h) file ... so ... why ? > > > > My patch removes the extern from the C file and put it on a header > > file with other extern's. > > What

  1   2   >