[U-Boot] [PATCH v2 2/3] arm, davinci: Add function lpsc_syncreset()

2011-10-13 Thread Christian Riesch
This patch adds a function lpsc_syncreset that allows setting a lpsc module into Sync Reset state. Signed-off-by: Christian Riesch Cc: Heiko Schocher Cc: Paulraj Sandeep Cc: Albert ARIBAUD --- arch/arm/cpu/arm926ejs/davinci/psc.c | 20 +++- arch/arm/include/asm/arch-

[U-Boot] [PATCH v2 1/3] arm, davinci: Rename AM1808 lowlevel functions to DA850

2011-10-13 Thread Christian Riesch
Rename arch/arm/cpu/arm926ejs/davinci/am1808_lowlevel.c and arch/arm/include/asm/arch-davinci/am1808_lowlevel.h to da850_lowlevel.c and da850_lowlevel.h since they apply not only to the AM1808 SoC but to all DA850 chips. The function names and #defines are changed likewise. Signed-off-by: Christia

[U-Boot] [PATCH v2 0/3] Remove duplicate functions for power and sleep configuration

2011-10-13 Thread Christian Riesch
Hi, these patches apply on top of Heiko Schocher's patches for the enbw_cmc board. They remove the duplicate functions for the power and sleep controller of the davinci devices. Changes for v2: - use -M option of git format-patch to detect the renaming of files - Removed one patch of the patchset

[U-Boot] [PATCH v2 3/3] arm, davinci: Remove the duplication of LPSC functions in da850_lowlevel.c

2011-10-13 Thread Christian Riesch
The LPSC functions defined in arch/arm/cpu/arm926ejs/davinci/da850_lowlevel.c are replaced by those already defined in arch/arm/cpu/arm926ejs/davinci/psc.c. Signed-off-by: Christian Riesch Cc: Heiko Schocher Cc: Paulraj Sandeep Cc: Albert ARIBAUD --- arch/arm/cpu/arm926ejs/davinci/da850_lowle

Re: [U-Boot] [PATCH] avr32: fix timer_init() return type

2011-10-13 Thread Andreas Bießmann
Dear Mike, Am 13.10.2011 07:11, schrieb Mike Frysinger: > The common.h header says this has to return an int, otherwise we get > build failures due to mismatched prototype and function definition. > > Signed-off-by: Mike Frysinger > --- > arch/avr32/cpu/interrupts.c |6 -- > 1 files cha

Re: [U-Boot] [PATCH 1/2] AVR32: use generic mmc framework

2011-10-13 Thread Sven Schnelle
Hi Wolfgang, On 10/12/2011 07:54 PM, Wolfgang Denk wrote: > Dear Sven Schnelle, > > In message<4e95a965.20...@stackframe.org> you wrote: >> >>> a) mmc init is not known? Is this not longer necessar >> 'mmc init' is only available if CONFIG_GENERIC_MMC isn't set, at least >> that's how i unterstan

Re: [U-Boot] [PATCH] sh: only add -mno-fdpic if the compiler supports it

2011-10-13 Thread phil . edworthy
Hi Mike, Mike Frysinger wrote on 13/10/2011 05:45:47: > Not all SuperH toolchains support -mno-fdpic. Chances are good that if > the flag doesn't work, it isn't defaulting to the FDPIC ABI, so the flag > isn't needed. So only add it if it is actually supported. > > Signed-off-by: Mike Frysinger

Re: [U-Boot] [PATCH v2 1/3] arm, davinci: Rename AM1808 lowlevel functions to DA850

2011-10-13 Thread Heiko Schocher
Hello Christian, Christian Riesch wrote: > Rename arch/arm/cpu/arm926ejs/davinci/am1808_lowlevel.c and > arch/arm/include/asm/arch-davinci/am1808_lowlevel.h to da850_lowlevel.c > and da850_lowlevel.h since they apply not only to the AM1808 SoC > but to all DA850 chips. The function names and #defi

Re: [U-Boot] [PATCH v2 2/3] arm, davinci: Add function lpsc_syncreset()

2011-10-13 Thread Heiko Schocher
Hello Christian, Christian Riesch wrote: > This patch adds a function lpsc_syncreset that allows setting a > lpsc module into Sync Reset state. > > Signed-off-by: Christian Riesch > Cc: Heiko Schocher > Cc: Paulraj Sandeep > Cc: Albert ARIBAUD > --- > arch/arm/cpu/arm926ejs/davinci/psc.c

Re: [U-Boot] [PATCH v2 3/3] arm, davinci: Remove the duplication of LPSC functions in da850_lowlevel.c

2011-10-13 Thread Heiko Schocher
Hello Christian, Christian Riesch wrote: > The LPSC functions defined in > arch/arm/cpu/arm926ejs/davinci/da850_lowlevel.c > are replaced by those already defined in > arch/arm/cpu/arm926ejs/davinci/psc.c. > > Signed-off-by: Christian Riesch > Cc: Heiko Schocher > Cc: Paulraj Sandeep > Cc: Alb

Re: [U-Boot] [PATCH 1/2] AVR32: use generic mmc framework

2011-10-13 Thread Sven Schnelle
On 10/12/2011 04:59 PM, Andreas Bießmann wrote: > Am 12.10.2011 16:51, schrieb Sven Schnelle: >> On 10/12/2011 04:29 PM, Andreas Bießmann wrote: >>> Am 12.10.2011 15:09, schrieb Sven Schnelle: >>> And does not work on device: >>> b) ext2ls is not reliable: >>> >>> ---8<--- >>> U-Boot> ext2ls mmc

