Re: [U-Boot] Bogus "Warning: failed to set MAC address" on Gumstix Overo

2011-09-08 Thread Wolfgang Denk
Dear Philip Balister, In message <4e68da54.4000...@balister.org> you wrote: > > My point is the code shouldn't be trying to write the MAC address, when > the MAC address is set in hardware. Earlier, (say 2011.6 but I am not > being scientific here) u-boots did the write thing (IE not print the

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

2011-09-08 Thread s-paulraj
Albert, Please pull u-boot-ti/master. I checked all the patches for checkpatch errors and fixed all the warnings that i could. Some cannot be fixed. Sanjeev's patches update thr GPIO for OMAP. The GPIO on OMAP was causing issues which now seem to be fixed. Regards, Sandeep The following changes

Re: [U-Boot] [PATCH 2/3] BeagleBoard: Added userbutton command

2011-09-08 Thread Kridner, Jason
On Sep 8, 2011, at 11:03 AM, "Albert ARIBAUD" wrote: > Hi Joel, > > Le 08/09/2011 16:56, Joel A Fernandes a écrit : > >>> Also, I agree with Albert: there should be no need for a separate >>> userbutton command. >>> >>> Please fix and resubmit. >> >> If this patch is to be dropped, then I'm

Re: [U-Boot] Bogus "Warning: failed to set MAC address" on Gumstix Overo

2011-09-08 Thread Philip Balister
On 09/08/2011 10:40 AM, Wolfgang Denk wrote: > Dear Philip Balister, > > In message<4e68c67f.2080...@opensdr.com> you wrote: >> When booting master on a Gumstix Overo based system, I get the following >> messages during ethernet initialization: >> >> Net: smc911x-0 >> Warning: failed to set MAC

Re: [U-Boot] [PATCH] net: Add \n before warning message so it prints on a new line.

2011-09-08 Thread Philip Balister
On 09/08/2011 11:01 AM, Albert ARIBAUD wrote: > Hi Philip, > > Le 07/09/2011 13:57, Philip Balister a écrit : >> Signed-off-by: Philip Balister >> --- >> net/eth.c | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/net/eth.c b/net/eth.c >> index dbd1e2d..67a8039 100644 >>

Re: [U-Boot] [PATCH 2/3] BeagleBoard: Added userbutton command

2011-09-08 Thread Albert ARIBAUD
Hi Joel, Le 08/09/2011 16:56, Joel A Fernandes a écrit : >> Also, I agree with Albert: there should be no need for a separate >> userbutton command. >> >> Please fix and resubmit. > > If this patch is to be dropped, then I'm not sure why the need to resubmit? I gather the "resubmit" means "resub

[U-Boot] [PATCH] MAKEALL: drop boards listed in boards.cfg

2011-09-08 Thread Wolfgang Denk
Pick them up automatically using $(boards_by_arch ...) Signed-off-by: Wolfgang Denk --- We have a problem with the MIPS boards here: We have separate lists for "mips4kc" and "au1xx0" (and again so for "mips4kc_el" and "au1xx0_el"), but in boards.cfg these are all listed with ARCH="mips". Is it

Re: [U-Boot] [PATCH] net: Add \n before warning message so it prints on a new line.

2011-09-08 Thread Albert ARIBAUD
Hi Philip, Le 07/09/2011 13:57, Philip Balister a écrit : > Signed-off-by: Philip Balister > --- > net/eth.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/net/eth.c b/net/eth.c > index dbd1e2d..67a8039 100644 > --- a/net/eth.c > +++ b/net/eth.c > @@ -305,7 +305,

Re: [U-Boot] [GIT PULL] Pull request: u-boot-imx

2011-09-08 Thread Albert ARIBAUD
Hi Stefano, Le 08/09/2011 16:34, Stefano Babic a écrit : > Hi Albert, > > Wolfgang has merged himself most of pending IMX patches. However, a few > are missing and for the rest we agree we go on the "normal" way via pull > request. With this pull request all IMX boards should be fixed. > > The fol

Re: [U-Boot] [PATCH 2/3] BeagleBoard: Added userbutton command

2011-09-08 Thread Joel A Fernandes
On Thu, Sep 8, 2011 at 3:05 AM, Wolfgang Denk wrote: > Dear Joel A Fernandes, > > In message <1313462214-3716-2-git-send-email-agnel.j...@gmail.com> you wrote: >> From: Jason Kridner >> >> Based on commit f1099c7c43caf5bac3bf6a65aa266fade4747072 >>     Author: Greg Turner >>     Date:   Tue May

Re: [U-Boot] [PATCH] OMAP3: Overo: Update GPMC timing for ethernet chip.

2011-09-08 Thread Enric Balletbò i Serra
2011/9/8 Steve Sakoman : > On Wed, Sep 7, 2011 at 4:57 AM, Philip Balister wrote: >> The existing timing does not quite meet the minimum requirements >> in the LAN9221 datasheet. The timing in this patch solves problems >> noticed on some parts. >> >> Signed-off-by: Philip Balister > > Acked-by:

Re: [U-Boot] Bogus "Warning: failed to set MAC address" on Gumstix Overo

