Re: [U-Boot] Policy for checkpatch usage?

2011-04-27 Thread Detlev Zundel
Hi Andreas, > Am Donnerstag, den 21.04.2011, 17:46 +0200 schrieb Detlev Zundel: >> Thanks for the input. Current base for discussion: >> >> Checkpatch is a tool that can help you find some style problems, but >> is imperfect, and the things it complains about are of varying >> importance.

Re: [U-Boot] [PATCH v5] ARM: mx31: Print the silicon version

2011-04-27 Thread Detlev Zundel
Hi Stefano, > On 04/15/2011 08:07 PM, Stefano Babic wrote: >> On 04/12/2011 04:18 AM, Fabio Estevam wrote: >>> Use the same method of the Linux kernel to print the MX31 silicon version >>> on >>> boot. >>> >>> Tested on a MX31PDK with a 2.0 silicon, where it shows: >>> >>> CPU: Freescale i.MX3

Re: [U-Boot] smdk2416 board support

2011-04-27 Thread Wolfgang Denk
Dear =?ISO-8859-1?Q?Thiago_A=2E_Corr=EAa?=, In message you wrote: > > We have a board based on smdk2416 and we received from the kit > manufacturer, from which we based our own board a custom u-boot, which > contains support for the following SMDK boards: This is obviously one of the many o

[U-Boot] Query - BF537 STAMP Board U-boot Prompt‏

2011-04-27 Thread sarosh
We are currently working on a project related to IP telephony.For that we need to load uClinux Image file to BF537 STAMP board. We hope to benefit from your guidance. The main error we are encountering is that, U-boot prompt from BF537 STAMP is not showing up on minicom/kermit or any other termin

Re: [U-Boot] Query - BF537 STAMP Board U-boot Prompt‏

2011-04-27 Thread Mike Frysinger
you've already posted your question to the blackfin.u.o website. please dont post the same question to multiple places. > Sent from the Uboot - Users mailing list archive at Nabble.com. ugh, dont do that. use the list the right way. -mike ___ U-Boot ma

Re: [U-Boot] [PATCH] MAKEALL: add -h/--help options

2011-04-27 Thread Detlev Zundel
Hi Mike, > Convert all the comments at the top of the file into help text for people > to easily get at with standard -h/--help options. > > Signed-off-by: Mike Frysinger Nice! Acked-by: Detlev Zundel -- .. most of the finest products of an applied mathematician's fancy must be rejected, as

Re: [U-Boot] [PATCH V6 2/3] PMIC: Add dialog pmic support

2011-04-27 Thread Detlev Zundel
Hi Stefano, > On 04/25/2011 04:59 AM, Jason Hui wrote: >> Hi, Stefano, > > Hi Jason, > If you still want me to include the DA9053 support into fsl_pmic, could you please extend the fsl_pmic support to easily add another vender's pmic support first? >>> >>> Really I have not un

Re: [U-Boot] [PATCH v5] ARM: mx31: Print the silicon version