Re: [U-Boot] [PATCH] AVR32: switch to generic mmc framework

2011-10-13 Thread Andreas Bießmann
Dear Reinhard, Dear Sven, Am 12.10.2011 19:01, schrieb Reinhard Meyer: > Dear Sven Schnelle, >> Hi List, >> > As I see it, all this should come as three patches: > > 1. switching *all* AVR32 and AT91 boards to use the *existing* generic driver > > 2. removal of the now obsolete legacy driver

[U-Boot] [PATCH v3] ppc4xx: Add Io64 board support

2011-10-13 Thread Dirk Eibach
Board support for the Guntermann & Drunck Io64. Signed-off-by: Dirk Eibach --- Changes for v2: - Added MAINTAINERS entry - Cleaned up issues reported by checkpatch Changes for v3: - migrated board specific init_func_fpga() to board_early_init_r() MAINTAINERS |1 +

Re: [U-Boot] SPL broken for pandaboard with v2011.09?

2011-10-13 Thread Aneesh V
Hi Steve, John, Sorry for the late reply. On Thursday 13 October 2011 05:12 AM, Steve Sakoman wrote: > On Wed, Oct 12, 2011 at 3:23 PM, John Rigby wrote: > >> My tree has a bunch of extra patches on top and is based on rc2. I >> know it works: >> >> http://git.linaro.org/gitweb?p=boot/u-boot-li

[U-Boot] Uboot with simpleImage elf.

2011-10-13 Thread Vineeth
Hi, For this particular project i cant burn more than 1 image in to the flash. and so i cant use uImage (which needs the DTB file). We were thinking of burning the simpleImage.elf file (ram image of kernel) ; tweaking uboot such that it will copy the simpleImage from the flash and put it on 0x4000

Re: [U-Boot] [PATCH v4] net: axi_ethernet: Add driver to u-boot

2011-10-13 Thread Michal Simek
Mike Frysinger wrote: > Acked-by: Mike Frysinger > -mike Thanks, Michal -- Michal Simek, Ing. (M.Eng) w: www.monstr.eu p: +42-0-721842854 Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/ Microblaze U-BOOT custodian ___ U-Bo

[U-Boot] Pull request - emaclite and axi ethernet driver

2011-10-13 Thread Michal Simek
Dear Wolfgang, I have composed new branch with emaclite changes and axi ethernet driver (I have got ACK from Mike). I have 3 more patches in my queue which I am going to send to mailing list for review. They will be based on this tree. Please pull these changes. Thanks, Michal The following

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

2011-10-13 Thread Anatolij Gustschin
From: 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 Signed-off-by: Helmut Raiger Acked-by: Marek Vasut Signed-off-by: Anatolij Gustschin Cc: Stefano Babic --- Changes sin

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

2011-10-13 Thread Anatolij Gustschin
From: 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 setu

[U-Boot] [PATCH 0/3] Microblaze/Xilinx network driver changes

2011-10-13 Thread Michal Simek
Hi, The patches are based on git://git.denx.de/u-boot-microblaze.git network-drivers branch. Thanks for your review. Thanks, Michal Michal Simek (3): net: emaclite: Use unsigned long for baseaddr net: emaclite: Move RX/TX ping pong initialization to board microblaze: Wire up axi_ethernet

[U-Boot] [PATCH 1/3 v2] net: emaclite: Use unsigned long for baseaddr

2011-10-13 Thread Michal Simek
Baseaddr should be unsigned long. Signed-off-by: Michal Simek --- v2: Fix merge confict - no function change --- drivers/net/xilinx_emaclite.c |4 ++-- include/netdev.h |2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/xilinx_emaclite.c b/dri

[U-Boot] [PATCH 2/3 v2] net: emaclite: Move RX/TX ping pong initialization to

2011-10-13 Thread Michal Simek
Init RX/TX ping pong directly from board not in the driver. Signed-off-by: Michal Simek --- v2: Fix merge confict - no function change --- .../xilinx/microblaze-generic/microblaze-generic.c | 18 -- drivers/net/xilinx_emaclite.c | 11 --- include

[U-Boot] [PATCH 3/3] microblaze: Wire up axi_ethernet driver initialization

2011-10-13 Thread Michal Simek
Initialize axi_ethernet driver. Signed-off-by: Michal Simek --- .../xilinx/microblaze-generic/microblaze-generic.c |6 ++ include/configs/microblaze-generic.h |4 2 files changed, 10 insertions(+), 0 deletions(-) diff --git a/board/xilinx/microblaze-generic/microb

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

2011-10-13 Thread Anatolij Gustschin
Hi Stefano, On Thu, 13 Oct 2011 11:16:28 +0200 Anatolij Gustschin wrote: > From: 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 > > Signed-off-by: Helmut Raiger >

[U-Boot] hwmon: problems with fan control

2011-10-13 Thread Eibach, Dirk
Commit "780f13a... hwmon: do not init sensors on startup" moved initialization of the dtt driver to cmd_dtt. I just realized that this means trouble for me, because the lm63 driver used on our boards is responsible for setting up the fans. Relying on cmd_dtt being called by preboot seems fragile t

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

2011-10-13 Thread Stefano Babic
On 10/13/2011 11:23 AM, Anatolij Gustschin wrote: > Hi Stefano, > > On Thu, 13 Oct 2011 11:16:28 +0200 > Anatolij Gustschin wrote: > >> From: Helmut Raiger >> >> This additionally updates mx31/generic.c by >> - replacing __REG() macro accesses with readl() and writel() >> - providing macros for

