Re: [U-Boot] [PATCH] Introduce generic TPM support in u-boot

2011-10-09 Thread Mike Frysinger
On Sunday 09 October 2011 22:53:26 Vadim Bendebury wrote: > Change-Id: I22a33c3e5b2e20eec9557a7621bd463b30389d73 these don't make sense outside of gerrit ;) > --- a/Makefile > +++ b/Makefile > > +ifneq ($(CONFIG_GENERIC_LPC_TPM),) please use: ifeq ($(CONFIG_...),y) > --- a/README > +++

Re: [U-Boot] [PATCH v5 01/20] sandbox: Add architecture header files

2011-10-09 Thread Simon Glass
Hi, On Sun, Oct 9, 2011 at 6:03 PM, Mike Frysinger wrote: > On Sunday 09 October 2011 15:28:16 Wolfgang Denk wrote: >> Simon Glass wrote: >> > 0 errors, 44 warnings for >> > 0001-sandbox-Add-architecture-header-files.patch: warning: >> > arch/sandbox/include/asm/bitops.h,30: Use of volatile is us

Re: [U-Boot] [PATCH v3 19/21] Use uintptr_t for 32/64-bit compatibility

2011-10-09 Thread Simon Glass
Hi Mike, On Sun, Oct 9, 2011 at 7:54 PM, Mike Frysinger wrote: > On Tuesday 04 October 2011 01:24:56 Simon Glass wrote: >> On Mon, Oct 3, 2011 at 11:57 AM, Mike Frysinger wrote: >> > On Monday, September 26, 2011 20:10:53 Simon Glass wrote: >> >> --- a/common/cmd_mem.c >> >> +++ b/common/cmd_mem.

Re: [U-Boot] [PATCH v3 20/21] Adjust dependency rules to permit per-file flags

2011-10-09 Thread Simon Glass
Hi Mike, On Sun, Oct 9, 2011 at 7:50 PM, Mike Frysinger wrote: > On Monday 03 October 2011 21:10:17 Simon Glass wrote: >> Hi, >> >> On Mon, Sep 26, 2011 at 5:10 PM, Simon Glass wrote: >> > The dependency rules are currently done in a shell 'for' loop. This does >> > not permit Makefile variables

Re: [U-Boot] [PATCH 0/2]: OMAP3 SPL updates

2011-10-09 Thread Tom Rini
On Sun, Oct 9, 2011 at 2:21 AM, Igor Grinberg wrote: > On 10/07/11 02:28, Tom Rini wrote: >> On Tue, Oct 4, 2011 at 7:59 AM, Tom Rini wrote: >>> Hey all, >>> >>> The following short series does two things.  First, it re-works where >>> we do memory initalization in SPL for OMAP3 to be board-speci

Re: [U-Boot] [PATCH v3 19/21] Use uintptr_t for 32/64-bit compatibility

2011-10-09 Thread Mike Frysinger
On Tuesday 04 October 2011 01:24:56 Simon Glass wrote: > On Mon, Oct 3, 2011 at 11:57 AM, Mike Frysinger wrote: > > On Monday, September 26, 2011 20:10:53 Simon Glass wrote: > >> --- a/common/cmd_mem.c > >> +++ b/common/cmd_mem.c > >> > >> printf ("\nMem error @ 0x%08

[U-Boot] [PATCH] Introduce generic TPM support in u-boot

2011-10-09 Thread Vadim Bendebury
TPM (Trusted Platform Module) is an integrated circuit and software platform that provides computer manufacturers with the core components of a subsystem used to assure authenticity, integrity and confidentiality. This driver supports version 1.2 of the TCG (Trusted Computing Group) specifications

Re: [U-Boot] [PATCH v3 20/21] Adjust dependency rules to permit per-file flags

2011-10-09 Thread Mike Frysinger
On Monday 03 October 2011 21:10:17 Simon Glass wrote: > Hi, > > On Mon, Sep 26, 2011 at 5:10 PM, Simon Glass wrote: > > The dependency rules are currently done in a shell 'for' loop. This does > > not permit Makefile variables to adjust preprocessor flags as is done > > with normal compile flags,

Re: [U-Boot] [PATCH 1/5 v1] integrator: pass configs for core modules

2011-10-09 Thread Linus Walleij
On Wed, Sep 21, 2011 at 1:00 PM, Linus Walleij wrote: > Albert do you want this series through patchwork or through > a git pull request? Ping on this... I'm happy to help. > The patchwork entries are here: > > http://patchwork.ozlabs.org/patch/115217/ > http://patchwork.ozlabs.org/patch/115218

Re: [U-Boot] [PATCH] cmd_bdinfo: replace print_str() with print_mhz()

2011-10-09 Thread Mike Frysinger
Tested-by: Mike Frysinger -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v5 01/20] sandbox: Add architecture header files