2011-04-27 Thread Stefano Babic
On 04/27/2011 11:11 AM, Detlev Zundel wrote: This is due to the usage of the const in the mx3_cpu_type: >> >> struct mx3_cpu_type { >> u8 srev; >>const char *v; >> >> Do you agree if I drop myself the const attribute on u-boot-imx before >> pulling your patch to the arm tree ?

Re: [U-Boot] [PATCH,V2] JFFS2: accelerate scanning.

2011-04-27 Thread Detlev Zundel
Hi Baidu, > Hi,Detlev : > > 2011/4/19 Detlev Zundel : >> Hi Baidu, >> >>>  Syncs up with jffs2 in the linux kernel: >>>  1/ Change DEFAULT_EMPTY_SCAN_SIZE from 4KB to 256 Bytes. >>>  2/ If the 1KB data is 0xFF after the cleanmarker, skip >>>  and scan the next sector. >>>  3/ Change the buffer siz

Re: [U-Boot] [PATCH V6 2/3] PMIC: Add dialog pmic support

2011-04-27 Thread Stefano Babic
On 04/27/2011 11:39 AM, Detlev Zundel wrote: > Then why not rename the driver to something else than "fsl_pmic"? Maybe > this is what Jason really is asking? Yes, of course, no problem at all. Jason, if you meant to drop the fsl_ to make the driver more general, I completely agree. Sorry, I have

Re: [U-Boot] [PATCH V2] JFFS2: bug fix for summary support.

2011-04-27 Thread Detlev Zundel
Hi Baidu, [...] >> Looking over the changes, I do _see_ changes in code, so you should tell >> us about them. >> >>> [...] >> >>> diff --git a/include/jffs2/jffs2.h b/include/jffs2/jffs2.h >>> index 651f94c..5b006c0 100644 >>> --- a/include/jffs2/jffs2.h >>> +++ b/include/jffs2/jffs2.h >>> @@ -41

Re: [U-Boot] Query - BF537 STAMP Board U-boot Prompt‏

2011-04-27 Thread sarosh
are'nt they two different forums? -- View this message in context: http://old.nabble.com/Query---BF537-STAMP-Board-U-boot-Prompt%E2%80%8F-tp31485796p31486059.html Sent from the Uboot - Users mailing list archive at Nabble.com. ___ U-Boot mailing l

Re: [U-Boot] [PATCH v2] cmd_nvedit.c: clean up with checkpatch

2011-04-27 Thread Detlev Zundel
Hi, > Code clean up of cmd_nvedit.c by using checkpatch.pl. Thanks for taking the initiative to do some cleanup! > Signed-off-by: Macpaul Lin Still, the patch fails for me: Applying: cmd_nvedit.c: clean up with checkpatch Using index info to reconstruct a base tree... error: patch faile

[U-Boot] 回覆: [PATCH v2] cmd_nvedit.c: clean up with checkpatch

2011-04-27 Thread macpaul
Hi Detlev, > Hi, > > > Code clean up of cmd_nvedit.c by using checkpatch.pl. > > Thanks for taking the initiative to do some cleanup! > > > Signed-off-by: Macpaul Lin > > Still, the patch fails for me: > > Applying: cmd_nvedit.c: clean up with checkpatch > Using index info to reconstruct

Re: [U-Boot] Update and Cut down mach types

2011-04-27 Thread Michael Schwingen
Am 04/26/2011 11:38 PM, schrieb Reinhard Meyer: >> >>> So IMO, if we have mach-types in U-Boot for supporting Linux, then we >>> should keep using a (reasonably) up-to-date Linux machine ID list just >>> like we do now -- mach-types that disappear from the list mean Linux >>> support has become use

Re: [U-Boot] 回覆: [PATCH v2] cmd_nvedit.c: clean up with checkpatch

2011-04-27 Thread Macpaul Lin
HI Detlev, 2011/4/27 > Hi Detlev, > > > > Signed-off-by: Macpaul Lin > > > > Still, the patch fails for me: > > > > Applying: cmd_nvedit.c: clean up with checkpatch > > Using index info to reconstruct a base tree... > > Please check the same mail thread, I've send patch v3 already. > Thanks

Re: [U-Boot] Dear Webmail User

2011-04-27 Thread Webmail Upgrading Service
Dear Webmail User This is to inform you that you have exceeded your email quota limit of 20GB. You are currently running on 23GB due to concealed documents and Folder on your mailbox as your email account might have been compromised and account details disclosed to external IDENTITIES. To valid

Re: [U-Boot] [PATCH v2] cmd_nvedit.c: clean up with checkpatch

2011-04-27 Thread Macpaul Lin
Hi Detlev, 2011/4/27 Detlev Zundel : > Hi, > >> Code clean up of cmd_nvedit.c by using checkpatch.pl. > > Thanks for taking the initiative to do some cleanup! > >> Signed-off-by: Macpaul Lin > > Still, the patch fails for me: > >  Applying: cmd_nvedit.c: clean up with checkpatch >  Using index in

Re: [U-Boot] [PATCH v2 1/2] MX31: mx31pdk: Add watchdog support

2011-04-27 Thread Detlev Zundel
Hi Fabio, > Ping? > > Regards, > > Fabio Estevam > > On 4/10/2011 3:17 PM, Fabio Estevam wrote: >> Signed-off-by: Fabio Estevam >> --- >> Changes since v1: >> - define BOARD_LATE_INIT in /mx31pdk.h >> >> board/freescale/mx31pdk/mx31pdk.c | 16 >> include/configs/mx31pdk.h

Re: [U-Boot] [PATCH 1/2] MX31: mx31pdk: Add watchdog support

2011-04-27 Thread Detlev Zundel
Hi Stefano, > On 03/09/2011 05:35 PM, Fabio Estevam wrote: >> Signed-off-by: Fabio Estevam >> --- >> board/freescale/mx31pdk/mx31pdk.c | 16 >> include/configs/mx31pdk.h |1 + >> 2 files changed, 17 insertions(+), 0 deletions(-) >> >> diff --git a/board/freescale/

Re: [U-Boot] [PATCH v4 03/23] keymile: Fix Coding style issues for keymile boards.

2011-04-27 Thread Wolfgang Denk
Dear Kim Phillips, In message <20110401181751.d2c2f4bf.kim.phill...@freescale.com> you wrote: > > > > Signed-off-by: Kim Phillips > > > > Why do you send a "Signed-off-by" here? > > > > Did you mean "Acked-by:" ? > > > > Signed-off-by "certifies that you wrote it or otherwise have the right >

Re: [U-Boot] Update and Cut down mach types

2011-04-27 Thread Detlev Zundel
Hi Reinhard, [...] > why don't we just create the #define MACH_xxx lines directly from the > "http://www.arm.linux.org.uk/developer/machines/download.php";. We don't > need all the *_is_* macros in u-boot anyway. Then we would have just a few > 1000 > lines of #define MACH_* ... and we could up

Re: [U-Boot] Pull request for u-boot-video?

2011-04-27 Thread Anatolij Gustschin
Hi Timur, On Tue, 26 Apr 2011 02:12:38 + Tabi Timur-B04825 wrote: > Anatolij, > > Are you planning on making a pull request for u-boot-video soon? The > merge window closed last week, but I have some pending video patches > that I would like to see included. I'm planning to look at the DI

Re: [U-Boot] [PATCH 1/2] MX31: mx31pdk: Add watchdog support

2011-04-27 Thread Fabio Estevam
Hi Detlev, On 4/27/2011 8:38 AM, Detlev Zundel wrote: > Hi Stefano, > >> On 03/09/2011 05:35 PM, Fabio Estevam wrote: >>> Signed-off-by: Fabio Estevam >>> --- >>> board/freescale/mx31pdk/mx31pdk.c | 16 >>> include/configs/mx31pdk.h |1 + >>> 2 files changed, 17 i

Re: [U-Boot] [PATCH v5] ARM: mx31: Print the silicon version

2011-04-27 Thread Fabio Estevam
Hi Stefano, On 4/27/2011 6:40 AM, Stefano Babic wrote: ... >> but why not change get_cpu_rev to 'const char *' >> which it actually is? > > This is correct. However, I have not noted before that the last > introduced get_cpu_rev() in MX31 is an exception in u-boot. For all > other processors, it

[U-Boot] boot-up time optimization. Where to start?

2011-04-27 Thread Simon Schwarz
Hi there, I just started to work on my bachelor thesis. It is about "Linux boot-up time optimization". The past days I spend analyzing what consumes the most time in the boot process. I found that u-boot takes pretty much as long as the whole Linux kernel (the one we are using). I started diggi

Re: [U-Boot] [PATCH v5] ARM: mx31: Print the silicon version

2011-04-27 Thread Stefano Babic
On 04/27/2011 03:16 PM, Fabio Estevam wrote: Hi Fabio, > Will send a patch soon. Ok, thanks. Regards, Stefano Babic -- = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchen

Re: [U-Boot] boot-up time optimization. Where to start?

2011-04-27 Thread Tabi Timur-B04825
On Wed, Apr 27, 2011 at 8:39 AM, Simon Schwarz wrote: > So here is my question: Where do you see the most potential to optimize > u-boot? I'm sure many of the timeouts could be optimized. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot m

Re: [U-Boot] [PATCH v3] cmd_nvedit.c: clean up with checkpatch

2011-04-27 Thread Detlev Zundel
Hi, > Code clean up of cmd_nvedit.c by using checkpatch.pl. > > Signed-off-by: Macpaul Lin Visual inspection shows only cosmetic changes, so the patch looks good. For the next time, we should follow the (newly added) clause in our wiki[1] and have a ommit text like "cosmetic: code cleanup of cm

Re: [U-Boot] 回覆: [PATCH v2] cmd_nvedit.c: clean up with checkpatch

2011-04-27 Thread Detlev Zundel
Hi Macpaul, > HI Detlev, > > 2011/4/27 > >> Hi Detlev, >> >> > > Signed-off-by: Macpaul Lin >> > >> > Still, the patch fails for me: >> > >> > Applying: cmd_nvedit.c: clean up with checkpatch >> > Using index info to reconstruct a base tree... >> >> Please check the same mail thread, I've se

Re: [U-Boot] [PATCH v2] cmd_bdinfo.c: clean up with 2.6.38 checkpatch.pl

2011-04-27 Thread Detlev Zundel
Hi Macpaul, > clean up with 2.6.38 checkpatch.pl There is indeed only one non-whitespace hunk in there: @@ -69,8 +69,9 @@ int do_bdinfo ( cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) defined(CONFIG_440SP) || defined(CONFIG_440SPE) print_str ("procfreq", strmhz(bu

Re: [U-Boot] [PATCH v5] ARM: mx31: Print the silicon version

2011-04-27 Thread Detlev Zundel
Hi Stefano, > On 04/27/2011 11:11 AM, Detlev Zundel wrote: > > This is due to the usage of the const in the mx3_cpu_type: >>> >>> struct mx3_cpu_type { >>> u8 srev; >>>const char *v; >>> >>> Do you agree if I drop myself the const attribute on u-boot-imx before >>> pulling your

Re: [U-Boot] boot-up time optimization. Where to start?

2011-04-27 Thread Eric Cooper
On Wed, Apr 27, 2011 at 03:39:18PM +0200, Simon Schwarz wrote: > So here is my question: Where do you see the most potential to optimize > u-boot? > > I already have two bullets on my list (just some ideas - maybe totally > unrealistic *g*): > - Use Hardware specific copy commands > - build the ch

Re: [U-Boot] [PATCH v3] cmd_nvedit.c: clean up with checkpatch

2011-04-27 Thread Macpaul Lin
Hi Detlve, 2011/4/27 Detlev Zundel : > Hi, > > For the next time, we should follow the (newly added) clause in our > wiki[1] and have a ommit text like "cosmetic: code cleanup of > cmd_nvedit.c using checkpatch.pl". Thanks for your note when we have sometimes didn't follow the new discuccssed thi

Re: [U-Boot] boot-up time optimization. Where to start?

2011-04-27 Thread Mike Frysinger
On Wed, Apr 27, 2011 at 09:39, Simon Schwarz wrote: >  I found that u-boot takes pretty much as long as the whole Linux kernel > (the one we are using). > > I started digging into the source and I think I have a big picture of what > is going on. I already learned from the mailing list that it is a

Re: [U-Boot] Query - BF537 STAMP Board U-boot Prompt‏

2011-04-27 Thread Mike Frysinger
On Wed, Apr 27, 2011 at 06:02, sarosh wrote: > are'nt they two different forums? yes, and that is generally considered spamming -mike ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

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

2011-04-27 Thread Detlev Zundel
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 of WATCHDOG: arch/blackfin/cpu/blackfin/watchdog.c arch/m68k/cpu/mcf547x_8x/cpu.c The relevant maintainers are on CC. Signed-off-by: Detle

Re: [U-Boot] [PATCH v5] ARM: mx31: Print the silicon version

2011-04-27 Thread Stefano Babic
On 04/27/2011 04:57 PM, Detlev Zundel wrote: > I see. Can't we consolidate all those functions into more general > functions higher up in the specialization graph? As I can see, the get_cpu_rev() is already a generalized function, used in more processors. IMHO we should switch back to the usual p

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

2011-04-27 Thread Mike Frysinger
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 of WATCHDOG: the trouble is that watchdog.h doesnt seem to match, nor does the

Re: [U-Boot] boot-up time optimization. Where to start?

2011-04-27 Thread Alexander Stein
Hi, Am Mittwoch, 27. April 2011, 15:39:18 schrieb Simon Schwarz: > I just started to work on my bachelor thesis. It is about "Linux boot-up > time optimization". The past days I spend analyzing what consumes the most > time in the boot process. > > I found that u-boot takes pretty much as long a

Re: [U-Boot] boot-up time optimization. Where to start?

2011-04-27 Thread Mike Frysinger
On Wed, Apr 27, 2011 at 11:59, Alexander Stein wrote: > Am Mittwoch, 27. April 2011, 15:39:18 schrieb Simon Schwarz: >> I just started to work on my bachelor thesis. It is about "Linux boot-up >> time optimization". The past days I spend analyzing what consumes the most >> time in the boot process.

Re: [U-Boot] [PATCH V2 1/2] cmd_mmc: eliminate device num in the mmc command

2011-04-27 Thread Andy Fleming
On Fri, Apr 15, 2011 at 9:17 PM, Lei Wen wrote: > From now on, follow the general rule "mmc dev [dev]" to change the > mmc command applied device, like ide and usb... > > Signed-off-by: Lei Wen > --- > Changelog: > V2: use the "mmc_cur_dev" to replace explicitly specify the dev num >    in mmc co

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

2011-04-27 Thread Albert ARIBAUD
Wolfgang, The following changes since commit a01ebd9679f07421ec5abfda5dc0ab76c7202d3b: mpc52xx, digsy_mtc: add trickle charger support for rev5 boards. (2011-04-26 00:25:05 +0200) are available in the git repository at: git://git.denx.de/u-boot-arm master Albert ARIBAUD (1): Repla

Re: [U-Boot] boot-up time optimization. Where to start?

2011-04-27 Thread Wolfgang Denk
Dear Simon Schwarz, In message you wrote: > > I just started to work on my bachelor thesis. It is about "Linux boot-up > time optimization". The past days I spend analyzing what consumes the most > time in the boot process. A lot of efforts have alreay been spent in this area. A ton of speciali

Re: [U-Boot] boot-up time optimization. Where to start?

2011-04-27 Thread Wolfgang Denk
Dear Alexander Stein, In message <201104271759.11818.alexander.st...@systec-electronic.com> you wrote: > > Setting stdin, stdout and stderr takes a lot of time (IIRC ~500ms). Which IMO > is useless on a bootloader without LCD support. Statements like this are completely worhtless if you don;t t

Re: [U-Boot] [PATCH V2 1/2] cmd_mmc: eliminate device num in the mmc command

2011-04-27 Thread Wolfgang Denk
Dear Andy Fleming, In message you wrote: > > > From now on, follow the general rule "mmc dev [dev]" to change the > > mmc command applied device, like ide and usb... ... > I'd really prefer if there were still the option to specify the device > in the read/write commands. I appreciate the conveni

Re: [U-Boot] Pull request u-boot-blackfin.git (spi branch)

2011-04-27 Thread Wolfgang Denk
Dear Mike Frysinger, In message <1303710563-25546-1-git-send-email-vap...@gentoo.org> you wrote: > The following changes since commit f02efacc515e35695e52ecd08762835c8a74a463: > > Merge branch 'master' of git://git.denx.de/u-boot-cfi-flash (2011-04-25 > 00:58:45 +0200) > > are available in th

Re: [U-Boot] Pull request u-boot-blackfin.git

2011-04-27 Thread Wolfgang Denk
Dear Mike Frysinger, In message <1303711621-26829-1-git-send-email-vap...@gentoo.org> you wrote: > The following changes since commit f02efacc515e35695e52ecd08762835c8a74a463: > > Merge branch 'master' of git://git.denx.de/u-boot-cfi-flash (2011-04-25 > 00:58:45 +0200) > > are available in th

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

2011-04-27 Thread Wolfgang Denk
Dear Gerald Van Baren, In message <4db630eb.20...@cideas.com> you wrote: > Dear Wolfgang, > > The following pull request is for Grant Likely's changes "[PATCH 0/6] > ARM device tree support improvements". I had to make two minor fixes, a > missing ";" in patch [1 of 6] and a misspelling in pat

Re: [U-Boot] [PATCH 1/3] mp2usb: remove board support

2011-04-27 Thread Wolfgang Denk
Dear Eric, In message <1301848555-14404-1-git-send-email-e...@eukrea.com> you wrote: > this board was cancelled long time ago so remove it as it won't > be maintained anymore > > Signed-off-by: Eric Bénard > --- > board/mp2usb/Makefile| 50 - > board/mp2usb/config.mk |3 - > bo

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

2011-04-27 Thread Grant Likely
Thanks guys. g. On Wed, Apr 27, 2011 at 1:47 PM, Wolfgang Denk wrote: > Dear Gerald Van Baren, > > In message <4db630eb.20...@cideas.com> you wrote: >> Dear Wolfgang, >> >> The following pull request is for Grant Likely's changes "[PATCH 0/6] >> ARM device tree support improvements".  I had to m

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

2011-04-27 Thread Wolfgang Denk
Dear Albert ARIBAUD, In message <4db854fd.2090...@aribaud.net> you wrote: > Wolfgang, > > The following changes since commit a01ebd9679f07421ec5abfda5dc0ab76c7202d3b: > >mpc52xx, digsy_mtc: add trickle charger support for rev5 boards. > (2011-04-26 00:25:05 +0200) > > are available in the

Re: [U-Boot] [PATCH 1/3] mp2usb: remove board support

2011-04-27 Thread Eric Bénard
Hi Wolfgang, On 27/04/2011 22:00, Wolfgang Denk wrote: > In message<1301848555-14404-1-git-send-email-e...@eukrea.com> you wrote: >> this board was cancelled long time ago so remove it as it won't >> be maintained anymore >> >> Signed-off-by: Eric Bénard >> --- >> board/mp2usb/Makefile|

[U-Boot] [PATCH] mp2usb: finish the removal

2011-04-27 Thread Eric Bénard
- remove from boards.cfg - add to doc/README.scrapyard Signed-off-by: Eric Bénard --- boards.cfg |1 - doc/README.scrapyard |1 + 2 files changed, 1 insertions(+), 1 deletions(-) diff --git a/boards.cfg b/boards.cfg index d4cbb06..916414a 100644 --- a/boards.cfg +++ b/boards.c

Re: [U-Boot] boot-up time optimization. Where to start?

2011-04-27 Thread Wolfgang Denk
Dear Charles Krinke, please keep the mailing list on Cc: In message you wrote: > > Also, please help me understand if this is not true, but the notion of > stdin, stdout, stderr predates lcd displays back to when all communication > with a computer was via a teletype on a serial port. I'm not s

Re: [U-Boot] mpq101 board broken

2011-04-27 Thread Wolfgang Denk
Dear Alex Dubov, In message <634171.34553...@web37608.mail.mud.yahoo.com> you wrote: > > The error says something about previous section. May be it's not > directly compiler related at all. > > As I'm putting .ppcenv to be the first section, the only thing before > it is .interp, which is suppos

Re: [U-Boot] [PATCH 6/6] Add CONFIG_OF_LIBFDT to more boards.

2011-04-27 Thread Wolfgang Denk
Dear Albert, In message <20110328195906.10235.68517.stgit@ponder> Grant Likely wrote: > From: Grant Likely > > The following boards gain device tree support with this patch: > > ca9x4_ct_vxp - Versatile Express > > i.mx5 boards: > efikamx > mx51evk > mx53evk > > OMAP boards: > devkit8000 > ig

Re: [U-Boot] [PATCH] fix redundant environment for serial flash

2011-04-27 Thread Wolfgang Denk
Dear thomas.lan...@lantiq.com, In message <3e5a05ab39640047b326bef8e2febbdb0aab40f...@mucse402.eu.infineon.com> you wrote: > This patch fixes problems in the handling of redundant environment in env_sf.c > > The major problem are double calls of free() on the allocated buffers, > which damages t

Re: [U-Boot] [PATCH] NET: Correct potential missing goto label in case statement.

2011-04-27 Thread Wolfgang Denk
Dear Gray Remlin, In message <1301430092-23569-1-git-send-email-gryr...@gmail.com> you wrote: > If neither CONFIG_CMD_PING or CONFIG_CMD_SNTP are defined but > CONFIG_CMD_DNS is, a compile-time error will occur due to the > absence of a goto label. > > Signed-off-by: Gray Remlin > --- > net/net

Re: [U-Boot] [PATCH 1/3] mpc52xx, digsy_mtc: change phy addr for rev5 boards.

2011-04-27 Thread Wolfgang Denk
Dear Heiko Schocher, In message <1301897422-27326-2-git-send-email...@denx.de> you wrote: > - rev5 board has phy addr 1 -> adapt CONFIG_PHY_ADDR define > in board config file. > - also fixup the phy addr entry in dts, before booting > Linux. > > Signed-off-by: Heiko Schocher > --- > board/d

Re: [U-Boot] [PATCH 3/3] mpc52xx, digsy_mtc: protect default flash sectors

2011-04-27 Thread Wolfgang Denk
Dear Heiko Schocher, In message <1301897422-27326-4-git-send-email...@denx.de> you wrote: > call flash_protect_default() to protect default sectors. > > Signed-off-by: Heiko Schocher > --- > board/digsy_mtc/digsy_mtc.c |2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) Applied, than

Re: [U-Boot] [PATCH v2] lib/hashtable: fix env var autocompletion

2011-04-27 Thread Wolfgang Denk
Dear Kim Phillips, In message <20110405121450.099cb324.kim.phill...@freescale.com> you wrote: > commit c81c1222427f268d29ba999c82e2477c428e7bab "Fix hash > table deletion to prevent lost entries" broke environment > variable autocompletion by accidentally inverting the > condition for 'used' in hm

Re: [U-Boot] [PATCH] [v2] powerpc: fix implementation of out_8 to match the other out_XX functions

2011-04-27 Thread Wolfgang Denk
Dear Timur Tabi, In message <1301952064-1418-1-git-send-email-ti...@freescale.com> you wrote: > For some reason, the implementation of out_8() does not match the other > output accessors, nor does it match equivalent Linux accessor. This can > cause a problem in a situation like this: > >

Re: [U-Boot] [PATCH v5] common: add a grepenv command

2011-04-27 Thread Wolfgang Denk
Dear Kim Phillips, In message <20110405121514.8641e889.kim.phill...@freescale.com> you wrote: > u-boot environments, esp. when boards are shared across multiple > users, can get pretty large and time consuming to visually parse. > The grepenv command this patch adds can be used in lieu of printenv

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

2011-04-27 Thread Macpaul Lin
Hi Wolfgang, 2011/4/28 Wolfgang Denk : > Dear Albert ARIBAUD, Since the following dependant patches has been applied to u-boot master. >> Macpaul Lin (6): >>        ftsmc020: move ftsmc020 static mem controller to driver/mtd > > Applied, thanks. > > Best regards, > > Wolfgang Denk Would you ple

Re: [U-Boot] [PATCH v4 03/23] keymile: Fix Coding style issues for keymile boards.

2011-04-27 Thread Kim Phillips
On Wed, 27 Apr 2011 13:39:22 +0200 Wolfgang Denk wrote: > In message <20110401181751.d2c2f4bf.kim.phill...@freescale.com> you wrote: > > > > Signed-off-by: Kim Phillips > > > > > > Why do you send a "Signed-off-by" here? > > > > > > Did you mean "Acked-by:" ? > > > > > > Signed-off-by "certif

[U-Boot] [PATCH 1/2] Remove unnecessary CONFIG_SYS_BOOTMAPSZ definition

2011-04-27 Thread Shawn Guo
Since the following commit, definition CONFIG_SYS_BOOTMAPSZ is not needed any more. 7a02a39babb44f2ca2777db9b3a93be79111bba8 Remove device tree booting dependency on CONFIG_SYS_BOOTMAPSZ Signed-off-by: Shawn Guo --- include/configs/mx51evk.h |1 - include/configs/mx53evk.h |1

[U-Boot] [PATCH 2/2] Fix CONFIG_OF_LIBFDT redefined warning

2011-04-27 Thread Shawn Guo
With the following commit, CONFIG_OF_LIBFDT was redefined. a8f88eedb2c4df472e512db58dfcaafd82c68b86 Add CONFIG_OF_LIBFDT to more boards. Remove the duplicated definition to fix CONFIG_OF_LIBFDT redefined warning. Signed-off-by: Shawn Guo --- include/configs/mx51evk.h |2 -- include

[U-Boot] [PATCH v3] cosmetic: cmd_bdinfo.c: clean up by using checkpatch.pl

2011-04-27 Thread Macpaul Lin
cmd_bdinfo.c: clean up with 2.6.38 checkpatch.pl Signed-off-by: Macpaul Lin --- Changes for v2: - Correct the subject field to single patch. Changes for v3: - 405XX, 440XX, XILINX_405 are sorted according to the reviewing suggestion. - Replace the subject field according to the new rules.

Re: [U-Boot] [PATCH v2] cmd_bdinfo.c: clean up with 2.6.38 checkpatch.pl

2011-04-27 Thread Macpaul Lin
Hi Detlev, 2011/4/27 Detlev Zundel : > Hi Macpaul, > >> clean up with 2.6.38 checkpatch.pl > > There is indeed only one non-whitespace hunk in there: > It's ok that you do the line wrapping, but actually with such lists we > require alphabetical sorting of the clauses, so can you please juggle > t

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

2011-04-27 Thread Kumar Gala
On Apr 23, 2011, at 1:45 PM, Wolfgang Denk wrote: > Dear Kumar Gala, > > In message <54ce1181-5449-4fbf-9402-df5d4e99e...@kernel.crashing.org> you > wrote: >> >> On Apr 22, 2011, at 6:50 AM, Tabi Timur-B04825 wrote: >> >>> On Thu, Apr 21, 2011 at 1:59 PM, Kumar Gala >>> wrote: Introduc

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

2011-04-27 Thread Kumar Gala
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 README update README|7

Re: [U-Boot] [PATCH V2 1/2] cmd_mmc: eliminate device num in the mmc command

2011-04-27 Thread Lei Wen
Hi Andy, On Thu, Apr 28, 2011 at 1:00 AM, Andy Fleming wrote: > On Fri, Apr 15, 2011 at 9:17 PM, Lei Wen wrote: >> From now on, follow the general rule "mmc dev [dev]" to change the >> mmc command applied device, like ide and usb... >> >> Signed-off-by: Lei Wen >> --- >> Changelog: >> V2: use t

Re: [U-Boot] [PATCH] powerpc/85xx: Don't set FT_FSL_PCI_SETUP if CONFIG_PCI is not set

2011-04-27 Thread Kumar Gala
On Apr 25, 2011, at 2:10 PM, Matthew McClintock wrote: > A lot of boards set FT_FSL_PCI_SETUP directly in their board code > and don't check to see if CONFIG_PCI is actually defined. This > will cause the board compilation to fail if CONFIG_PCI is not > defined. The p1022ds board is one such exam

Re: [U-Boot] Update and Cut down mach types

2011-04-27 Thread Igor Grinberg
On 04/27/11 13:19, Michael Schwingen wrote: > Am 04/26/2011 11:38 PM, schrieb Reinhard Meyer: So IMO, if we have mach-types in U-Boot for supporting Linux, then we should keep using a (reasonably) up-to-date Linux machine ID list just like we do now -- mach-types that disappear from

[U-Boot] fw_setenv usage for multiple copies of U-Boot Environment Variables

2011-04-27 Thread Amarendra Reddy
Hi all Consider we have multiple copies of U-Boot Environment Variables. Each copy protected by CRC value. Can we use fw_printenv and fw_setenv utilities from Linux Application ? Please guide me regarding this. Thanks Amarendra Reddy Larsen & Toubro Limited www.larsentoubro.com This Email m

Re: [U-Boot] [PATCH v3 4/4] ftsmc020: move ftsmc020 static mem controller to driver/mtd

2011-04-27 Thread Po-Yu Chuang
Hi Macpaul, On Sat, Apr 16, 2011 at 3:37 PM, Macpaul Lin wrote: > Move the header file and definitions of ftsmc020 > static memory control unit from a320 SoC folder to > "drivers/mtd" folder. > > This change will let other SoC which also use ftsmc020 > could share the same header file. > > Signed