2011-09-08 Thread Wolfgang Denk
Dear Philip Balister, In message <4e68c67f.2080...@opensdr.com> you wrote: > When booting master on a Gumstix Overo based system, I get the following > messages during ethernet initialization: > > Net: smc911x-0 > Warning: failed to set MAC address > > (Note that patch [1] is needed to move t

[U-Boot] [GIT PULL] Pull request: u-boot-imx

2011-09-08 Thread Stefano Babic
Hi Albert, Wolfgang has merged himself most of pending IMX patches. However, a few are missing and for the rest we agree we go on the "normal" way via pull request. With this pull request all IMX boards should be fixed. The following changes since commit 63cf97e825b8d821e2d3d6d6c6604634ac24810a:

Re: [U-Boot] [PATCH] OMAP3: Overo: Update GPMC timing for ethernet chip.

2011-09-08 Thread Steve Sakoman
On Wed, Sep 7, 2011 at 4:57 AM, Philip Balister wrote: > The existing timing does not quite meet the minimum requirements > in the LAN9221 datasheet. The timing in this patch solves problems > noticed on some parts. > > Signed-off-by: Philip Balister Acked-by: Steve Sakoman Tested-by: Steve Sak

[U-Boot] Mailbox

2011-09-08 Thread MRS CLARA
I am happy to inform you that your draft of 800,000usd which has been under my custody has been cleared out for delivery to you and the courier company in charge is Fedex courier company(United Kingdom). Am presently away for some official dutties and i hope to stay for more than 6weeks. Kindl

[U-Boot] Bogus "Warning: failed to set MAC address" on Gumstix Overo

2011-09-08 Thread Philip Balister
When booting master on a Gumstix Overo based system, I get the following messages during ethernet initialization: Net: smc911x-0 Warning: failed to set MAC address (Note that patch [1] is needed to move the Warning to a new line, otherwise it is after the previous line with no space) In this

[U-Boot] [PATCH] MAKEALL: drop non-existent "versatile" configuration

2011-09-08 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk Cc: Albert ARIBAUD --- MAKEALL |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/MAKEALL b/MAKEALL index 2e39d2c..3d6b43e 100755 --- a/MAKEALL +++ b/MAKEALL @@ -333,7 +333,6 @@ LIST_ARM9=" \ spear320\

Re: [U-Boot] [PATCH 1/1] mmc: omap: enable high capacity

2011-09-08 Thread T Krishnamoorthy, Balaji
On Sun, Sep 4, 2011 at 1:49 AM, Andy Fleming wrote: > Isn't high-capacity capability also connected to board capabilities? > Is it safe to enable this across all users of this driver? Hi, Host contoller IP supports High capacity. If High capacity card is connected to the board, then this flag is

Re: [U-Boot] [PATCH 0/3] omap: gpio: Use generic API (instead of custom)

2011-09-08 Thread Luca Ceresoli
Premi, Sanjeev wrote: > [sp] Actually, I realized that PATCH 1 in the series depends upon: > http://marc.info/?l=u-boot&m=131515805732292&w=2 > > I had mentioned 2 patches in the cover letter. Of the two, > one is already included in u-boot-arm/master. This one is > still no

[U-Boot] [PATCH 4/6] YAFFS2: fs/yaffs2/yaffs_nand.[hc] - fix build warnings

2011-09-08 Thread Wolfgang Denk
Fix these: yaffs_guts.c: In function 'yaffs_Scan': yaffs_guts.c:5436: warning: pointer targets in passing argument 4 of 'yaffs_QueryInitialBlockState' differ in signedness yaffs_guts.c: In function 'yaffs_ScanBackwards': yaffs_guts.c:6017: warning: pointer targets in passing argument 4 of 'yaffs_

[U-Boot] [PATCH 5/6] YAFFS2: fs/yaffs2/yaffs_guts.c - fix build warnings

2011-09-08 Thread Wolfgang Denk
Fix these: yaffs_guts.c: At top level: yaffs_guts.c:400: warning: 'yaffs_SkipFullVerification' defined but not used Testing shows no changes of the image sizes. Signed-off-by: Wolfgang Denk --- fs/yaffs2/yaffs_guts.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/

[U-Boot] [PATCH 3/6] YAFFS2: fs/yaffs2/Makefile - fix build warnings

2011-09-08 Thread Wolfgang Denk
Drop the "-DNO_Y_INLINE" setting to fix these: yaffs_guts.h:806: warning: 'yaffs_GetBlockInfo' defined but not used Impact on image size is negligible - for the VCMA9 board the text segment size grew from 496353 to 496357 bytes (i. e. 0.0008%); total image size even remained constant. Signed-off-

[U-Boot] [PATCH 6/6] YAFFS2: fs/yaffs2/yaffs_guts.c - fix build warnings

2011-09-08 Thread Wolfgang Denk
Fix these: yaffs_guts.c: In function 'yaffs_ReadDataFromFile': yaffs_guts.c:4622: warning: pointer targets in passing argument 3 of 'yaffs_AddrToChunk' differ in signedness yaffs_guts.c:4622: warning: pointer targets in passing argument 4 of 'yaffs_AddrToChunk' differ in signedness yaffs_guts.c:

[U-Boot] [PATCH 0/6] YAFFS2: fix build warnings

2011-09-08 Thread Wolfgang Denk
The following set of patches fixes the build warnings from the YAFFS2 code in a minimal-invasive way. No attempts are made to fix the coding style (imported from Linux), so the patches are not checkpatch clean. A more thorough clean up is probably a waste of efforts - instead, the code should be

[U-Boot] [PATCH 2/6] YAFFS2: fs/yaffs2/yaffscfg.c - fix build warnings

2011-09-08 Thread Wolfgang Denk
Fix these: yaffscfg.c: In function 'cmd_yaffs_mread_file': yaffscfg.c:316: warning: format '%08x' expects type 'unsigned int', but argument 3 has type 'char *' yaffscfg.c: In function 'cmd_yaffs_ls': yaffscfg.c:371: warning: format '%7d' expects type 'int', but argument 3 has type 'off_t' Signed

[U-Boot] [PATCH 1/6] YAFFS2: cmd_yaffs2.c - fix build warnings

2011-09-08 Thread Wolfgang Denk
Fix these: cmd_yaffs2.c: In function 'do_ywr': cmd_yaffs2.c:69: warning: format '%x' expects type 'unsigned int', but argument 2 has type 'ulong' cmd_yaffs2.c:69: warning: format '%x' expects type 'unsigned int', but argument 3 has type 'ulong' Signed-off-by: Wolfgang Denk --- common/cmd_yaffs

Re: [U-Boot] [PATCH 1/2] net/eth.c: throw BUG for eth_get_dev_by_name(NULL)

2011-09-08 Thread Wolfgang Denk
Dear Helmut Raiger, In message <1314008237-24180-1-git-send-email-helmut.rai...@hale.at> you wrote: > eth_get_dev_by_name() is not safe to use for devname being NULL > as it uses strcmp. This patch makes it fail with a BUG(). > > Signed-off-by: Helmut Raiger > --- > V2: use BUG_ON() instead of g

[U-Boot] [PATCH] smc911x: Fix build warnings

2011-09-08 Thread Wolfgang Denk
Commit 6af1d41 "smc911x MII made available" was missing a few "const" qualifiers. Fix the resulting in build warnings: smc911x.c: In function 'smc911x_initialize': smc911x.c:297: warning: passing argument 2 of 'miiphy_register' from incompatible pointer type smc911x.c:297: warning: passing argum

[U-Boot] [PATCH v6 1/3] net: Adds Fast Ethernet Controller driver for Armada100

2011-09-08 Thread Ajay Bhargav
This patch adds support for Fast Ethernet Controller driver for Armada100 series. Signed-off-by: Ajay Bhargav --- Changes for v2: - removed random MAC generation - driver init function changed to register as per new naming convention - code cleanup (Thanks to Wolfgang, Mar

[U-Boot] [PATCH v6 2/3] Armada100: Enable Ethernet support for GplugD

2011-09-08 Thread Ajay Bhargav
This patch enables ethernet support for Marvell GplugD board. Network related commands works. Signed-off-by: Ajay Bhargav --- Changes for v2: - armada100_fec_initialize changed to armada100_fec_register Changes for v3: - fec base address as argument to armada100_fec_register Chang

[U-Boot] [PATCH v6 3/3] Armada100: Enable 88E3015 PHY support for GplugD

2011-09-08 Thread Ajay Bhargav
This patch adds support for 88E3015 PHY for Marvell GplugD board. This patch depends on series of patch which adds support for Marvell GuruPlug-Display. Signed-off-by: Ajay Bhargav --- Changes for v2: - Not changed Changes for v3: - code cleanup; removed unwated cast Changes for v

Re: [U-Boot] [PATCH v3] ns16550: change to allow 32 bit access to registers

2011-09-08 Thread Dave Aldridge
Hi Wolfgang On 07/09/11 22:22, Wolfgang Denk wrote: > Dear Dave Aldridge, > > In message <1314877212-31552-1-git-send-email-fovs...@gmail.com> you wrote: >> If CONFIG_SYS_NS16550_MEM32 is defined then 32 bit memory >> mapped access will be used to read/write the uart registers. >> >> This is espe

[U-Boot] [PATCH RFC] env: add command to set individual variables to default

2011-09-08 Thread Gerlando Falauto
Here I am proposing an implementation for setting individual variables to their default values as outlined in http://www.denx.de/wiki/U-Boot/TaskSetEnvironmentDefaults For instance, to reset defautl values for variables bootcmd and bootdelay: => env default bootcmd bootdelay There are a few issue

Re: [U-Boot] [PATCH 2/3] BeagleBoard: Added userbutton command

2011-09-08 Thread Wolfgang Denk
Dear Joel A Fernandes, In message <1313462214-3716-2-git-send-email-agnel.j...@gmail.com> you wrote: > From: Jason Kridner > > Based on commit f1099c7c43caf5bac3bf6a65aa266fade4747072 > Author: Greg Turner > Date: Tue May 25 09:19:06 2010 -0500 > > New u-boot command for status o

<    1   2