2011-10-09 Thread Mike Frysinger
On Sunday 09 October 2011 15:28:16 Wolfgang Denk wrote: > Simon Glass wrote: > > 0 errors, 44 warnings for > > 0001-sandbox-Add-architecture-header-files.patch: warning: > > arch/sandbox/include/asm/bitops.h,30: Use of volatile is usually wrong: > > see Documentation/volatile-considered-harmful.txt

Re: [U-Boot] [PATCH v2] powerpc/mpc8548cds: Update config file

2011-10-09 Thread Tabi Timur-B04825
Kumar Gala wrote: > * The only things not explained is the env address change. I can see > it matches our other boards MPC8572DS, etc. So not exactly critical to > me. There's no explanation for increasing the malloc size. > * If Wolfgang wants to remove CONFIG_PCI_SCAN_SHOW so be it, but as >

Re: [U-Boot] [PATCH v2] powerpc/mpc8548cds: Update config file

2011-10-09 Thread Kumar Gala
On Oct 9, 2011, at 6:41 PM, Tabi Timur-B04825 wrote: > On Sun, Oct 9, 2011 at 5:55 PM, Kumar Gala wrote: >> >> - Increase the size of malloc space. >> - Enable e1000 network card. >> - Show pci devices on startup. >> - Change the location of env address. >> - Use hwconfig to turn off ECC by def

Re: [U-Boot] [PATCH v2] powerpc/mpc8548cds: Update config file

2011-10-09 Thread Tabi Timur-B04825
On Sun, Oct 9, 2011 at 5:55 PM, Kumar Gala wrote: > > - Increase the size of malloc space. > - Enable e1000 network card. > - Show pci devices on startup. > - Change the location of env address. > - Use hwconfig to turn off ECC by default. This patch is missing all of the explanations that were p

[U-Boot] [GIT PULL] Pull request u-boot-85xx.git (updated)

2011-10-09 Thread Kumar Gala
[ fixed mpc8548cds config patch to pass checkpatch ] The following changes since commit 0841ca90f22d73b0ea4642ef1ce33d879bb2f3ff: Wolfgang Denk (1): arm920t/s3c24x0/usb_ohci.c: fix warning: variable ... set but not used are available in the git repository at: git://git.denx.de/u-boot

[U-Boot] [PATCH v2] powerpc/mpc8548cds: Update config file

2011-10-09 Thread Kumar Gala
From: chenhui zhao - Increase the size of malloc space. - Enable e1000 network card. - Show pci devices on startup. - Change the location of env address. - Use hwconfig to turn off ECC by default. [Kumar Gala] Fixed white space formating for CONFIG_EXTRA_ENV_SETTINGS Signed-off-by: Zhao Chenhui

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

2011-10-09 Thread Kumar Gala
On Oct 9, 2011, at 1:34 PM, Wolfgang Denk wrote: > Dear Kumar Gala, > > In message you > wrote: >> The following changes since commit 08e4f700ac450ddc5d4819e23df5bc46eb767464: >> Marek Vasut (1): >> SPL: Make path to start.S configurable >> >> are available in the git repository at: >> g

Re: [U-Boot] [PATCH v5 01/20] sandbox: Add architecture header files

2011-10-09 Thread Simon Glass
Hi Wolfgang, On Sun, Oct 9, 2011 at 12:28 PM, Wolfgang Denk wrote: > Dear Simon Glass, > > In message <1318031631-13643-2-git-send-email-...@chromium.org> you wrote: >> This adds required header files for the sandbox architecture, and a basic >> description of what sandbox is (README.sandbox). >>

