Re: [U-Boot] [beagleboard] [PATCH] BeagleBoard: fixed typo in typecast

2011-04-29 Thread Dirk Behme
On 21.04.2011 01:52, Jason Kridner wrote: > Without this patch, you should get a warning. > > Signed-off-by: Jason Kridner > --- > board/ti/beagle/beagle.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/board/ti/beagle/beagle.c b/board/ti/beagle/beagle.c > index 5

Re: [U-Boot] [PATCH v4 2/5] Add Ethernet hardware MAC address framework to usbnet

2011-04-29 Thread Mike Frysinger
On Thursday, April 21, 2011 19:38:06 Wolfgang Denk wrote: > Simon Glass wrote: > > >> + * base_name - base name for device (NULL for "eth") > > > > > > This is an atitifical decision for the API which is difficult to > > > understand. It just makes the code and understanding it more > > > diffi

Re: [U-Boot] [PATCH 2/2][v3] powerpc: make espi can read more than 0xFFFA bytes

2011-04-29 Thread Mike Frysinger
On Tuesday, April 26, 2011 02:31:32 Xie Shaohui-B21989 wrote: > From: Xie Shaohui-B21989 > >espi flash read returns invalid data if the read length is more than > >0xFFFA bytes, it supports maximum transaction of 2^16 bytes at a time, > >resister spcom[TRANLEN] is 16 bits. If the transaction length

Re: [U-Boot] [PATCH 2/2][v2] powerpc: make espi can read more than 0xFFFA bytes

2011-04-29 Thread Mike Frysinger
On Friday, April 22, 2011 05:23:59 Xie Shaohui-B21989 wrote: > From: Mike Frysinger > >On Thu, Apr 21, 2011 at 2:05 AM, Shaohui Xie wrote: > >> espi flash read returns invalid data if the read length is more than > >> 0xFFFA bytes, it supports maximum transaction of 2^16 bytes at a time, > >> resis

Re: [U-Boot] [PATCH v2 3/6] TFTP: rename "server" to "remote"

2011-04-29 Thread Mike Frysinger
On Monday, April 18, 2011 12:19:51 Luca Ceresoli wrote: > With the upcoming TFTP server implementation, the remote node can be > either a client or a server, so avoid ambiguities. the summary made me worried because i thought you were going to rename the env var "server" to "remote". could you t

Re: [U-Boot] [PATCH v2 6/6] TFTP: add tftpsrv command

2011-04-29 Thread Mike Frysinger
On Monday, April 18, 2011 12:19:54 Luca Ceresoli wrote: > +#ifdef CONFIG_CMD_TFTPSRV > +int do_tftpsrv(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) should be static > +U_BOOT_CMD( > + tftpsrv,2, 1, do_tftpsrv, > + "boot image via network acting as a TFTP se

Re: [U-Boot] [RFC] ks8851_mll: add ethernet support

2011-04-29 Thread Mike Frysinger
On Wednesday, March 16, 2011 05:17:19 Raffaele Recalcati wrote: > This driver has been adapted to u-boot starting from ks8851_mll linux > driver. The smsc911x u-boot has been used as reference. if you're basing this driver on other code, then the copyright info in the header should be preserved f

Re: [U-Boot] [PATCH 2/9] mvgbe: Support preserving the existing MAC address

