[U-Boot] [PATCH] powerpc/mpc83xx: set TXEQA/TXEQE value for mpc837XE sata

2011-11-16 Thread Chang-Ming.Huang
From: Jerry Huang In the current u-boot code, the value of these fields are the reserved value (0b100), through the signal integrity measurement on freescale's board with these reserved setting, the signal eye is out of the recommended spec for non-transition amplitude at 500mV nominal. According

[U-Boot] [PATCH 3/3] mpc83xx: km83xx_i2c - fix gcc 4.6 compiler warning

2011-11-16 Thread Kim Phillips
Configuring for kmeter1 board... km83xx_i2c.c: In function 'i2c_make_abort': km83xx_i2c.c:44:8: warning: variable 'dummy' set but not used [-Wunused-but-set-variable] Signed-off-by: Kim Phillips --- board/keymile/km83xx/km83xx_i2c.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-)

[U-Boot] [PATCH 1/3] mpc83xx: spd_sdram - fix gcc 4.6 compiler warning

2011-11-16 Thread Kim Phillips
Configuring for sbc8349 board... spd_sdram.c: In function 'spd_sdram': spd_sdram.c:152:41: warning: variable 'trfc_high' set but not used [-Wunused-but-set-variable] Signed-off-by: Kim Phillips --- arch/powerpc/cpu/mpc83xx/spd_sdram.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-)

[U-Boot] [PATCH 2/3] mpc83xx: mpc8360emds - fix gcc 4.6 compiler warning

2011-11-16 Thread Kim Phillips
Configuring for MPC8360EMDS_66_HOST_33 - Board: MPC8360EMDS, Options: CLKIN_66MHZ,PCI,PCI_33M,PQ_MDS_PIB=1 mpc8360emds.c: In function 'board_eth_init': mpc8360emds.c:178:12: warning: array subscript is above array bounds [-Warray-bounds] Signed-off-by: Kim Phillips --- board/freescale/mpc8360e

[U-Boot] [PATCH] common: tsi148 - fix gcc 4.6 compiler warning

2011-11-16 Thread Kim Phillips
Configuring for vme8349 board... cmd_tsi148.c: In function 'tsi148_init': cmd_tsi148.c:56:17: warning: variable 'lastError' set but not used [-Wunused-but-set-variable] Signed-off-by: Kim Phillips --- common/cmd_tsi148.c |7 ++- 1 files changed, 2 insertions(+), 5 deletions(-) diff --g

Re: [U-Boot] [PATCH 3/3] mpc83xx: km83xx_i2c - fix gcc 4.6 compiler warning

2011-11-16 Thread Heiko Schocher
Hello Kim, Kim Phillips wrote: > Configuring for kmeter1 board... > km83xx_i2c.c: In function 'i2c_make_abort': > km83xx_i2c.c:44:8: warning: variable 'dummy' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Kim Phillips > --- > board/keymile/km83xx/km83xx_i2c.c |3 +-- >

Re: [U-Boot] [PATCH] powerpc/mpc83xx: set TXEQA/TXEQE value for mpc837XE sata

2011-11-16 Thread Kim Phillips
On Wed, 16 Nov 2011 15:36:25 +0800 wrote: > From: Jerry Huang > > In the current u-boot code, the value of these fields are the reserved > value (0b100), through the signal integrity measurement on freescale's > board with these reserved setting, the signal eye is out of the recommended > spec

Re: [U-Boot] [PATCH v2] netspace_v2: Read Ethernet MAC address from EEPROM

2011-11-16 Thread Simon Guinot
Hi Prafulla, On Tue, Nov 08, 2011 at 10:31:14PM +0100, Simon Guinot wrote: > From: Simon Guinot > > Signed-off-by: Simon Guinot > --- > > Changes in v2: > - fix asm-offsets.s removal > - make MAC address setup conditional to CONFIG_SYS_I2C_EEPROM_ADDR > > board/LaCie/netspace_v2/netspace_v

[U-Boot] [PATCH v3] AT91: pio: Add PIO3 features

2011-11-16 Thread Hong Xu
This patch adds the support for new PIO controller introduced by some AT91 SoCs. New features include * More peripheral multiplexing * Pull-down, Schmitt trigger, Debouncer * More irq trigger mode (may be not interesting in U-Boot) Signed-off-by: Hong Xu --- Change for v3 * CONFIG_CPU_HAS_PIO

Re: [U-Boot] [PATCH 6/6] i.mx: i.mx6q: Add the initial support for i.mx6q ARM2 board

2011-11-16 Thread Dirk Behme
On 12.11.2011 11:36, Jason Liu wrote: > Add the initial support for Freescale i.MX6Q Armadillo2 board > Support: MMC boot from slot 0/1, debug UART(UART4), usdhc. ... > diff --git a/include/configs/mx6qarm2.h b/include/configs/mx6qarm2.h > new file mode 100644 > index 000..a4c7b31 > --- /dev/nu

Re: [U-Boot] [PATCH 0/6] i.mx: add the initial i.mx6q core/board support

2011-11-16 Thread Dirk Behme
On 12.11.2011 11:36, Jason Liu wrote: > This patch-set add the initial support for freescale i.mx6q support. > freescale i.mx6q is a quad core built on arm cortex_a9 complex. With the mx6qarm2 config changes proposed some minutes ago: Tested-by: Dirk Behme Thanks Dirk > The patch-set has been

Re: [U-Boot] [PATCH 6/6] i.mx: i.mx6q: Add the initial support for i.mx6q ARM2 board