Re: [U-Boot] [PATCH v2 0/7] Add cache line alignment support

2011-10-09 Thread Anton Staaf
On Sun, Oct 9, 2011 at 2:22 PM, Wolfgang Denk wrote: > Dear Anton Staaf, > > In message <1317763491-7274-1-git-send-email-robot...@chromium.org> you wrote: >> The cache line alignment issue has gone around a couple of times now.  This >> patch set implements all of the details that we have discuss

Re: [U-Boot] [PATCH 1/2] ppc: Include in common.h

2011-10-09 Thread Anton Staaf
On Sun, Oct 9, 2011 at 12:11 PM, Wolfgang Denk wrote: > Dear Anton Staaf, > > In message > you > wrote: >> >> > This is needed for the patch "cache: add default setting for >> > CONFIG_SYS_CACHELINE_SIZE" from Anton Staaf. As cache.h defines >> > CONFIG_SYS_CACHELINE_SIZE for PPC targets. >> >

Re: [U-Boot] [PATCH v2 0/7] Add cache line alignment support

2011-10-09 Thread Wolfgang Denk
Dear Anton Staaf, In message <1317763491-7274-1-git-send-email-robot...@chromium.org> you wrote: > The cache line alignment issue has gone around a couple of times now. This > patch set implements all of the details that we have discussed about the > implementation of ALLOC_CACHE_ALIGN_BUFFER oth

Re: [U-Boot] [PATCH] README: fix typos and such.

2011-10-09 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1318187362-9798-1-git-send-email...@denx.de> you wrote: > Reported-by: Michael Jones > Signed-off-by: Wolfgang Denk > --- > README | 13 + > 1 files changed, 5 insertions(+), 8 deletions(-) Applied, thanks. Best regards, Wolfgang Denk -- DENX S

Re: [U-Boot] [PATCH] README: fix documentation of CONFIG_SHOW_BOOT_PROGRESS

2011-10-09 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1317974690-6739-1-git-send-email...@denx.de> you wrote: > SOme previous changes added code right in the middle of the > description of CONFIG_SHOW_BOOT_PROGRESS. Move this text down. > Fix formatting while we are at it. > > Signed-off-by: Wolfgang Denk > --- > R

Re: [U-Boot] [PATCH 1/7] arch/arm/lib/board.c: fix warning: variable ... set but not used

2011-10-09 Thread Wolfgang Denk
Dear Simon Glass, In message you wrote: > > Sorry if I have missed a mailing list patch, but in my version of the > source there is a line a little lower down which uses bd: > >mmc_initialize(bd); > > Of course it can just become gd->bd so there is no problem removing > the variable.

Re: [U-Boot] [PATCH 7/7] common/usb.c: fix warning: variable ... set but not used

2011-10-09 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1317849453-8216-7-git-send-email...@denx.de> you wrote: > Fix: > usb.c: In function 'usb_parse_config': > usb.c:331:17: warning: variable 'ch' set but not used > [-Wunused-but-set-variable] > usb.c: In function 'usb_hub_port_connect_change': > usb.c:1123:29: warnin

Re: [U-Boot] [PATCH 6/7] drivers/usb/host/ohci-hcd.c: fix warning: variable ... set but not used

2011-10-09 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1317849453-8216-6-git-send-email...@denx.de> you wrote: > Fix: > ohci-hcd.c: In function 'dl_transfer_length': > ohci-hcd.c:968:8: warning: variable 'tdINFO' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > Cc: Remy Bohmer > ---

Re: [U-Boot] [PATCH 5/7] common/cmd_usb.c: fix warning: variable ... set but not used

2011-10-09 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1317849453-8216-5-git-send-email...@denx.de> you wrote: > Fix: > cmd_usb.c: In function 'usb_show_tree_graph': > cmd_usb.c:284:29: warning: variable 'port' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > Cc: Remy Bohmer > --- >

Re: [U-Boot] [PATCH 4/7] drivers/net/smc91111.c: fix warning: variable ... set but not used

2011-10-09 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1317849453-8216-4-git-send-email...@denx.de> you wrote: > Fix: > smc9.c: In function 'smc_phy_configure': > smc9.c:1194:6: warning: variable 'failed' set but not used > [-Wunused-but-set-variable] > smc9.c:1190:7: warning: variable 'phyaddr' set but not