Re: [U-Boot] [PATCH] sparc: fix unknown escape sequence warnings

2011-10-13 Thread Daniel Hellstrom
On 10/13/2011 07:47 AM, Mike Frysinger wrote: > I don't know what exactly the code was going for, but the object code > is the same before/after my change, and in looking at the env strings, > this seems to be OK. > > Otherwise gcc warns: > cc1: warning: unknown escape sequence: '\$' > cc1: warnin

Re: [U-Boot] [PATCH] avr32: fix timer_init() return type

2011-10-13 Thread Reinhard Meyer
Dear Andreas Bießmann, >> if (set_interrupt_handler(0, &timer_interrupt_handler, 3)) >> -return; >> +return 0; > > NAK, this is an error and should return a negative value (though the > return value is currently not evaluated). Agreed > > BTW there is another patch

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

2011-10-13 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] sparc and .udiv/.rem/.urem/.umul/etc... link errors

2011-10-13 Thread Daniel Hellstrom
On 10/13/2011 07:33 AM, Mike Frysinger wrote: > it seems that these symbols are usually implemented in the C library on sparc > systems, but under u-boot, we don't have that luxury. Linux itself simply > imports the relevant asm files from glibc into their lib subdir. without > these, building u-

Re: [U-Boot] [PATCH v3] DaVinci: correct MDSTAT.STATE mask

2011-10-13 Thread Christian Riesch
Hi, On Fri, Sep 23, 2011 at 4:29 PM, Sergei Shtylyov wrote: > MDSTAT.STATE occupies bits 0..5 according to all available documentation, so > fix > the mask which previously was leaving out the intermediate state indicator > bit. > > While at it, introduce two #define's for that mask -- unfortun

Re: [U-Boot] [PATCH v2 1/1] pci: move pcidelay code to new location just before PCI bus scan

2011-10-13 Thread Matthias Fuchs
On 12.10.2011 10:44, Anatolij Gustschin wrote: > PCI cards might need some time after reset to respond. On some > boards (mpc5200 or mpc8260 based) the PCI bus reset is deasserted > at pci_init_board() time, so we currently can not use available > "pcidelay" option for waiting before PCI bus scan s

Re: [U-Boot] [PATCH v2 1/1] pci: move pcidelay code to new location just before PCI bus scan

2011-10-13 Thread Stefan Roese
On Thursday 13 October 2011 14:50:02 Matthias Fuchs wrote: > On 12.10.2011 10:44, Anatolij Gustschin wrote: > > PCI cards might need some time after reset to respond. On some > > boards (mpc5200 or mpc8260 based) the PCI bus reset is deasserted > > at pci_init_board() time, so we currently can not

[U-Boot] [PATCH 3/3] ppc4xx: fix PMC440 painit command

2011-10-13 Thread matthias . fuchs
From: Matthias Fuchs This patch fixes the PMC440 BSP command painit. The implementation was broken since the step to the new environment handling. Signed-off-by: Matthias Fuchs --- board/esd/pmc440/cmd_pmc440.c | 17 +++-- 1 files changed, 15 insertions(+), 2 deletions(-) diff -

[U-Boot] [PATCH 2/3] ppc4xx: remove invalid access to PCI_BRDGOPT2 register

2011-10-13 Thread matthias . fuchs
From: Matthias Fuchs This patch removes an invalid call to pci_write_config_dword to PCI_BRDGOPT2 register. This function must not be used from pci_target_init and is also at the wrong place. The correct call is done later in pci_target_init via pci_hose_write_config_dword. Signed-off-by: Matthi

Re: [U-Boot] [PATCH v2 1/3] davinci: emac: add new features to autonegotiate for EMAC

2011-10-13 Thread Hadli, Manjunath
Hi Mike, On Wed, Oct 12, 2011 at 20:39:29, Mike Frysinger wrote: > On Wednesday 12 October 2011 09:27:59 manjunath.ha...@ti.com wrote: > > --- a/include/miiphy.h > > +++ b/include/miiphy.h > > NAK to all of these changes. you should be migrating to linux/mii.h. we've > gone through a lot of tr

Re: [U-Boot] [PATCH v2 1/1] pci: move pcidelay code to new location just before PCI bus scan

2011-10-13 Thread Matthias Fuchs
On 13.10.2011 15:03, Stefan Roese wrote: >> >> Tested on PMC440. Works fine. >> >> Acked-by: Matthias Fuchs > > Thanks. In this case a: > > Tested-by: ... > > would be even better. :) > Even both if you like :-) Matthias ___ U-Boot mailing list U-Bo

[U-Boot] [PATCH 1/3] ppc4xx: use CONFIG_PCI_BOOTDELAY instead of private implementation

2011-10-13 Thread matthias . fuchs
From: Matthias Fuchs This patch switches PMC440 board code to the CONFIG_PCI_BOOTDELAY option instead of using a private implemention. This relies on Anatolji's patch that moves the pcidelay handling behind pci_target_init. Signed-off-by: Matthias Fuchs --- board/esd/pmc440/pmc440.c | 15 ---

[U-Boot] [PATCH v3 0/3] These patches add features relating davinci emac driver.

2011-10-13 Thread manjunath.hadli
From: Manjunath Hadli Changes for revision v2: 1: Used a #define for maximum number of phy's as suggested by Laurence. 2: Dropped a patch, da830: emac: add support for RMII, it has some issues, Will come back with this patch soon. 3: fixed checkpatch warnings. Changes for revision v3: 1: m

[U-Boot] [PATCH v3 1/3] davinci: emac: add new features to autonegotiate for EMAC