2011-11-16 Thread Jason Hui
On Wed, Nov 16, 2011 at 7:34 PM, Dirk Behme wrote: > On 12.11.2011 11:36, Jason Liu wrote: >> >> Add the initial support for Freescale i.MX6Q Armadillo2 board >> Support: MMC boot from slot 0/1, debug UART(UART4), usdhc. > > ... >> >> diff --git a/include/configs/mx6qarm2.h b/include/configs/mx6qa

Re: [U-Boot] [PATCH v2] netspace_v2: Read Ethernet MAC address from EEPROM

2011-11-16 Thread Prafulla Wadaskar
> -Original Message- > From: Simon Guinot [mailto:si...@sequanux.org] > Sent: Wednesday, November 16, 2011 4:02 PM > To: Prafulla Wadaskar > Cc: u-boot@lists.denx.de > Subject: Re: [U-Boot] [PATCH v2] netspace_v2: Read Ethernet MAC > address from EEPROM > > Hi Prafulla, > > On Tue, Nov

[U-Boot] Possible fix for dns command

2011-11-16 Thread Mathias Adam
Hello, I'm currently using u-boot on a TI DM816x evaluation board (u-boot as included in TI's EZSDK 5.02). I manually enabled CONFIG_CMD_DNS. However, running dhcp followed by dns only gave timeouts. I found (using #define DEBUG) that DNS UDP queries are being sent to the MAC of the DHCP server

Re: [U-Boot] [RFC PATCH 4/8] arm: printf() is not available in the SPL

2011-11-16 Thread Tom Rini
On Wed, Nov 16, 2011 at 12:37 AM, Christian Riesch wrote: > Hi Tom, > thanks for your comments! > > On Tue, Nov 15, 2011 at 6:50 PM, Tom Rini wrote: >> On Tue, Nov 15, 2011 at 3:37 AM, Christian Riesch >> wrote: >>> >>> Signed-off-by: Christian Riesch >>> Cc: Wolfgang Denk >>> --- >>>  arch/ar

[U-Boot] Please pull u-boot-ti/master

2011-11-16 Thread s-paulraj
The following changes since commit 75acc4d7c1c9081e06d1197c6da01361cf1bce92: Heiko Schocher (1): arm, davinci: add DAVINCI_MMC_CLKID are available in the git repository at: git://git.denx.de/u-boot-ti.git master Sanjeev Premi (2): part_efi: Fix compile errors omap3evm: Ad

Re: [U-Boot] [PATCH 09/12] drivers/i2c/davinci_i2c.c: Fix GCC 4.6 warning

2011-11-16 Thread Tom Rini
On Tue, Nov 15, 2011 at 4:21 PM, Anatolij Gustschin wrote: > Fix: > davinci_i2c.c: In function 'flush_rx': > davinci_i2c.c:81:6: warning: variable 'dummy' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Anatolij Gustschin > Cc: Heiko Schocher > --- >  drivers/i2c/davinci_i2c.c

Re: [U-Boot] [PATCH 04/12] drivers/net/enc28j60.c: Fix GCC 4.6 warnings

2011-11-16 Thread Mike Frysinger
On Tuesday 15 November 2011 18:20:55 Anatolij Gustschin wrote: > - eir_reg = enc_r8(enc, CTL_REG_EIR); > + (void)enc_r8(enc, CTL_REG_EIR); is that (void) actually necessary ? last i looked, gcc didn't actually care. which means you should drop the (void). -mike signatu

Re: [U-Boot] __bss_end__ and the like

2011-11-16 Thread Mike Frysinger
On Wednesday 16 November 2011 01:02:50 Simon Glass wrote: > grep -rs __bss_end__ * |wc > 2151011 11552 > grep -rs "__bss_end " * |wc > 39 1952175 > > I think we should standardise on one or other of these. To me it feels > like __bss_end is correct, since we have __bss_start

Re: [U-Boot] [PATCH 1/2] Fix warnings in arch/arm/cpu/arm920t/s3c24x0/usb_ohci.c

2011-11-16 Thread Mike Frysinger
On Tuesday 15 November 2011 23:17:06 Simon Glass wrote: > --- a/arch/arm/cpu/arm920t/s3c24x0/usb_ohci.c > +++ b/arch/arm/cpu/arm920t/s3c24x0/usb_ohci.c > > - __u32 datab[4]; > - __u8 *data_buf = (__u8 *) datab; > + union { > + __u32 word[4]; > + __u16 hword[8]; >

Re: [U-Boot] [PATCH v2 2/3] tegra2: Add Avionic Design Plutux support.

2011-11-16 Thread Stephen Warren
Thierry Reding wrote at Wednesday, November 16, 2011 12:14 AM: > * Thierry Reding wrote: > > * Simon Glass wrote: > [...] > > > > > > If you are not implementing MMC, just remove it and all the MMC code > > > below. > > > > > > > I don't understand. What makes you think I don't want MMC? Both the

Re: [U-Boot] [PATCH] tegra2: Change CONFIG_SYS_TEXT_BASE to 0x00108000.

2011-11-16 Thread Stephen Warren
Thierry Reding wrote at Tuesday, November 15, 2011 11:49 PM: > * PGP Signed by an unknown key > > * Stephen Warren wrote: > > Thierry Reding wrote at Tuesday, November 15, 2011 12:19 AM: > > > NVIDIA's flashing tools assume that the bootloader is loaded at address > > > 0x00108000. Instead of requ

Re: [U-Boot] [PATCH v2 2/3] tegra2: Add Avionic Design Plutux support.

2011-11-16 Thread Tom Warren
Thierry, -Original Message- From: Thierry Reding [mailto:thierry.red...@avionic-design.de] Sent: Wednesday, November 16, 2011 12:14 AM To: Simon Glass Cc: u-boot@lists.denx.de; Tom Warren Subject: Re: [U-Boot] [PATCH v2 2/3] tegra2: Add Avionic Design Plutux support. * PGP Signed by an u

Re: [U-Boot] [PATCH] tegra2: Change CONFIG_SYS_TEXT_BASE to 0x00108000.

2011-11-16 Thread Thierry Reding
* Stephen Warren wrote: > Thierry Reding wrote at Tuesday, November 15, 2011 11:49 PM: > > * PGP Signed by an unknown key > > > > * Stephen Warren wrote: > > > Thierry Reding wrote at Tuesday, November 15, 2011 12:19 AM: > > > > NVIDIA's flashing tools assume that the bootloader is loaded at addre

Re: [U-Boot] [PATCH v2 2/3] tegra2: Add Avionic Design Plutux support.

2011-11-16 Thread Thierry Reding
* Stephen Warren wrote: > Thierry Reding wrote at Wednesday, November 16, 2011 12:14 AM: > > * Thierry Reding wrote: > > > * Simon Glass wrote: > > [...] > > > > > > > > If you are not implementing MMC, just remove it and all the MMC code > > > > below. > > > > > > > > > > I don't understand. What

[U-Boot] [STATUS] Custodians - please lend a hand

2011-11-16 Thread Wolfgang Denk
Hm... it seems a subject as "Help needed - urgently" ends in too many spam filters, so I'm reposting. This is actually serious. In message <2015150119.1f8e41689...@gemini.denx.de> I wrote: > > Hello all, > > I guess most of you will already have noticed that my activity on the > mailing li

Re: [U-Boot] Please pull u-boot-mmc.git

2011-11-16 Thread Wolfgang Denk
Dear Andy Fleming, In message <1321134885-20735-1-git-send-email-aflem...@freescale.com> you wrote: > The following changes since commit 5721385b187b3154c7768e6c182501022f4e2e45: > > Merge branch 'master' of git://git.denx.de/u-boot-mpc83xx (2011-11-08 > 07:44:52 +0100) > > are available in t

Re: [U-Boot] [i2c] Pull request

2011-11-16 Thread Wolfgang Denk
Dear Heiko Schocher, In message <4eba1018.9030...@denx.de> you wrote: > Hello Wolfgang, > > The following changes since commit 5721385b187b3154c7768e6c182501022f4e2e45: > > Merge branch 'master' of git://git.denx.de/u-boot-mpc83xx (2011-11-08 > 07:44:52 +0100) > > are available in the git re

Re: [U-Boot] Pull request: u-boot-nds32

2011-11-16 Thread Wolfgang Denk
Dear Macpaul Lin, In message you wrote: > Dear Wolfgang, > > Could you please pull these 5 patches to your tree. > > Thanks, > Macpaul > > > The following changes since commit 5721385b187b3154c7768e6c182501022f4e2e45: > > Merge branch 'master' of git://git.denx.de/u-boot-mpc83xx > (2011-1

[U-Boot] Auto insurance (11/16/2011)

2011-11-16 Thread Dave Gilmore
Hello, Our company represents an online platform of over 36.4 million people, most of which are located in the US and Canada. What we do allows us to present our online platform with a first choice when they search for anything on any of the major search engines. We seek a preferred source to sen

Re: [U-Boot] [GIT PULL] Pull request u-boot-85xx.git

2011-11-16 Thread Wolfgang Denk
Dear Kumar Gala, In message you wrote: > The following changes since commit 5721385b187b3154c7768e6c182501022f4e2e45: > > Merge branch 'master' of git://git.denx.de/u-boot-mpc83xx (2011-11-08 > 07:44:52 +0100) > > are available in the git repository at: > > git://git.denx.de/u-boot-mpc85

Re: [U-Boot] Pull request: u-boot-fdt

2011-11-16 Thread Wolfgang Denk
Dear Jerry Van Baren, In message <4ebff810.6080...@cideas.com> you wrote: > Dear Wolfgang, > > Please pull the bugfixes from the following changes since commit > 5721385b187b3154c7768e6c182501022f4e2e45: > >Merge branch 'master' of git://git.denx.de/u-boot-mpc83xx (2011-11-08 > 07:44:52 +0

Re: [U-Boot] Pull request: u-boot-arm/master

2011-11-16 Thread Wolfgang Denk
Dear Simon Glass, In message you wrote: > > We have built up quite a large number of patches which are currently > sitting in the Chromium tree - something like 200. If we continue to > dribble these out at 10-20 a month it will take a while. Current > mainline U-Boot support for Tegra is prett

[U-Boot] [PATCH] powerpc/85xx: verify the localbus device tree address before booting the OS

2011-11-16 Thread Timur Tabi
The localbus controller node in the device tree is typically a root node, even though the controller is part of CCSR. If we were to put the lbc node under the SOC node, then the 'ranges' property in the lbc node would translate through the 'ranges' property of the parent SOC node, and we don't wan

Re: [U-Boot] [STATUS] Custodians - please lend a hand

2011-11-16 Thread Stefano Babic
On 11/16/2011 07:51 PM, Wolfgang Denk wrote: Wolfgang, >> >> Please let's try if this works. If you have any suggestions how to >> help better, please don't hesitate to tell us. I have tried with a couple of patches (network related), and then I wanted to test if push works. As u-boot-staging

Re: [U-Boot] Pull request: u-boot-arm/master

2011-11-16 Thread Wolfgang Denk
Dear Albert ARIBAUD, In message <4ec2d979.6090...@aribaud.net> you wrote: > Hi Wolfgang, > > The following changes since commit 5721385b187b3154c7768e6c182501022f4e2e45: > >Merge branch 'master' of git://git.denx.de/u-boot-mpc83xx (2011-11-08 > 07:44:52 +0100) > > are available in the git

Re: [U-Boot] Pull request: u-boot-video/master

2011-11-16 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <2016000607.43a91f9b@wker> you wrote: > Hello Wolfgang, > > The following changes since commit 5721385b187b3154c7768e6c182501022f4e2e45: > > Merge branch 'master' of git://git.denx.de/u-boot-mpc83xx (2011-11-08 > 07:44:52 +0100) > > are available in th

Re: [U-Boot] [STATUS] Custodians - please lend a hand

2011-11-16 Thread Wolfgang Denk
Dear Stefano Babic, In message <4ec4102d.2020...@denx.de> you wrote: > > I have tried with a couple of patches (network related), and then I > wanted to test if push works. As u-boot-staging should be open for all > custodians, I was expecting I should use the same mechanism, and then I > tried

Re: [U-Boot] [PATCH] FPGA: drivers/fpga/ivm_core.c: GCC4.6 fix build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefano Babic, In message <1320576159-1921-1-git-send-email-sba...@denx.de> you wrote: > Fix: > ivm_core.c: In function 'ispVMLCOUNT': > ivm_core.c:2105:16: warning: unused variable 'usByte' > > Signed-off-by: Stefano Babic > CC: Wolfgang Denk > --- > drivers/fpga/ivm_core.c |2 -- >

Re: [U-Boot] [PATCH] davinci, mmc: fix gcc 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Heiko Schocher, In message <1320038153-24364-1-git-send-email...@denx.de> you wrote: > Fix: > davinci_mmc.c: In function 'dmmc_wait_fifo_status': > davinci_mmc.c:72:7: warning: variable 'mmcstatus1' set but not used > [-Wunused-but-set-variable] > davinci_mmc.c: In function 'dmmc_busy_wait':

Re: [U-Boot] [PATCH 1/3] drivers/block/ahci.c: Fix GCC 4.6 build warning

2011-11-16 Thread Wolfgang Denk
Dear Kumar Gala, In message <1320855670-7289-1-git-send-email-ga...@kernel.crashing.org> you wrote: > Fix: > > ahci.c: In function 'ata_scsiop_read10': > ahci.c:564:6: warning: variable 'lba' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Kumar Gala > --- > drivers/block/a

Re: [U-Boot] [u-boot-release] [PATCH] powerpc/85xx: verify the localbus device tree address before booting the OS

2011-11-16 Thread Swarthout Edward L-SWARTHOU
> ... it's possible for the 'reg' property to be wrong. Instead of all this, why doesn't u-boot just fix it? I have the same annoyance with ccsrbar. U-boot controls the ccsrbar address via a user configured #define. So instead of having different dts's based on the u-boot ccsrbar, why can't u-bo

Re: [U-Boot] [PATCH 2/3] drivers/bios_emulator/x86emu/ops.c: Fix GCC 4.6 build warning

2011-11-16 Thread Wolfgang Denk
Dear Kumar Gala, In message <1320855670-7289-2-git-send-email-ga...@kernel.crashing.org> you wrote: > Fix: > > x86emu/ops.c: In function 'x86emuOp_int3': > x86emu/ops.c:3521:9: warning: variable 'tmp' set but not used > [-Wunused-but-set-variable] > x86emu/ops.c: In function 'x86emuOp_int_IMM':

Re: [U-Boot] [u-boot-release] [PATCH] powerpc/85xx: verify the localbus device tree address before booting the OS

2011-11-16 Thread Timur Tabi
Swarthout Edward L-SWARTHOU wrote: >> ... it's possible for the 'reg' property to be wrong. > > Instead of all this, why doesn't u-boot just fix it? > > I have the same annoyance with ccsrbar. > > U-boot controls the ccsrbar address via a user configured #define. > So instead of having different

Re: [U-Boot] [PATCH 3/3] drivers/block/sata_sil3114.c: Fix GCC 4.6 build warning

2011-11-16 Thread Wolfgang Denk
Dear Kumar Gala, In message <1320855670-7289-3-git-send-email-ga...@kernel.crashing.org> you wrote: > Fix: > > sata_sil3114.c: In function 'scan_sata': > sata_sil3114.c:793:7: warning: array subscript is above array bounds > [-Warray-bounds] > sata_sil3114.c:794:7: warning: array subscript is a

Re: [U-Boot] [PATCH 01/13] drivers/block/sym53c8xx.c: Fix GCC 4.6 build warning

2011-11-16 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320866946-28235-1-git-send-email...@denx.de> you wrote: > Fix: > sym53c8xx.c: In function 'scsi_write_dsp': > sym53c8xx.c:456:16: warning: variable 'val' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > drivers/block/sym53c

Re: [U-Boot] [PATCH 02/13] board/evb64260/i2c.c: Coding Style cleanup

2011-11-16 Thread Wolfgang Denk
Dear Heiko Schocher, In message <4ebb6ff4.5060...@denx.de> you wrote: > > > Make checkpatch-clean ... > > - unsigned int actualFreq, actualN=0, actualM=0; > > + unsigned int actualFreq, actualN = 0, actualM = 0; > > Hmm.. we don't use Upper and Lower case mixed for variable names. > This sho

Re: [U-Boot] [PATCH 02/13] board/evb64260/i2c.c: Coding Style cleanup

2011-11-16 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320866946-28235-2-git-send-email...@denx.de> you wrote: > Make checkpatch-clean > > Signed-off-by: Wolfgang Denk > Cc: Eran Man > --- > board/evb64260/i2c.c | 48 ++-- > 1 files changed, 22 insertions(+), 26 deletio

Re: [U-Boot] [u-boot-release] [PATCH] powerpc/85xx: verify the localbus device tree address before booting the OS

2011-11-16 Thread Scott Wood
On 11/16/2011 01:58 PM, Swarthout Edward L-SWARTHOU wrote: >> ... it's possible for the 'reg' property to be wrong. > > Instead of all this, why doesn't u-boot just fix it? > > I have the same annoyance with ccsrbar. > > U-boot controls the ccsrbar address via a user configured #define. > So ins

Re: [U-Boot] [PATCH 03/13] board/evb64260/i2c.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320866946-28235-3-git-send-email...@denx.de> you wrote: > Fix: > i2c.c: In function 'i2c_init': > i2c.c:23:15: warning: variable 'actualFreq' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > Cc: Eran Man > --- > board/evb64260/i

Re: [U-Boot] [PATCH 04/13] board/evb64260/evb64260.c: Fix GC 4.6 build warning

2011-11-16 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320866946-28235-4-git-send-email...@denx.de> you wrote: > Fix: > evb64260.c: In function 'debug_led': > evb64260.c:363:6: warning: variable 'dummy' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > board/evb64260/evb64260.c

Re: [U-Boot] [PATCH 05/13] board/evb64260/eth.c: Fix GCC 4.6 build warning

2011-11-16 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320866946-28235-5-git-send-email...@denx.de> you wrote: > Fix: > eth.c: In function 'gt6426x_handle_SMI': > eth.c:130:15: warning: variable 'psr' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > board/evb64260/eth.c | 37

Re: [U-Boot] [PATCH 06/13] board/evb64260/sdram_init.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320866946-28235-6-git-send-email...@denx.de> you wrote: > Fix: > sdram_init.c: In function 'setup_sdram_common': > sdram_init.c:333:49: warning: variable 'ecc' set but not used > [-Wunused-but-set-variable] > sdram_init.c: In function 'setup_sdram': > sdram_init.c:

Re: [U-Boot] [PATCH 07/13] board/evb64260/zuma_pbb_mbox.c: CodingStyle cleanup

2011-11-16 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320866946-28235-7-git-send-email...@denx.de> you wrote: > Make (mostly) checkpatch-clean > > Signed-off-by: Wolfgang Denk > --- > board/evb64260/zuma_pbb_mbox.c | 256 > +--- > 1 files changed, 132 insertions(+), 124 deletio

Re: [U-Boot] [PATCH 08/13] board/evb64260/zuma_pbb_mbox.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320866946-28235-8-git-send-email...@denx.de> you wrote: > Fix: > zuma_pbb_mbox.c: In function 'zuma_mbox_dump': > zuma_pbb_mbox.c:115:2: warning: dereferencing type-punned pointer will > break strict-aliasing rules [-Wstrict-aliasing] > zuma_pbb_mbox.c:117:2: warni

Re: [U-Boot] [PATCH 09/13] common/cmd_fdc.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320866946-28235-9-git-send-email...@denx.de> you wrote: > Fix: > cmd_fdc.c: In function 'fdc_read_data': > cmd_fdc.c:435:6: warning: variable 'flags' set but not used > [-Wunused-but-set-variable] > cmd_fdc.c:432:16: warning: variable 'pcn' set but not used > [-Wun

Re: [U-Boot] [PATCH 10/13] common/cmd_universe.c: Fix GCC 4.6 build warning

2011-11-16 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320866946-28235-10-git-send-email...@denx.de> you wrote: > Fix: > cmd_universe.c: In function 'universe_init': > cmd_universe.c:49:17: warning: variable 'lastError' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > --- > common/cm

Re: [U-Boot] [PATCH 11/13] drivers/video/ct69000.c: Fix GC 4.6 build warning

2011-11-16 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320866946-28235-11-git-send-email...@denx.de> you wrote: > Convert custom debug code to use standard debug() facility. > This also fixes these build warning: > ct69000.c: In function 'FindAndSetPllParamIntoXrRegs': > ct69000.c:706:28: warning: variable 'new_pixcloc

Re: [U-Boot] [PATCH 12/13] board/esd/cpci750/cpci750.c: Fix error handling

2011-11-16 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320866946-28235-12-git-send-email...@denx.de> you wrote: > ThE code recorded error conditions but did not pass these on to the > higher level caller. Fixing this fixes also this build warning: > cpci750.c: In function 'do_loadpci': > cpci750.c:569:6: warning: vari

Re: [U-Boot] [PATCH 13/13] board/esd/cpci750/sdram_init.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Mike Frysinger, In message <20091545.07892.vap...@gentoo.org> you wrote: > > static int check_dimm (uchar slot, AUX_MEM_DIMM_INFO * dimmInfo) > > { > > +#if 0 > > unsigned long spd_checksum; > > - > > +#endif > > forgot to just delete ? Indeed. Will do in commit. Best regards,

Re: [U-Boot] [PATCH 13/13] board/esd/cpci750/sdram_init.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1320866946-28235-13-git-send-email...@denx.de> you wrote: > Fix: > sdram_init.c: In function 'check_dimm': > sdram_init.c:267:50: warning: variable 'trrd_clocks' set but not used > [-Wunused-but-set-variable] > sdram_init.c:267:37: warning: variable 'tras_clocks' se

Re: [U-Boot] [PATCH 1/2] arch/powerpc/cpu/mpc512x/pci.c: Fix GCC 4.6 warnings

2011-11-16 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1320963704-568-1-git-send-email-ag...@denx.de> you wrote: > Fix: > pci.c: In function 'pci_init_board': > pci.c:55:26: warning: variable 'pci_conf' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Anatolij Gustschin > --- > arch/powerpc/cpu/

[U-Boot] Re

2011-11-16 Thread Ma. Jesusa M. Marco
Dear beneficiary, This is to re-notify you of the $600,000.00 USD that was deposited here in the western union office in your favour is available for pickup. Contact us via email for your M.T.C.N Numbers. Contact Person: Mr.David Churchill Tel. +44 703 186 0768 Tel. +44 703 596 3267 Email: ( d

Re: [U-Boot] [PATCH 2/2] arch/powerpc/cpu/mpc512x/i2c.c: Fix GCC 4.6 warnings

2011-11-16 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1320963704-568-2-git-send-email-ag...@denx.de> you wrote: > Fix: > i2c.c: In function 'wait_for_bb': > i2c.c:81:16: warning: variable 'temp' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Anatolij Gustschin > --- > arch/powerpc/cpu/mpc512x

Re: [U-Boot] [PATCH 01/19] usb_ohci.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380105-6159-1-git-send-email...@denx.de> you wrote: > Fix: > usb_ohci.c: In function 'dl_transfer_length': > usb_ohci.c:756:8: warning: variable 'tdINFO' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Stefan Roese > --- > arch/powerpc/cpu/p

Re: [U-Boot] [PATCH 03/19] 4xx_enet.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380118-6257-1-git-send-email...@denx.de> you wrote: > Fix: > 4xx_enet.c: In function 'ppc_4xx_eth_init': > 4xx_enet.c:875:6: warning: variable 'ethgroup' set but not used > [-Wunused-but-set-variable] > > I used "__maybe_unused" here intentionally, since all ot

Re: [U-Boot] [PATCH 04/19] 4xx_pcie.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380124-6304-1-git-send-email...@denx.de> you wrote: > Fix: > 4xx_pcie.c: In function 'pcie_read_config': > 4xx_pcie.c:230:6: warning: variable 'address' set but not used > [-Wunused-but-set-variable] > 4xx_pcie.c: In function 'pcie_write_config': > 4xx_pcie.c:29

Re: [U-Boot] [PATCH 05/19] 4xx_ibm_ddr2_autocalib.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380131-6351-1-git-send-email...@denx.de> you wrote: > Fix: > 4xx_ibm_ddr2_autocalib.c: In function 'get_membase': > 4xx_ibm_ddr2_autocalib.c:157:8: warning: variable 'bxcf' set but not used > [-Wunused-but-set-variable] > 4xx_ibm_ddr2_autocalib.c: In function 'D

Re: [U-Boot] [PATCH 06/19] cmd_ecctest.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380138-6398-1-git-send-email...@denx.de> you wrote: > Fix: > cmd_ecctest.c: In function 'inject_ecc_error': > cmd_ecctest.c:116:6: warning: variable 'val' set but not used > [-Wunused-but-set-variable] > cmd_ecctest.c: In function 'rewrite_ecc_parity': > cmd_ecc

Re: [U-Boot] [PATCH] mvgbe: remove setting of ethaddr within the driver

2011-11-16 Thread Michael Walle
Hi Wolfgang, Am Donnerstag 10 November 2011, 12:44:26 schrieb Wolfgang Denk: > Dear Prafulla, > > In message <48935aa530a6457b76b9915a1fe7faf9.squir...@ssl.serverraum.org> Michael Walle wrote: > > >> So was there any conclusion, now? Never touch mvgbe.c anymore? > > > > > > Let's remove this b

Re: [U-Boot] [PATCH 07/19] 44x_spd_ddr.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380144-6445-1-git-send-email...@denx.de> you wrote: > Fix: > 44x_spd_ddr.c: In function 'program_cfg0': > 44x_spd_ddr.c:384:16: warning: variable 'dimm_64bit' set but not used > [-Wunused-but-set-variable] > 44x_spd_ddr.c:383:16: warning: variable 'dimm_32bit' s

Re: [U-Boot] [PATCH 08/19] 44x_spd_ddr2.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380150-6492-1-git-send-email...@denx.de> you wrote: > 44x_spd_ddr2.c: In function 'initdram': > 44x_spd_ddr2.c:450:17: warning: variable 'dimm_spd' set but not used > [-Wunused-but-set-variable] > 44x_spd_ddr2.c: In function 'program_copt1': > 44x_spd_ddr2.c:100

Re: [U-Boot] [PATCH 09/19] 40x_spd_sdram.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380157-6539-1-git-send-email...@denx.de> you wrote: > Fix: > 40x_spd_sdram.c: In function 'spd_sdram': > 40x_spd_sdram.c:137:6: warning: variable 'sdram0_b3cr' set but not used > [-Wunused-but-set-variable] > 40x_spd_sdram.c:136:6: warning: variable 'sdram0_b2cr

Re: [U-Boot] [PATCH 10/19] 20001122-1.c: Fix GCC 4.6 build warning

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380163-6587-1-git-send-email...@denx.de> you wrote: > Fix: > 20001122-1.c: In function 'fpu_post_test_math1': > 20001122-1.c:37:22: warning: variable 'p' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Stefan Roese > --- > post/lib_powerpc/f

Re: [U-Boot] [PATCH 11/19] sata_dwc.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380169-6634-1-git-send-email...@denx.de> you wrote: > Fix: > sata_dwc.c: In function 'sata_dwc_softreset': > sata_dwc.c:444:5: warning: variable 'status' set but not used > [-Wunused-but-set-variable] > sata_dwc.c:443:6: warning: variable 'serror' set but not us

Re: [U-Boot] [PATCH 12/19] nand_boot.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380174-6687-1-git-send-email...@denx.de> you wrote: > Fix: > nand_boot.c: In function 'nand_read_page': > nand_boot.c:190:6: warning: variable 'stat' set but not used > [-Wunused-but-set-variable] > nand_boot.c: In function 'nand_boot': > nand_boot.c:271:6: warn

Re: [U-Boot] [PATCH 13/19] dlvision-10g.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380180-6735-1-git-send-email...@denx.de> you wrote: > Fix: > dlvision-10g.c: In function 'print_fpga_info': > dlvision-10g.c:90:11: warning: variable 'feature_compression' set but not > used [-Wunused-but-set-variable] > > Signed-off-by: Stefan Roese > --- >

Re: [U-Boot] [PATCH 14/19] cmd_pci405.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380186-6782-1-git-send-email...@denx.de> you wrote: > Fix: > cmd_pci405.c: In function 'do_loadpci': > cmd_pci405.c:45:6: warning: variable 'status' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Stefan Roese > --- > board/esd/pci405/cmd_pc

Re: [U-Boot] [PATCH 15/19] cmd_dasa_sim.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380200-6833-1-git-send-email...@denx.de> you wrote: > Fix: > cmd_dasa_sim.c: In function 'updatePci9054': > cmd_dasa_sim.c:135:6: warning: variable 'val' set but not used > [-Wunused-but-set-variable] > cmd_dasa_sim.c: In function 'clearPci9054': > cmd_dasa_sim.

Re: [U-Boot] [PATCH 16/19] iop480_uart.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380207-6889-1-git-send-email...@denx.de> you wrote: > Fix: > iop480_uart.c: In function 'serial_init': > iop480_uart.c:137:16: warning: variable 'val' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Stefan Roese > --- > arch/powerpc/cpu/ppc4

Re: [U-Boot] [PATCH 17/19] auto_update.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380213-6938-1-git-send-email...@denx.de> you wrote: > Fix: > ../common/auto_update.c: In function 'au_check_header_valid': > ../common/auto_update.c:94:16: warning: variable 'checksum' set but not used > [-Wunused-but-set-variable] > ../common/auto_update.c: In

Re: [U-Boot] [PATCH 18/19] mip405.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380219-6986-1-git-send-email...@denx.de> you wrote: > Fix: > mip405.c: In function 'init_sdram': > mip405.c:250:4: warning: variable 'tctp_clocks' set but not used > [-Wunused-but-set-variable] > mip405.c: In function 'initdram': > mip405.c:629:9: warning: varia

Re: [U-Boot] [PATCH 19/19] pip405.c: Fix GCC 4.6 build warnings

2011-11-16 Thread Wolfgang Denk
Dear Stefan Roese, In message <1321380225-7033-1-git-send-email...@denx.de> you wrote: > Fix: > pip405.c: In function 'board_early_init_f': > pip405.c:192:16: warning: variable 'tctp_clocks' set but not used > [-Wunused-but-set-variable] > pip405.c:182:16: warning: variable 'dataout' set but not

Re: [U-Boot] [PATCH] mvgbe: remove setting of ethaddr within the driver

2011-11-16 Thread Wolfgang Denk
Dear Michael Walle, In message <20162115.0.mich...@walle.cc> you wrote: > > > Could you please revise your assessment? I would really appreciate if > > we could clean this up. > Regarding Prafulla's NAK? Yes - I'd really appreciate if Prafulla woudl withdraw his NAK, and agree on a clea

Re: [U-Boot] [PATCH 01/12] drivers/spi/atmel_spi.c: Fix GCC 4.6 warning

2011-11-16 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1321399263-13826-2-git-send-email-ag...@denx.de> you wrote: > Fix: > atmel_spi.c: In function 'spi_xfer': > atmel_spi.c:139:7: warning: variable 'ret' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Anatolij Gustschin > --- > drivers/spi/at

Re: [U-Boot] [PATCH 02/12] drivers/mtd/dataflash.c: Fix GCC 4.6 warning

2011-11-16 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1321399263-13826-3-git-send-email-ag...@denx.de> you wrote: > Fix: > dataflash.c: In function 'AT91F_DataflashInit': > dataflash.c:42:6: warning: variable 'last_part' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Anatolij Gustschin > --- >

Re: [U-Boot] [PATCH 03/12] drivers/net/at91_emac.c: Fix GCC 4.6 warning

2011-11-16 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1321399263-13826-4-git-send-email-ag...@denx.de> you wrote: > Fix: > at91_emac.c: In function 'at91emac_write_hwaddr': > at91_emac.c:477:15: warning: variable 'dev' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Anatolij Gustschin > --- >

Re: [U-Boot] [PATCH v4 3/4] tegra2: Modify MMC driver to handle power and cd GPIOs

2011-11-16 Thread Thierry Reding
* Stephen Warren wrote: > Pass the GPIO numbers for power and card detect to tegra2_mmc_init(), and > modify that function to perform all required GPIO initialization. This > removes the need for board files to perform these operations. > > Move board_mmc_getcd() into tegra2_mmc.c now that the dri

Re: [U-Boot] [PATCH 04/12] drivers/net/enc28j60.c: Fix GCC 4.6 warnings

2011-11-16 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1321399263-13826-5-git-send-email-ag...@denx.de> you wrote: > Fix: > enc28j60.c: In function 'enc_receive': > enc28j60.c:435:5: warning: variable 'eir_reg' set but not used > [-Wunused-but-set-variable] > enc28j60.c: In function 'enc_poll': > enc28j60.c:503:5:

Re: [U-Boot] [PATCH 04/12] drivers/net/enc28j60.c: Fix GCC 4.6 warnings

2011-11-16 Thread Wolfgang Denk
Dear Mike Frysinger, In message <20161058.57150.vap...@gentoo.org> you wrote: > > On Tuesday 15 November 2011 18:20:55 Anatolij Gustschin wrote: > > - eir_reg = enc_r8(enc, CTL_REG_EIR); > > + (void)enc_r8(enc, CTL_REG_EIR); > > is that (void) actually necessary ? last i l

[U-Boot] unsubscribe

2011-11-16 Thread Joe Halpin
___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 05/12] board/cm41xx/flash.c: Fix GCC 4.6 warnings

2011-11-16 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1321399263-13826-6-git-send-email-ag...@denx.de> you wrote: > Fix: > flash.c: In function 'flash_erase': > flash.c:209:6: warning: variable 'flag' set but not used > [-Wunused-but-set-variable] > flash.c: In function 'write_data': > flash.c:373:6: warning: vari

Re: [U-Boot] [PATCH 06/12] board/cm4008/flash.c: Fix GCC 4.6 warnings

2011-11-16 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1321399263-13826-7-git-send-email-ag...@denx.de> you wrote: > Fix: > flash.c: In function 'flash_erase': > flash.c:209:6: warning: variable 'flag' set but not used > [-Wunused-but-set-variable] > flash.c: In function 'write_data': > flash.c:373:6: warning: vari

Re: [U-Boot] [PATCH 07/12] drivers/video/da8xx-fb.c: Fix GCC 4.6 warning

2011-11-16 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1321399263-13826-8-git-send-email-ag...@denx.de> you wrote: > Fix: > da8xx-fb.c: In function 'video_hw_init': > da8xx-fb.c:688:6: warning: variable 'ret' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Anatolij Gustschin > --- > drivers/vid

Re: [U-Boot] [PATCH 08/12] drivers/video/bus_vcxk.c: Fix GCC 4.6 warning

2011-11-16 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1321399263-13826-9-git-send-email-ag...@denx.de> you wrote: > Fix: > bus_vcxk.c: In function 'vcxk_display_bitmap': > bus_vcxk.c:396:16: warning: variable 'compression' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Anatolij Gustschin > ---

Re: [U-Boot] [PATCH 10/12] arch/arm/cpu/armv7/s5p-common/pwm.c: fix GCC 4.6 warning

2011-11-16 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1321399263-13826-11-git-send-email-ag...@denx.de> you wrote: > Fix: > pwm.c: In function 'pwm_config': > pwm.c:85:16: warning: variable 'timer_rate_hz' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Anatolij Gustschin > --- > arch/arm/cpu/

Re: [U-Boot] [PATCH 11/12] fs/yaffs2/yaffs_guts.c: Fix some GCC 4.6 warnings

2011-11-16 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1321399263-13826-12-git-send-email-ag...@denx.de> you wrote: > Fix: > yaffs_guts.c: In function 'yaffs_CheckChunkErased': > yaffs_guts.c:854:6: warning: variable 'result' set but not used > yaffs_guts.c: In function 'yaffs_UpdateObjectHeader': > yaffs_guts.c:34

Re: [U-Boot] [PATCH 12/12] drivers/rtc/s3c24x0_rtc.c: fix GCC 4.6 warnings

2011-11-16 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <1321399263-13826-13-git-send-email-ag...@denx.de> you wrote: > Fix: > s3c24x0_rtc.c: In function 'rtc_get': > s3c24x0_rtc.c:67:53: warning: variable 'a_armed' set but not used > s3c24x0_rtc.c:67:45: warning: variable 'a_year' set but not used > s3c24x0_rtc.c:67

Re: [U-Boot] [STATUS] Help needed - urgently

2011-11-16 Thread Kumar Gala
On Nov 15, 2011, at 9:01 AM, Wolfgang Denk wrote: > Hello all, > > I guess most of you will already have noticed that my activity on the > mailing list has significantly declined recently. I'm sorry for that, > but I find myself in a situation where I have even less time > available for U-Boot t

Re: [U-Boot] [PATCH 2/2] Fix warnings in yaffs

2011-11-16 Thread Wolfgang Denk
Dear Simon Glass, In message <1321417027-5639-2-git-send-email-...@chromium.org> you wrote: > Sorry if this is already fixed somewhere - I could not find it. > > This fixes the warnings show below. > > yaffs_tagscompat.c: In function > 'yaffs_TagsCompatabilityReadChunkWithTagsFromNAND': > yaffs

  1   2   >