[U-Boot-Users] [PATCH] common/cmd_load.c - Minor Code Style update & code cleanup

2008-08-07 Thread Gururaja Hebbar K R
- os_data_header Variable is a carry over feature & unused. So removed all instance of this variable - Minor Code Style Update Signed-off-by: Gururaja Hebbar <[EMAIL PROTECTED]> --- common/cmd_load.c | 15 --- 1 files changed, 8 insertions(+), 7 deletions(-) diff --git a/commo

[U-Boot-Users] [PATCH] (Resubmit) ADD ARM AMBA PL031 RTC Support

2008-08-03 Thread Gururaja Hebbar K R
- Add ARM AMBA PL031 RTC Support Signed-off-by: Gururaja Hebbar <[EMAIL PROTECTED]> --- drivers/rtc/Makefile|1 + drivers/rtc/rtc_pl031.c | 123 +++ 2 files changed, 124 insertions(+), 0 deletions(-) create mode 100755 drivers/rtc/rtc_pl031.c

[U-Boot-Users] Resubmit : [PATCH] Correct ARM Versatile Timer Initialization

2008-08-03 Thread Gururaja Hebbar K R
- According to ARM Dual-Timer Module (SP804) TRM (ARM DDI0271), -- Timer Value Register @ TIMER Base + 4 is Read-only. -- Prescale Value (Bits 3-2 of TIMER Control register) can only be one of 00,01,10. 11 is undefined. Signed-off-by: Gururaja Hebbar <[EMAIL PROTECTED]> --- cpu/arm92

Re: [U-Boot-Users] Please Help me to Setup & Send Patch using M$ Office

2008-08-01 Thread Gururaja Hebbar K R
Hi, > Type: man git-send-email But i am using a company email address and it uses Microsoft Exchange server. I will check about configuring it but not sure. TIA Regards Gururaja - This SF.Net email is sponsored by the M

Re: [U-Boot-Users] [PATCH] Correct ARM Versatile Timer Initialization

2008-08-01 Thread Gururaja Hebbar K R
Hi, > In message > <[EMAIL PROTECTED] > > you wrote: > > > > Yes. Sorry for the mistake. It was my M$ Office which was > wrapping the > > mail @ column 76 & hence the issue. > > I have changed it to 81 & i have Re-Submitted the Patch. Hopefully. > > Its Corrected Now. > > Changed it to 81?

Re: [U-Boot-Users] [PATCH] Correct ARM Versatile Timer Initialization

2008-08-01 Thread Gururaja Hebbar K R
Hi, >The original source file has nothing to do with this problem. >Yes, but that is NOT the problem. The problem is that your mailer is wrapping long lines when it sends your patch, thus corrpting it. >The source line may be long, but it is not broken. >It is your mailer which corrupts the patch

[U-Boot-Users] [PATCH] (Resubmit) ADD ARM AMBA PL031 RTC Support

2008-07-31 Thread Gururaja Hebbar K R
Signed-off-by: Gururaja Hebbar <[EMAIL PROTECTED]> --- drivers/rtc/Makefile|1 + drivers/rtc/rtc_pl031.c | 123 +++ 2 files changed, 124 insertions(+), 0 deletions(-) create mode 100755 drivers/rtc/rtc_pl031.c diff --git a/drivers/rtc/Makefile

[U-Boot-Users] Resubmit : [PATCH] Correct ARM Versatile Timer Initialization

2008-07-31 Thread Gururaja Hebbar K R
Signed-off-by: Gururaja Hebbar <[EMAIL PROTECTED]> --- cpu/arm926ejs/versatile/timer.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/cpu/arm926ejs/versatile/timer.c b/cpu/arm926ejs/versatile/timer.c index 32872d2..9659b67 100644 --- a/cpu/arm926ejs/versatile/timer.c ++

Re: [U-Boot-Users] [PATCH] (Resubmit) ADD ARM AMBA PL031 RTC Support