2011-10-13 Thread manjunath.hadli
From: Manjunath Hadli add more features like DUPLEX, 100MB link speed etc to auto negotiate in EMAC driver. EMAC controller autonegotiates for these features with PHYs which are on the board. Signed-off-by: Sudhakar Rajashekhara Signed-off-by: Manjunath Hadli --- drivers/net/davinci_emac.c |

[U-Boot] [PATCH v3 2/3] davinci: emac: add support for more than 1 PHYs

2011-10-13 Thread manjunath.hadli
From: Manjunath Hadli add support for more than 1 PHYs. Many of the davinci platforms have more than 1 PHYs on thier board. This patch extends support in davinci emac driver for upto 3 PHYs. Signed-off-by: Sudhakar Rajashekhara Signed-off-by: Manjunath Hadli --- drivers/net/davinci_emac.c |

[U-Boot] [PATCH v3 3/3] davinci: remove obsolete macro CONFIG_EMAC_MDIO_PHY_NUM

2011-10-13 Thread manjunath.hadli
From: Manjunath Hadli remove macro CONFIG_EMAC_MDIO_PHY_NUM and depending macro EMAC_MDIO_PHY_NUM as they are no longer needed with the support for more than 1 PHYs in davinci emac driver. Signed-off-by: Manjunath Hadli --- arch/arm/cpu/arm926ejs/davinci/et1011c.c |6 ++ arch/arm/