2011-04-29 Thread Mike Frysinger
On Thursday, March 17, 2011 15:46:56 Michael Spang wrote: > while (!eth_getenv_enetaddr(s, dev->enetaddr)) { this logic in the mvgbe is plain broken. it should not be touching the env at all. if there is no eeprom to read, then the mvgbe driver should leave dev- >enetaddr alone.

Re: [U-Boot] [PATCH 2/9] mvgbe: Support preserving the existing MAC address

2011-04-29 Thread Mike Frysinger
On Tuesday, April 26, 2011 00:23:47 Michael Spang wrote: > On Mon, Apr 25, 2011 at 7:37 AM, Tabi Timur-B04825 wrote: > > On Sun, Apr 24, 2011 at 6:50 PM, Wolfgang Denk wrote: > >> And please add documentation for the new CONFIG_PRESERVE_LOCAL_MAC to > >> the README. > > > > We have something simil

Re: [U-Boot] [PATCH v7 01/10] nds32: add header files support for nds32

2011-04-29 Thread Chih-Min Chao
On Sat, Apr 30, 2011 at 7:09 AM, Wolfgang Denk wrote: > Dear Macpaul Lin, > > In message you wrote: >> >> I think we still have to discuss about the typedef's. >> What does the "new" typedef means? > > It means adding any new code to U-Boot which includes "typedef"s. > >> According to the checkpa

[U-Boot] Webmail Quote The Set-Limit überschritten

2011-04-29 Thread System Administrator
Ihr Postfach hat die Lagerung Grenze, die 20GB ist wie vom Administrator festgelegt, Sie sind derzeit auf 20.9GB, können Sie nicht in der Lage zu senden oder zu empfangen, bis Sie neue E-Mail erneut validieren Ihr Postfach überschritten. Zur erneuten Überprüfung Ihrer Mailbox bitte hier k

[U-Boot] [PATCH 3/3] serial: drop useless ctlr field

2011-04-29 Thread Mike Frysinger
The multi serial support has a "ctlr" field which almost no one uses, but everyone is forced to set to useless strings. So punt it. Funny enough, the only code that actually reads this field (the mpc8xx driver) has a typo where it meant to look for the SCC driver. Fix it while converting the che

[U-Boot] [PATCH 1/3] serial: push default_serial_console to drivers

2011-04-29 Thread Mike Frysinger
Rather than sticking arch/board/driver specific logic in the common serial code, push it all out to the respective drivers. The serial drivers declare these funcs weak so that boards can still override things with their own definition. Signed-off-by: Mike Frysinger CC: Heiko Schocher CC: Anatol

[U-Boot] [PATCH 2/3] serial: drop serial_register return value

2011-04-29 Thread Mike Frysinger
The serial_register function never fails (always return 0), so change it to a void function to avoid wasting overhead on it. Signed-off-by: Mike Frysinger --- common/serial.c |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/common/serial.c b/common/serial.c index 505c2

[U-Boot] (no subject)

2011-04-29 Thread Western Union
Attention: Beneficiary, Greetings to you by the power resting on me. I was advice to send your first payment of US$7,000 to you as instructed by G-20 Committee and Mr. David Cameron (The British Prime Minister) and will keep sending you US$7,000 twice a week Tuesdays and Fridays until the payment

Re: [U-Boot] [PATCH 5/7] JFFS2: Change DEFAULT_EMPTY_SCAN_SIZE to 256 Bytes

2011-04-29 Thread Baidu Liu
Hi,Detlev 2011/4/30 Detlev Zundel : > Hi Baidu, > >> Hi,Detlev >>                               default:                                       /* unknown */ -                                     putLabeledWord("UNKOWN COMPRESSION METHOD = ", jNode->compr); +              

Re: [U-Boot] [PATCH 3/7] JFFS2: Calculate buf_len before we read data from flash

2011-04-29 Thread Baidu Liu
Hi, Detlev : 2011/4/30 Detlev Zundel : > Hi Baidu, > > [...] > diff --git a/fs/jffs2/jffs2_1pass.c b/fs/jffs2/jffs2_1pass.c index 8eb77b1..be6ac78 100644 --- a/fs/jffs2/jffs2_1pass.c +++ b/fs/jffs2/jffs2_1pass.c @@ -1643,6 +1643,8 @@ jffs2_1pass_build_lists(struct part_inf

Re: [U-Boot] [PATCH 1/7] JFFS2: Bug fix for summary support

2011-04-29 Thread Baidu Liu
Hi,Detlev : 2011/4/30 Detlev Zundel : > Hi Baidu, > >> Hi,  Detlev : >> >> 2011/4/29 Detlev Zundel : >>> Hi, >>>  1/ Get the latest DIRENT  For example, if you create a file in linux jffs2 which config summary  support, then you delete the file , you will not see the file  in  

Re: [U-Boot] [PATCH] README: Clarify difference of CONFIG_WATCHDOG and CONFIG_HW_WATCHDOG

2011-04-29 Thread Mike Frysinger
On Friday, April 29, 2011 07:06:56 Detlev Zundel wrote: > > On Wed, Apr 27, 2011 at 11:25, Detlev Zundel wrote: > >> Now that we have the documentation, the code should be changed to > >> reflect it ;) > >> > >> Asd far as I can see, these are the places where HW_WATCHDOG is used > > > >> instead

Re: [U-Boot] [PATCH] GPIO: Tegra2: add GPIO driver for Seaboard and Harmony

2011-04-29 Thread Mike Frysinger
same as the SPI patch, if this is a driver for GPIO hardware on a processor, then it should be labeled as such. do not mention specific boards. changes that update board config headers to enable the new driver should be split out into a dedicated patch. -mike signature.asc Description: This i

Re: [U-Boot] [PATCH] spi: Tegra2: Add SPI driver for SPIFLASH on Seaboard

2011-04-29 Thread Mike Frysinger
On Thursday, April 28, 2011 10:55:13 Tom Warren wrote: > Signed-off-by: Tom Warren > --- > This patch adds support for the SPIFLASH peripheral on Tegra2 (SPI 1). > Probe, erase, read and write are all supported, as well as low-level > SPI commands via 'sspi'. > > Note that, at this time, only Sea

Re: [U-Boot] [PATCH 01/30] lib, vsprintf: introduce strict_strtoul

2011-04-29 Thread Wolfgang Denk
Dear Valentin Longchamp, In message you wrote: > From: Heiko Schocher > > as checkpatch proposes to use strict_strtoul instead of > simple_strtoul, introduce it. Same comment as prvious version: please provide full reference / attribution. Best regards, Wolfgang Denk -- DENX Software Eng

Re: [U-Boot] [PATCH] lib, vsprintf: introduce strict_strtoul

2011-04-29 Thread Wolfgang Denk
Dear Valentin Longchamp, In message you wrote: > From: Heiko Schocher > > as checkpatch proposes to use strict_strtoul instead of > simple_strtoul, introduce it. > > Signed-off-by: Heiko Schocher > cc: Wolfgang Denk > cc: Detlev Zundel > cc: Valentin Longchamp > cc: Holger Brunck > Sign

Re: [U-Boot] [PATCH] tools/env: document current cross-compilation issues and workaround

2011-04-29 Thread Wolfgang Denk
Dear Luca Ceresoli, In message <1302250280-3423-1-git-send-email-luca.ceres...@comelit.it> you wrote: > Signed-off-by: Luca Ceresoli > --- > Cross-compiling fw_{print,set}env has caused people to waste a lot of time: > - http://www.mail-archive.com/u-boot@lists.denx.de/msg42317.html > - http:/

Re: [U-Boot] [PATCH v7 01/10] nds32: add header files support for nds32

2011-04-29 Thread Wolfgang Denk
Dear Macpaul Lin, In message you wrote: > > I think we still have to discuss about the typedef's. > What does the "new" typedef means? It means adding any new code to U-Boot which includes "typedef"s. > According to the checkpatch result, "typedef" warning exists in 4 files. > arch/nds32/inclu

[U-Boot] [PATCH] [v5] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Timur Tabi
The compatible property for the L2 cache node (on 85xx systems that don't have a CPC) was using a value for the property length that did not match the actual length of the property. Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/fdt.c | 29 +++-- 1 files changed

Re: [U-Boot] Second serial port access ...

2011-04-29 Thread Mike Frysinger
On Friday, April 29, 2011 11:23:10 Robert Sciuk wrote: > I'm attempting to extend u-boot to request some IPMI information from a > BMC attached to a uart, and was wondering how best to approach the > problem. My first concern is whether to add "cmd" type code or use the > u-boot API, and the secon

Re: [U-Boot] [PATCH] Handle most LDSCRIPT setting centrally

2011-04-29 Thread Wolfgang Denk
Dear Scott Wood, In message <20110406233136.ga13...@schlenkerla.am.freescale.net> you wrote: > Currently, some linker scripts are found by common code in config.mk. > Some are found using CONFIG_SYS_LDSCRIPT, but the code for that is > sometimes in arch config.mk and sometimes in board config.mk.

Re: [U-Boot] [PATCH] [v4] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Tabi Timur-B04825
Wolfgang Denk wrote: > This gets lost in the code. That's why I added the comments: /* MPC, where == 4-digit number */ > If you keep the if / else, you have to add braces for the multiline > statements. Ok. -- Timur Tabi Linux kernel developer at Freescale ___

Re: [U-Boot] [PATCH] [v4] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Wolfgang Denk
Dear Tabi Timur-B04825, In message <4dbb3e01.7000...@freescale.com> you wrote: > Wolfgang Denk wrote: > > Why do we need this "if" at all? tolower() on a digit is a nop, so you > > can omit the first branch. > > Because cpu->name looks like one of two ways: > > 8578 > > or > >

Re: [U-Boot] [PATCH v4 00/23] keymile board update

2011-04-29 Thread Wolfgang Denk
Dear Heiko Schocher, In message <1301642195-15280-1-git-send-email...@denx.de> you wrote: > The following patchset updates the support for the keymile > boards. > > - fix a lot of Codingstyle issues for this boards > - heavy rework of the headerfiles, common board code > - add support for 4 new m

Re: [U-Boot] [PATCH] [v4] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Tabi Timur-B04825
Wolfgang Denk wrote: > Why do we need this "if" at all? tolower() on a digit is a nop, so you > can omit the first branch. Because cpu->name looks like one of two ways: 8578 or P4080 In the case of "8578", we want to convert that to "mpc8578". In the case of "P4080",

Re: [U-Boot] [PATCH] [v4] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Wolfgang Denk
Dear Timur Tabi, In message <1304113875-6749-1-git-send-email-ti...@freescale.com> you wrote: > The compatible property for the L2 cache node (on 85xx systems that don't > have a CPC) was using a value for the property length that did not match > the actual length of the property. > > Signed-off-

Re: [U-Boot] [PATCH] [v2] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Wolfgang Denk
Dear Timur Tabi, In message <4dbb2723.4050...@freescale.com> you wrote: > > I disagree. It's quite clear what I'm trying to do. I'm trying to insert a This is your opinion. I disagree. > NULL character into a string. Since device tree properties use a NULL to > delimit multiple strings, it's

[U-Boot] [PATCH] [v4] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Timur Tabi
The compatible property for the L2 cache node (on 85xx systems that don't have a CPC) was using a value for the property length that did not match the actual length of the property. Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/fdt.c | 20 1 files changed, 12 inse

[U-Boot] [PATCH] [v3] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Timur Tabi
The compatible property for the L2 cache node (on 85xx systems that don't have a CPC) was using a value for the property length that did not match the actual length of the property. Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/fdt.c | 20 1 files changed, 12 inse

Re: [U-Boot] [PATCH] [v2] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Timur Tabi
Wolfgang Denk wrote: >> > Except that his version works and your version doesn't. With your version >> > sprintf will stop reading the format string after "controller". > Yes, and this is why I call this a dirty hack, as it's obfuscating > what's going on and what the intended result is. I disag

Re: [U-Boot] [PATCH] [v2] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Wolfgang Denk
Dear Scott Wood, In message <20110429154457.0fee3...@schlenkerla.am.freescale.net> you wrote: > > > > len = sprintf(compat_buf, > > > - "fsl,mpc%s-l2-cache-controller", cpu->name); > > > + "fsl,mpc%s-l2-cache-controller" "%c" "cache

Re: [U-Boot] musb: usb storage: USB device not accepting new address error

2011-04-29 Thread Wolfgang Denk
Dear Alper YILDIRIM, In message <31267662.p...@talk.nabble.com> you wrote: > > I found the offending patch, it is the following commit that > makes usb broken on OMAPL138. When i revert the patch the > problem no more occurs. > > 3c0659b535b075be124c3d2a0714e55e65c46737 > ARM: Avoid compiler op

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

2011-04-29 Thread Wolfgang Denk
Dear Andy Fleming, In message <1304065909-16710-1-git-send-email-aflem...@freescale.com> you wrote: > are available in the git repository at: > > git://www.denx.de/git/u-boot-mmc.git master > > Jason Liu (1): > fsl_esdhc: Fix multi-block read restriction on i.MX53 eSDHCv2 > > John Rigby

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

2011-04-29 Thread Paulraj, Sandeep
> > nhk8815: move config targets from Makefile to boards.cfg > > > > are available in the git repository at: > > > >git://git.denx.de/u-boot-ti.git master > > > > Ben Gardiner (1): > >da850evm: fix NAND WSTROBE and TA timings > > > > board/davinci/da8xxevm/da850evm.c |4

[U-Boot] [GIT PULL] Pull request u-boot-mpc85xx.git

2011-04-29 Thread Kumar Gala
[ w/ L2 patch dropped ] The following changes since commit a000b7950da938d2df37ec5e081cd0680e6e4bbe: common: add a grepenv command (2011-04-28 01:00:07 +0200) are available in the git repository at: git://git.denx.de/u-boot-mpc85xx.git master Andy Fleming (1): tsec: Fix MDIO on device

Re: [U-Boot] [PATCH] [v2] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Scott Wood
On Fri, 29 Apr 2011 22:30:58 +0200 Wolfgang Denk wrote: > Dear Timur Tabi, > > In message <1304089126-11945-1-git-send-email-ti...@freescale.com> you wrote: > > The compatible property for the L2 cache node (on 85xx systems that don't > > have a CPC) was using a value for the property length tha

Re: [U-Boot] [GIT PULL] Pull request u-boot-mpc85xx.git (updated)

2011-04-29 Thread Wolfgang Denk
Dear Kumar Gala, In message <40b22acd-651c-4765-ba19-421d3abe1...@kernel.crashing.org> you wrote: > > Did another push to drop: > > * powerpc/85xx: fix compatible property for the L2 cache node Please send a new pull req when all is ready. Thanks. Best regards, Wolfgang Denk -- DENX Softwa

Re: [U-Boot] [PATCH] [v2] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Wolfgang Denk
Dear Kumar Gala, In message you wrote: > > I was thinking the patch wasn't going to have any additional commentary. The rule is that we allow _a_few_working_days_ of review time normally - not just a few minutes. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgan

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

2011-04-29 Thread s-paulraj
The following changes since commit e2b1fb98dab08f18e456fd8d846440fe81567785: Sandeep Paulraj (1): Merge branch 'master' of git://git.denx.de/u-boot-arm are available in the git repository at: git://git.denx.de/u-boot-ti.git master Ben Gardiner (1): da850evm: fix NAND WSTROBE an

Re: [U-Boot] [GIT PULL] Pull request u-boot-mpc85xx.git (updated)

2011-04-29 Thread Kumar Gala
On Apr 29, 2011, at 3:35 PM, Wolfgang Denk wrote: > Dear Kumar Gala, > > In message you > wrote: >> >> The following changes since commit a000b7950da938d2df37ec5e081cd0680e6e4bbe: >> >> common: add a grepenv command (2011-04-28 01:00:07 +0200) >> >> are available in the git repository at:

Re: [U-Boot] [PATCH] [v2] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Kumar Gala
On Apr 29, 2011, at 3:33 PM, Wolfgang Denk wrote: > Dear Kumar Gala, > > In message <8188c4ac-9553-4f2d-9370-9416d63af...@freescale.com> you wrote: >> >> On Apr 29, 2011, at 9:58 AM, Timur Tabi wrote: >> >>> The compatible property for the L2 cache node (on 85xx systems that don't >>> have a C

Re: [U-Boot] [GIT PULL] Pull request u-boot-mpc85xx.git (updated)

2011-04-29 Thread Wolfgang Denk
Dear Kumar Gala, In message you wrote: > > The following changes since commit a000b7950da938d2df37ec5e081cd0680e6e4bbe: > > common: add a grepenv command (2011-04-28 01:00:07 +0200) > > are available in the git repository at: > git://git.denx.de/u-boot-mpc85xx.git master NAK. I refuse to

Re: [U-Boot] [GIT PULL] Pull request u-boot-mpc85xx.git (updated)

2011-04-29 Thread Kumar Gala
On Apr 29, 2011, at 10:34 AM, Kumar Gala wrote: > The following changes since commit a000b7950da938d2df37ec5e081cd0680e6e4bbe: > > common: add a grepenv command (2011-04-28 01:00:07 +0200) > > are available in the git repository at: > git://git.denx.de/u-boot-mpc85xx.git master Did another p

Re: [U-Boot] [PATCH] [v2] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Wolfgang Denk
Dear Kumar Gala, In message <8188c4ac-9553-4f2d-9370-9416d63af...@freescale.com> you wrote: > > On Apr 29, 2011, at 9:58 AM, Timur Tabi wrote: > > > The compatible property for the L2 cache node (on 85xx systems that don't > > have a CPC) was using a value for the property length that did not ma

Re: [U-Boot] [PATCH] [v2] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Wolfgang Denk
Dear Timur Tabi, In message <1304089126-11945-1-git-send-email-ti...@freescale.com> you wrote: > The compatible property for the L2 cache node (on 85xx systems that don't > have a CPC) was using a value for the property length that did not match > the actual length of the property. > > Signed-off

Re: [U-Boot] [GIT PULL] Pull request u-boot-mpc85xx.git (updated)

2011-04-29 Thread Scott Wood
On Fri, 29 Apr 2011 15:16:59 -0500 Kumar Gala wrote: > > On Apr 29, 2011, at 12:36 PM, Tabi Timur-B04825 wrote: > > > > > What about this patch: > > > > http://patchwork.ozlabs.org/patch/71614/ > > > > It was ACK'd by gvb: > > > > http://lists.denx.de/pipermail/u-boot/2011-April/089791.html

Re: [U-Boot] [GIT PULL] Pull request u-boot-mpc85xx.git (updated)

2011-04-29 Thread Kumar Gala
On Apr 29, 2011, at 12:36 PM, Tabi Timur-B04825 wrote: > > What about this patch: > > http://patchwork.ozlabs.org/patch/71614/ > > It was ACK'd by gvb: > > http://lists.denx.de/pipermail/u-boot/2011-April/089791.html I'd rather we just fix the addresses up rather than verifying them. - k __

[U-Boot] PPC405EX CHIP_21 erratum

2011-04-29 Thread Steven A. Falco
We've been bitten by the PPC405EX CHIP_21 erratum. I've looked through the U-Boot code, but it doesn't appear that there is a work-around for this one. The following patch is my adaptation of AMCC's suggestion as to the fix. But I have to say that I don't care for it, if for no other reason that

Re: [U-Boot] [GIT PULL] Pull request u-boot-mpc85xx.git (updated)

2011-04-29 Thread Tabi Timur-B04825
On Fri, Apr 29, 2011 at 10:34 AM, Kumar Gala wrote: > Timur Tabi (7): >      powerpc/86xx: remove empty board_early_init_f() >      powerpc/fsl: add 'pixis_reset dump' command >      powerpc/85xx: Implement work-around for P4080 erratum SERDES-A005 >      powerpc/85xx: Display SERDES 8 erratum war

Re: [U-Boot] [PATCH 4/7] JFFS2: Improve error checking

2011-04-29 Thread Detlev Zundel
Hi Baidu, > Hi, Detlev > > >>> @@ -1470,9 +1471,16 @@ jffs2_1pass_build_lists(struct part_info * part) >>>       /* lcd_off(); */ >>> >>>       /* if we are building a list we need to refresh the cache. */ >>> -     jffs_init_1pass_list(part); >>> +     if(! jffs_init_1pass_list(part)) >>> +      

Re: [U-Boot] [PATCH 3/7] JFFS2: Calculate buf_len before we read data from flash

2011-04-29 Thread Detlev Zundel
Hi Baidu, [...] >>> diff --git a/fs/jffs2/jffs2_1pass.c b/fs/jffs2/jffs2_1pass.c >>> index 8eb77b1..be6ac78 100644 >>> --- a/fs/jffs2/jffs2_1pass.c >>> +++ b/fs/jffs2/jffs2_1pass.c >>> @@ -1643,6 +1643,8 @@ jffs2_1pass_build_lists(struct part_info * part) >>>                       case JFFS2_NODE

Re: [U-Boot] [PATCH 1/7] JFFS2: Bug fix for summary support

2011-04-29 Thread Detlev Zundel
Hi Baidu, > Hi, Detlev : > > 2011/4/29 Detlev Zundel : >> Hi, >> >>>  1/ Get the latest DIRENT >>>  For example, if you create a file in linux jffs2 which config summary >>>  support, then you delete the file , you will not see the file  in >>>  linux jffs2. But you can also see this file in uboo

Re: [U-Boot] [PATCH] ehci-pci: Fix PCI EHCI driver for 36-bit

2011-04-29 Thread Kumar Gala
On Apr 18, 2011, at 9:47 PM, Zhao Chenhui wrote: > Convert the PCI base address into a virtual address. > > Signed-off-by: Zhao Chenhui > Signed-off-by: Li Yang > --- > drivers/usb/host/ehci-pci.c |5 ++--- > 1 files changed, 2 insertions(+), 3 deletions(-) Remy, any comments or acceptanc

[U-Boot] Second serial port access ...

2011-04-29 Thread Robert Sciuk
I'm attempting to extend u-boot to request some IPMI information from a BMC attached to a uart, and was wondering how best to approach the problem. My first concern is whether to add "cmd" type code or use the u-boot API, and the second concern, is to how to configure and access the 2nd serial por

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

2011-04-29 Thread Kumar Gala
The following changes since commit a000b7950da938d2df37ec5e081cd0680e6e4bbe: common: add a grepenv command (2011-04-28 01:00:07 +0200) are available in the git repository at: git://git.denx.de/u-boot-mpc85xx.git master Andy Fleming (1): tsec: Fix MDIO on devices with eTSEC2 Dipen Dudh

Re: [U-Boot] [PATCH] fsl-ddr: Fix mixed-case macro names

2011-04-29 Thread Kumar Gala
On Apr 14, 2011, at 12:39 PM, Kyle Moffett wrote: > Signed-off-by: Kyle Moffett > --- > arch/powerpc/cpu/mpc8xxx/ddr/util.c | 18 +- > 1 files changed, 9 insertions(+), 9 deletions(-) applied to 85xx - k ___ U-Boot mailing list U-Boo

Re: [U-Boot] [PATCH] [v2] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Kumar Gala
On Apr 29, 2011, at 9:58 AM, Timur Tabi wrote: > The compatible property for the L2 cache node (on 85xx systems that don't > have a CPC) was using a value for the property length that did not match > the actual length of the property. > > Signed-off-by: Timur Tabi > --- > arch/powerpc/cpu/mpc85

Re: [U-Boot] [PATCH V2] MX31: change return value of get_cpu_rev

2011-04-29 Thread Detlev Zundel
Hi Stefano, > Drop warnings in get_cpu_rev and changes the return value > (a u32 instead of char * is returned) of the function > to be coherent with other processors. > > Signed-off-by: Stefano Babic > CC: Detlev Zundel > CC: Fabio Estevam > --- > > Changes: > - commit message > - "unknown" wi

Re: [U-Boot] [PATCH 6/7] JFFS2: scanning performance improvement

2011-04-29 Thread Baidu Liu
Hi,Detlev >> +                             } >> + > > This has style-problems.  Actually all of your patches have style > problems.  Checking them as one single patch gives: > > total: 32 errors, 22 warnings, 369 lines checked > > Please rework the whole series. I do not know why my patch has sty

Re: [U-Boot] [PATCH 5/7] JFFS2: Change DEFAULT_EMPTY_SCAN_SIZE to 256 Bytes

2011-04-29 Thread Baidu Liu
Hi, Detlev : >>                               default: >>                                       /* unknown */ >> -                                     putLabeledWord("UNKOWN COMPRESSION >> METHOD = ", jNode->compr); >> +                                     putLabeledWord("UNKNOWN COMPRESSION >>

Re: [U-Boot] [PATCH 4/7] JFFS2: Improve error checking

2011-04-29 Thread Baidu Liu
Hi, Detlev >> @@ -1470,9 +1471,16 @@ jffs2_1pass_build_lists(struct part_info * part) >>       /* lcd_off(); */ >> >>       /* if we are building a list we need to refresh the cache. */ >> -     jffs_init_1pass_list(part); >> +     if(! jffs_init_1pass_list(part)) >> +             return 0; >> +

[U-Boot] [PATCH] [v2] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Timur Tabi
The compatible property for the L2 cache node (on 85xx systems that don't have a CPC) was using a value for the property length that did not match the actual length of the property. Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/fdt.c | 13 +++-- 1 files changed, 7 insertions(+

Re: [U-Boot] [PATCH] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Timur Tabi
Kumar Gala wrote: > Can we do: > ""fsl,mpc%s-l2-cache-controller" "%c" "cache", > cpu->name, 0); > > just for readability? > Sure, but couldn't you have told me that before you asked me to post it upstream? :-) -- Timur Tabi Linux kernel developer at Freescale _

Re: [U-Boot] [PATCH] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Kumar Gala
On Apr 29, 2011, at 9:08 AM, Timur Tabi wrote: > The compatible property for the L2 cache node (on 85xx systems that don't > have a CPC) was using a value for the property length that did not match > the actual length of the property. > > Signed-off-by: Timur Tabi > --- > arch/powerpc/cpu/mpc85

Re: [U-Boot] [PATCH 3/7] JFFS2: Calculate buf_len before we read data from flash

2011-04-29 Thread Baidu Liu
Hi,Detlev 2011/4/29 Detlev Zundel : > Hi Baidu, > >>  1/ We should calculate the buf_len before we call >>  get_fl_mem(). > > If I read your change correctly, then do you mean the following? > >  When we know what we want to read, we can calculate buf_len to be the >  maximum size of the data to b

Re: [U-Boot] [PATCH 3/7] JFFS2: Calculate buf_len before we read data from flash

2011-04-29 Thread Baidu Liu
Hi,Detlev 2011/4/29 Detlev Zundel : > Hi Baidu, > >>  1/ We should calculate the buf_len before we call >>  get_fl_mem(). > > If I read your change correctly, then do you mean the following? > >  When we know what we want to read, we can calculate buf_len to be the >  maximum size of the data to b

Re: [U-Boot] [u-boot-release] [PATCH] tsec: Fix MDIO on devices with eTSEC2

2011-04-29 Thread Timur Tabi
Andy Fleming wrote: > The tsec driver was defining the default MDIO address as > the TSEC_BASE + 0x520, but on eTSEC2 controllers, the first > TSEC's registers are separated from the MDIO registers. Use > the existing MDIO_BASE_ADDR, instead. > > Signed-off-by: Andy Fleming Acked-by: Timur Tabi

[U-Boot] [PATCH] powerpc/85xx: fix compatible property for the L2 cache node

2011-04-29 Thread Timur Tabi
The compatible property for the L2 cache node (on 85xx systems that don't have a CPC) was using a value for the property length that did not match the actual length of the property. Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/fdt.c | 13 +++-- 1 files changed, 7 insertions(+

[U-Boot] [PATCH V2] MX31: change return value of get_cpu_rev

2011-04-29 Thread Stefano Babic
Drop warnings in get_cpu_rev and changes the return value (a u32 instead of char * is returned) of the function to be coherent with other processors. Signed-off-by: Stefano Babic CC: Detlev Zundel CC: Fabio Estevam --- Changes: - commit message - "unknown" will be printed together with the

Re: [U-Boot] [PATCH 0/5] introduce nand write.ubi, and drop ffs for jffs2 too

2011-04-29 Thread Artem Bityutskiy
On Fri, 2011-04-29 at 13:58 +0200, Detlev Zundel wrote: > Hi Ben, > > > It was found that on da850evm, where the NAND ECC used does not map all 0xff > > data to 0xff ECC, that flashing UBI and JFFS2 image from U-boot with nand > > write[.e] command resulted in alot of ECC errors... for UBI the res

Re: [U-Boot] [PATCH 1/7] JFFS2: Bug fix for summary support

2011-04-29 Thread Baidu Liu
Hi, Detlev : 2011/4/29 Detlev Zundel : > Hi, > >>  1/ Get the latest DIRENT >>  For example, if you create a file in linux jffs2 which config summary >>  support, then you delete the file , you will not see the file  in >>  linux jffs2. But you can also see this file in uboot after you reset >>  

[U-Boot] Hot sale LED lightings

2011-04-29 Thread Sunbow-Sophia
Dear Trading manager We are a professional manufacturer of LED lighting based in China . We hope our products will be helpful for your business. Our products cover a very wide range of LED products, such as LED SMD strips(12V/24V),LED SMD strips (220V),LED SMD strips(super brightness), LED Corn

Re: [U-Boot] [PATCH 6/7] JFFS2: scanning performance improvement

2011-04-29 Thread Detlev Zundel
Hi Baidu, > 1/ Sync with kernel. > If the 256-(struct jffs2_unknown_node *) bytes are > 0xff after the cleanmarker. We get the conclusion that > the sector is empty. > > Signed-off-by: Baidu Liu > --- > fs/jffs2/jffs2_1pass.c | 19 +-- > 1 files changed, 13 insertions(+), 6

Re: [U-Boot] [PATCH 5/7] JFFS2: Change DEFAULT_EMPTY_SCAN_SIZE to 256 Bytes

2011-04-29 Thread Detlev Zundel
Hi Baidu, > 1/ Syncs up with jffs2 in the linux kernel: > If the first 256 Bytes is 0xff,we get the conclusion > that the sector is empty. > > Signed-off-by: Baidu Liu > --- > fs/jffs2/jffs2_1pass.c | 11 ++- > fs/jffs2/jffs2_nand_1pass.c | 13 ++--- > include/jffs2/j

Re: [U-Boot] [PATCH 4/7] JFFS2: Improve error checking

2011-04-29 Thread Detlev Zundel
Hi Baidu, > Check the return value when we do malloc. > > Signed-off-by: Baidu Liu > --- > fs/jffs2/jffs2_1pass.c | 12 ++-- > fs/jffs2/jffs2_nand_1pass.c |5 - > 2 files changed, 14 insertions(+), 3 deletions(-) > > diff --git a/fs/jffs2/jffs2_1pass.c b/fs/jffs2/jffs2_1p

Re: [U-Boot] [PATCH 3/7] JFFS2: Calculate buf_len before we read data from flash

2011-04-29 Thread Detlev Zundel
Hi Baidu, > 1/ We should calculate the buf_len before we call > get_fl_mem(). If I read your change correctly, then do you mean the following? When we know what we want to read, we can calculate buf_len to be the maximum size of the data to be read. Without this, we usually read EMPTY_S

Re: [U-Boot] [PATCH 1/7] JFFS2: Bug fix for summary support

2011-04-29 Thread Detlev Zundel
Hi, > 1/ Get the latest DIRENT > For example, if you create a file in linux jffs2 which config summary > support, then you delete the file , you will not see the file in > linux jffs2. But you can also see this file in uboot after you reset > the system. That is because all the nodes in jffs

[U-Boot] [GIT PULL] Pull request u-boot-mpc85xx.git

2011-04-29 Thread Kumar Gala
The following changes since commit a000b7950da938d2df37ec5e081cd0680e6e4bbe: common: add a grepenv command (2011-04-28 01:00:07 +0200) are available in the git repository at: git://git.denx.de/u-boot-mpc85xx.git master Andy Fleming (1): tsec: Fix MDIO on devices with eTSEC2 Dipen Dudh

Re: [U-Boot] [PATCH] MX31: change return value of get_cpu_rev

2011-04-29 Thread Detlev Zundel
Hi Stefano, [...] >>> @@ -129,7 +129,7 @@ char *get_cpu_rev(void) >>> for (i = 0; i < ARRAY_SIZE(mx31_cpu_type); i++) >>> if (srev == mx31_cpu_type[i].srev) >>> return mx31_cpu_type[i].v; >>> - return "unknown"; >>> + return srev; >> >> Hm, so we d

Re: [U-Boot] [PATCH] powerpc/85xx: Bump up the CONFIG_SYS_BOOTM_LEN to 64M on FSL 85xx boards

2011-04-29 Thread Kumar Gala
On Apr 28, 2011, at 9:55 PM, Kumar Gala wrote: > CONFIG_SYS_BOOTMAPSZ has been 64M on these boards for some time so we > should also allow the kernel image to be up to 64M decompressed. This > also matches what we pass to the OS based on the ePAPR specification. > > Signed-off-by: Kumar Gala >

Re: [U-Boot] [PATCH] powerpc/85xx: Add support to save environment in SD card on p1_p2_rdb

2011-04-29 Thread Kumar Gala
On Apr 9, 2011, at 12:49 PM, Kumar Gala wrote: > From: Priyanka Jain > > If we boot from a SD card use it for the environment as well. > > Signed-off-by: Priyanka Jain > Signed-off-by: Poonam Aggrwal > Signed-off-by: Kumar Gala > --- > include/configs/P1_P2_RDB.h |6 +- > 1 files cha

Re: [U-Boot] [PATCH] powerpc/85xx: Added PMUXCR1 and PMUXCR2 defines for P1010/P1014 SoC

2011-04-29 Thread Kumar Gala
On Apr 9, 2011, at 12:49 PM, Kumar Gala wrote: > From: Dipen Dudhat > > Signed-off-by: Dipen Dudhat > Signed-off-by: Kumar Gala > --- > arch/powerpc/include/asm/immap_85xx.h | 71 + > 1 files changed, 71 insertions(+), 0 deletions(-) applied to 85xx - k

Re: [U-Boot] [PATCH] powerpc/85xx: Change CS timing parameters before changing CS properties on IFC

2011-04-29 Thread Kumar Gala
On Apr 9, 2011, at 12:54 PM, Kumar Gala wrote: > From: Dipen Dudhat > > To make sure that machine change operation work successfully, change > timing parameters first before changing machine for chip select on IFC. > > Signed-off-by: Dipen Dudhat > Signed-off-by: Kumar Gala > --- > arch/powe

Re: [U-Boot] [PATCH] powerpc/85xx: Add support for 2nd USB controller on p1_p2_rdb

2011-04-29 Thread Kumar Gala
On Apr 9, 2011, at 1:22 PM, Kumar Gala wrote: > From: Ramneek Mehresh > > Second USB controller only works for SPI and SD boot because of pin muxing > > Signed-off-by: Ramneek Mehresh > --- > arch/powerpc/include/asm/immap_85xx.h |1 + > board/freescale/p1_p2_rdb/p1_p2_rdb.c | 14 +++

Re: [U-Boot] [PATCH] powerpc/85xx: Add device tree fixup for bman portal

2011-04-29 Thread Kumar Gala
On Apr 9, 2011, at 12:55 PM, Kumar Gala wrote: > From: Haiying Wang > > Fix fdt bportal to pass the bman revision number to kernel via device tree. > > Signed-off-by: Haiying Wang > Signed-off-by: Kumar Gala > --- > arch/powerpc/cpu/mpc85xx/fdt.c |1 + > arch/powerpc/cpu/mpc85xx/p

Re: [U-Boot] [PATCH 2/2] p4080/serdes: Implement the XAUI workaround for SERDES9 erratum

2011-04-29 Thread Kumar Gala
On Apr 9, 2011, at 3:40 PM, Kumar Gala wrote: > From: Emil Medve > > Signed-off-by: Emil Medve > Signed-off-by: Timur Tabi > Signed-off-by: Kumar Gala > --- > arch/powerpc/cpu/mpc85xx/cmd_errata.c |3 + > arch/powerpc/cpu/mpc85xx/fsl_corenet_serdes.c | 107 +++

Re: [U-Boot] [PATCH 1/2] powerpc/85xx: fsl_corenet_serdes code rework

2011-04-29 Thread Kumar Gala
On Apr 9, 2011, at 3:40 PM, Kumar Gala wrote: > From: Emil Medve > > Rework and add some new APIs to the fsl_corenet_serdes code for use by > erratum and drivers. > > * Rename serdes_get_bank() to serdes_get_bank_by_lane() > * Add serdes_get_first_lane returns which SERDES lane is used by devi

Re: [U-Boot] [PATCH] powerpc: Add P3041DS/P5020DS board support (uses corenet_ds code)

2011-04-29 Thread Kumar Gala
On Apr 9, 2011, at 1:47 PM, Kumar Gala wrote: > The P3041DS & P5020DS boards are almost identical (except for the > processor in them). Additionally they are based on the P4080DS board > design so we use the some board code for all 3 boards. > > Some ngPIXIS (FPGA) registers where reserved on P

Re: [U-Boot] [PATCH] powerpc/85xx: Enable Internal USB PHY for p2040, p3041, p5010 and p5020

2011-04-29 Thread Kumar Gala
On Apr 12, 2011, at 2:10 PM, Kumar Gala wrote: > From: Roy Zang > > The P2040, P3041, P5010, and P5020 all have internal USB PHYs that we > need to enable for them to function. > > Signed-off-by: Roy Zang > Signed-off-by: Kumar Gala > --- > arch/powerpc/cpu/mpc85xx/cpu_init.c | 17 ++

Re: [U-Boot] [PATCH v2] powerpc/85xx: Change timebase divisor to be defined per processor

2011-04-29 Thread Kumar Gala
On Apr 27, 2011, at 10:39 PM, Kumar Gala wrote: > Introduce new CONFIG_SYS_FSL_TBCLK_DIV on 85xx platforms because > different SoCs have different divisor amounts. All the PQ3 parts are > /8, the P4080/P4080 is /16, and P2040/P3041/P5020 are /32. > > Signed-off-by: Kumar Gala > --- > * Added R

Re: [U-Boot] [PATCH] powerpc/85xx: handle both "secX.Y" and "sec-vX.Y" properties

2011-04-29 Thread Kumar Gala
On Apr 21, 2011, at 1:52 PM, Kumar Gala wrote: > From: Kim Phillips > > versioned SEC properties changed names during development, so > for now search and update LIODNs for both "secX.Y" and > "sec-vX.Y" based properties. > > Signed-off-by: Kim Phillips > Signed-off-by: Kumar Gala > --- > ar

Re: [U-Boot] [PATCH] powerpc/85xx: Enable ESDHC111 erratum on P2040/P3041/P5010/P5020 SoCs

2011-04-29 Thread Kumar Gala
On Apr 21, 2011, at 1:52 PM, Kumar Gala wrote: > From: Lei Xu > > The workaround for ESDHC111 should also be applied on > P2040/P3041/P5010/P5020 SoCs. > > Signed-off-by: Lei Xu > Signed-off-by: Roy Zang > Signed-off-by: Kumar Gala > --- > arch/powerpc/include/asm/config_mpc85xx.h |4 ++

Re: [U-Boot] [PATCH 4/4] powerpc/85xx: Implement work-around for P4080 erratum SERDES-A001

2011-04-29 Thread Kumar Gala
On Apr 22, 2011, at 8:34 AM, Kumar Gala wrote: > From: Timur Tabi > > Bank powerdown through RCW[SRDS_LPD_Bn] for XAUI on FM2 and SGMII on FM1 > are swapped. > > Erratum SERDES-A001 says that if bank two is kept disabled and after bank > three is enabled, then the PLL for bank three won't lock

  1   2   >