Re: [U-Boot] [PATCH 3/7] fs/jffs2/jffs2_1pass.c: fix warning: variable ... set but not used

2011-10-09 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1317849453-8216-3-git-send-email...@denx.de> you wrote: > Fix: > jffs2_1pass.c: In function 'jffs2_1pass_read_inode': > jffs2_1pass.c:699:7: warning: variable 'ret' set but not used > [-Wunused-but-set-variable] > jffs2_1pass.c: In function 'jffs2_1pass_build_lists

Re: [U-Boot] [PATCH 2/7] CFI: fix warning: variable ... set but not used

2011-10-09 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1317849453-8216-2-git-send-email...@denx.de> you wrote: > Fix: > cfi_mtd.c: In function 'cfi_mtd_init': > cfi_mtd.c:226:19: warning: variable 'mtd_list' set but not used > [-Wunused-but-set-variable] > cfi_mtd.c: In function 'cfi_mtd_init': > cfi_mtd.c:226:19: warn

Re: [U-Boot] [PATCH 1/7] arch/arm/lib/board.c: fix warning: variable ... set but not used

2011-10-09 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1317849453-8216-1-git-send-email...@denx.de> you wrote: > Fix: > board.c:445:8: warning: variable 'bd' set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Wolfgang Denk > Cc: Albert ARIBAUD > --- > arch/arm/lib/board.c |2 -- > 1 files changed

Re: [U-Boot] [PATCH V3] cmd_time: add time command

2011-10-09 Thread Wolfgang Denk
Dear Che-Liang Chiou, In message <1317980448-25774-1-git-send-email-clch...@chromium.org> you wrote: > The 'time' command runs and reports execution time of commands. > > Sample usage: > > u-boot# time crc 0x1000 1000 > CRC32 for 1000 ... 1fff ==> ae94dc4b > > time:

Re: [U-Boot] [PATCH] common/image.c: remove duplicated IH_TYPE entries.

2011-10-09 Thread Wolfgang Denk
Dear Wolfgang Denk, In message <1317969361-24887-1-git-send-email...@denx.de> you wrote: > Also rearrange IH_TYPE table slightly. > > Signed-off-by: Wolfgang Denk > --- > common/image.c |9 +++-- > 1 files changed, 3 insertions(+), 6 deletions(-) Applied, thanks. Best regards, Wolfga

Re: [U-Boot] [PATCH v4 1/4] USB: move ehci.h and ehci-core.h into /include/usb/

2011-10-09 Thread Jana Rapava
2011/10/9 Marek Vasut > On Sunday, September 25, 2011 07:25:07 PM Jana Rapava wrote: > > Signed-off-by: Jana Rapava > > Cc: Marek Vasut > > Cc: Remy Bohmer > > Cc: Fabio Estevam > > --- > > Changes for v2: > > - changed to proper patch > > Changes for v3: > > - merged other USB pa

Re: [U-Boot] [PATCH 2/2] ppc: Fix DBSR_IAx defines in

2011-10-09 Thread Wolfgang Denk
Dear Stefan Roese, In message <1317985549-11206-1-git-send-email...@denx.de> you wrote: > While adding asm/cache.h to common.h for PPC targets, I got an > error about multiple definitions of some DBSR_ macros. While > scanning these defines, I noticed that some where defined not > correctly for al

Re: [U-Boot] [PATCH 1/2] ppc: Include in common.h

2011-10-09 Thread Wolfgang Denk
Dear Stefan Roese, In message <1317985534-11135-1-git-send-email...@denx.de> you wrote: > This is needed for the patch "cache: add default setting for > CONFIG_SYS_CACHELINE_SIZE" from Anton Staaf. As cache.h defines > CONFIG_SYS_CACHELINE_SIZE for PPC targets. > > This will remove the following

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

2011-10-09 Thread Stany MARCEL
>>> From: Stany MARCEL [mailto:stany.mar...@novasys-ingenierie.com] >>> Sent: Wednesday, September 14, 2011 8:44 PM >>> To: u-boot@lists.denx.de >>> Cc: Jin Zhengxiong-R64188; Jin Zhengxiong-R64188; Stany MARCEL >>> Subject: [PATCH 1/6] ColdFire: Cleanup lds files for multiple defined >>> symbols