Re: [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable

2011-10-13 Thread Marek Vasut
On Thursday, October 13, 2011 06:31:09 AM Simon Glass wrote: > On Wed, Oct 12, 2011 at 5:27 PM, Marek Vasut wrote: > > On Wednesday, October 12, 2011 11:17:07 PM Simon Glass wrote: > >> Hi Marek, > >> > >> On Wed, Oct 12, 2011 at 4:19 AM, Marek Vasut wrote: > >> > On Wednesday, October 12, 2011

Re: [U-Boot] [PATCH] imx: fix coding style

2011-10-13 Thread Fabio Estevam
On Wed, Oct 12, 2011 at 7:20 AM, Stefano Babic wrote: 136/mx31/timer.c >> +++ b/arch/arm/cpu/arm1136/mx31/timer.c >> @@ -43,7 +43,10 @@ >> >>  DECLARE_GLOBAL_DATA_PTR; >> >> -/* "time" is measured in 1 / CONFIG_SYS_HZ seconds, "tick" is internal >> timer period */ >> +/* "time" is measured in 1 /

[U-Boot] [PATCH] mpc5200: digsy_mtc: fix detection of Coral-PA

2011-10-13 Thread Anatolij Gustschin
A delay of approximately 250 ms after PCI bus reset in pci_mpc5xxx_init() is needed to recognize the Coral-PA controller on the graphic extention board. Signed-off-by: Anatolij Gustschin --- To make it actually work another patch "pci: move pcidelay code to new location just before PCI bus scan"

Re: [U-Boot] [PATCH] imx: fix coding style

2011-10-13 Thread Stefan Roese
On Thursday 13 October 2011 17:17:03 Fabio Estevam wrote: > > This style for multiline comments is wrong. Please fix it. > > After applying the patch I read: > > /* "time" is measured in 1 / CONFIG_SYS_HZ seconds, > * "tick" is internal timer period > */ > > Could you please let me know what i

Re: [U-Boot] [PATCH] avr32: fix timer_init() return type

2011-10-13 Thread Mike Frysinger
On Thursday 13 October 2011 06:27:38 Reinhard Meyer wrote: > Dear Andreas Bießmann, > > BTW there is another patch pending -> > > http://patchwork.ozlabs.org/patch/117688/ > > I'll apply that one in the year 2015 ;) > > NO.. its on my list for applying into next WFM -mike signature.asc Descrip

[U-Boot] [PATCH v2] imx: fix coding style

2011-10-13 Thread Fabio Estevam
Fix checkpatch warning and errors in several i.MX related files. While at it also address a checkpatch warning at arch/arm/cpu/armv7/mx5/soc.c regarding the usage of extern in a C file. Signed-off-by: Fabio Estevam --- Changes since v1: - Fix multi-line style - Add comment in the changelog regar

Re: [U-Boot] [PATCH] sparc: fix unknown escape sequence warnings

2011-10-13 Thread Mike Frysinger
On Thursday 13 October 2011 04:49:59 Daniel Hellstrom wrote: > On 10/13/2011 07:47 AM, Mike Frysinger wrote: > > I don't know what exactly the code was going for, but the object code > > is the same before/after my change, and in looking at the env strings, > > this seems to be OK. > > > > Otherwi

Re: [U-Boot] sparc and .udiv/.rem/.urem/.umul/etc... link errors

2011-10-13 Thread Mike Frysinger
On Thursday 13 October 2011 04:45:27 Daniel Hellstrom wrote: > On 10/13/2011 07:33 AM, Mike Frysinger wrote: > > it seems that these symbols are usually implemented in the C library on > > sparc systems, but under u-boot, we don't have that luxury. Linux > > itself simply imports the relevant asm

[U-Boot] [PATCH] common: fix missing function pointer relocation in fixup_cmdtable()

2011-10-13 Thread Daniel Schwierzeck
The command auto-completion does not work on architectures relying on CONFIG_NEEDS_MANUAL_RELOC like MIPS. Cause is the missing function pointer fixup for cmd_tbl_t::complete function in fixup_cmdtable(). This patch adds the missing pointer fixup in case of CONFIG_AUTO_COMPLETE is defined. Signed

Re: [U-Boot] [PATCH 01/15 V3] iMX28: Initial support for iMX28 CPU

2011-10-13 Thread Stefano Babic
On 09/30/2011 11:42 AM, Marek Vasut wrote: > This patch supports: > - Timers > - Debug UART > - Clock Hi Marek, /opt/tools/bin/checkpatch.pl --no-tree ~/Desktop/U-Boot-01-15-V3-iMX28-Initial-support-for-iMX28-CPU.patch WARNING: line over 80 characters #634: FILE: arch/arm/cpu/arm926ejs/mx28/mx28.

Re: [U-Boot] [PATCH 00/12] Define ARCH_DMA_MINALIGN for all architectures

2011-10-13 Thread Anton Staaf
On Wed, Oct 12, 2011 at 8:47 PM, Kumar Gala wrote: > > On Oct 12, 2011, at 4:01 PM, Anton Staaf wrote: > >> ARCH_DMA_MINALIGN is a new define borrowed from the Linux kernel.  It is >> used to define the minimum alignement requirement for a DMA buffer.  This >> series of patches ensures that it wil

Re: [U-Boot] checkpatch compliance

2011-10-13 Thread Joe Hershberger
On Wed, Oct 12, 2011 at 8:03 PM, Mike Frysinger wrote: > On Wednesday 12 October 2011 20:56:09 Joe Hershberger wrote: >> WARNING:NEW_TYPEDEFS: do not add new typedefs >> This seems rather limiting... I'm not sure why even Linux would want >> this, at least when it applies to typedefs of structs.  

Re: [U-Boot] [PATCH] tools: checkpatch.pl from Linux added to tools

2011-10-13 Thread Joe Hershberger
On Wed, Oct 12, 2011 at 10:27 PM, Mike Frysinger wrote: > On Tuesday 11 October 2011 21:41:19 Joe Hershberger wrote: >> --- /dev/null >> +++ b/.checkpatch.conf >> >> +# Not Linux, so don't expect a Linux tree. >> +--no-tree Why depend on or have to specify where a linux tree is for building u-boo

Re: [U-Boot] [PATCH v3 1/6] cache: add ALLOC_CACHE_ALIGN_BUFFER macro

2011-10-13 Thread Anton Staaf
On Wed, Oct 12, 2011 at 5:55 PM, Mike Frysinger wrote: > On Wednesday 12 October 2011 19:55:59 Anton Staaf wrote: >>  doc/README.arm-caches |    2 + > > seems like most of the advice in this is arch independent Agreed. It probably makes sense to change the name of the file. I'm not sure that an

Re: [U-Boot] [PATCH 01/15 V3] iMX28: Initial support for iMX28 CPU

2011-10-13 Thread Marek Vasut
On Thursday, October 13, 2011 06:19:44 PM Stefano Babic wrote: > On 09/30/2011 11:42 AM, Marek Vasut wrote: > > This patch supports: > > - Timers > > - Debug UART > > - Clock > > Hi Marek, > > /opt/tools/bin/checkpatch.pl --no-tree > ~/Desktop/U-Boot-01-15-V3-iMX28-Initial-support-for-iMX28-CPU.p

Re: [U-Boot] [PATCH v3 1/6] cache: add ALLOC_CACHE_ALIGN_BUFFER macro

2011-10-13 Thread Mike Frysinger
On Thursday 13 October 2011 14:06:41 Anton Staaf wrote: > On Wed, Oct 12, 2011 at 5:55 PM, Mike Frysinger wrote: > > On Wednesday 12 October 2011 19:55:59 Anton Staaf wrote: > >> doc/README.arm-caches |2 + > > > > seems like most of the advice in this is arch independent > > Agreed. It prob

Re: [U-Boot] checkpatch compliance

2011-10-13 Thread Mike Frysinger
On Thursday 13 October 2011 13:14:34 Joe Hershberger wrote: > On Wed, Oct 12, 2011 at 8:03 PM, Mike Frysinger wrote: > > On Wednesday 12 October 2011 20:56:09 Joe Hershberger wrote: > >> WARNING:NEW_TYPEDEFS: do not add new typedefs > >> This seems rather limiting... I'm not sure why even Linux wo

Re: [U-Boot] [PATCH v3 1/6] cache: add ALLOC_CACHE_ALIGN_BUFFER macro

2011-10-13 Thread Anton Staaf
On Thu, Oct 13, 2011 at 11:15 AM, Mike Frysinger wrote: > On Thursday 13 October 2011 14:06:41 Anton Staaf wrote: >> On Wed, Oct 12, 2011 at 5:55 PM, Mike Frysinger wrote: >> > On Wednesday 12 October 2011 19:55:59 Anton Staaf wrote: >> >>  doc/README.arm-caches |    2 + >> > >> > seems like most

Re: [U-Boot] [PATCH] tools: checkpatch.pl from Linux added to tools

2011-10-13 Thread Mike Frysinger
On Thursday 13 October 2011 13:25:51 Joe Hershberger wrote: > On Wed, Oct 12, 2011 at 10:27 PM, Mike Frysinger wrote: > > On Tuesday 11 October 2011 21:41:19 Joe Hershberger wrote: > >> --- /dev/null > >> +++ b/.checkpatch.conf > >> > >> +# Not Linux, so don't expect a Linux tree. > >> +--no-tree

Re: [U-Boot] [PATCH] tools: checkpatch.pl from Linux added to tools

2011-10-13 Thread Joe Hershberger
On Thu, Oct 13, 2011 at 1:20 PM, Mike Frysinger wrote: > On Thursday 13 October 2011 13:25:51 Joe Hershberger wrote: >> On Wed, Oct 12, 2011 at 10:27 PM, Mike Frysinger wrote: >> > On Tuesday 11 October 2011 21:41:19 Joe Hershberger wrote: >> >> --- /dev/null >> >> +++ b/.checkpatch.conf >> >> >>

[U-Boot] [PATCHv9] new tool mkenvimage: generates an env image from an arbitrary config file

2011-10-13 Thread David Wagner
This tool takes a key=value configuration file (same as would a `printenv' show) and generates the corresponding environment image, ready to be flashed. use case: flash the environment with an external tool Signed-off-by: David Wagner Acked-by; Mike Frysinger Tested-by: Thomas Petazzoni --- (M

Re: [U-Boot] [PATCH] tools: checkpatch.pl from Linux added to tools

2011-10-13 Thread Mike Frysinger
On Thursday 13 October 2011 14:30:12 Joe Hershberger wrote: > On Thu, Oct 13, 2011 at 1:20 PM, Mike Frysinger wrote: > > On Thursday 13 October 2011 13:25:51 Joe Hershberger wrote: > >> On Wed, Oct 12, 2011 at 10:27 PM, Mike Frysinger wrote: > >> > On Tuesday 11 October 2011 21:41:19 Joe Hershberge

Re: [U-Boot] [PATCH v2] powerpc/p3060qds: Add board related support for P3060QDS platform

2011-10-13 Thread Tabi Timur-B04825
On Thu, Oct 6, 2011 at 4:23 PM, Wolfgang Denk wrote: > Checkpatch says: > > WARNING: static const char * array should probably be static const > char * const > #1511: FILE: board/freescale/p3060qds/p3060qds.c:84: > +               static const char *freq[] = {"100", "125", "Reserved", > > > Pleas

Re: [U-Boot] [PATCH 1/2] image: Support relative-addresses property in FIT images

2011-10-13 Thread Wolfgang Denk
Dear Stephen Warren, In message <74cdbe0f657a3d45afbb94109fb122ff173a2c7...@hqmail01.nvidia.com> you wrote: > > > > OK. Wouldn't we also need e.g. IH_TYPE_STANDALONE_REL, IH_TYPE_RAMDISK_REL > > > and perhaps other too? > > > > I don't think so - either these don't have (and use) load and entry

Re: [U-Boot] [PATCH 1/6] ColdFire: Cleanup lds files for multiple defined symbols

2011-10-13 Thread Wolfgang Denk
Dear "Stany MARCEL", In message <1512c13c0c5ea947ae50b9b9ac314f4701b00...@atlas.pactenovation.fr> you wrote: > > Each time a file is edited do I have to actualizes end dates ? Or leave them > as they are ? If the file has a copyright entry for yourself, you would udate this to include the curr

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-13 Thread Wolfgang Denk
Dear Anton Staaf, In message you wrote: > > Would you be OK with a build warning for the lack of definition of > CONFIG_SYS_CACHELINE_SIZE like I have now if it only happened > once per board? ... Yes, that would be great. > ... I could move it from common.h to a c file that is al

Re: [U-Boot] [PATCH V2 1/2] mmc: change magic number to macro define

2011-10-13 Thread Wolfgang Denk
Dear Lei Wen, In message Graeme Russ wrote: > > > So do you means I should keep the EXT_CSD_HS_TIMING? Or I change like below? > > > > from > > err = mmc_switch(mmc, EXT_CSD_CMD_SET_NORMAL, EXT_CSD_HS_TIMING, 1); > > to: > > err = mmc_switch(mmc, EXT_CSD_CMD_SET_NORMAL, &ext_csd.hs_timing - >

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-13 Thread Anton Staaf
On Thu, Oct 13, 2011 at 1:03 PM, Wolfgang Denk wrote: > Dear Anton Staaf, > > In message > you > wrote: >> >> Would you be OK with a build warning for the lack of definition of >> CONFIG_SYS_CACHELINE_SIZE like I have now if it only happened >> once per board? ... > > Yes, that would be great.

Re: [U-Boot] Interrupt issue about bootvx command

2011-10-13 Thread Wolfgang Denk
Dear Peter Pan, please stop top posting / full quoting. In message you wrote: > > And the following is my updated patch. My git does not have "git > send-mail" supported, so I have to paste the formatted commit below. > Sorry for that. Well, you'd better spend the time installing a full-feat

Re: [U-Boot] [PATCH v3 5/6] fdt: ARM: Add fdtcontroladdr to set device tree address in environment

2011-10-13 Thread Mike Frysinger
On Tuesday 11 October 2011 18:26:10 Simon Glass wrote: > --- a/arch/arm/lib/board.c > +++ b/arch/arm/lib/board.c > > +/** > + * Decode the value of an environment variable and return it. > + * > + * @param name Name of environemnt variable > + * @param base Number base

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-13 Thread Wolfgang Denk
Dear Anton Staaf, In message you wrote: > > Turns out I was able to define ARCH_DMA_MINALIGN for all U-Boot > architectures and use that > instead of the actual cache line size to generate correctly aligned > buffers. My latest > patch set implements this. So we shouldn't have any warnings bei

Re: [U-Boot] [PATCH v3 6/6] fdt: add decode helper library

2011-10-13 Thread Mike Frysinger
On Tuesday 11 October 2011 18:26:11 Simon Glass wrote: > --- /dev/null > +++ b/include/fdtdec.h > > +/* > + * A typedef for a physical address. Note that fdt data is always big > + * endian even on a litle endian machine. > + */ > +#ifdef CONFIG_PHYS_64BIT > +typedef u64 addr_t; > +#define ADDR_T_N

[U-Boot] [PATCH] powerpc/p3060: remove all references to RCW bits EC1_EXT, EC2_EXT, and EC3

2011-10-13 Thread Timur Tabi
The EC1_EXT, EC2_EXT, and EC3 bits in the RCW don't officially exist on the P3060 and should always be set to zero. Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/p3060_serdes.c | 20 arch/powerpc/include/asm/immap_85xx.h | 10 -- drivers/net/fm/p3060.c

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-13 Thread Mike Frysinger
On Thursday 13 October 2011 16:03:37 Wolfgang Denk wrote: > Anton Staaf wrote: > > ... I could move it from common.h to a c file that is always > > > > built. Perhaps I could add a checks.c file to libgeneric? I'm not > > really sure if that's the right place for it. Do you have a

Re: [U-Boot] checkpatch compliance

2011-10-13 Thread Wolfgang Denk
Dear Joe Hershberger, In message you wrote: > > WARNING:NEW_TYPEDEFS: do not add new typedefs > This seems rather limiting... I'm not sure why even Linux would want > this, at least when it applies to typedefs of structs. It makes sense > if it's a new typedef for int or something. See the Cod

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-13 Thread Anton Staaf
On Thu, Oct 13, 2011 at 1:31 PM, Wolfgang Denk wrote: > Dear Anton Staaf, > > In message > you > wrote: >> >> Turns out I was able to define ARCH_DMA_MINALIGN for all U-Boot >> architectures and use that >> instead of the actual cache line size to generate correctly aligned >> buffers.  My late

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-13 Thread Anton Staaf
On Thu, Oct 13, 2011 at 1:36 PM, Mike Frysinger wrote: > On Thursday 13 October 2011 16:03:37 Wolfgang Denk wrote: >> Anton Staaf wrote: >> >              ... I could move it from common.h to a c file that is always >> > >> > built.  Perhaps I could add a checks.c file to libgeneric?  I'm not >> >

Re: [U-Boot] [PATCH 01/15 V3] iMX28: Initial support for iMX28 CPU

2011-10-13 Thread Wolfgang Denk
Dear Marek Vasut, In message <201110132014.55510.marek.va...@gmail.com> you wrote: > > the patch is big and will need approval to be let into ML. Removing the space > in > char * const argv[] shall do the trick, can you please do that by hand ? It will do the trick and trigger a new checkpatch

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-13 Thread Wolfgang Denk
Dear Anton Staaf, In message you wrote: > > I believe it is correct yes. But it would be good to have each > architecture custodian > weigh in on the architectures I couldn't test directly. Also, Mike > has submitted a > patch that adds the asm/cache.h file for blackfin from the Linux > kernel

[U-Boot] [PATCH 0/9] tegra2: Tidy up boot path

2011-10-13 Thread Simon Glass
On Tegra2 the AVP runs the normal U-Boot code to a point, then halts and the A9 takes over. The current Tegra2 boot path is fairly complex, since it has a separate path and code for the Cortex-A9 and the AVP. In fact, they can largely execute the same code path. This series cleans up this logic an

[U-Boot] [PATCH 9/9] tegra2: Don't use board pointer before it is set up

2011-10-13 Thread Simon Glass
In board_init_f() the gd->bd pointer is not valid when dram_init() is called. This only avoids dying because DRAM is at zero on Tegra2. The common ARM routine sets up the banks in the same way anyway, so we can just remove this code. Signed-off-by: Simon Glass --- arch/arm/cpu/armv7/tegra2/board

[U-Boot] [PATCH 1/9] tegra2: Add arch_cpu_init() to fire up Cortex-A9

2011-10-13 Thread Simon Glass
We want to move away from a special Tegra2 start-up, and just use arch_cpu_init() instead. However, if we run board_init_f() from boot we need to build it for ARMv4T, since the Tegra's AVP start-up CPU does not support ARMv7. The effect of this is to do the AVP init earlier, and in arch_cpu_init()

[U-Boot] [PATCH 8/9] tegra2: Remove unneeded 'dynamic ram size' message

2011-10-13 Thread Simon Glass
This message is not required, since it is followed by an 'official' U-Boot message. U-Boot 2011.03-00048-gd7cb0d3 (May 11 2011 - 17:17:23) TEGRA2 Board: NVIDIA Seaboard dynamic ram_size = 1073741824 DRAM: 1 GiB becomes: TEGRA2 Board: NVIDIA Seaboard DRAM: 1 GiB This is a separate commit sinc

[U-Boot] [PATCH 5/9] tegra2: Remove unneeded boot code

2011-10-13 Thread Simon Glass
Since we have cache support built in we can remove Tegra's existing cache initialization code amd other related dead code. Signed-off-by: Simon Glass --- arch/arm/cpu/armv7/start.S| 12 --- arch/arm/cpu/armv7/tegra2/ap20.h |7 +-- arch/arm/cpu/armv7/tegra2/board.c

[U-Boot] [PATCH 7/9] tegra2: Remove unused low-level Tegra2 UART code

2011-10-13 Thread Simon Glass
This was used by the AVP in early boot but is no longer used. Unless we plan to enable it somehow it is not needed. In any case we should try to use the ns16550 driver instead as it has the same code. Signed-off-by: Simon Glass --- drivers/serial/Makefile|1 - drivers/serial/serial_t

[U-Boot] [PATCH 4/9] tegra2: Enable instruction cache

2011-10-13 Thread Simon Glass
Since low-level init is skipped, the instruction cache is never enabled on Tegra2. This explicitly calls this initialization as soon as the A9 is initialized. Signed-off-by: Simon Glass --- board/nvidia/common/board.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/boa

[U-Boot] [PATCH 2/9] tegra2: Simplify tegra_start() boot path

2011-10-13 Thread Simon Glass
The Tegra2 boot path is more complicated than it needs to be. Since we want to move to building most of U-Boot with ARMv7 and only a small part with ARMv4T (for AVP) it should be as simple as possible. This makes tegra2_start() into a simple function which either does AVP init or A9 init depending

[U-Boot] [PATCH 3/9] arm: Move CP15 init out of cpu_init_crit()

2011-10-13 Thread Simon Glass
Some SOCs have do not start up with their 'main' CPU. The first U-Boot code may then be executed with a CPU which does not have a CP15, or not a useful one. Here we split the initialization of CP15 into a separate call, which can be performed later if required. Once the main CPU is running, you s

[U-Boot] [PATCH 6/9] tegra2: Remove unneeded config option

2011-10-13 Thread Simon Glass
CONFIG_ENABLE_CORTEXA9 and CONFIG_SKIP_RELOCATE_UBOOT are not needed, so remove them. Signed-off-by: Simon Glass --- include/configs/tegra2-common.h |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/include/configs/tegra2-common.h b/include/configs/tegra2-common.h index 3

Re: [U-Boot] [PATCH v3 5/6] fdt: ARM: Add fdtcontroladdr to set device tree address in environment

2011-10-13 Thread Simon Glass
Hi Mike, On Thu, Oct 13, 2011 at 1:27 PM, Mike Frysinger wrote: > On Tuesday 11 October 2011 18:26:10 Simon Glass wrote: >> --- a/arch/arm/lib/board.c >> +++ b/arch/arm/lib/board.c >> >> +/** >> + * Decode the value of an environment variable and return it. >> + * >> + * @param name              

[U-Boot] [PATCH 2/2] common: cosmetic: CONFIG_BOOTFILE checkpatch compliance

2011-10-13 Thread Joe Hershberger
Remove MK_STR from places that consume CONFIG_BOOTFILE to force all definitions to be string literals. Signed-off-by: Joe Hershberger Cc: Joe Hershberger Cc: Wolfgang Denk --- common/env_common.c |2 +- common/env_embedded.c|2 +- doc/README.usb

[U-Boot] [PATCH 1/2] common: cosmetic: CONFIG_ROOTPATH checkpatch compliance

2011-10-13 Thread Joe Hershberger
Remove MK_STR from places that consume CONFIG_ROOTPATH to force all definitions to be string literals. Signed-off-by: Joe Hershberger Cc: Joe Hershberger Cc: Wolfgang Denk --- common/env_common.c |2 +- common/env_embedded.c |2 +- include/configs/DB64360.h

[U-Boot] [PATCH 0/8][v2] DaVinci EMAC for AM35x and HTKW mcx board support

2011-10-13 Thread Ilya Yanok
Hello, These series bring support of DaVinci EMAC interface on TI AM35xx processors as well as support for HTKW mcx board which uses it. Changes: 1. Common defines and register definitions moved to drivers/net/davinci_emac.h 2. Added coding/decoding of internal RAM addresses. 3. DaVinci-specifi

[U-Boot] [PATCH 5/8] davinci_emac: hardcode 100Mbps for AM35xx and RMII

2011-10-13 Thread Ilya Yanok
For some reason code setting the speed based on the PHY feedback causes troubles on AM3517 so hardcode 100Mbps for now. Signed-off-by: Ilya Yanok --- drivers/net/davinci_emac.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/drivers/net/davinci_emac.c b/drivers/net/dav

[U-Boot] [PATCH 4/8] davinci_emac: fix for running with dcache enabled

2011-10-13 Thread Ilya Yanok
DaVinci EMAC is present on TI AM35xx SoCs (ARMv7) which run with D-Cache enabled by default. So we have to take care and flush/invalidate the cache before/after the DMA operations. Please note that the receive buffer alignment to 32 byte boundary comes from the old driver version I don't know if i

[U-Boot] [PATCH 7/8] AM3517: move AM3517 specific mux defines to generic header

2011-10-13 Thread Ilya Yanok
AM3517 specific CONTROL_PADCONF_* defines moved from board-specific files to Signed-off-by: Ilya Yanok --- arch/arm/include/asm/arch-omap3/mux.h | 41 + board/logicpd/am3517evm/am3517evm.h | 40 board/ti/am3517crane/am3517cr

[U-Boot] [PATCH 2/8] davinci_emac: use internal addresses in buffer descriptors

2011-10-13 Thread Ilya Yanok
On AM35xx CPPI RAM had different addresses when accessed from the CPU and from the EMAC. We need to account this to deal with the buffer descriptors correctly. Signed-off-by: Ilya Yanok --- drivers/net/davinci_emac.c | 39 ++- 1 files changed, 30 insertions(

[U-Boot] [PATCH 1/8] davinci_emac: move arch-independent defines to separate header

2011-10-13 Thread Ilya Yanok
DaVinci EMAC is found not only on DaVinci SoCs but on some OMAP3 SoCs also. This patch moves common defines from arch-davinci/emac_defs.h to drivers/net/davinci_emac.h Signed-off-by: Ilya Yanok --- arch/arm/include/asm/arch-davinci/emac_defs.h | 293 --- drivers/net/davinci_

  1   2   >