2008-07-31 Thread Gururaja Hebbar K R
Hi, >Please move such comments *below* the '---' line. Anything ABOVE '---' >will become the commit message that goes into the repository, only stuff below the '---' gets ignores as comment. Yes i will do & re-submit >> drivers/rtc/Makefile|1 + >> drivers/rtc/rtc_pl031.c | 123 +

Re: [U-Boot-Users] [PATCH] Correct ARM Versatile Timer Initialization

2008-07-31 Thread Gururaja Hebbar K R
Hi, In message <[EMAIL PROTECTED]> you wrote: >> Hi, >> >> >> @@ -50,8 +50,7 @@ static ulong lastdec; int timer_init (void) { >> >> *(volatile ulong *)(CFG_TIMERBASE + 0) =3D3D CFG_TIMER_RELOAD; >> >> /*=20 TimerLoad */ >> >^ >> >> - *(volatile ulong *)(CFG_TIMERBASE + 4) =3D3D

Re: [U-Boot-Users] [PATCH] Correct ARM Versatile Timer Initialization

2008-07-29 Thread Gururaja Hebbar K R
Hi, >> @@ -50,8 +50,7 @@ static ulong lastdec; int timer_init (void) { >> *(volatile ulong *)(CFG_TIMERBASE + 0) =3D CFG_TIMER_RELOAD; /* >> TimerLoad */ >^ >> - *(volatile ulong *)(CFG_TIMERBASE + 4) =3D CFG_TIMER_RELOAD; /* >> TimerValue */ >^ > >Your patch is line-

[U-Boot-Users] [PATCH] Correct ARM Versatile Timer Initialization

2008-07-24 Thread Gururaja Hebbar K R
Hi, Sorry. Re-sending as uboot-mailing list was not properly Included last time Regards Gururaja - According to ARM Dual-Timer Module (SP804) TRM (ARM DDI0271), --Timer Value Register @ TIMER Base + 4 is Read-only. So removed code which writes to this register --Prescale Value (Bits 3-2 of T

Re: [U-Boot-Users] [PATCH] (Resubmit) ADD ARM AMBA PL031 RTC Support

2008-07-21 Thread Gururaja Hebbar K R
Hi j, >> Could you please send your patch as inline, otherwise it's really difficult to comment use git-send-email as example >> and also please read this http://www.denx.de/wiki/UBoot/Patches >> and http://lwn.net/Articles/139918/ Sorry for the whitespace mistakes. i have now corrected it. B

Re: [U-Boot-Users] [PATCH] (Resubmit) ADD ARM AMBA PL031 RTCSupport

2008-07-21 Thread Gururaja Hebbar K R
Hi, > >>Are you going to add a board which will use it? > >> Right now i am using it on arm Versatile board. Should i send a patch > >> to that also > >>If you do I can test it > >>Regards > >>Peter Please find attached my changes to arm versatile config & board file to enable rtc module

Re: [U-Boot-Users] [PATCH] (Resubmit) ADD ARM AMBA PL031 RTCSupport

2008-07-20 Thread Gururaja Hebbar K R
Hi j, Could you please reply so that i can submit a reformatted patch if required. >> First your patch is line wrapped by your mailler. That's why i had also attached the respective patch file. that i generated using git-diff command. >> Secondly please be care ful of the 80 chars lim

Re: [U-Boot-Users] [PATCH] (Resubmit) ADD ARM AMBA PL031 RTCSupport

2008-07-17 Thread Gururaja Hebbar K R
>>From: Jean-Christophe PLAGNIOL-VILLARD >>Subject: Re: [U-Boot-Users] [PATCH] (Resubmit) ADD ARM AMBA PL031 RTCSupport >>Hi, Hello >> First your patch is line wrapped by your mailler. That's why i had also attached the respective patch file. that i generated using git-diff command. >>

[U-Boot-Users] [PATCH] (Resubmit) : Remove unused code from uboot-1.3.3/lib_arm/bootm.c

2008-07-16 Thread Gururaja Hebbar K R
Hi, Sorry i had generated the patch from with uboot/lib_arm directory and hence i think it failed. now i have created the diff for the root directory where the uboot-1.3.3 folder resides. sorry for the trouble. Thanks in advance. Regards Gururaja Signed off by: Gururaja Hebbar <[EMAIL PROT

[U-Boot-Users] [PATCH] (Resubmit) ADD ARM AMBA PL031 RTC Support

2008-07-16 Thread Gururaja Hebbar K R
Hi, This patch adds support to arm amba pl031 rtc chip. User needs to define 3 variables in board config to make use of this driver & needs to call rtc_init in order to start rtc. #define CONFIG_RTC_PL031 1 /* use ARM AMBA PL031 RTC */ #define CFG_RTC_PL031_BASE 0x101e8000 /* RTC Registers B

[U-Boot-Users] How to call rtc_init from arm boot sequence in uboot

2008-07-16 Thread Gururaja Hebbar K R
HI all, I am working on customized arm9 versatile board. I am trying to write a driver for arm amba pl031 rtc module. i need to setup rtc controller to enable it. I need to call it during boot sequence. But in lib_arm/board.c we do not have a call to rtc_init func. i checked blackfin boot up seq

[U-Boot-Users] [PATCH] : Remove unused code from uboot-1.3.3/lib_arm/bootm.c

2008-07-15 Thread Gururaja Hebbar K R
Hi, This patch removes an unused part of the code. This code is only used in lib_avr32/bootm.c and in lib_arm/bootm.c it is under #if 0 Comments are welcome. Regards Gururaja Signed off by: Gururaja Hebbar <[EMAIL PROTECTED]> --- bootm_orig.c2008-07-15 16:51:03.296875000 -0700 +++

Re: [U-Boot-Users] [PATCH v2] MMC: Consolidate MMC/SD command

2008-05-22 Thread Gururaja Hebbar K R
Hi Haavard Skinnemoen, while you are working on atmel mci mmc driver, i would like to know whether you can tell me anything about the bug mentioned here Possible Bug in at32ap mmc driver atmel_mci.c related to incorrect usage of variables. http://www.nabble.com/Possible-Bug-in-at32ap-mmc-driv

Re: [U-Boot-Users] [PATCH] Remove duplicate #undef SHOW_INFO inside drivers\usb\usb_ohci.c

2008-04-17 Thread Gururaja Hebbar K R
Hi, Thanks and sorry for the mistake. will sure make it a point not to repeat this. Regards Gururaja -Original Message- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] Sent: Thursday, April 17, 2008 9:14 PM To: Gururaja Hebbar K R Cc: u-boot-users@lists.sourceforge.net Subject: Re

[U-Boot-Users] [PATCH] remove error meesage in usb_ohci.c when debug is enabled

2008-04-02 Thread Gururaja Hebbar K R
Hi, In u-boot-1.3.1\drivers\usb\usb_ohci.c driver, if debug is enabled (#define DEBUG), make comes out with an error as below. arm-none-linux-gnueabi-gcc -g -Os -fno-strict-aliasing -fno-common -ffixed-r8 -msoft-float -D__KERNEL__ -DTEXT_BASE=0x20 -I/home/guru/project/u_boot/Ubuild/

[U-Boot-Users] [PATCH] Remove duplicate #undef SHOW_INFO inside drivers\usb\usb_ohci.c

2008-04-01 Thread Gururaja Hebbar K R
Hi, In drivers\usb\usb_ohci.c file, SHOW_INFO is undef at 2 locations. @line 77 and @line 112. Below patch removes them for code size savings. comments welcome. Thanks in advance Signed-off-by: gururaja hebbar <[EMAIL PROTECTED]> --- usb_orig.c 2007-12-06 01:21:19.0 -0800 +++ usb_

[U-Boot-Users] ARM march architecture version for arm926ejs in config.mk

2008-04-01 Thread Gururaja Hebbar K R
Hi, i have seen questions on the mailing list about march for arm926ejs being mentioned as -march=armv4. http://www.nabble.com/-PATCH--ARM%3A-Fix-arm-architecture-version-for-ar m926ejs-td5195486.html#a5210157 http://www.nabble.com/-patch-u-boot-git-2-3--better-ARM9-compiler-option s-td14962153.

Re: [U-Boot-Users] how to enable if, then command support in uboot

2008-03-28 Thread Gururaja Hebbar K R
olve this. thanks in advance Regards Gururaja -Original Message- From: Markus Klotzbücher [mailto:[EMAIL PROTECTED] Sent: Friday, March 28, 2008 3:08 AM To: Gururaja Hebbar K R Cc: u-boot-users@lists.sourceforge.net Subject: Re: [U-Boot-Users] how to enable

Re: [U-Boot-Users] how to enable if, then command support in uboot

2008-03-28 Thread Gururaja Hebbar K R
add missing info Uboot version : 1.3.1 for ARM926EJS processor based board using arm toolchain from CodeSourcery Regards Gururaja -Original Message- From: Gururaja Hebbar K R Sent: Friday, March 28, 2008 6:37 PM To: 'u-boot-users@lists.sourceforge.net' Subject: how to enabl

[U-Boot-Users] how to enable if, then command support in uboot

2008-03-28 Thread Gururaja Hebbar K R
Hi all, I am trying to execute some commands stored in environment as below #define CONFIG_EXTRA_ENV_SETTINGS \ "boot_mmc=" \ "if mmcinit && " \ "fatload mmc 0 ${ramfs_addr} ${ramfs_file} && " \

Re: [U-Boot-Users] multiple definition of __udivsi3 when using divide operation on unsigned long numbers

2008-03-25 Thread Gururaja Hebbar K R
Hi Dirk, Thanks for the information. i looked at he nand_util.c and used the same do_div in my code and was able to compile without any issues. Regards Gururaja > Don't do this. Compiler/Linker then tries to use math from libraries coming with your tool chain. Normally, U-Boot should be compl

[U-Boot-Users] multiple definition of __udivsi3 when using divide operation on unsigned long numbers

2008-03-24 Thread Gururaja Hebbar K R
Hi, I am trying to compile uboot 1.3.1 for arm926ejs processor. i am using CodeSourcery arm-none-linux-gnueabi toolchain (gcc version 4.2.1 (CodeSourcery Sourcery G++ Lite 2007q3-51)) . i am trying to write a driver for our on board mmc controller. i need to do multiply and divide operation as b

[U-Boot-Users] Possible Bug in at32ap mmc driver atmel_mci.c

2008-03-17 Thread Gururaja Hebbar K R
Hi, I am trying to port mmc driver to my board with little assistance from atmel_mci driver. While i was trying to understand the mmc_cmd function, i think i found a bug in it. File -> uboot-1.3.1\cpu\at32ap\atmel_mci.c Function -> static int mmc_cmd(unsigned long cmd, unsigned long arg,

[U-Boot-Users] copy command taking incorrect/invalid arguments

2008-03-01 Thread Gururaja Hebbar K R
Hi, i am working a customized variant of arm versatile board. I have spansion s29gl18n 16MB flash on board. i am using Uboot-1.3.1 Version and CFI flash driver to work on flash related commands our board reset vector points to 0x2c00 By mistake today i entered "cp.b ox80 0x2c00 0x1

Re: [U-Boot-Users] [PATCH] ARM926: Display Ethernet info in do_bdinfo only if CONFIG_CMD_NET is defined

2008-02-04 Thread Gururaja Hebbar K R
Hi Wolfgang, Thanks. Will sure learn about GIT Process. Regards Gururaja -Original Message- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] Sent: Monday, February 04, 2008 2:19 PM To: Gururaja Hebbar K R Cc: u-boot-users@lists.sourceforge.net Subject: Re: [U-Boot-Users] [PATCH

Re: [U-Boot-Users] [PATCH] ARM926: Display Ethernet info in do_bdinfo only if CONFIG_CMD_NET is defined

2008-01-30 Thread Gururaja Hebbar K R
MIPS etc) are eliminated and only ARM is left out. So it doent effect any other systems. Kindly correct me if i am wrong Regards Gururaja -Original Message- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] Sent: Tuesday, January 08, 2008 11:51 PM To: Gururaja Hebbar K R Cc: u-b