Re: [U-Boot] [PATCH 1/2] USB: Rework USB keyboard driver

2011-10-09 Thread Wolfgang Denk
Dear Marek Vasut, In message <201110092112.18014.marek.va...@gmail.com> you wrote: > > > > +#undef USB_KBD_DEBUG > > > > Please do not undef what is not defined [or what a user might want to > > define on the command line]. > > Command line ? This is a placeholder in case the user wants to deb

Re: [U-Boot] [PATCH][V3] drivers: add the support for Silicon Image SATA controller

2011-10-09 Thread Wolfgang Denk
Dear Tang Yuantian-B29983, In message <3c06c26914daca4bb3a368f78ca0b3a70ea...@039-sn1mpn1-005.039d.mgd.msft.net> you wrote: > > Did you say the following warnings: > - > WARNING: Use #include instead of > #79: FILE: drivers/block/sata_sil.c:26: > +#include We should add this to a U-

Re: [U-Boot] [PATCH 2/2] api: export LCD and video to external apps

2011-10-09 Thread Wolfgang Denk
Dear Che-liang Chiou, In message you wrote: > Hi Wolfgang Denk, > > I have added following comments to this call in patch V2. > + /*+* This only clears me= > ssages on screen, not on > serial port. It is+* equivalent to a no-op if n= > o display is > available.+

Re: [U-Boot] [PATCH v4] OMAP3: mvblx: Initial support for mvBlueLYNX-X

2011-10-09 Thread Wolfgang Denk
Dear Michael Jones, In message <4e8ec11b.8040...@matrix-vision.de> you wrote: > > But while we're on the topic- Most of those old warnings were because my > board/matrix_vision/mvblx/mvblx.h was based on board/ti/beagle/beagle.h, > which would've itself had lots of checkpatch warnings from long

Re: [U-Boot] [PATCH v2 1/5] usb:gadget:s5p USB Device Controller (UDC) implementation

2011-10-09 Thread Wolfgang Denk
Dear Lukasz Majewski, In message <20111007093950.17d94287@lmajewski.digital.local> you wrote: > > Most warnings are related to lines over 80 characters. > This code is a debug code. As fair as I remember debug print code shall > not be break in Linux kernel for easier debugging. Can you please p

Re: [U-Boot] [PATCH v5 01/20] sandbox: Add architecture header files

2011-10-09 Thread Wolfgang Denk
Dear Simon Glass, In message <1318031631-13643-2-git-send-email-...@chromium.org> you wrote: > This adds required header files for the sandbox architecture, and a basic > description of what sandbox is (README.sandbox). > > This commit generates a list of 44 checkpatch warnings: This should go t

Re: [U-Boot] checkpatch compliance

2011-10-09 Thread Wolfgang Denk
Dear Joe Hershberger, In message you wrote: > > I'm attempting to make the files I touched in several recent > patch-series chechkpatch.pl compliant. > > I've hit several cases which fail and probably shouldn't. For each of > these cases, should the warning / error just be ignored or reported

Re: [U-Boot] [PATCH 1/2] USB: Rework USB keyboard driver

2011-10-09 Thread Marek Vasut
On Sunday, October 09, 2011 08:54:07 PM Wolfgang Denk wrote: > Dear Marek Vasut, > > In message <1317990657-17214-2-git-send-email-marek.va...@gmail.com> you wrote: > > Also, fix usb drivers which use extern new. > > > > Signed-off-by: Marek Vasut > > Cc: Ajay Kumar Gupta > > Cc: Bryan Wu > >

Re: [U-Boot] [PATCH 1/2] ppc: Include in common.h

2011-10-09 Thread Wolfgang Denk
Dear Anton Staaf, In message you wrote: > > > This is needed for the patch "cache: add default setting for > > CONFIG_SYS_CACHELINE_SIZE" from Anton Staaf. As cache.h defines > > CONFIG_SYS_CACHELINE_SIZE for PPC targets. > > > > This will remove the following warnings/errors: > > > > include/co

[U-Boot] [PATCH] README: fix typos and such.

2011-10-09 Thread Wolfgang Denk
Reported-by: Michael Jones Signed-off-by: Wolfgang Denk --- README | 13 + 1 files changed, 5 insertions(+), 8 deletions(-) diff --git a/README b/README index e0a4d53..c5cfb13 100644 --- a/README +++ b/README @@ -627,7 +627,7 @@ The following options need to be configured:

Re: [U-Boot] [PATCH] README: fix documentation of CONFIG_SHOW_BOOT_PROGRESS

2011-10-09 Thread Wolfgang Denk
Dear Michael Jones, In message <4e8ed1ef.8020...@matrix-vision.de> you wrote: > > > SOme previous changes added code right in the middle of the > > typo- double capitalization Thanks for pointing out. > > + bytes are output before the console is initialised, the > > two spaces betw

Re: [U-Boot] Can I boot at image from a jffs2 filesystem

2011-10-09 Thread Wolfgang Denk
Dear Owain, In message <0e168e936a32d3409718ad71277afe210988085...@mchp057a.global-ad.net> you wrote: > > I am trying to squeeze new features onto a uboot based system where we have> > run out of flash. > In tandum with general optimisation of apps, removal of redundant code etc > > I wanted t

Re: [U-Boot] [PATCH 1/2] USB: Rework USB keyboard driver

2011-10-09 Thread Wolfgang Denk
Dear Marek Vasut, In message <1317990657-17214-2-git-send-email-marek.va...@gmail.com> you wrote: > Also, fix usb drivers which use extern new. > > Signed-off-by: Marek Vasut > Cc: Ajay Kumar Gupta > Cc: Bryan Wu > Cc: Cliff Cai > Cc: Mike Frysinger > Cc: Remy Bohmer > Cc: Wolfgang Denk >

Re: [U-Boot] [PATCH v4 1/4] USB: move ehci.h and ehci-core.h into /include/usb/

2011-10-09 Thread Marek Vasut
On Sunday, September 25, 2011 07:25:07 PM Jana Rapava wrote: > Signed-off-by: Jana Rapava > Cc: Marek Vasut > Cc: Remy Bohmer > Cc: Fabio Estevam > --- > Changes for v2: > - changed to proper patch > Changes for v3: > - merged other USB patches from u-boot-pxa/efikasb > - offs

Re: [U-Boot] [PATCH] image: Implement IH_TYPE_KERNEL_REL

2011-10-09 Thread Wolfgang Denk
Dear Stephen Warren, In message <74cdbe0f657a3d45afbb94109fb122ff173b23b...@hqmail01.nvidia.com> you wrote: > > > But please define "first bank" - does it mena the firs one > > initialized, or the lowest start address, or the lowest chip select, > > or ... ? > > It's the following currently: >

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

2011-10-09 Thread Wolfgang Denk
Dear Kumar Gala, In message you wrote: > The following changes since commit 08e4f700ac450ddc5d4819e23df5bc46eb767464: > Marek Vasut (1): > SPL: Make path to start.S configurable > > are available in the git repository at: > git://git.denx.de/u-boot-mpc85xx master > > Andy Fleming (

Re: [U-Boot] [PATCH 6/9] powerpc/mpc8548cds: Update config file

2011-10-09 Thread Wolfgang Denk
Dear Kumar Gala, In message <9c8cc940-a952-4323-8a48-bd6fb389a...@kernel.crashing.org> you wrote: > > > total: 0 errors, 1 warnings, 50 lines checked > > I don't agree with this checkpatch warning. The extra space adds to = > readability so leaving this alone. Indentation must be done by TABs.

Re: [U-Boot] [PATCH 04/15] Only delay boot if keyboard open

2011-10-09 Thread Mike Frysinger
On Saturday 08 October 2011 19:29:27 Pali Rohár wrote: > On Thursday 01 September 2011 09:58:06 you wrote: > > On Thursday, September 01, 2011 07:34:30 Pali Rohár wrote: > > > board/nokia/rx51/rx51.c |7 +++ > > > include/configs/nokia_rx51.h |1 + > > > > this is a new board port

Re: [U-Boot] [PATCH 03/15] Nokia RX-51 aka N900 support

2011-10-09 Thread Mike Frysinger
On Saturday 08 October 2011 20:20:47 Pali Rohár wrote: > On Thursday 01 September 2011 09:57:13 you wrote: > > On Thursday, September 01, 2011 07:34:29 Pali Rohár wrote: > > > --- /dev/null > > > +++ b/board/nokia/rx51/rx51.c > > > > > > +GraphicDevice gdev; > > > > no camel case > > GraphicDevi

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

2011-10-09 Thread Mike Frysinger
On Sunday 09 October 2011 06:41:47 Ilya Yanok wrote: > On 07.10.2011 21:34, Mike Frysinger wrote: > >> +#ifdef DAVINCI_EMAC_DCACHE > >> +static inline void davinci_flush(void *addr, int size) > >> +{ > >> + flush_dcache_range((unsigned long)addr, > >> + (unsigned long)addr + size)

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

2011-10-09 Thread Laurence Withers
On Sun, Oct 09, 2011 at 02:41:47PM +0400, Ilya Yanok wrote: > arm926ejs doesn't have {invalidate,flush}_dcache_range(), so we have to > add this not to break the driver on DaVinci boards (maybe we need to add > empty cache functions on arm926ejs instead?) Even better would be working cache functio

Re: [U-Boot] [PATCH 1/2] USB: Rework USB keyboard driver

2011-10-09 Thread Marek Vasut
On Sunday, October 09, 2011 11:01:36 AM Remy Bohmer wrote: > Hi Marek, > > >> > > it's hard to evaluate each piece by itself when it's just one patch > >> > > smooshing it all together > >> > > >> > Well how would you rework crap code piece by piece? > >> > >> - fix style > >> - abstract out stu

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

2011-10-09 Thread Ilya Yanok
Hi Mike, On 07.10.2011 21:34, Mike Frysinger wrote: >> +#ifdef DAVINCI_EMAC_DCACHE >> +static inline void davinci_flush(void *addr, int size) >> +{ >> +flush_dcache_range((unsigned long)addr, >> +(unsigned long)addr + size); >> +} >> + >> +static inline void davinci_invalid

Re: [U-Boot] [PATCH v2 2/2] omap4_panda: Build in cmd_gpio support on panda

2011-10-09 Thread Igor Grinberg
Hi Joe, On 10/06/11 23:50, Joe Hershberger wrote: > Enable the gpio command > > Signed-off-by: Joe Hershberger > Cc: Joe Hershberger > Cc: Sandeep Paulraj > --- > Changes for v2: > - Correct a bad merge... sorry. > > board/ti/panda/panda_mux_data.h |2 +- > include/configs/omap4_panda.

Re: [U-Boot] [PATCH 0/2]: OMAP3 SPL updates

2011-10-09 Thread Igor Grinberg
On 10/07/11 02:28, Tom Rini wrote: > On Tue, Oct 4, 2011 at 7:59 AM, Tom Rini wrote: >> Hey all, >> >> The following short series does two things. First, it re-works where >> we do memory initalization in SPL for OMAP3 to be board-specific. >> This is needed since in many cases (beagleboard, om

Re: [U-Boot] [PATCH 1/2] USB: Rework USB keyboard driver

2011-10-09 Thread Remy Bohmer
Hi Marek, >> > > it's hard to evaluate each piece by itself when it's just one patch >> > > smooshing it all together >> > >> > Well how would you rework crap code piece by piece? >> >> - fix style >> - abstract out stuff > > Not like there was so much abstraction it couldn't be squashed into this

Re: [U-Boot] [PATCH 07/11] MX5: mx53evk: use new pmic driver

2011-10-09 Thread Jason Hui
Hi, Stefano, On Sat, Oct 8, 2011 at 6:07 PM, Stefano Babic wrote: > Switch to new pmic generic driver. > > Signed-off-by: Stefano Babic > CC: Jason Liu > --- >  board/freescale/mx53evk/mx53evk.c |   15 +-- >  include/configs/mx53evk.h         |    5 +++-- >  2 files changed, 12 inse

[U-Boot] (no subject)

2011-10-09 Thread victor casunuran
how to reset or re program this part item s29gl064n90tf103.. can you please help me thanks vic ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot