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

[Anatolij Gustschin]

Hello Wolfgang,

The following changes since commit 99ffccbd3e5b7bc715e2eed6ea6d36f4020b56d8:

  Flush cache after the OS image is loaded into the memory. (2011-09-05 
16:07:44 +0200)

are available in the git repository at:
  git://git.denx.de/u-boot-video.git master

Wolfgang Denk (1):
  VIDEO: mb86r0xgdc.c: fix warning: unused variable 'i'

 drivers/video/mb86r0xgdc.c |1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

please pull. Thanks!

Anatolij
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] VIDEO: mb86r0xgdc.c: fix warning: unused variable 'i'

[Anatolij Gustschin]

Hello Wolfgang,

On Mon,  5 Sep 2011 21:51:33 +0200
Wolfgang Denk  wrote:

> Fix build warning:
> 
> Configuring for jadecpu board...
> mb86r0xgdc.c: In function 'dsp_init':
> mb86r0xgdc.c:60: warning: unused variable 'i'
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Anatolij Gustschin 
> ---
>  drivers/video/mb86r0xgdc.c |1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)

applied to u-boot-video/master. Thanks!

Anatolij
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v11 5/9] nds32/lib: add generic funcs in NDS32 lib

[馬克泡]

Hi, Mike and Wolfgang,

2011/9/1 Mike Frysinger :
> On Thursday, September 01, 2011 01:52:46 Macpaul Lin wrote:

[... skipped and will discuss these later]

>> +void board_init_f(ulong bootflag)
>> +{
>> ...
>> +     /* Pointer is writable since we allocated a register for it */
>> +     gd = (gd_t *) ((CONFIG_SYS_INIT_SP_ADDR) & ~0x07);
>> +
>> +     /* compiler optimization barrier needed for GCC >= 3.4 */
>> +     __asm__ __volatile__("" : : : "memory");
>> +
>> +     memset((void *)gd, 0, sizeof(gd_t));
>
> err, is that barrier really needed ?  sounds like a bug in your compiler.

I'm not sure if this is really a bug which has been fixed or this is
still a common problem.
I've found there are architectures x86, mips, m68k, nios2, powerpc
have this code to do
optimization barrier.
I'll try to verify this with our toolchain department, but I think
this might be a common problem.
Could someone give a comment on this?

> as for the memset, it should be:
> memset((void *)gd, 0, GENERATED_GBL_DATA_SIZE);

It's the similar case.
Since we have generated header file for global data, this code might be good.
However, there are other architecture remain the same code which do
memset with sizeof(gd_t).
Included m68k, sparc, arm, avr32 and powerpc.
Do I miss something of the discussion before in the mailing list?
Should these also be a clean up for other architectures, too?

>
>> +     addr_sp -= sizeof(bd_t);
>
> sizeof(bd_t) -> GENERATED_BD_INFO_SIZE
>
> this shows up multiple times
>
> also, did i miss something, or do you not zero out your board info ?
>
>> +     addr_sp -= sizeof(gd_t);
>
> sizeof(gd_t) -> GENERATED_GBL_DATA_SIZE
>
> this shows up multiple times
> -mike
>

No, I didn't zero out the board info just like the implementation of
other architecture.
Dose this lead a problem?

-- 
Best regards,
Macpaul Lin
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 3/3 v3] ARM: ARM926EJS - Add cache operations

[Marek Vasut]

On Tuesday, September 06, 2011 08:19:26 AM Albert ARIBAUD wrote:
> Le 05/09/2011 21:50, Marek Vasut a écrit :
> >> Le 02/09/2011 13:57, Marek Vasut a écrit :
> >>> On Friday, September 02, 2011 01:43:39 PM Simon Guinot wrote:
>  On Fri, Sep 02, 2011 at 12:23:54PM +0200, Marek Vasut wrote:
> > On Friday, September 02, 2011 12:22:05 PM Simon Guinot wrote:
> >> Hi Hong,
> >> 
> >> On Mon, Aug 15, 2011 at 03:08:51PM +0800, Hong Xu wrote:
> >>> Hi Marek,
> >>> 
> >>> On 08/11/2011 12:47 PM, Marek Vasut wrote:
>  On Thursday, August 11, 2011 05:27:37 AM Hong Xu wrote:
> > Add a new file arch/arm/cpu/arm926ejs/cache.c and put cache
> > operations into this file.
> > 
> > Signed-off-by: Hong Xu
> > ---
> > 
> > V2:
> >  Fixed a typo when CONFIG_SYS_DCACHE_OFF is defined
> > 
> > V3:
> >  Undo changes in include/configs/at91sam9260ek.h
> >  It's for testing purpose
> > 
> > arch/arm/cpu/arm926ejs/Makefile |   2 +-
> > arch/arm/cpu/arm926ejs/cache.c   |   142
> > 
> > +++ 2 files changed,
> > 143 insertions(+), 1 deletions(-)
> > 
> > create mode 100644 arch/arm/cpu/arm926ejs/cache.c
> >> 
> >> What is the status for this patch ?
> >> 
> >> I have failed to find any trace for this patch in patchwork or
> >> in the arm git repository...
> > 
> > We discussed this should be moved to arch/arm/cpu/armv5 (Aneesh
> > V's idea ... I dunno what's this guys' full name ;-D). I tested
> > this patch on certain type of CPU and it works really good. I'd be
> > really glad if we got this mainline!
>  
>  Yes. Moreover some drivers (as mvgbe) really needs such functions to
>  fix the DMA operations when D-cache is enabled.
> >>> 
> >>> I have some fixes already queued, yes.
> >>> 
>  Simon
> >> 
> >> Hi folks,
> >> 
> >> I'm trying to locate the latest version of this set's patches in
> >> patchwork and I can't find them at all in any state. What am I missing?
> > 
> > Hi Albert,
> > 
> > you're missing the move from cpu/arm926ejs to cpu/armv5/ . Though I'd
> > prefer to have it in arm926 for now as we still didn't verify the other
> > cores are ok with it (I changed my opinion just recently). For example,
> > xscale isn't. So maybe we can pick the patches as they are.
> 
> Seems like we're having two problems there:
> 
> 1) at least some cpus or cores (xscale) do not implement all of what was
> thought to be armv5 cache operations, which suggests we should indeed
> keep this patch in arm926ejs;
> 
> 2) that many cpus and cores implement a good subset of armv5 cache
> operations, which warrants having an armv5 directory where we would
> define them once only for all to use.
> 
> I think the Right Way is to create an armv5 subtree with armv5-common
> operations, but with the capability of overruling these with cpu-level
> variants, themselves being possibly overridden by core-level variants.

All right, but what about the CPUs that don't adapt so fast? There will be CPUs 
with broken cache-ops. How do you intend to solve that ?

> 
> Now as to where to put this:
> 
> As ARMv5 is an ISA, not a cpu or core, I'm uncomfortable with creating
> arch/arm/cpu/armv5 anyway. I'd prefer an ISA subtree, either in parallel
> with the cpu subtree: arch/arm/isa [/armv5], or, and I would slightly
> prefer that even though some paths would become rather long, a hierarchy
> withe cpus stand below their isa: arch/arm/isa [/armv5/cpu/arm926ejs].
> To reduce path length, we could remove the 'cpu' part and consider that
> anything below the 'isa' is a cpu, just like currently anything below
> the cpu is a core.
> 

This is completely out of scope for this patch. My proposal would be to merge 
this, then start mucking with this moving files around.

> To sum it up, we would have
> 
> arch/arm/isa/armv5 (where ARMv5t ISA common code would reside, including
> cache ops)
> 
> arch/arm/isa/armv5/arm926ejs (where ARM926EJ-S cpu common code would
> reside, including cache ops)
> 
> arch/arm/isa/armv5/arm926ejs/orion5x (a personal favorite :) where
> Orion5x core code would reside, including cache ops)
> 
> Maybe we could even make do without the .../isa/... level and put ISAs
> directly under ARM -- I don't think any ARM ISA will ever be named
> 'include' or 'lib' or 'Makefile'. :)
> 
> Comments?

I'll have to think about it, but there is my proposal above. Please consider 
it, 
it'll help while dodging the trouble for a while until this is settled.

> 
> > Cheers!
> 
> Amicalement,

Cheers
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 3/3 v3] ARM: ARM926EJS - Add cache operations

[Albert ARIBAUD]

Le 05/09/2011 21:50, Marek Vasut a écrit :
>> Le 02/09/2011 13:57, Marek Vasut a écrit :
>>> On Friday, September 02, 2011 01:43:39 PM Simon Guinot wrote:
 On Fri, Sep 02, 2011 at 12:23:54PM +0200, Marek Vasut wrote:
> On Friday, September 02, 2011 12:22:05 PM Simon Guinot wrote:
>> Hi Hong,
>>
>> On Mon, Aug 15, 2011 at 03:08:51PM +0800, Hong Xu wrote:
>>> Hi Marek,
>>>
>>> On 08/11/2011 12:47 PM, Marek Vasut wrote:
 On Thursday, August 11, 2011 05:27:37 AM Hong Xu wrote:
> Add a new file arch/arm/cpu/arm926ejs/cache.c and put cache
> operations into this file.
>
> Signed-off-by: Hong Xu
> ---
>
> V2:
>  Fixed a typo when CONFIG_SYS_DCACHE_OFF is defined
>
> V3:
>  Undo changes in include/configs/at91sam9260ek.h
>  It's for testing purpose
>
> arch/arm/cpu/arm926ejs/Makefile |   2 +-
> arch/arm/cpu/arm926ejs/cache.c   |   142
>
> +++ 2 files changed,
> 143 insertions(+), 1 deletions(-)
>
> create mode 100644 arch/arm/cpu/arm926ejs/cache.c
>>
>> What is the status for this patch ?
>>
>> I have failed to find any trace for this patch in patchwork or
>> in the arm git repository...
>
> We discussed this should be moved to arch/arm/cpu/armv5 (Aneesh
> V's idea ... I dunno what's this guys' full name ;-D). I tested
> this patch on certain type of CPU and it works really good. I'd be
> really glad if we got this mainline!

 Yes. Moreover some drivers (as mvgbe) really needs such functions to
 fix the DMA operations when D-cache is enabled.
>>>
>>> I have some fixes already queued, yes.
>>>

 Simon
>>
>> Hi folks,
>>
>> I'm trying to locate the latest version of this set's patches in
>> patchwork and I can't find them at all in any state. What am I missing?
>
> Hi Albert,
>
> you're missing the move from cpu/arm926ejs to cpu/armv5/ . Though I'd prefer 
> to have it in arm926 for now as we still didn't verify the other cores are ok 
> with it (I changed my opinion just recently). For example, xscale isn't. So 
> maybe we can pick the patches as they are.

Seems like we're having two problems there:

1) at least some cpus or cores (xscale) do not implement all of what was 
thought to be armv5 cache operations, which suggests we should indeed 
keep this patch in arm926ejs;

2) that many cpus and cores implement a good subset of armv5 cache 
operations, which warrants having an armv5 directory where we would 
define them once only for all to use.

I think the Right Way is to create an armv5 subtree with armv5-common 
operations, but with the capability of overruling these with cpu-level 
variants, themselves being possibly overridden by core-level variants.

Now as to where to put this:

As ARMv5 is an ISA, not a cpu or core, I'm uncomfortable with creating 
arch/arm/cpu/armv5 anyway. I'd prefer an ISA subtree, either in parallel 
with the cpu subtree: arch/arm/isa [/armv5], or, and I would slightly 
prefer that even though some paths would become rather long, a hierarchy 
withe cpus stand below their isa: arch/arm/isa [/armv5/cpu/arm926ejs]. 
To reduce path length, we could remove the 'cpu' part and consider that 
anything below the 'isa' is a cpu, just like currently anything below 
the cpu is a core.

To sum it up, we would have

arch/arm/isa/armv5 (where ARMv5t ISA common code would reside, including 
cache ops)

arch/arm/isa/armv5/arm926ejs (where ARM926EJ-S cpu common code would 
reside, including cache ops)

arch/arm/isa/armv5/arm926ejs/orion5x (a personal favorite :) where 
Orion5x core code would reside, including cache ops)

Maybe we could even make do without the .../isa/... level and put ISAs 
directly under ARM -- I don't think any ARM ISA will ever be named 
'include' or 'lib' or 'Makefile'. :)

Comments?

> Cheers!

Amicalement,
-- 
Albert.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] mx35pdk vs. mx35_3stack

[Teemu Keskinarkaus]

Hi,

-Original Message-
From: Stefano Babic [mailto:sba...@denx.de] 

Take into account that the u-boot.bin in the last release must be stored on the 
NOR flash of your board - the board has both NAND and NOR flash.
The DIP switch on the main board and on the Debug extension must be set to boot 
from the NOR flash.

Yes, I just realized that it might be the cause. I our hw doesn't have NOR, 
only NAND.  I looked in mx35pdk.h config-header, but couldn't find place where 
to config U-boot for NAND. I'm compiling U-boot myself and not using 
precompiled binary.

-Teemu
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v11 3/9] nds32/core N1213: NDS32 N12 core family N1213

[馬克泡]

Hi Mike and Albert,

2011/9/1 Mike Frysinger :
> On Thursday, September 01, 2011 01:52:44 Macpaul Lin wrote:
>> --- /dev/null
>> +++ b/arch/nds32/cpu/n1213/u-boot.lds
>>
>> +SECTIONS
>> +{
>> +     . = 0x;
>
> shouldnt this be CONFIG_SYS_TEXT_BASE or CONFIG_SYS_MONITOR_BASE or something
> other than 0 ?
> -mike

After checking the binary with NDS32 architecture, I've found the
dominate parameter is
CONFIG_SYS_TEXT_BASE or CONFIG_SYS_MONITOR_BASE.
So this line is not necessary.
However I also found there are the same configurations in ARM's linker script.
I just curios about is this a necessity for ARM?

I'll remove this in PATCH v12. Thanks.

-- 
Best regards,
Macpaul Lin
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] Good news?! China Daily Europe advertising - Two-for-One expiring 15th Sept

[China Daily Europe (European Edition - Weekly)]

Hi there!

We thought you might be interested to know that the TWO-FOR-ONE advertising 
offer for China Daily Europe Edition (weekly) has been reinstated briefly and 
is finishing on 15th Sept 2011 (original details below). This £2500.00 offer 
gives you exposure to the entire AB1 distribution in two weekly hard copy 
editions of your choice, and exposure at the heavily trafficked China Daily 
Online for the duration of your entries, with clickthroughs/tracking. Usual 
list price for this package is £5500.00 fyi, and it will enable you to receive 
enquiries for your products and services from the Chinese and Euro-Chinese 
marketplaces with excellent ROI. In addition, the first five to accept are 
receiving a free web banner without charge!

Can this fit into budget? Simply reply with the word 'BOOK!' and we will get 
back to you before the offer deadline cutoff! We think you'll be glad you did!

Best wishes, 
Chris Brown
@ SKS for China Daily Europe / +44 (0) 203 286 8737

W: http://europe.chinadaily.com.cn/


* original email


Hi there

How are you doing? Getting into summertime now? :))

Good news, I think?! As you might recall, your organisation has been invited to 
advertise in China Daily Europe Edition (weekly).

We now have ten two-for-one offers to send out on a first-come-first-served 
basis, and we thought we would send you one of them! For China Daily European 
Edition (weekly, in English), it's usually £2500.00 for the full tabloid size 
ad page, full colour, BUT you can have two of these included when you book 
before 7th July (although the editions in which you appear can be later in the 
year at any time). This offer also includes the online edition in the European 
area at the site which gets 2.5 million unique business users per week, thus 
perfect for you to reach the C-level subscribers and senior executives 
interested in China, and vice-versa, for your business development, direct 
sales and branding!

To take advantage of this limited time Two-for-One offer, please simply reply 
to this email with the word "BOOK!", and we'll be delighted to welcome you 
aboard by return before 7th July!

Interested?!

Best wishes, 

Chris Brown 
@ SKS for China Daily Europe

T: +44 (0)203 286 8737 ddi / or via accts team +44 (0) 207 607 0717 ddi

W: http://europe.chinadaily.com.cn/

Downloads:

for CDEW:
China Daily Europe Flyer (weekly) - 
http://www.sksassociates.co.uk/cd/ChinaDailyEuropeanWeeklyMediaFlyerJune2011.pdf
China Daily Europe - airline distribution - 
http://www.sksassociates.co.uk/cd/ChinaDailyEuropeanWeekly-airlines.xls
China Daily Europe - hotel distribution - 
http://www.sksassociates.co.uk/cd/ChinaDailyEuropeanWeekly-hotels.xls
China Daily Europe - corporate receptions distribution - 
http://www.sksassociates.co.uk/cd/ChinaDailyEuropeanWeekly-corporates.xls

and for China:
China Daily China Edition (daily) - 
http://www.sksassociates.co.uk/cd/2011ChinaDailyChinaADRATE.pdf
China Daily website - 
http://www.sksassociates.co.uk/cd/2011ChinaDailyWebsiteMediaKit.pdf


Contracted exclusively by: 
SKS/Enhanced Media & Communications Ltd
1 Farnham Road, Guildford, Surrey GU2 4RG UK


"Bringing the top drawer to you!"


** original email





Hi there

We know how busy you are, so just a quick note, if we may, to let you know 
about China Daily European Edition, advertising in which is now available for 
the first time ever!

>From China Daily, now 30 years old, as you might see it's a brand new and 
>exciting opportunity, especially if your organisation is intending to reach 
>European CEOs/CFOs/Senior Executives interested in China, and full page 
>advertisements are now invited (flyer link below). It's weekly, it's in 
>English, it's a business tabloid-size newspaper, it's distributed to its 
>41,000 C-level subscription base each week, and also offers some private 
>distribution to corporate receptions, business hotels and airlines (approx 
>10,000 per week also). In addition, the online edition (included) at the 
>European section of the website benefits from 2.5 million unique visitors per 
>week. You might also like to know that the European edition is now already a 
>year old, and we have spent the entire year building the subscription base and 
>distribution, now available for your advertising.

AND, if you would like to reach the Chinese market itself in English (with 
Chinese subscript), China Daily (daily broadsheet) will do this effectively for 
you, as, indeed, will the China Daily website, now benefiting from 3.6 million 
unique visitors per day, media pack links here:

China Daily China Edition (daily, broadsheet) - 
http://sksassociates.co.uk/cd/2011ChinaDailyChinaADRATE.pdf (large file)

China Daily website - 
http://sksassociates.co.uk/cd/2011ChinaDailyWebsiteMediaKit.pdf

You may decide that, for a new opportunity such as China Daily European Weekly, 
you might like to try a 'test' advertisement. We appreciate this sentiment, and 
there

[U-Boot] [PATCH v2] fix compile warning for env tools

[Luka Perkov]

Patch fixes this issue:

fw_env.c: In function ‘fw_setenv’:
fw_env.c:492:5: warning: format ‘%u’ expects argument of type ‘unsigned int’, 
but argument 3 has type ‘size_t’ [-Wformat]
fw_env.c: In function ‘flash_write_buf’:
fw_env.c:806:6: warning: format ‘%u’ expects argument of type ‘unsigned int’, 
but argument 3 has type ‘size_t’ [-Wformat]

Signed-off-by: Luka Perkov 
---

diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c
index ed6b53f..cd0cc31 100644
--- a/tools/env/fw_env.c
+++ b/tools/env/fw_env.c
@@ -488,7 +488,7 @@ int fw_setenv(int argc, char *argv[])
value = (char *)malloc(len - strlen(name));
if (!value) {
fprintf(stderr,
-   "Cannot malloc %u bytes: %s\n",
+   "Cannot malloc %zu bytes: %s\n",
len - strlen(name), strerror(errno));
return -1;
}
@@ -802,7 +802,7 @@ static int flash_write_buf (int dev, int fd, void *buf, 
size_t count,
data = malloc (erase_len);
if (!data) {
fprintf (stderr,
-"Cannot malloc %u bytes: %s\n",
+"Cannot malloc %zu bytes: %s\n",
 erase_len, strerror (errno));
return -1;
}
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Patches to fix more ARM boards

[stefano babic]

Am 05/09/2011 21:26, schrieb Wolfgang Denk:
> Dear Albert & Stefano,
>
> earlier today I posted a number of patches to remove some more dead
> ARM boards:
>
> [PATCH 00/10] ARM board removal, next round
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107465
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107469
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107467
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107471
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107464
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107473
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107472
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107470
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107466
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107474
>
> In addition, a number of other patches are neede to fix more build
> breakage on ARM, i. e.
>
> 09/05 Stefano Babic  [PATCH V4] Makefile : fix generation of cpu related 
> asm-offsets.h
> http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107505
>
> 09/05 Wolfgang Denk  [PATCH] da8xxevm: Fix warning: unused variable 'val'
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107515
>
> 09/05 Wolfgang Denk  [PATCH] omap24xx: fix 'reset_timer_masked' 
> declaration error
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107521
>
> 09/05 Wolfgang Denk  harmony: fix out of tree building
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107526
>
> 08/30 Stefano Babic  [U-Boot] [PATCH 1/2] MX31: fix missing mxc_get_clk() 
> call
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/106825
>
> 08/30 Stefano Babic  [U-Boot] [PATCH 2/2] I2C: mxc: fix compilation for 
> MX31
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/106826
>
> 08/30 Stefano Babic  [PATCH 1/2] Makefile: drop imx31_phycore from 
> general Makefile
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/106827
>
> 08/30 Stefano Babic  [PATCH 2/2] Makefile: remove mx31pdk exception
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/106828
>
> And maybe some more.
>
> Normally these should go through your iMX and ARM repositories.  To
> save time and finally get a -rc1 out, I would like to pull these
> directly, if this is OK with you.  As far as I can tell these re all
> straightforward fixes, without bigger impact on architectural things.
>
> Do you agree?

Agree, that's fine for me.

Stefano

-- 

=
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] minor improvements for env tools

[Wolfgang Denk]

Dear Luka Perkov,

In message <20110905203848.ga12...@w500.gigaset.lan> you wrote:
> Patch from openwrt commit r20062 regarding erase_len.
> 
> Correct types inside two fprintf functions.
> 
> Signed-off-by: Luka Perkov < lists ->-to->- lukaperkov.net >

Also, please fix you SoB line. "lists ->-to->- lukaperkov.net" is not
a valid mail address.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
"Who alone has reason to *lie  himself  out*  of  actuality?  He  who
*suffers* from it." - Friedrich Nietzsche
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] minor improvements for env tools

[Wolfgang Denk]

Dear Luka Perkov,

In message <20110905203848.ga12...@w500.gigaset.lan> you wrote:
> Patch from openwrt commit r20062 regarding erase_len.
> 
> Correct types inside two fprintf functions.
> 
> Signed-off-by: Luka Perkov < lists ->-to->- lukaperkov.net >
> ---
> 
> diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c
> index ed6b53f..4e4f6aa 100644
> --- a/tools/env/fw_env.c
> +++ b/tools/env/fw_env.c
> @@ -488,7 +488,7 @@ int fw_setenv(int argc, char *argv[])
>   value = (char *)malloc(len - strlen(name));
>   if (!value) {
>   fprintf(stderr,
> - "Cannot malloc %u bytes: %s\n",
> + "Cannot malloc %zu bytes: %s\n",
>   len - strlen(name), strerror(errno));
>   return -1;
>   }
> @@ -779,7 +779,10 @@ static int flash_write_buf (int dev, int fd, void *buf, 
> size_t count,
>   erase_offset = (offset / blocklen) * blocklen;
>  
>   /* Maximum area we may use */
> - erase_len = top_of_range - erase_offset;
> + if (mtd_type == MTD_NANDFLASH)
> + erase_len = top_of_range - erase_offset;
> + else
> + erase_len = blocklen;

This is obviously not a printf() type fix.  Please submit as separate
patch, with exact descrition what this is good for, i. e. which
problem it fixes.

Thanks.


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
"In matrimony, to hesitate is sometimes to be saved."- Butler
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

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

[Paulraj, Sandeep]

> 
> Sanjeev Premi (3):
>   omap: gpio: Use generic API
>   omap: gpio: generic changes after changing API
>   omap: gpio: Adapt board files to use generic API
> 
>  arch/arm/cpu/armv7/omap-common/gpio.c |  113 
> -
>  arch/arm/cpu/armv7/omap4/clocks.c |5 +-
>  arch/arm/include/asm/omap_gpio.h  |   15 -
>  board/cm_t35/leds.c   |6 +-
>  board/comelit/dig297/dig297.c |   12 ++--
>  board/isee/igep0020/igep0020.c|   12 ++--
>  board/logicpd/zoom2/debug_board.c |   10 ++--
>  board/logicpd/zoom2/led.c |   38 ++--
>  board/logicpd/zoom2/zoom2.c   |   10 ++--
>  board/overo/overo.c   |   54 
>  board/ti/beagle/beagle.c  |   44 +++---
>  board/ti/beagle/led.c |   22 +++---
>  board/ti/evm/evm.c|   12 ++--
>  doc/README.omap3  |   20 +++---
>  14 files changed, 205 insertions(+), 168 deletions(-)


Albert, Wolfgang,

After sufficient review, can we perhaps add this patch set to this release?

When I submitted my pull request I know omap3 beagle was compiling. However 
after I rebased with the mainline, the builds broke.

Regards,
Sandeep
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] minor improvements for env tools

[Luka Perkov]

Patch from openwrt commit r20062 regarding erase_len.

Correct types inside two fprintf functions.

Signed-off-by: Luka Perkov < lists ->-to->- lukaperkov.net >
---

diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c
index ed6b53f..4e4f6aa 100644
--- a/tools/env/fw_env.c
+++ b/tools/env/fw_env.c
@@ -488,7 +488,7 @@ int fw_setenv(int argc, char *argv[])
value = (char *)malloc(len - strlen(name));
if (!value) {
fprintf(stderr,
-   "Cannot malloc %u bytes: %s\n",
+   "Cannot malloc %zu bytes: %s\n",
len - strlen(name), strerror(errno));
return -1;
}
@@ -779,7 +779,10 @@ static int flash_write_buf (int dev, int fd, void *buf, 
size_t count,
erase_offset = (offset / blocklen) * blocklen;
 
/* Maximum area we may use */
-   erase_len = top_of_range - erase_offset;
+   if (mtd_type == MTD_NANDFLASH)
+   erase_len = top_of_range - erase_offset;
+   else
+   erase_len = blocklen;
 
blockstart = erase_offset;
/* Offset inside a block */
@@ -802,7 +805,7 @@ static int flash_write_buf (int dev, int fd, void *buf, 
size_t count,
data = malloc (erase_len);
if (!data) {
fprintf (stderr,
-"Cannot malloc %u bytes: %s\n",
+"Cannot malloc %zu bytes: %s\n",
 erase_len, strerror (errno));
return -1;
}
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 3/3] omap: gpio: Adapt board files to use generic API

[Sanjeev Premi]

This patch contains updates the sources in the board files
to use the generic API.

Signed-off-by: Sanjeev Premi 
---

 While most changes were done via script mentioned in the
 patch 0/3, some of the changes were manual - as the script
 failed to match few instances.


 board/cm_t35/leds.c   |   10 +++---
 board/comelit/dig297/dig297.c |   12 
 board/isee/igep0020/igep0020.c|   12 
 board/logicpd/zoom2/debug_board.c |   10 +++---
 board/logicpd/zoom2/led.c |   38 +-
 board/logicpd/zoom2/zoom2.c   |   10 +++---
 board/overo/overo.c   |   54 ++--
 board/ti/beagle/beagle.c  |   44 +++---
 board/ti/beagle/led.c |   22 +++---
 board/ti/evm/evm.c|   12 
 10 files changed, 112 insertions(+), 112 deletions(-)

diff --git a/board/cm_t35/leds.c b/board/cm_t35/leds.c
index 71c5b0d..48ad598 100644
--- a/board/cm_t35/leds.c
+++ b/board/cm_t35/leds.c
@@ -20,26 +20,26 @@
  */
 #include 
 #include 
-#include 
+#include 
 
 static unsigned int leds[] = { GREEN_LED_GPIO };
 
 void __led_init(led_id_t mask, int state)
 {
-   if (omap_request_gpio(leds[mask]) != 0) {
+   if (gpio_request(leds[mask], "") != 0) {
printf("%s: failed requesting GPIO%u\n", __func__, leds[mask]);
return;
}
 
-   omap_set_gpio_direction(leds[mask], 0);
+   gpio_direction_output(leds[mask], 0);
 }
 
 void __led_set(led_id_t mask, int state)
 {
-   omap_set_gpio_dataout(leds[mask], state == STATUS_LED_ON);
+   gpio_set_value(leds[mask], state == STATUS_LED_ON);
 }
 
 void __led_toggle(led_id_t mask)
 {
-   omap_set_gpio_dataout(leds[mask], !omap_get_gpio_datain(leds[mask]));
+   gpio_set_value(leds[mask], !gpio_get_value(leds[mask]));
 }
diff --git a/board/comelit/dig297/dig297.c b/board/comelit/dig297/dig297.c
index a7071cd..c81ce58 100644
--- a/board/comelit/dig297/dig297.c
+++ b/board/comelit/dig297/dig297.c
@@ -42,7 +42,7 @@
 #include 
 #include 
 #include 
-#include 
+#include 
 #include 
 #include "dig297.h"
 
@@ -177,13 +177,13 @@ static void setup_net_chip(void)
   &ctrl_base->gpmc_nadv_ale);
 
/* Make GPIO 12 as output pin and send a magic pulse through it */
-   if (!omap_request_gpio(NET_LAN9221_RESET_GPIO)) {
-   omap_set_gpio_direction(NET_LAN9221_RESET_GPIO, 0);
-   omap_set_gpio_dataout(NET_LAN9221_RESET_GPIO, 1);
+   if (!gpio_request(NET_LAN9221_RESET_GPIO, "")) {
+   gpio_direction_output(NET_LAN9221_RESET_GPIO, 0);
+   gpio_set_value(NET_LAN9221_RESET_GPIO, 1);
udelay(1);
-   omap_set_gpio_dataout(NET_LAN9221_RESET_GPIO, 0);
+   gpio_set_value(NET_LAN9221_RESET_GPIO, 0);
udelay(31000);  /* Should be >= 30ms according to datasheet */
-   omap_set_gpio_dataout(NET_LAN9221_RESET_GPIO, 1);
+   gpio_set_value(NET_LAN9221_RESET_GPIO, 1);
}
 }
 #endif /* CONFIG_CMD_NET */
diff --git a/board/isee/igep0020/igep0020.c b/board/isee/igep0020/igep0020.c
index 36cc924..2279cc0 100644
--- a/board/isee/igep0020/igep0020.c
+++ b/board/isee/igep0020/igep0020.c
@@ -24,7 +24,7 @@
 #include 
 #include 
 #include 
-#include 
+#include 
 #include 
 #include 
 #include 
@@ -81,13 +81,13 @@ static void setup_net_chip(void)
&ctrl_base->gpmc_nadv_ale);
 
/* Make GPIO 64 as output pin and send a magic pulse through it */
-   if (!omap_request_gpio(64)) {
-   omap_set_gpio_direction(64, 0);
-   omap_set_gpio_dataout(64, 1);
+   if (!gpio_request(64, "")) {
+   gpio_direction_output(64, 0);
+   gpio_set_value(64, 1);
udelay(1);
-   omap_set_gpio_dataout(64, 0);
+   gpio_set_value(64, 0);
udelay(1);
-   omap_set_gpio_dataout(64, 1);
+   gpio_set_value(64, 1);
}
 }
 #endif
diff --git a/board/logicpd/zoom2/debug_board.c 
b/board/logicpd/zoom2/debug_board.c
index a4ddf29..33aa600 100644
--- a/board/logicpd/zoom2/debug_board.c
+++ b/board/logicpd/zoom2/debug_board.c
@@ -22,7 +22,7 @@
 #include 
 #include 
 #include 
-#include 
+#include 
 
 #define DEBUG_BOARD_CONNECTED  1
 #define DEBUG_BOARD_NOT_CONNECTED  0
@@ -33,14 +33,14 @@ static void zoom2_debug_board_detect (void)
 {
int val = 0;
 
-   if (!omap_request_gpio(158)) {
+   if (!gpio_request(158, "")) {
/*
 * GPIO to query for debug board
 * 158 db board query
 */
-   omap_set_gpio_direction(158, 1);
-   val = omap_get_gpio_datain(158);
-   omap_free_gpio(158);
+   gpio_direction_input(158);
+   val = gpio_get_value(158);
+   gpio_free(158);
}
 
if (

[U-Boot] [PATCH 2/3] omap: gpio: generic changes after changing API

[Sanjeev Premi]

This patch contains the generic changes required after
change to generic API in the previous patch.

Signed-off-by: Sanjeev Premi 
---

 This patch could have been merged with previous one, but
 has been kept separate to illustrate (and review) the
 changes to generic code.

 arch/arm/cpu/armv7/omap4/clocks.c |5 +++--
 doc/README.omap3  |   20 ++--
 2 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/arch/arm/cpu/armv7/omap4/clocks.c 
b/arch/arm/cpu/armv7/omap4/clocks.c
index eda960c..4f0e0cd 100644
--- a/arch/arm/cpu/armv7/omap4/clocks.c
+++ b/arch/arm/cpu/armv7/omap4/clocks.c
@@ -31,6 +31,7 @@
  */
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -481,8 +482,8 @@ static void do_scale_tps62361(u32 reg, u32 volt_mv)
 * VSEL1 is grounded on board. So the following selects
 * VSEL1 = 0 and VSEL0 = 1
 */
-   omap_set_gpio_direction(TPS62361_VSEL0_GPIO, 0);
-   omap_set_gpio_dataout(TPS62361_VSEL0_GPIO, 1);
+   gpio_direction_output(TPS62361_VSEL0_GPIO, 0);
+   gpio_set_value(TPS62361_VSEL0_GPIO, 1);
 
temp = TPS62361_I2C_SLAVE_ADDR |
(reg << PRM_VC_VAL_BYPASS_REGADDR_SHIFT) |
diff --git a/doc/README.omap3 b/doc/README.omap3
index 460950d..1768cdd 100644
--- a/doc/README.omap3
+++ b/doc/README.omap3
@@ -98,24 +98,24 @@ gpio
 
 To set a bit :
 
-   if (!omap_request_gpio(N)) {
-   omap_set_gpio_direction(N, 0);
-   omap_set_gpio_dataout(N, 1);
+   if (!gpio_request(N, "")) {
+   gpio_direction_output(N, 0);
+   gpio_set_value(N, 1);
}
 
 To clear a bit :
 
-   if (!omap_request_gpio(N)) {
-   omap_set_gpio_direction(N, 0);
-   omap_set_gpio_dataout(N, 0);
+   if (!gpio_request(N, "")) {
+   gpio_direction_output(N, 0);
+   gpio_set_value(N, 0);
}
 
 To read a bit :
 
-   if (!omap_request_gpio(N)) {
-   omap_set_gpio_direction(N, 1);
-   val = omap_get_gpio_datain(N);
-   omap_free_gpio(N);
+   if (!gpio_request(N, "")) {
+   gpio_direction_input(N);
+   val = gpio_get_value(N);
+   gpio_free(N);
}
if (val)
printf("GPIO N is set\n");
-- 
1.7.0.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 1/3] omap: gpio: Use generic API

[Sanjeev Premi]

Convert all OMAP specific functions to use the common API
definitions in include/asm/gpio.h. In the process, made
few additional changes:
 - Use -EINVAL consistently. -1 was used in many places.
 - Removed one-liner static functions that were used only
   once. Replaced the content as necessary.
 - Combines implementation of functions omap_get_gpio_dataout()
   and omap_get_gpio_datain(). To do so, new static function
   _get_gpio_direction() was added.

Signed-off-by: Sanjeev Premi 
---

 Changes since RFC:
 - Rebased against u-boot-arm.git
 - Dummy header is no longer necessary due to additional
   changes on the arm tree.
 - Added function _get_gpio_direction().
 - Fold omap_get_gpio_dataout() and omap_get_gpio_datain()
   into single gpio_get_value().

 arch/arm/cpu/armv7/omap-common/gpio.c |  113 -
 arch/arm/include/asm/omap_gpio.h  |   15 -
 2 files changed, 82 insertions(+), 46 deletions(-)

diff --git a/arch/arm/cpu/armv7/omap-common/gpio.c 
b/arch/arm/cpu/armv7/omap-common/gpio.c
index 4749524..5b14c0f 100644
--- a/arch/arm/cpu/armv7/omap-common/gpio.c
+++ b/arch/arm/cpu/armv7/omap-common/gpio.c
@@ -40,6 +40,9 @@
 #include 
 #include 
 
+#define OMAP_GPIO_DIR_OUT  0
+#define OMAP_GPIO_DIR_IN   1
+
 static inline const struct gpio_bank *get_gpio_bank(int gpio)
 {
return &omap_gpio_bank[gpio >> 5];
@@ -53,17 +56,17 @@ static inline int get_gpio_index(int gpio)
 static inline int gpio_valid(int gpio)
 {
if (gpio < 0)
-   return -1;
+   return -EINVAL;
if (gpio < 192)
return 0;
-   return -1;
+   return -EINVAL;
 }
 
 static int check_gpio(int gpio)
 {
if (gpio_valid(gpio) < 0) {
printf("ERROR : check_gpio: invalid GPIO %d\n", gpio);
-   return -1;
+   return -EINVAL;
}
return 0;
 }
@@ -89,14 +92,29 @@ static void _set_gpio_direction(const struct gpio_bank 
*bank, int gpio,
__raw_writel(l, reg);
 }
 
-void omap_set_gpio_direction(int gpio, int is_input)
+/**
+ * Get the direction of the GPIO by reading the GPIO_OE register
+ * corresponding to the specified bank.
+ */
+static int _get_gpio_direction(const struct gpio_bank *bank, int gpio)
 {
-   const struct gpio_bank *bank;
+   void *reg = bank->base;
+   u32 v;
 
-   if (check_gpio(gpio) < 0)
-   return;
-   bank = get_gpio_bank(gpio);
-   _set_gpio_direction(bank, get_gpio_index(gpio), is_input);
+   switch (bank->method) {
+   case METHOD_GPIO_24XX:
+   reg += OMAP_GPIO_OE;
+   break;
+   default:
+   return -EINVAL;
+   }
+
+   v = __raw_readl(reg);
+
+   if (v & (1 << gpio))
+   return OMAP_GPIO_DIR_IN;
+   else
+   return OMAP_GPIO_DIR_OUT;
 }
 
 static void _set_gpio_dataout(const struct gpio_bank *bank, int gpio,
@@ -121,20 +139,27 @@ static void _set_gpio_dataout(const struct gpio_bank 
*bank, int gpio,
__raw_writel(l, reg);
 }
 
-void omap_set_gpio_dataout(int gpio, int enable)
+/**
+ * Set value of the specified gpio
+ */
+void gpio_set_value(int gpio, int value)
 {
const struct gpio_bank *bank;
 
if (check_gpio(gpio) < 0)
return;
bank = get_gpio_bank(gpio);
-   _set_gpio_dataout(bank, get_gpio_index(gpio), enable);
+   _set_gpio_dataout(bank, get_gpio_index(gpio), value);
 }
 
-int omap_get_gpio_datain(int gpio)
+/**
+ * Get value of the specified gpio
+ */
+int gpio_get_value(int gpio)
 {
const struct gpio_bank *bank;
void *reg;
+   int input;
 
if (check_gpio(gpio) < 0)
return -EINVAL;
@@ -142,7 +167,17 @@ int omap_get_gpio_datain(int gpio)
reg = bank->base;
switch (bank->method) {
case METHOD_GPIO_24XX:
-   reg += OMAP_GPIO_DATAIN;
+   input = _get_gpio_direction(bank, get_gpio_index(gpio));
+   switch (input) {
+   case OMAP_GPIO_DIR_IN:
+   reg += OMAP_GPIO_DATAIN;
+   break;
+   case OMAP_GPIO_DIR_OUT:
+   reg += OMAP_GPIO_DATAOUT;
+   break;
+   default:
+   return -EINVAL;
+   }
break;
default:
return -EINVAL;
@@ -151,32 +186,45 @@ int omap_get_gpio_datain(int gpio)
& (1 << get_gpio_index(gpio))) != 0;
 }
 
-int omap_get_gpio_dataout(int gpio)
+/**
+ * Set gpio direction as input
+ */
+int gpio_direction_input(unsigned gpio)
 {
-   struct gpio_bank *bank;
-   void *reg;
+   const struct gpio_bank *bank;
 
if (check_gpio(gpio) < 0)
return -EINVAL;
+
bank = get_gpio_bank(gpio);
-   reg = bank->base;
-   switch (bank->method) {
-   case METHOD_GPIO_24XX:
-   reg += OMAP_GPIO_DATAOUT;
-   

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

[Sanjeev Premi]

The OMAP boards use a custom api for GPIO operations. While
it works, it doesn't help when when we don't know existence
of the customization.

I earlier encountered the problem when looking for GPIO
related changes when submitting prev set of patches. Since
the search for gpio_request() in omap sources returned empty,
I had assumed that it isn't supported.

This patchset aims to use generic API instead of custom one.
After making the API level changes, most of the board specific
files were updated via this bash script (to minimize errors).

  [script]
  list=search.lst

  [ -f ${list} ] && \rm -rf ${list}

  grep -l -R "omap_.*_gpio" . > ${list}

  sed -i '/\.sh$/d'${list}  # Protect this script

  sed -i '/\.o$/d' ${list}  # Object files
  sed -i '/\.orig$/d'  ${list}  # ongoing merge
  sed -i '/\.rej$/d'   ${list}  # ongoing merge

  sed -i '/\.git/d'${list}  # git packs
  sed -i '/patch/d'${list}  # patches
  sed -i '/diff/d' ${list}  # saved diffs
  sed -i '/save/d' ${list}  # any specifically saved content

  for f in `cat ${list}` ;
  do
echo "Fixing $f..."

sed -r -i 's/omap_request_gpio\s*\((\w+)\)/gpio_request(\1, "")/g' "$f"
sed -r -i 's/omap_free_gpio/gpio_free/g' "$f"
sed -r -i 
's/omap_set_gpio_direction\s*\((\w+)\s*,\s*1\)/gpio_direction_input(\1)/g' "$f"
sed -r -i 
's/omap_set_gpio_direction\s*\((\w+)\s*,\s*(\w+)\)/gpio_direction_output(\1, 
\2)/g' "$f"
sed -r -i 's/omap_set_gpio_dataout/gpio_set_value/g' "$f"
sed -r -i 's/omap_get_gpio_datain/gpio_get_value/g' "$f"
sed -r -i 's/omap_get_gpio_dataout/gpio_get_value/g' "$f"
sed -r -i 's/asm\/arch\/gpio.h/asm\/gpio.h/g' "$f"
  done
  [/script]

Few manual changes were still required to ensure that sources
compile without errors for all these configurations:
 - omap3_evm_config
 - am3517_evm_config
 - am3517_crane_config
 - omap4_panda_config
 - omap3_zoom1_config
 - omap4_sdp4430_config
 - igep0020_config
 - cm_t35_config
 - devkit8000_config
 - igep0030_config
 - dig297_config
 - omap3_beagle_config
 - omap3_pandora_config
 - omap3_sdp3430_config
 - omap3_zoom2_config
 - omap3_overo_config

 All these boards define "CONFIG_OMAP34XX" - therefore likely
 to be impacted with this change. The list was created using:
 grep -l -R "OMAP34XX" include/configs | cut -d "/" -f 3 | sed 's/\.h/_config/g'

 Some additional patches would be required for successful
 build. Some of them are under discussion, but they status
 shouldn't impact the contents of this patch set:
 1) http://marc.info/?l=u-boot&m=131515805732292&w=2
 2) http://marc.info/?l=u-boot&m=131521839308281&w=2

Changes since RFC[1]:
 1) While the original RFC was based against the master of
u-boot.git. This series is based against master of
u-boot-arm.git

 2) Used better logic to search and fix 'possibly' impacted
files i.e. the script embedded above.

 3) In order to avoid rejection of patch series due to long
recipent list, board maintainers will be sent separate
(off-the-list) notification.

 [1] http://marc.info/?l=u-boot&m=131473403131657&w=2


Sanjeev Premi (3):
  omap: gpio: Use generic API
  omap: gpio: generic changes after changing API
  omap: gpio: Adapt board files to use generic API

 arch/arm/cpu/armv7/omap-common/gpio.c |  113 -
 arch/arm/cpu/armv7/omap4/clocks.c |5 +-
 arch/arm/include/asm/omap_gpio.h  |   15 -
 board/cm_t35/leds.c   |6 +-
 board/comelit/dig297/dig297.c |   12 ++--
 board/isee/igep0020/igep0020.c|   12 ++--
 board/logicpd/zoom2/debug_board.c |   10 ++--
 board/logicpd/zoom2/led.c |   38 ++--
 board/logicpd/zoom2/zoom2.c   |   10 ++--
 board/overo/overo.c   |   54 
 board/ti/beagle/beagle.c  |   44 +++---
 board/ti/beagle/led.c |   22 +++---
 board/ti/evm/evm.c|   12 ++--
 doc/README.omap3  |   20 +++---
 14 files changed, 205 insertions(+), 168 deletions(-)

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] ARM: PXA: remove broken "zylonite" board.

[Marek Vasut]

On Monday, September 05, 2011 10:00:30 PM Wolfgang Denk wrote:
> Signed-off-by: Wolfgang Denk 
> Cc: Albert ARIBAUD 
> Cc: Marek Vasut 
> ---

Acked-by: Marek Vasut 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] OMAP: Remove omapfb.debug=y from Beagle and Overo env settings

[Paulraj, Sandeep]

> 
> Le 11/08/2011 21:28, Joel A Fernandes a écrit :
> > From: Steve Sakoman
> >
> > The kernel DSS2 code is mature now, and keeping this setting hurts
> performance
> >
> > Signed-off-by: Steve Sakoman
> > Signed-off-by: Joel A Fernandes
> > ---
> > Changes since v1:
> > Added myself to SOB
> 
> I see no indication of this anywhere, but this change seems to be
> already in git, from a patch submitted by Steve Sakoman on december 2010:
> 
>  boot.git;a=commit;h=2490e591b4b5450d2b011deac7b07345a1f1daf4>

I just submitted it as part of the TI pull request the other day

> 
> ... although patchwork does not know it yet:
> 
> 

I'll update.

--Sandeep

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] ARM: PXA: remove broken "zylonite" board.

[Wolfgang Denk]

Signed-off-by: Wolfgang Denk 
Cc: Albert ARIBAUD 
Cc: Marek Vasut 
---
I discussed this on IRC with Marek; he says the Zylonite board is
broken beyond repair and should be removed. So be it, then.  -- wd

 board/zylonite/Makefile|   51 
 board/zylonite/config.mk   |4 -
 board/zylonite/flash.c |  432 --
 board/zylonite/lowlevel_init.S |  373 --
 board/zylonite/nand.c  |  562 
 board/zylonite/zylonite.c  |   82 --
 boards.cfg |1 -
 doc/README.scrapyard   |1 +
 include/configs/zylonite.h |  238 -
 9 files changed, 1 insertions(+), 1743 deletions(-)
 delete mode 100644 board/zylonite/Makefile
 delete mode 100644 board/zylonite/config.mk
 delete mode 100644 board/zylonite/flash.c
 delete mode 100644 board/zylonite/lowlevel_init.S
 delete mode 100644 board/zylonite/nand.c
 delete mode 100644 board/zylonite/zylonite.c
 delete mode 100644 include/configs/zylonite.h

diff --git a/board/zylonite/Makefile b/board/zylonite/Makefile
deleted file mode 100644
index 3e57e49..000
--- a/board/zylonite/Makefile
+++ /dev/null
@@ -1,51 +0,0 @@
-#
-# (C) Copyright 2000-2006
-# Wolfgang Denk, DENX Software Engineering, w...@denx.de.
-#
-# See file CREDITS for list of people who contributed to this
-# project.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License as
-# published by the Free Software Foundation; either version 2 of
-# the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
-# MA 02111-1307 USA
-#
-
-include $(TOPDIR)/config.mk
-
-LIB= $(obj)lib$(BOARD).o
-
-COBJS  := zylonite.o nand.o
-SOBJS  := lowlevel_init.o
-
-SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)
-OBJS   := $(addprefix $(obj),$(COBJS))
-SOBJS  := $(addprefix $(obj),$(SOBJS))
-
-$(LIB):$(obj).depend $(OBJS) $(SOBJS)
-   $(call cmd_link_o_target, $(OBJS) $(SOBJS))
-
-clean:
-   rm -f $(SOBJS) $(OBJS)
-
-distclean: clean
-   rm -f $(LIB) core *.bak $(obj).depend
-
-#
-
-# defines $(obj).depend target
-include $(SRCTREE)/rules.mk
-
-sinclude $(obj).depend
-
-#
diff --git a/board/zylonite/config.mk b/board/zylonite/config.mk
deleted file mode 100644
index 954f46e..000
--- a/board/zylonite/config.mk
+++ /dev/null
@@ -1,4 +0,0 @@
-#CONFIG_SYS_TEXT_BASE = 0x0
-#CONFIG_SYS_TEXT_BASE = 0xa170
-#CONFIG_SYS_TEXT_BASE = 0xa308
-CONFIG_SYS_TEXT_BASE = 0xa3008000
diff --git a/board/zylonite/flash.c b/board/zylonite/flash.c
deleted file mode 100644
index 3ee0ab8..000
--- a/board/zylonite/flash.c
+++ /dev/null
@@ -1,432 +0,0 @@
-/*
- * (C) Copyright 2001
- * Kyle Harris, Nexus Technologies, Inc. khar...@nexus-tech.net
- *
- * (C) Copyright 2001
- * Wolfgang Denk, DENX Software Engineering, w...@denx.de.
- *
- * See file CREDITS for list of people who contributed to this
- * project.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
- * MA 02111-1307 USA
- */
-
-#include 
-#include 
-
-
-flash_info_t flash_info[CONFIG_SYS_MAX_FLASH_BANKS];   /* info for FLASH chips 
   */
-
-/* Board support for 1 or 2 flash devices */
-#define FLASH_PORT_WIDTH32
-#undef FLASH_PORT_WIDTH16
-
-#ifdef FLASH_PORT_WIDTH16
-#define FLASH_PORT_WIDTH   ushort
-#define FLASH_PORT_WIDTHV  vu_short
-#define SWAP(x)   __swab16(x)
-#else
-#define FLASH_PORT_WIDTH   ulong
-#define FLASH_PORT_WIDTHV  vu_long
-#define SWAP(x)   __swab32(x)
-#endif
-
-#define FPW   FLASH_PORT_WIDTH
-#define FPWV   FLASH_PORT_WIDTHV
-
-#define mb() __asm__ __volatile__ ("" : : : "memory")
-
-/*-

[U-Boot] [PATCH] VIDEO: mb86r0xgdc.c: fix warning: unused variable 'i'

[Wolfgang Denk]

Fix build warning:

Configuring for jadecpu board...
mb86r0xgdc.c: In function 'dsp_init':
mb86r0xgdc.c:60: warning: unused variable 'i'

Signed-off-by: Wolfgang Denk 
Cc: Anatolij Gustschin 
---
 drivers/video/mb86r0xgdc.c |1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/drivers/video/mb86r0xgdc.c b/drivers/video/mb86r0xgdc.c
index 5425c9d..345a73b 100644
--- a/drivers/video/mb86r0xgdc.c
+++ b/drivers/video/mb86r0xgdc.c
@@ -57,7 +57,6 @@ static void dsp_init(struct mb86r0x_gdc_dsp *dsp, char 
*modestr,
u16 l2dx, l2dy, l2wx, l2wy, l2ww, l2wh;
unsigned long div;
int bpp;
-   u32 i;
 
bpp = video_get_params(&var_mode, modestr);
 
-- 
1.7.6

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 3/3 v3] ARM: ARM926EJS - Add cache operations

[Marek Vasut]

> Le 02/09/2011 13:57, Marek Vasut a écrit :
> > On Friday, September 02, 2011 01:43:39 PM Simon Guinot wrote:
> > > On Fri, Sep 02, 2011 at 12:23:54PM +0200, Marek Vasut wrote:
> > > > On Friday, September 02, 2011 12:22:05 PM Simon Guinot wrote:
> > > > > Hi Hong,
> > > > > 
> > > > > On Mon, Aug 15, 2011 at 03:08:51PM +0800, Hong Xu wrote:
> > > > > > Hi Marek,
> > > > > > 
> > > > > > On 08/11/2011 12:47 PM, Marek Vasut wrote:
> > > > > > > On Thursday, August 11, 2011 05:27:37 AM Hong Xu wrote:
> > > > > > > > Add a new file arch/arm/cpu/arm926ejs/cache.c and put cache
> > > > > > > > operations into this file.
> > > > > > > > 
> > > > > > > > Signed-off-by: Hong Xu
> > > > > > > > ---
> > > > > > > > 
> > > > > > > > V2:
> > > > > > > >     Fixed a typo when CONFIG_SYS_DCACHE_OFF is defined
> > > > > > > > 
> > > > > > > > V3:
> > > > > > > >     Undo changes in include/configs/at91sam9260ek.h
> > > > > > > >     It's for testing purpose
> > > > > > > >    
> > > > > > > > arch/arm/cpu/arm926ejs/Makefile |       2 +-
> > > > > > > > arch/arm/cpu/arm926ejs/cache.c   |   142
> > > > > > > > 
> > > > > > > > +++ 2 files changed,
> > > > > > > > 143 insertions(+), 1 deletions(-)
> > > > > > > > 
> > > > > > > > create mode 100644 arch/arm/cpu/arm926ejs/cache.c
> > > > > 
> > > > > What is the status for this patch ?
> > > > > 
> > > > > I have failed to find any trace for this patch in patchwork or
> > > > > in the arm git repository...
> > > > 
> > > > We discussed this should be moved to arch/arm/cpu/armv5 (Aneesh
> > > > V's idea ... I dunno what's this guys' full name ;-D). I tested
> > > > this patch on certain type of CPU and it works really good. I'd be
> > > > really glad if we got this mainline!
> > > 
> > > Yes. Moreover some drivers (as mvgbe) really needs such functions to
> > > fix the DMA operations when D-cache is enabled.
> > 
> > I have some fixes already queued, yes.
> > 
> > > 
> > > Simon
> 
> Hi folks,
> 
> I'm trying to locate the latest version of this set's patches in 
> patchwork and I can't find them at all in any state. What am I missing?

Hi Albert,

you're missing the move from cpu/arm926ejs to cpu/armv5/ . Though I'd prefer to 
have it in arm926 for now as we still didn't verify the other cores are ok with 
it (I changed my opinion just recently). For example, xscale isn't. So maybe we 
can pick the patches as they are.

Cheers!
> 
> Amicalement,
> -- 
> Albert.

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 3/3 v3] ARM: ARM926EJS - Add cache operations

[Albert ARIBAUD]

Le 02/09/2011 13:57, Marek Vasut a écrit :
> On Friday, September 02, 2011 01:43:39 PM Simon Guinot wrote:
>> On Fri, Sep 02, 2011 at 12:23:54PM +0200, Marek Vasut wrote:
>>> On Friday, September 02, 2011 12:22:05 PM Simon Guinot wrote:
 Hi Hong,

 On Mon, Aug 15, 2011 at 03:08:51PM +0800, Hong Xu wrote:
> Hi Marek,
>
> On 08/11/2011 12:47 PM, Marek Vasut wrote:
>> On Thursday, August 11, 2011 05:27:37 AM Hong Xu wrote:
>>> Add a new file arch/arm/cpu/arm926ejs/cache.c and put cache
>>> operations into this file.
>>>
>>> Signed-off-by: Hong Xu
>>> ---
>>>
>>> V2:
>>> Fixed a typo when CONFIG_SYS_DCACHE_OFF is defined
>>>
>>> V3:
>>> Undo changes in include/configs/at91sam9260ek.h
>>> It's for testing purpose
>>> 
>>>arch/arm/cpu/arm926ejs/Makefile |2 +-
>>>arch/arm/cpu/arm926ejs/cache.c  |  142
>>>
>>> +++ 2 files changed, 143
>>> insertions(+), 1 deletions(-)
>>>
>>>create mode 100644 arch/arm/cpu/arm926ejs/cache.c

 What is the status for this patch ?

 I have failed to find any trace for this patch in patchwork or in the
 arm git repository...
>>>
>>> We discussed this should be moved to arch/arm/cpu/armv5 (Aneesh V's idea
>>> ... I dunno what's this guys' full name ;-D). I tested this patch on
>>> certain type of CPU and it works really good. I'd be really glad if we
>>> got this mainline!
>>
>> Yes. Moreover some drivers (as mvgbe) really needs such functions to fix
>> the DMA operations when D-cache is enabled.
>
> I have some fixes already queued, yes.
>
>>
>> Simon

Hi folks,

I'm trying to locate the latest version of this set's patches in 
patchwork and I can't find them at all in any state. What am I missing?

Amicalement,
-- 
Albert.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] harmony / seaboard: fix out of tree building

[Wolfgang Denk]

Out of tree building of the "harmony" and "seaboard" boards failed
like this:

Configuring for harmony board...
Assembler messages:
Fatal error: can't create 
/work/wd/tmp-arm/board/nvidia/harmony/../common/board.o: No such file or 
directory
make[1]: *** [/work/wd/tmp-arm/board/nvidia/harmony/../common/board.o] Error 2

Signed-off-by: Wolfgang Denk 
Cc: Albert ARIBAUD 
Cc: Tom Warren 
---
This replaces [PATCH] harmony: fix out of tree building;
both Nvidia boards have the same problem, so fix them in one go.

 board/nvidia/harmony/Makefile  |4 
 board/nvidia/seaboard/Makefile |4 
 2 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/board/nvidia/harmony/Makefile b/board/nvidia/harmony/Makefile
index 9fb6b57..ebd8e02 100644
--- a/board/nvidia/harmony/Makefile
+++ b/board/nvidia/harmony/Makefile
@@ -24,6 +24,10 @@
 
 include $(TOPDIR)/config.mk
 
+ifneq ($(OBJTREE),$(SRCTREE))
+$(shell mkdir -p $(obj)../common)
+endif
+
 LIB= $(obj)lib$(BOARD).o
 
 COBJS  := $(BOARD).o
diff --git a/board/nvidia/seaboard/Makefile b/board/nvidia/seaboard/Makefile
index 9fb6b57..ebd8e02 100644
--- a/board/nvidia/seaboard/Makefile
+++ b/board/nvidia/seaboard/Makefile
@@ -24,6 +24,10 @@
 
 include $(TOPDIR)/config.mk
 
+ifneq ($(OBJTREE),$(SRCTREE))
+$(shell mkdir -p $(obj)../common)
+endif
+
 LIB= $(obj)lib$(BOARD).o
 
 COBJS  := $(BOARD).o
-- 
1.7.6

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Patches to fix more ARM boards

[Albert ARIBAUD]

Hi Wolfgang,

Le 05/09/2011 21:26, Wolfgang Denk a écrit :
> Dear Albert&  Stefano,
>
> earlier today I posted a number of patches to remove some more dead
> ARM boards:
>
> [PATCH 00/10] ARM board removal, next round
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107465
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107469
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107467
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107471
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107464
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107473
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107472
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107470
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107466
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107474
>
> In addition, a number of other patches are neede to fix more build
> breakage on ARM, i. e.
>
> 09/05 Stefano Babic  [PATCH V4] Makefile : fix generation of cpu related 
> asm-offsets.h
>  http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107505
>
> 09/05 Wolfgang Denk  [PATCH] da8xxevm: Fix warning: unused variable 'val'
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107515
>
> 09/05 Wolfgang Denk  [PATCH] omap24xx: fix 'reset_timer_masked' 
> declaration error
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107521
>
> 09/05 Wolfgang Denk  harmony: fix out of tree building
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107526
>
> 08/30 Stefano Babic  [U-Boot] [PATCH 1/2] MX31: fix missing mxc_get_clk() 
> call
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/106825
>
> 08/30 Stefano Babic  [U-Boot] [PATCH 2/2] I2C: mxc: fix compilation for 
> MX31
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/106826
>
> 08/30 Stefano Babic  [PATCH 1/2] Makefile: drop imx31_phycore from 
> general Makefile
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/106827
>
> 08/30 Stefano Babic  [PATCH 2/2] Makefile: remove mx31pdk exception
>   http://article.gmane.org/gmane.comp.boot-loaders.u-boot/106828
>
> And maybe some more.
>
> Normally these should go through your iMX and ARM repositories.  To
> save time and finally get a -rc1 out, I would like to pull these
> directly, if this is OK with you.  As far as I can tell these re all
> straightforward fixes, without bigger impact on architectural things.
>
> Do you agree?

That's fine with me.

> Best regards,
>
> Wolfgang Denk

Amicalement,
-- 
Albert.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] Patches to fix more ARM boards

[Wolfgang Denk]

Dear Albert & Stefano,

earlier today I posted a number of patches to remove some more dead
ARM boards:

[PATCH 00/10] ARM board removal, next round
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107465
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107469
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107467
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107471
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107464
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107473
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107472
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107470
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107466
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107474

In addition, a number of other patches are neede to fix more build
breakage on ARM, i. e.

09/05 Stefano Babic  [PATCH V4] Makefile : fix generation of cpu related 
asm-offsets.h
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107505

09/05 Wolfgang Denk  [PATCH] da8xxevm: Fix warning: unused variable 'val'
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107515

09/05 Wolfgang Denk  [PATCH] omap24xx: fix 'reset_timer_masked' declaration 
error
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107521

09/05 Wolfgang Denk  harmony: fix out of tree building
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/107526

08/30 Stefano Babic  [U-Boot] [PATCH 1/2] MX31: fix missing mxc_get_clk() 
call
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/106825

08/30 Stefano Babic  [U-Boot] [PATCH 2/2] I2C: mxc: fix compilation for MX31
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/106826

08/30 Stefano Babic  [PATCH 1/2] Makefile: drop imx31_phycore from general 
Makefile
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/106827

08/30 Stefano Babic  [PATCH 2/2] Makefile: remove mx31pdk exception
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/106828

And maybe some more.

Normally these should go through your iMX and ARM repositories.  To
save time and finally get a -rc1 out, I would like to pull these
directly, if this is OK with you.  As far as I can tell these re all
straightforward fixes, without bigger impact on architectural things.

Do you agree?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
"Ja, mach' nur einen Plan,sei nur ein grosses Licht
und mach' dann noch 'nen zweiten Plan,geh'n tun sie beide nicht."
 -- Bert Brecht, Dreigroschenoper
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] OMAP: Remove omapfb.debug=y from Beagle and Overo env settings

[Albert ARIBAUD]

Le 11/08/2011 21:28, Joel A Fernandes a écrit :
> From: Steve Sakoman
>
> The kernel DSS2 code is mature now, and keeping this setting hurts performance
>
> Signed-off-by: Steve Sakoman
> Signed-off-by: Joel A Fernandes
> ---
> Changes since v1:
> Added myself to SOB

I see no indication of this anywhere, but this change seems to be 
already in git, from a patch submitted by Steve Sakoman on december 2010:



... although patchwork does not know it yet:



Amicalement,
-- 
Albert.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] De administrador de sistemas

[junaida]

Un virus DGTFX se ha detectado en sus carpetas. Su cuenta de correo tiene
que ser actualizado a la nueva garantizados DGTFX anti-virus versión 2011
para evitar daños a nuestro registro de correo web y sus
archivos importantes. De
revalidar su buzón de
correo por favor clickhere


 administrador del sistema

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] Setting CONFIG_SYS_TEXT_BASE when CONFIG_NAND_SPL is selected

[Fabio Estevam]

Hi,

On a MX31PDK the only boot media is NAND flash, so I was trying to get
rid of the
conditional setting of CONFIG_SYS_TEXT_BASE.

CONFIG_NAND_SPL is set in nand_spl/board/freescale/mx31pdk/Makefile.

,so I tried the following:

diff --git a/board/freescale/mx31pdk/config.mk b/board/freescale/mx31pdk/config.
index de2c642..b9932ec 100644
--- a/board/freescale/mx31pdk/config.mk
+++ b/board/freescale/mx31pdk/config.mk
@@ -1,5 +1,2 @@
-ifdef CONFIG_NAND_SPL
 CONFIG_SYS_TEXT_BASE = 0x87ec
-else
-CONFIG_SYS_TEXT_BASE = 0x87f0
-endif
+

,but this causes the board not to boot.

Does anyone have any ideas as to why the patch above does not work?

Thanks,

Fabio Estevam
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] MX31: mx31pdk: Remove nand_spl config.mk file

[Fabio Estevam]

No need to have a config.mk to only store a single line.

Signed-off-by: Fabio Estevam 
---
 nand_spl/board/freescale/mx31pdk/Makefile  |2 +-
 nand_spl/board/freescale/mx31pdk/config.mk |1 -
 2 files changed, 1 insertions(+), 2 deletions(-)
 delete mode 100644 nand_spl/board/freescale/mx31pdk/config.mk

diff --git a/nand_spl/board/freescale/mx31pdk/Makefile 
b/nand_spl/board/freescale/mx31pdk/Makefile
index e6ec10a..87784d2 100644
--- a/nand_spl/board/freescale/mx31pdk/Makefile
+++ b/nand_spl/board/freescale/mx31pdk/Makefile
@@ -1,7 +1,7 @@
 CONFIG_NAND_SPL= y
+PAD_TO := 2048
 
 include $(TOPDIR)/config.mk
-include $(TOPDIR)/nand_spl/board/$(BOARDDIR)/config.mk
 
 nandobj:= $(OBJTREE)/nand_spl/
 
diff --git a/nand_spl/board/freescale/mx31pdk/config.mk 
b/nand_spl/board/freescale/mx31pdk/config.mk
deleted file mode 100644
index 68afbf1..000
--- a/nand_spl/board/freescale/mx31pdk/config.mk
+++ /dev/null
@@ -1 +0,0 @@
-PAD_TO := 2048
-- 
1.6.0.4


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH v5 1/3] mx25: Introduce sys_proto.h

[Fabio Estevam]

Create a sys_proto.h for MX25 as done on other MX25 processors.

Signed-off-by: Fabio Estevam 
---
Changes since v4:
- Omit "extern" annotation in function prototype
Changes since v3:
- No changes. This patch was introduced in v4
 arch/arm/cpu/arm926ejs/mx25/generic.c  |1 +
 arch/arm/include/asm/arch-mx25/imx-regs.h  |4 
 arch/arm/include/asm/arch-mx25/sys_proto.h |   27 +++
 3 files changed, 28 insertions(+), 4 deletions(-)
 create mode 100644 arch/arm/include/asm/arch-mx25/sys_proto.h

diff --git a/arch/arm/cpu/arm926ejs/mx25/generic.c 
b/arch/arm/cpu/arm926ejs/mx25/generic.c
index 76e4b5c..3feb449 100644
--- a/arch/arm/cpu/arm926ejs/mx25/generic.c
+++ b/arch/arm/cpu/arm926ejs/mx25/generic.c
@@ -31,6 +31,7 @@
 #ifdef CONFIG_MXC_MMC
 #include 
 #endif
+#include 
 
 /*
  *  get the system pll clock in Hz
diff --git a/arch/arm/include/asm/arch-mx25/imx-regs.h 
b/arch/arm/include/asm/arch-mx25/imx-regs.h
index 9e30f7c..33bdb01 100644
--- a/arch/arm/include/asm/arch-mx25/imx-regs.h
+++ b/arch/arm/include/asm/arch-mx25/imx-regs.h
@@ -34,10 +34,6 @@
 #define _IMX_REGS_H
 
 #ifndef __ASSEMBLY__
-#ifdef CONFIG_FEC_MXC
-extern void mx25_fec_init_pins(void);
-extern void imx_get_mac_from_fuse(unsigned char *mac);
-#endif
 
 /* Clock Control Module (CCM) registers */
 struct ccm_regs {
diff --git a/arch/arm/include/asm/arch-mx25/sys_proto.h 
b/arch/arm/include/asm/arch-mx25/sys_proto.h
new file mode 100644
index 000..1c59c7b
--- /dev/null
+++ b/arch/arm/include/asm/arch-mx25/sys_proto.h
@@ -0,0 +1,27 @@
+/*
+ * (C) Copyright 2009
+ * Stefano Babic, DENX Software Engineering, sba...@denx.de.
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef _SYS_PROTO_H_
+#define _SYS_PROTO_H_
+
+#ifdef CONFIG_FEC_MXC
+void mx25_fec_init_pins(void);
+void imx_get_mac_from_fuse(unsigned char *mac);
+#endif /* CONFIG_FEC_MXC */
+
+#endif /* _SYS_PROTO_H_ */
-- 
1.7.1


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH v5 3/3] MX25: Add initial support for MX25PDK

[Fabio Estevam]

Add the initial support for MX25PDK booting from SD card via internal boot.

Signed-off-by: Fabio Estevam 
---
Chanegs since v4:
- No changes
Changes since v3:
- No changes
Changes since v2:
- No changes
Changes since v1:
- Keep mx25pdk entry to boards.cfg sorted
- Removed unused defines in mx25pdk.h
- Do not use the whole RAM space for memory test
 MAINTAINERS |1 +
 board/freescale/mx25pdk/Makefile|   49 ++
 board/freescale/mx25pdk/imximage.cfg|   73 +
 board/freescale/mx25pdk/lowlevel_init.S |   21 ++
 board/freescale/mx25pdk/mx25pdk.c   |   62 ++
 boards.cfg  |1 +
 include/configs/mx25pdk.h   |  106 +++
 7 files changed, 313 insertions(+), 0 deletions(-)
 create mode 100644 board/freescale/mx25pdk/Makefile
 create mode 100644 board/freescale/mx25pdk/imximage.cfg
 create mode 100644 board/freescale/mx25pdk/lowlevel_init.S
 create mode 100644 board/freescale/mx25pdk/mx25pdk.c
 create mode 100644 include/configs/mx25pdk.h

diff --git a/MAINTAINERS b/MAINTAINERS
index f895e9a..7f07b60 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -645,6 +645,7 @@ Kristoffer Ericson 
 
 Fabio Estevam 
 
+   mx25pdk i.MX25
mx31pdk i.MX31
mx53ard i.MX53
mx53smd i.MX53
diff --git a/board/freescale/mx25pdk/Makefile b/board/freescale/mx25pdk/Makefile
new file mode 100644
index 000..4d45ae3
--- /dev/null
+++ b/board/freescale/mx25pdk/Makefile
@@ -0,0 +1,49 @@
+#
+# Copyright (C) 2007, Guennadi Liakhovetski 
+#
+# (C) Copyright 2011 Freescale Semiconductor, Inc.
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License as
+# published by the Free Software Foundation; either version 2 of
+# the License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+# MA 02111-1307 USA
+#
+
+include $(TOPDIR)/config.mk
+
+LIB= $(obj)lib$(BOARD).o
+
+COBJS  := mx25pdk.o
+SOBJS  := lowlevel_init.o
+
+SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)
+OBJS   := $(addprefix $(obj),$(COBJS))
+SOBJS  := $(addprefix $(obj),$(SOBJS))
+
+$(LIB):$(obj).depend $(OBJS) $(SOBJS)
+   $(call cmd_link_o_target, $(OBJS) $(SOBJS))
+
+clean:
+   rm -f $(SOBJS) $(OBJS)
+
+distclean: clean
+   rm -f $(LIB) core *.bak .depend
+
+#
+
+# defines $(obj).depend target
+include $(SRCTREE)/rules.mk
+
+sinclude $(obj).depend
+
+#
diff --git a/board/freescale/mx25pdk/imximage.cfg 
b/board/freescale/mx25pdk/imximage.cfg
new file mode 100644
index 000..f7af7ff
--- /dev/null
+++ b/board/freescale/mx25pdk/imximage.cfg
@@ -0,0 +1,73 @@
+#
+# (C) Copyright 2009
+# Stefano Babic DENX Software Engineering sba...@denx.de.
+#
+# See file CREDITS for list of people who contributed to this
+# project.
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License as
+# published by the Free Software Foundation; either version 2 of
+# the License or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# Refer docs/README.imxmage for more details about how-to configure
+# and create imximage boot image
+#
+# The syntax is taken as close as possible with the kwbimage
+
+# Boot Device : one of
+# spi, sd (the board has no nand neither onenand)
+
+BOOT_FROM  sd
+
+# Device Configuration Data (DCD)
+#
+# Each entry must have the format:
+# Addr-type   AddressValue
+#
+# where:
+#  Addr-type register length (1,2 or 4 bytes)
+#  Address   absolute address of the register
+#  value value to be stored in the register
+
+# EIM config-CS5 init -- CPLD
+DATA 4 0xB8002050 0xD843
+DATA 4 0xB8002054 0x22252521
+DATA 4 0xB8002058 0x0A00
+
+# DDR2 init
+DATA 4 0xB8001004 0x0076E83A
+DATA 4 0xB8001010 0x0204
+DATA 4 0xB8001000 0x9221
+DATA 4 0x8f00 0x12344321
+DATA 4 0xB8001000 0xB221
+DATA 1 0x8200 0xda
+DATA 1 0x8300 0xda
+DATA 1 0x81000400 0xda
+DATA 1 0x8333 0xda
+
+DATA 4 0xB8001000 0x9221
+DATA 1 0x8400 0x12345678
+
+DATA 4 0xB8001000 0

[U-Boot] [PATCH v5 2/3] MX25: tx25: Avoid the usage of extern in C file

[Fabio Estevam]

Avoid the usage of extern in C file as pointed out by checkpatch.

Signed-off-by: Fabio Estevam 
---
Chanegs since v4:
- No changes
Changes since v3:
- Place mx25_uart1_init_pins inside sys_proto.h
Changes since v2:
- Place the extern in the MX25 imx-regs.h instead of inside the MX27 imx-regs.h
Changes since v1:
- No changes
 arch/arm/include/asm/arch-mx25/sys_proto.h |1 +
 board/karo/tx25/tx25.c |2 --
 2 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/arch/arm/include/asm/arch-mx25/sys_proto.h 
b/arch/arm/include/asm/arch-mx25/sys_proto.h
index 1c59c7b..f9c1420 100644
--- a/arch/arm/include/asm/arch-mx25/sys_proto.h
+++ b/arch/arm/include/asm/arch-mx25/sys_proto.h
@@ -23,5 +23,6 @@
 void mx25_fec_init_pins(void);
 void imx_get_mac_from_fuse(unsigned char *mac);
 #endif /* CONFIG_FEC_MXC */
+void mx25_uart1_init_pins(void);
 
 #endif /* _SYS_PROTO_H_ */
diff --git a/board/karo/tx25/tx25.c b/board/karo/tx25/tx25.c
index 25b99e8..c34672a 100644
--- a/board/karo/tx25/tx25.c
+++ b/board/karo/tx25/tx25.c
@@ -141,8 +141,6 @@ void tx25_fec_init(void)
 int board_init()
 {
 #ifdef CONFIG_MXC_UART
-   extern void mx25_uart1_init_pins(void);
-
mx25_uart1_init_pins();
 #endif
/* board id for linux */
-- 
1.7.1


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 2/2] phylib: remove a couple of redundant code lines

[Vladimir Zapolskiy]

This change slightly improves readability of the phydev speed/duplex
assignment logic.

Signed-off-by: Vladimir Zapolskiy 
---
 drivers/net/phy/phy.c |7 ++-
 1 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c
index 8da7688..833a051 100644
--- a/drivers/net/phy/phy.c
+++ b/drivers/net/phy/phy.c
@@ -318,13 +318,10 @@ static int genphy_parse_link(struct phy_device *phydev)
lpa = phy_read(phydev, MDIO_DEVAD_NONE, MII_ADVERTISE);
lpa &= phy_read(phydev, MDIO_DEVAD_NONE, MII_LPA);
 
-   if (lpa & (LPA_100FULL | LPA_100HALF)) {
+   if (lpa & (LPA_100FULL | LPA_100HALF))
phydev->speed = SPEED_100;
 
-   if (lpa & LPA_100FULL)
-   phydev->duplex = DUPLEX_FULL;
-
-   } else if (lpa & LPA_10FULL)
+   if (lpa & (LPA_100FULL | LPA_10FULL))
phydev->duplex = DUPLEX_FULL;
} else {
u32 bmcr = phy_read(phydev, MDIO_DEVAD_NONE, MII_BMCR);
-- 
1.7.5.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 1/2] phylib: reset mii bus only if reset handler is registered

[Vladimir Zapolskiy]

This change allows to cope with a mii bus device registered using
miiphy_register(), which doesn't assign a default reset handler.

Signed-off-by: Vladimir Zapolskiy 
---
 drivers/net/phy/phy.c |3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c
index ce69c19..8da7688 100644
--- a/drivers/net/phy/phy.c
+++ b/drivers/net/phy/phy.c
@@ -692,7 +692,8 @@ struct phy_device *phy_connect(struct mii_dev *bus, int 
addr,
struct phy_device *phydev;
 
/* Reset the bus */
-   bus->reset(bus);
+   if (bus->reset)
+   bus->reset(bus);
 
/* Wait 15ms to make sure the PHY has come out of hard reset */
udelay(15000);
-- 
1.7.5.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 0/2] phylib: a couple of minor fixes in generic phy library

[Vladimir Zapolskiy]

Hello,

recently I decided to port one (not integrated to U-boot yet) MAC to
use phylib, and found that it serves quite good. I managed to find
only one bug related to miiphy_register() and phylib cooperative
usage, the small fix is provided. Additionally let me send a tiny
readability improvement change.

Vladimir Zapolskiy (2):
  phylib: reset mii bus only if reset handler is registered
  phylib: remove a couple of redundant code lines

 drivers/net/phy/phy.c |   10 --
 1 files changed, 4 insertions(+), 6 deletions(-)

-- 
1.7.5.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] doc: provide a correct board_init_r definition path

[Vladimir Zapolskiy]

This is a trivial fix in the documentation, which corrects
board_init_r() source reference.

Signed-off-by: Vladimir Zapolskiy 
---
 doc/README.console |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/doc/README.console b/doc/README.console
index 25c4f1d..b353f9f 100644
--- a/doc/README.console
+++ b/doc/README.console
@@ -74,7 +74,7 @@ You can use the following functions to access the console:
 fgetc  (like getc but redirected to a file)
 
 Remember that all FILE-related functions CANNOT be used before
-U-Boot relocation (done in 'board_init_r' in common/board.c).
+U-Boot relocation (done in 'board_init_r' in arch/*/lib/board.c).
 
 HOW CAN I USE STANDARD FILE INTO APPLICATIONS?
 --
-- 
1.7.5.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] harmony: fix out of tree building

[Wolfgang Denk]

Out of tree building of the "harmony" board failed like this:

Configuring for harmony board...
Assembler messages:
Fatal error: can't create 
/work/wd/tmp-arm/board/nvidia/harmony/../common/board.o: No such file or 
directory
make[1]: *** [/work/wd/tmp-arm/board/nvidia/harmony/../common/board.o] Error 2

Signed-off-by: Wolfgang Denk 
Cc: Albert ARIBAUD 
Cc: Tom Warren 
---
 board/nvidia/harmony/Makefile |4 
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/board/nvidia/harmony/Makefile b/board/nvidia/harmony/Makefile
index 9fb6b57..ebd8e02 100644
--- a/board/nvidia/harmony/Makefile
+++ b/board/nvidia/harmony/Makefile
@@ -24,6 +24,10 @@
 
 include $(TOPDIR)/config.mk
 
+ifneq ($(OBJTREE),$(SRCTREE))
+$(shell mkdir -p $(obj)../common)
+endif
+
 LIB= $(obj)lib$(BOARD).o
 
 COBJS  := $(BOARD).o
-- 
1.7.6

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

[Stefano Babic]

On 09/05/2011 05:48 PM, Fabio Estevam wrote:
> On Mon, Sep 5, 2011 at 12:40 PM, Marek Vasut  wrote:
> ...
>

Hi Fabio,

>>> ./scripts/checkpatch.pl -F u-boot/board/karo/tx25/tx25.c
>>> 
>>> WARNING: externs should be avoided in .c files
>>> #144: FILE: home/fabio/denx/u-boot/board/karo/tx25/tx25.c:144:
>>> + extern void mx25_uart1_init_pins(void);
>>
>> But you're using extern in _header_ (.h) file ... so ... why ?
>>
> 
> My patch removes the extern from the C file and put it on a header
> file with other extern's.
> 
> The checkpatch warning I showed happens with the original file, and
> not after my patch is applied.
> 

I think checkpatch warnings because the original file has already some
extern. In the most of u-boot code we do not mark the prototypes with
extern.
I made this simple test - I create a new file (from sys_proto.h) and I
run checkpatch on it:

+#ifndef _SYS_PROTO_H_
+#define _SYS_PROTO_H_
+
+u32 get_cpu_rev(void);
+#define is_soc_rev(rev)((get_cpu_rev() & 0xFF) - rev)
+void sdelay(unsigned long);
+void set_chipselect_size(int const);
+#endif
-- 

checpatch reports neither errors nor warnings:

checkpatch.pl --no-tree 0001-tmp.patch
total: 0 errors, 0 warnings, 31 lines checked

I think you can try moving the prototypes in sys_proto.h, and run
checkpatch on it without extern. I expect to see no warnings at all.

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

[Marek Vasut]

On Monday, September 05, 2011 06:02:21 PM Wolfgang Denk wrote:
> Dear Fabio Estevam,
> 
> In message 
 you wrote:
> > > But you're using extern in _header_ (.h) file ... so ... why ?
> > 
> > My patch removes the extern from the C file and put it on a header
> > file with other extern's.
> 
> What Marek probably wants to tell yet (in a bit a complicated way) is
> that you should omit the "extern" in the prototype declaration. Just
> remove all "extern" from the header file.

No, I was asking if there's some special reason for this or it's plain wrong. I 
see the later's the case.

Thanks, cheers!
> 
> Best regards,
> 
> Wolfgang Denk
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

[Wolfgang Denk]

Dear Fabio Estevam,

In message  
you wrote:
>
> > But you're using extern in _header_ (.h) file ... so ... why ?
> >
> 
> My patch removes the extern from the C file and put it on a header
> file with other extern's.

What Marek probably wants to tell yet (in a bit a complicated way) is
that you should omit the "extern" in the prototype declaration. Just
remove all "extern" from the header file.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
One essential to success is that you desire be an all-obsessing  one,
your thoughts and aims be co-ordinated, and your energy be concentra-
ted and applied without letup.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] UBI problems on current u-boot

[Holger Brunck]

Hi Stefan,
sorry for the previous mail, but I hit the send button too fast ;-)

On 09/05/2011 04:37 PM, Stefan Roese wrote:
> 
> On Friday 02 September 2011 15:32:40 Holger Brunck wrote:
>> I address this question to you because one of your commits is connected to
>> this problem, but other hints from other readers are also welcome ;-) .
> 
> I'll try to look into this later this week.
> 

Thanks. I also wanted to do some further investigations, if I find the time this
week.

> BTW: Is this problem reproducible on one of your systems?
> 

yes we find a way to reproduce the bug on one of our boards. We need a special
bit pattern in one UBI PEB, force a bitflip and afterwards the problem is 
present.

Best regards
Holger
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] omap24xx: fix 'reset_timer_masked' declaration error

[Wolfgang Denk]

Commit 17659d7 "Timer: Remove reset_timer_masked()" introduced a
static declaration for reset_timer_masked() which causes build errors:

timer.c:45: error: static declaration of 'reset_timer_masked' follows 
non-static declaration
include/asm/u-boot-arm.h:70: error: previous declaration of 
'reset_timer_masked' was here

Signed-off-by: Wolfgang Denk 
Cc: Graeme Russ 
Cc: Cc: Albert ARIBAUD 
Cc: Sandeep Paulraj 
---
 arch/arm/cpu/arm1136/omap24xx/timer.c |   20 +---
 1 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/arch/arm/cpu/arm1136/omap24xx/timer.c 
b/arch/arm/cpu/arm1136/omap24xx/timer.c
index 73bf4a7..e929ae4 100644
--- a/arch/arm/cpu/arm1136/omap24xx/timer.c
+++ b/arch/arm/cpu/arm1136/omap24xx/timer.c
@@ -41,13 +41,6 @@
 
 DECLARE_GLOBAL_DATA_PTR;
 
-static void reset_timer_masked (void)
-{
-   /* reset time */
-   gd->lastinc = READ_TIMER;   /* capture current incrementer value 
time */
-   gd->tbl = 0;/* start "advancing" time stamp from 0 
*/
-}
-
 int timer_init (void)
 {
int32_t val;
@@ -57,7 +50,9 @@ int timer_init (void)
val = (CONFIG_SYS_PTV << 2) | BIT5 | BIT1 | BIT0;   /* mask 
to enable timer*/
*((int32_t *) (CONFIG_SYS_TIMERBASE + TCLR)) = val; /* start timer 
*/
 
-   reset_timer_masked(); /* init the timestamp and lastinc value */
+   /* reset time */
+   gd->lastinc = READ_TIMER;   /* capture current incrementer value */
+   gd->tbl = 0;/* start "advancing" time stamp */
 
return(0);
 }
@@ -84,10 +79,13 @@ void __udelay (unsigned long usec)
}
 
tmp = get_timer (0);/* get current timestamp */
-   if ( (tmo + tmp + 1) < tmp )/* if setting this forward will roll 
time stamp */
-   reset_timer_masked ();  /* reset "advancing" timestamp to 0, 
set lastinc value */
-   else
+   if ((tmo + tmp + 1) < tmp) {/* if setting this forward will roll */
+   /* time stamp, then reset time */
+   gd->lastinc = READ_TIMER;   /* capture incrementer value */
+   gd->tbl = 0;/* start time stamp */
+   } else {
tmo += tmp; /* else, set advancing stamp wake up 
time */
+   }
while (get_timer_masked () < tmo)/* loop till event */
/*NOP*/;
 }
-- 
1.7.6

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

[Fabio Estevam]

On Mon, Sep 5, 2011 at 12:40 PM, Marek Vasut  wrote:
...

>> ./scripts/checkpatch.pl -F u-boot/board/karo/tx25/tx25.c
>> 
>> WARNING: externs should be avoided in .c files
>> #144: FILE: home/fabio/denx/u-boot/board/karo/tx25/tx25.c:144:
>> +     extern void mx25_uart1_init_pins(void);
>
> But you're using extern in _header_ (.h) file ... so ... why ?
>

My patch removes the extern from the C file and put it on a header
file with other extern's.

The checkpatch warning I showed happens with the original file, and
not after my patch is applied.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] UBI problems on current u-bo

[Holger Brunck]

Hi Stefan,

On 09/05/2011 04:37 PM, Stefan Roese wrote:
>> I address this question to you because one of your commits is connected to
>> this problem, but other hints from other readers are also welcome ;-) .
> 
> I'll try to look into this later this week.
> 
> BTW: Is this problem reproducible on one of your systems?
> 
> Best regards,
> Stefan
> 
> --
> DENX Software Engineering GmbH,  MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: off...@denx.de

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

[Marek Vasut]

On Monday, September 05, 2011 05:38:26 PM Fabio Estevam wrote:
> On Mon, Sep 5, 2011 at 12:05 PM, Marek Vasut  wrote:
> > On Monday, September 05, 2011 03:15:44 PM Fabio Estevam wrote:
> >> On Mon, Sep 5, 2011 at 10:05 AM, Marek Vasut 
> >> wrote: ...
> >> 
> >> >> extern void mx25_fec_init_pins(void);
> >> >> extern void imx_get_mac_from_fuse(unsigned char *mac);
> >> >> extern void mx25_uart1_init_pins(void);
> >> > 
> >> > Ok, this might be a stupid one, but ...
> >> > 
> >> > why use externs in header files ?
> >> 
> >> This will make checkpatch happy :-)
> > 
> > I'm not quite sure I understand ... ?
> 
> ./scripts/checkpatch.pl -F u-boot/board/karo/tx25/tx25.c
> 
> WARNING: externs should be avoided in .c files
> #144: FILE: home/fabio/denx/u-boot/board/karo/tx25/tx25.c:144:
> + extern void mx25_uart1_init_pins(void);

But you're using extern in _header_ (.h) file ... so ... why ?
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

[Fabio Estevam]

On Mon, Sep 5, 2011 at 12:05 PM, Marek Vasut  wrote:
> On Monday, September 05, 2011 03:15:44 PM Fabio Estevam wrote:
>> On Mon, Sep 5, 2011 at 10:05 AM, Marek Vasut  wrote:
>> ...
>>
>> >> extern void mx25_fec_init_pins(void);
>> >> extern void imx_get_mac_from_fuse(unsigned char *mac);
>> >> extern void mx25_uart1_init_pins(void);
>> >
>> > Ok, this might be a stupid one, but ...
>> >
>> > why use externs in header files ?
>>
>> This will make checkpatch happy :-)
>>
> I'm not quite sure I understand ... ?

./scripts/checkpatch.pl -F u-boot/board/karo/tx25/tx25.c

WARNING: externs should be avoided in .c files
#144: FILE: home/fabio/denx/u-boot/board/karo/tx25/tx25.c:144:
+   extern void mx25_uart1_init_pins(void);
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] omap3: gpio: build warnings and errors

[Paulraj, Sandeep]

> 
> However, I couldn't find the definition of omap_get_gpio_dataout()
> used in board/ti/beagle/led.c.
> 
> The change was introduced in:
>   b8bc8973: led: Remove state-saving of led for...
> 
> After much search, I did find a patch that contains the defintion
> of omap_get_gpio_dataout() submitted yesterday:
>   http://marc.info/?l=u-boot&m=131515805732292&w=2
> 
> Can you pull this patch immediately (assuming it is correct)?
> ... for now, it will assume it right and update my patchset.

I will.


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] da8xxevm: Fix warning: unused variable 'val'

[Wolfgang Denk]

Signed-off-by: Wolfgang Denk 
Cc: Albert ARIBAUD 
Cc: Sandeep Paulraj 
Cc: Sudhakar Rajashekhara 
---
 board/davinci/da8xxevm/da850evm.c |4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/board/davinci/da8xxevm/da850evm.c 
b/board/davinci/da8xxevm/da850evm.c
index 2f950e7..46924d3 100644
--- a/board/davinci/da8xxevm/da850evm.c
+++ b/board/davinci/da8xxevm/da850evm.c
@@ -280,12 +280,14 @@ u32 get_board_rev(void)
 
 int board_init(void)
 {
+#ifdef CONFIG_USE_NOR
u32 val;
+#endif
+
 #ifndef CONFIG_USE_IRQ
irq_init();
 #endif
 
-
 #ifdef CONFIG_NAND_DAVINCI
/*
 * NAND CS setup - cycle counts based on da850evm NAND timings in the
-- 
1.7.6

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] OMAP3 Beagle: Minor config cleanup

[s-paulraj]

From: Sandeep Paulraj 

This patch removes a hardcoded MAC address


Signed-off-by: Sandeep Paulraj 
---
 include/configs/omap3_beagle.h |1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/include/configs/omap3_beagle.h b/include/configs/omap3_beagle.h
index 18c6deb..57d4bb3 100644
--- a/include/configs/omap3_beagle.h
+++ b/include/configs/omap3_beagle.h
@@ -215,7 +215,6 @@
"loadaddr=0x8020\0" \
"rdaddr=0x8100\0" \
"usbtty=cdc_acm\0" \
-   "usbethaddr=de:ad:be:ef\0" \
"bootfile=uImage.beagle\0" \
"console=ttyS2,115200n8\0" \
"mpurate=auto\0" \
-- 
1.6.0.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] arm: Add Prep subcommand support to bootm

[Andreas Bießmann]

Dear Simon,

Am 05.09.2011 16:06, schrieb Simon Schwarz:
> Dear Andreas,
> 
> On 09/05/2011 12:58 PM, Andreas Bießmann wrote:
>> Dear Simon,
>>
>> Am Mo 29 Aug 2011 18:08:13 CEST, Simon Schwarz schrieb:



>>
>>> +}
>>> +
>>> +/* Main Entry point for arm bootm implementation
>>> + *
>>> + * Modeled after the powerpc implementation
>>> + * DIFFERENCE: Instead of calling prep and go at the end
>>> + * they are called if subommand is equal 0.
>>
>> s/subommand/subcommand/
> done
>>
>>> + */
>>> +int do_bootm_linux(int flag, int argc, char *argv[], bootm_headers_t
>>> *images)
>>> +{
>>> +if (flag&  BOOTM_STATE_OS_CMDLINE)
>>> +boot_cmdline_linux(images);
>>> +
>>> +if (flag&  BOOTM_STATE_OS_BD_T)
>>> +boot_bd_t_linux(images);
>>
>> NAK, remove these two functions. Since the ARM linux boot requirements
>> are different to powerpc we do not need these two states of bootm at all.
>>
>> The powerpc entry_32.S (in linux) show they need commandline pointer
>> apart from 'residual board info' pointer. The arm implementation in
>> head.S (also linux source) says:
>>
>> ---8<---
>>   * This is normally called from the decompressor code.  The requirements
>>   * are: MMU = off, D-cache = off, I-cache = dont care, r0 = 0,
>>   * r1 = machine nr, r2 = atags or dtb pointer.
>> --->8---
>>
>> For arm we do not need to prepare the cmdline apart from 'bd_t', we just
>> need to setup the ATAGS (ord FDT) which contains all information we
>> need. This could all be done in the prep state.
>>
> 
> changed.
> 
> But they are shown in bootm help message regardles of the architecture.
> Shouldn't we add #ifdefs in the help message then? (or, and I really
> hate to bring this up, change the way the helpmessage is created if the
> function is arch dependent)

No, I don't think we should change the bootm command. We could either
return 'not implemented' or 'everything is ok' here for these two flags.
But we do not need extra empty functions for that.

How about:

/* OS_CMDLINE is not needed by ARM cause of ... the bootargs (==cmdline)
is set in ATAGS/FDT ... */
if (flag&  BOOTM_STATE_OS_CMDLINE)
return 0; /* pretend everything is ok */

/* OS_BD_T is not needed by ARM casue of ... */
if (flag&  BOOTM_STATE_OS_BD_T)
return 0; /* pretend everything is ok */

...

best regards

Andreas Bießmann
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

[Marek Vasut]

On Monday, September 05, 2011 03:15:44 PM Fabio Estevam wrote:
> On Mon, Sep 5, 2011 at 10:05 AM, Marek Vasut  wrote:
> ...
> 
> >> extern void mx25_fec_init_pins(void);
> >> extern void imx_get_mac_from_fuse(unsigned char *mac);
> >> extern void mx25_uart1_init_pins(void);
> > 
> > Ok, this might be a stupid one, but ...
> > 
> > why use externs in header files ?
> 
> This will make checkpatch happy :-)
> 
I'm not quite sure I understand ... ?

> Regards,
> 
> Fabio Estevam
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] UBIFS: Change ubifsload to set the filesize variable

[Marek Vasut]

On Monday, September 05, 2011 04:27:28 PM Wolfgang Denk wrote:
> Dear Marek Vasut,
> 
> In message <201109051512.48148.marek.va...@gmail.com> you wrote:
> > > + setenv("filesize", buf);
> > 
> > maybe you want to check set_local_var() too ?
> 
> Yes, we should clean up all these auto-generated environment variables
> one day.  But what you propose here is not a good approach:
> 
> (1) not all boards use the hush shell, and set_local_var() is only
> available there.
> (2) if you make such a change, then you must make it everywhere, and
> you must make sure not to break existent use of these things.
> 
> Both is definitely out of the scope of this patch.

You're definitelly right on this thing.

> 
> Best regards,
> 
> Wolfgang Denk
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] i2c_no_probes: Fix initialization warnings

[Premi, Sanjeev]

> -Original Message-
> From: Heiko Schocher [mailto:h...@denx.de] 
> Sent: Monday, September 05, 2011 7:13 PM
> To: Premi, Sanjeev
> Cc: u-boot@lists.denx.de; Albert ARIBAUD
> Subject: Re: [U-Boot] [PATCH] i2c_no_probes: Fix 
> initialization warnings
> 
> Hello Sanjeev,

[snip]...[snip]

> >   Compile tested for oma3_beagle only.
> 
> Compiling for example for the alpr board with your patch:
> 
> [hs@pollux u-boot]$ ./MAKEALL alpr
> Configuring for alpr board...
> cmd_i2c.c:115: warning: braces around scalar initializer
> cmd_i2c.c:115: warning: (near initialization for 'i2c_no_probes[0]')
> fpga.c:226: warning: initialization from incompatible pointer type
>textdata bss dec hex filename
>  231746   18612   68328  318686   4dcde ./u-boot
> 
> - SUMMARY 
> Boards compiled: 1
> Boards with warnings or errors: 1 ( alpr )
> --
> [hs@pollux u-boot]$
> 
> -> so Sorry, NACK for this patch.

[sp] Didn't look at this reponse while I was responding to your
 earlier mail.

 One difference, I see notice between the warnings I got
 while building for beagle and the one you see:

 (beagle)
 cmd_i2c.c:109:1: warning: missing braces around initializer

 (alpr)
 cmd_i2c.c:115: warning: braces around scalar initializer

 The difference seems to be length of the array.

 (beagle)
 #define CONFIG_SYS_I2C_NOPROBES  {{0x0, 0x0}}

 (alpr)
 #define CONFIG_SYS_I2C_NOPROBES {{0x69}}

 So, temporarily, I changed the definition for beagle as
 #define CONFIG_SYS_I2C_NOPROBES  {{0x0}}

 ...but it succeeded.

 I am using "Codesourcery 2011.03-41". I will try using
 an older toolchain as well. Only trying to find of the
 original patch itself was right OR not.

~sanjeev

> 
> bye,
> Heiko
> -- 
> DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH V2] arm: Add Prep subcommand support to bootm

[Simon Schwarz]

Adds prep subcommand to bootm implementation of ARM. When bootm is called with
the subcommand prep the function stops right after ATAGS creation and before
announce_and_cleanup.

This is used in command "cmd_spl export"

Signed-off-by: Simon Schwarz 
---

V2 changes:
nothing

V3 changes:
nothing

changes after slicing this from patch
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/106422:
DEL Prototype declaration - not necessary if reordered
DEL Most #ifdefs - relying on -ffunction-sections and --gc-sections to handle 
unused
CHG reorganized bootm. powerpc implementation was the model
ADD get_board_serial fake implementation - this is needed if setup_serial_tag
is compiled but the board doesn't support it - tradeoff for removing
#ifdefs
CHG changed back to former not checking for zero approach

V2 changes (after split):
CHG wrong comment using old savebp
DEL file-wide kernel-entry forward-defintion (moved to funciton using it)
DEL get_board_serial added #ifdef CONFIG_SERIAL_TAG to setup_serial_tag
instead
DEL boot_bd_t_linux and boot_cmdline_linux, do_bootm_linux returns -1 if
they are called
CHG boot_prep_linux: error text emitted
CHG some typos and style problems
CHG logic in do_bootm_linux to do it exactly like ppc no == 0
DEL extern from udc_disconnect - gc should do it

squash! 7d85f60647af20c52e40a8422cdb8eb91a94d0a2
---
 arch/arm/lib/bootm.c |  320 --
 1 files changed, 154 insertions(+), 166 deletions(-)

diff --git a/arch/arm/lib/bootm.c b/arch/arm/lib/bootm.c
index 802e833..e5afc7e 100644
--- a/arch/arm/lib/bootm.c
+++ b/arch/arm/lib/bootm.c
@@ -1,4 +1,8 @@
-/*
+/* Copyright (C) 2011
+ * Corscience GmbH & Co. KG - Simon Schwarz 
+ *  - Added prep subcommand support
+ *  - Reorganized source - modeled after powerpc version
+ *
  * (C) Copyright 2002
  * Sysgo Real-Time Solutions, GmbH 
  * Marius Groeger 
@@ -32,31 +36,15 @@
 
 DECLARE_GLOBAL_DATA_PTR;
 
-#if defined (CONFIG_SETUP_MEMORY_TAGS) || \
-defined (CONFIG_CMDLINE_TAG) || \
-defined (CONFIG_INITRD_TAG) || \
-defined (CONFIG_SERIAL_TAG) || \
-defined (CONFIG_REVISION_TAG)
-static void setup_start_tag (bd_t *bd);
-
-# ifdef CONFIG_SETUP_MEMORY_TAGS
-static void setup_memory_tags (bd_t *bd);
-# endif
-static void setup_commandline_tag (bd_t *bd, char *commandline);
-
-# ifdef CONFIG_INITRD_TAG
-static void setup_initrd_tag (bd_t *bd, ulong initrd_start,
- ulong initrd_end);
-# endif
-static void setup_end_tag (bd_t *bd);
-
 static struct tag *params;
-#endif /* CONFIG_SETUP_MEMORY_TAGS || CONFIG_CMDLINE_TAG || CONFIG_INITRD_TAG 
*/
 
-static ulong get_sp(void);
-#if defined(CONFIG_OF_LIBFDT)
-static int bootm_linux_fdt(int machid, bootm_headers_t *images);
-#endif
+static ulong get_sp(void)
+{
+   ulong ret;
+
+   asm("mov %0, sp" : "=r"(ret) : );
+   return ret;
+}
 
 void arch_lmb_reserve(struct lmb *lmb)
 {
@@ -80,85 +68,6 @@ void arch_lmb_reserve(struct lmb *lmb)
gd->bd->bi_dram[0].start + gd->bd->bi_dram[0].size - sp);
 }
 
-static void announce_and_cleanup(void)
-{
-   printf("\nStarting kernel ...\n\n");
-
-#ifdef CONFIG_USB_DEVICE
-   {
-   extern void udc_disconnect(void);
-   udc_disconnect();
-   }
-#endif
-   cleanup_before_linux();
-}
-
-int do_bootm_linux(int flag, int argc, char *argv[], bootm_headers_t *images)
-{
-   bd_t*bd = gd->bd;
-   char*s;
-   int machid = bd->bi_arch_number;
-   void(*kernel_entry)(int zero, int arch, uint params);
-
-#ifdef CONFIG_CMDLINE_TAG
-   char *commandline = getenv ("bootargs");
-#endif
-
-   if ((flag != 0) && (flag != BOOTM_STATE_OS_GO))
-   return 1;
-
-   s = getenv ("machid");
-   if (s) {
-   machid = simple_strtoul (s, NULL, 16);
-   printf ("Using machid 0x%x from environment\n", machid);
-   }
-
-   show_boot_progress (15);
-
-#ifdef CONFIG_OF_LIBFDT
-   if (images->ft_len)
-   return bootm_linux_fdt(machid, images);
-#endif
-
-   kernel_entry = (void (*)(int, int, uint))images->ep;
-
-   debug ("## Transferring control to Linux (at address %08lx) ...\n",
-  (ulong) kernel_entry);
-
-#if defined (CONFIG_SETUP_MEMORY_TAGS) || \
-defined (CONFIG_CMDLINE_TAG) || \
-defined (CONFIG_INITRD_TAG) || \
-defined (CONFIG_SERIAL_TAG) || \
-defined (CONFIG_REVISION_TAG)
-   setup_start_tag (bd);
-#ifdef CONFIG_SERIAL_TAG
-   setup_serial_tag (¶ms);
-#endif
-#ifdef CONFIG_REVISION_TAG
-   setup_revision_tag (¶ms);
-#endif
-#ifdef CONFIG_SETUP_MEMORY_TAGS
-   setup_memory_tags (bd);
-#endif
-#ifdef CONFIG_CMDLINE_TAG
-   setup_commandline_tag (bd, commandline);
-#endif
-#ifdef CONFIG_INITRD_TAG
-   if (images->rd_start && images->rd_end)
-   setup_initrd_tag (bd, images->rd_start, images->rd_end);
-#endif
-   setup_

Re: [U-Boot] [PATCH] omap3: beagle: Fix build warning

[Premi, Sanjeev]

> -Original Message-
> From: Heiko Schocher [mailto:h...@denx.de] 
> Sent: Monday, September 05, 2011 7:01 PM
> To: Premi, Sanjeev
> Cc: Albert ARIBAUD; u-boot@lists.denx.de; Wolfgang Denk
> Subject: Re: [U-Boot] [PATCH] omap3: beagle: Fix build warning
> 
> Hello Sanjeev,
> 
> Premi, Sanjeev wrote:
> >> -Original Message-
> >> From: Albert ARIBAUD [mailto:albert.u.b...@aribaud.net] 
> >> Sent: Monday, September 05, 2011 5:06 PM
> >> To: Premi, Sanjeev
> >> Cc: u-boot@lists.denx.de; Heiko Schocher; Wolfgang Denk
> >> Subject: Re: [U-Boot] [PATCH] omap3: beagle: Fix build warning
> >>
[snip]...[snip]

> >>> --- a/include/configs/omap3_beagle.h
> >>> +++ b/include/configs/omap3_beagle.h
> >>> @@ -118,7 +118,7 @@
> >>>   #define CONFIG_I2C_MULTI_BUS1
> >>>
> >>>   /* Probe all devices */
> >>> -#define CONFIG_SYS_I2C_NOPROBES  {0x0, 0x0}
> >>> +#define CONFIG_SYS_I2C_NOPROBES  {{0x0, 0x0}}
> >>>
> >>>   /* USB */
> >>>   #define CONFIG_MUSB_UDC 1
> >> Won't all board configs which use CONFIG_SYS_I2C_NOPROBES 
> suffer from 
> >> the same bug? I would hate to see an endless trickle of 
> >> individual board 
> >> config patches, and would much prefer a single patch to fix 
> >> all boards 
> > 
> > [sp] Actually, not all boards suffer from this issue.
> >  But yes, I can make a single patch and submit it
> >  in next couple of hours.
> 
> Isn;t this issue introduced just from commit
> 
> authorJason Kridner 
>  Sat, 23 Jul 2011 04:42:44 + (23:42 -0500)
> committer Albert ARIBAUD 
>  Sun, 4 Sep 2011 09:36:21 + (11:36 +0200)
> commitf74fc4ae6d6257ecdbc0049f6aa2e96212207f05
> 
> so ~1day old ... Hmm, I think, this is just a single board bugfix,
> as this warning only raises, if CONFIG_I2C_MULTI_BUS and
> CONFIG_SYS_I2C_NOPROBES is defined ... as introduced for the beagle
> board through above commit ... other boards should be clean, as I tend
> to do a MAKEALL after applying patches from ML ... but if you find

[sp] Yes, you are right about the issue, and hence I posted it as
 single patch against beagleboard only.  The description includes
 a bash script that I used to "blindly" extent the fix to other
 boards.

 I guess, MAKEALL succeeds because the appropriate config options
 may not be enabled for the boards.

 Will dig on a few cases (of the ones changed) as to why the build
 succeeds.

> time and can check this, it would be nice!

[sp] I only have a arm codesourcery toolchain installed. Would be able
 to check only ARM boards (of the 31) impacted by the updated patch.

 Can do by tomorrow...

> 
> bye,
> Heiko
> -- 
> DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] UBI problems on current u-bo

[Stefan Roese]

Hi Holger,

On Friday 02 September 2011 15:32:40 Holger Brunck wrote:
> I address this question to you because one of your commits is connected to
> this problem, but other hints from other readers are also welcome ;-) .

I'll try to look into this later this week.

BTW: Is this problem reproducible on one of your systems?

Best regards,
Stefan

--
DENX Software Engineering GmbH,  MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: off...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] NULL base_name for hwaddr env lookup

[Heiko Schocher]

Hello MARCEL,

stany MARCEL wrote:
> in net/eth.c there is this function call with test:
> 
>  if (eth_write_hwaddr(dev, NULL, eth_number))
> 
> If I understand correctly this test will always be true as NULL is used.
> 
> I think it should be "eth" instead of NULL.
> 
> Am I wrong ?

No.

> If not I can submit a patch.

See fix for this issue here:

http://patchwork.ozlabs.org/patch/112361/

bye,
Heiko
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH V4] Makefile : fix generation of cpu related asm-offsets.h

[Stefano Babic]

commit 0edf8b5b2fa0d210ebc4d6da0fd1aceeb7e44e47 breaks
building on a different directory with the O= parameter.
The patch wil fix this issue, generating always asm-offsets.h before
the other targets.

Signed-off-by: Stefano Babic 
CC: Matthias Weisser 
CC: Wolfgang Denk 
---

Note: V2 breaks *all* not ARM boards
Put the generated asm-offsets.h into include/generated,
because this directory is common to all architectures.

Changes since V3: the empty asm-offsets.s was generated under src,
and not in the directory indicated by $(obj), required for out-of-tree 
buildeing.

 Makefile|   20 +++-
 arch/arm/cpu/arm1136/mx35/Makefile  |2 --
 arch/arm/cpu/arm926ejs/mb86r0x/Makefile |2 --
 arch/arm/cpu/arm926ejs/mx25/Makefile|2 --
 arch/arm/cpu/arm926ejs/mx27/Makefile|2 --
 arch/arm/cpu/armv7/mx5/Makefile |2 --
 arch/arm/cpu/armv7/mx5/lowlevel_init.S  |2 +-
 arch/arm/include/asm/arch-mx25/macro.h  |2 +-
 board/freescale/mx35pdk/lowlevel_init.S |2 +-
 board/logicpd/imx27lite/lowlevel_init.S |2 +-
 board/syteco/jadecpu/lowlevel_init.S|2 +-
 board/syteco/zmx25/lowlevel_init.S  |2 +-
 rules.mk|   10 --
 13 files changed, 25 insertions(+), 27 deletions(-)

diff --git a/Makefile b/Makefile
index 6abc319..cfa8071 100644
--- a/Makefile
+++ b/Makefile
@@ -464,7 +464,8 @@ updater:
 # parallel sub-makes creating .depend files simultaneously.
 depend dep:$(TIMESTAMP_FILE) $(VERSION_FILE) \
$(obj)include/autoconf.mk \
-   $(obj)include/generated/generic-asm-offsets.h
+   $(obj)include/generated/generic-asm-offsets.h \
+   $(obj)include/generated/asm-offsets.h
for dir in $(SUBDIRS) $(CPUDIR) $(dir $(LDSCRIPT)) ; do \
$(MAKE) -C $$dir _depend ; done
 
@@ -527,6 +528,21 @@ $(obj)lib/asm-offsets.s:   $(obj)include/autoconf.mk.dep \
$(CFLAGS) $(CFLAGS_$(BCURDIR)/$(@F)) $(CFLAGS_$(BCURDIR)) \
-o $@ $(src)lib/asm-offsets.c -c -S
 
+$(obj)include/generated/asm-offsets.h: $(obj)include/autoconf.mk.dep \
+   $(obj)$(CPUDIR)/$(SOC)/asm-offsets.s
+   @echo Generating $@
+   tools/scripts/make-asm-offsets $(obj)$(CPUDIR)/$(SOC)/asm-offsets.s $@
+
+$(obj)$(CPUDIR)/$(SOC)/asm-offsets.s:  $(obj)include/autoconf.mk.dep
+   @mkdir -p $(obj)$(CPUDIR)/$(SOC)
+   if [ -f $(src)$(CPUDIR)/$(SOC)/asm-offsets.c ];then \
+   $(CC) -DDO_DEPS_ONLY \
+   $(CFLAGS) $(CFLAGS_$(BCURDIR)/$(@F)) $(CFLAGS_$(BCURDIR)) \
+   -o $@ $(src)$(CPUDIR)/$(SOC)/asm-offsets.c -c -S; \
+   else \
+   touch $@; \
+   fi
+
 #
 else   # !config.mk
 all $(obj)u-boot.hex $(obj)u-boot.srec $(obj)u-boot.bin \
@@ -916,6 +932,8 @@ clean:
   $(obj)arch/blackfin/cpu/init.{lds,elf}
@rm -f $(obj)include/bmp_logo.h
@rm -f $(obj)lib/asm-offsets.s
+   @rm -f $(obj)include/generated/asm-offsets.h
+   @rm -f $(obj)$(CPUDIR)/$(SOC)/asm-offsets.s
@rm -f 
$(obj)nand_spl/{u-boot.lds,u-boot-nand_spl.lds,u-boot-spl,u-boot-spl.map,System.map}
@rm -f $(obj)onenand_ipl/onenand-{ipl,ipl.bin,ipl.map}
@rm -f 
$(obj)mmc_spl/{u-boot.lds,u-boot-spl,u-boot-spl.map,u-boot-spl.bin,u-boot-mmc-spl.bin}
diff --git a/arch/arm/cpu/arm1136/mx35/Makefile 
b/arch/arm/cpu/arm1136/mx35/Makefile
index 284cdc5..469397c 100644
--- a/arch/arm/cpu/arm1136/mx35/Makefile
+++ b/arch/arm/cpu/arm1136/mx35/Makefile
@@ -39,8 +39,6 @@ all:  $(obj).depend $(LIB)
 $(LIB):$(OBJS)
$(call cmd_link_o_target, $(OBJS))
 
-$(OBJS) : $(TOPDIR)/include/asm/arch/asm-offsets.h
-
 
 #
 
diff --git a/arch/arm/cpu/arm926ejs/mb86r0x/Makefile 
b/arch/arm/cpu/arm926ejs/mb86r0x/Makefile
index 974d0be..bab048b 100644
--- a/arch/arm/cpu/arm926ejs/mb86r0x/Makefile
+++ b/arch/arm/cpu/arm926ejs/mb86r0x/Makefile
@@ -37,8 +37,6 @@ all:  $(obj).depend $(LIB)
 $(LIB):$(OBJS)
$(call cmd_link_o_target, $(OBJS))
 
-$(OBJS) : $(TOPDIR)/include/asm/arch/asm-offsets.h
-
 #
 
 # defines $(obj).depend target
diff --git a/arch/arm/cpu/arm926ejs/mx25/Makefile 
b/arch/arm/cpu/arm926ejs/mx25/Makefile
index 9219c06..3c2a65e 100644
--- a/arch/arm/cpu/arm926ejs/mx25/Makefile
+++ b/arch/arm/cpu/arm926ejs/mx25/Makefile
@@ -34,8 +34,6 @@ all:  $(obj).depend $(LIB)
 $(LIB):$(OBJS)
$(call cmd_link_o_target, $(OBJS))
 
-$(OBJS) : $(TOPDIR)/include/asm/arch/asm-offsets.h
-
 #
 
 # defines $(obj).depend target
diff --git a/arch/arm/cpu/arm926ejs/mx27/Makefile 
b/arch/arm/cpu/arm926ejs/mx27/Makefile
index 7ac1a21..0e112b3 100644
--

Re: [U-Boot] [PATCH V3] Makefile : fix generation of cpu related asm-offsets.h

[Stefano Babic]

On 09/05/2011 03:52 PM, Wolfgang Denk wrote:
> Dear Stefano,
>

Hi Wolfgang,

> In message <20110905134419.d820f158d...@gemini.denx.de> I wrote:
>>
>> This one is not that much better:
> 
> Maybe it helps debugging - here is an observation:
> 
>> sed: can't read 
>> /work/wd/tmp-arm/arch/arm/cpu/arm1136/omap24xx/asm-offsets.s: No such file 
>> or directory
>> sed: can't read 
>> /work/wd/tmp-arm/arch/arm/cpu/arm926ejs/davinci/asm-offsets.s: No such file 
>> or directory
>> sed: can't read /work/wd/tmp-arm/arch/arm/cpu/ixp//asm-offsets.s: No such 
>> file or directory
> 
> Mind the inconsistency - see the double slash after "ixp".

An evident error is :

+   else \
+   touch $(src)$(CPUDIR)/$(SOC)/asm-offsets.s; \
+   fi

Then an asm-offsets.s not in the out-of-tree directory is generated. I
send V4 of the patch.

Changing that I can compile clean the da850evm board, and mostly of
other boards you report broken. However, I cannot compile the harmony,
but is seem to me this is an unrelated issue:

I did it with:
make O=~/Projects/imx/tmp  harmony

and I get this error:
common/board.o ../common/board.c -c
Assembler messages:
Fatal error: can't create
/home/stefano/Projects/imx/tmp/board/nvidia/harmony/../common/board.o:
No such file or directory

This looks related that the Makefile in the board directory refers
always to the common code in ../common, that is not found using the
out-of-tree.

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] UBIFS: Change ubifsload to set the filesize variable

[Wolfgang Denk]

Dear Marek Vasut,

In message <201109051512.48148.marek.va...@gmail.com> you wrote:
>
> > +   setenv("filesize", buf);
> 
> maybe you want to check set_local_var() too ?

Yes, we should clean up all these auto-generated environment variables
one day.  But what you propose here is not a good approach:

(1) not all boards use the hush shell, and set_local_var() is only
available there.
(2) if you make such a change, then you must make it everywhere, and
you must make sure not to break existent use of these things.

Both is definitely out of the scope of this patch.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Quote from a recent meeting:   "We are going to continue having these
meetings everyday until I find out why no work is getting done."
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] NULL base_name for hwaddr env lookup

[Stany MARCEL]

On Mon, Sep 5, 2011 at 3:53 PM, stany MARCEL
 wrote:
> in net/eth.c there is this function call with test:
>
>  if (eth_write_hwaddr(dev, NULL, eth_number))
>
> If I understand correctly this test will always be true as NULL is used.
>
> I think it should be "eth" instead of NULL.
>
> Am I wrong ?
>
> If not I can submit a patch.
>
> Regards,
>
> Stany
> ___
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>

Ok it's corrected by a commit few minutes ago.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/2] powerpc/85xx: standardize display of address map size (32-bit vs. 36-bit)

[Wolfgang Denk]

Dear Timur Tabi,

In message <4e60dcec.9090...@freescale.com> you wrote:
> Kumar Gala wrote:
> > Both.  I'm think for your patch we'd add some general config option for 
> > extra print info.
> 
> So you want to see this instead:
> 
> /*
>  * Display whether this is a 32-bit build or a 36-bit build.
>  */
> #ifdef CONFIG_DISPLAY_ADDR_SIZE
> #ifdef CONFIG_PHYS_64BIT
>   puts("ADDR:  36-bit address map\n");
> #else
>   puts("ADDR:  32-bit address map\n");
> #endif
> #endif

Please dump this completely.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
"A complex system that works is invariably found to have evolved from
a simple system that worked." - John Gall, _Systemantics_
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/2] powerpc/85xx: standardize display of address map size (32-bit vs. 36-bit)

[Wolfgang Denk]

Dear Kumar Gala,

In message <6a841e78-5d44-4190-9899-1976234ef...@freescale.com> you wrote:
> 
>  The FSL PPC board ports are not the typical use case.  I view our board
> ports having a few different purposes:
> * Examples for people building their own boards

Especially of this you have additional responsibility not to set bad
examples.

> Having the additional information printed on boot is extremely useful to
> FSL for our board ports for supporting customers.  So my take is the

"extremely useful" is a really big statement.  I don't buy it.

> Any concerns w/that proposal?

I want to clean this up, and I'm extremely unhappy if there are
"examples for people building their own boards" that use excessively
verbose boot messages.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
In my experience the best way to get something done  is to give it to
someone who is busy.   - Terry Pratchett, _Going_Postal_
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/2] powerpc/85xx: standardize display of address map size (32-bit vs. 36-bit)

[Wolfgang Denk]

Dear Timur Tabi,

In message <4e6002b4.90...@freescale.com> you wrote:
>
> > What sort of "confusion" do you have?  I see two situations:  in
> > 99.99% of all cases U-Boot is just a means to boot an OS, and nobody
> > cares a bit about the actual U-Boot output, as long as the OS is
> > runnign after a few seconds, and rather sooner than later. 
> 
> The confusion stems mostly from the fact that we've been displaying this
> information for years.  If suddenly we stop displaying it, people are going to
> think that something is wrong.  Also, you need to know which U-Boot you're
> running before you boot the kernel, because the device tree needs to match.  
> No
> one is going to think to use the 'bdinfo' command to find it.

Software changes, output changes.  Document your changes if you want
to make sure your users happy.  Feel free to send them detailled
changed logs.

> The problem is that this isn't standard.  "make P1022DS" builds a 32-bit 
> U-Boot.
>  But "make P4080DS" builds a 36-bit U-Boot.  So we've been displaying the
> address map size at boot time in some situations, but not all.  My patch makes
> this standard.

But it's not you who defines what the standard looks like.

> I agree with adding that information to the bdinfo command.  I'm just worried
> about having it available only there.

You will get used to it, and so will your users.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Being schizophrenic is better than living alone.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] Flush cache after the OS image is loaded into the memory.

[Wolfgang Denk]

Dear Kumar Gala,

In message <1314794723-25547-1-git-send-email-ga...@kernel.crashing.org> you 
wrote:
> From: Diana CRACIUN 
> 
> Since we are loading an executable image into memory we need flush it
> out of the cache to possible maintain coherence on CPUs with split
> instruction and data caches.  We do this for other executable image
> loading command.
> 
> On PowerPC once we do this we no longer need to explicitly flush the
> dcache on multi-core systems in the BOOTM_STATE_OS_PREP phase.  We now
> treat the BOOTM_STATE_OS_PREP as a no-op to maintain backwards
> compatibility with the bootm subcommand.
> 
> Signed-off-by: James Yang 
> Signed-off-by: Diana CRACIUN 
> Signed-off-by: Kumar Gala 
> ---
>  arch/powerpc/lib/bootm.c |   20 ++--
>  common/cmd_bootm.c   |3 +++
>  2 files changed, 9 insertions(+), 14 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Children are natural mimics who act like their parents despite  every
effort to teach them good manners.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] arm: Add Prep subcommand support to bootm

[Simon Schwarz]

Dear Andreas,

On 09/05/2011 12:58 PM, Andreas Bießmann wrote:
> Dear Simon,
>
> Am Mo 29 Aug 2011 18:08:13 CEST, Simon Schwarz schrieb:
>> Adds prep subcommand to bootm implementation of ARM. When bootm is called 
>> with
>> the subcommand prep the function stops right after ATAGS creation and before
>> announce_and_cleanup.
>>
>> This is used in savebp command
>
> savebp? I thought this command is now 'spl' with some subcommands.
fixed.
>
>>
>> Signed-off-by: Simon Schwarz
>> 
>
> this four times '-' will not be recognized as comment section by git am
>
fixed
>>
>> V2 changes:
>> nothing
>>
>> V3 changes:
>> nothing
>>
>> changes after slicing this from patch
>> http://article.gmane.org/gmane.comp.boot-loaders.u-boot/106422:
>> DEL Prototype declaration - not necessary if reordered
>> DEL Most #ifdefs - relying on -ffunction-sections and --gc-sections to 
>> handle unused
>> CHG reorganized bootm. powerpc implementation was the model
>> ADD get_board_serial fake implementation - this is needed if setup_serial_tag
>>  is compiled but the board doesn't support it - tradeoff for removing
>>  #ifdefs
>> ---
>>   arch/arm/lib/bootm.c |  330 
>> +
>>   1 files changed, 168 insertions(+), 162 deletions(-)
>>
>> diff --git a/arch/arm/lib/bootm.c b/arch/arm/lib/bootm.c
>> index 802e833..5f112e2 100644
>> --- a/arch/arm/lib/bootm.c
>> +++ b/arch/arm/lib/bootm.c
>> @@ -1,4 +1,8 @@
>> -/*
>> +/* Copyright (C) 2011
>> + * Corscience GmbH&  Co. KG - Simon Schwarz
>> + *  - Added prep subcommand support
>> + *  - Reorganized source - modeled after powerpc version
>> + *
>>* (C) Copyright 2002
>>* Sysgo Real-Time Solutions, GmbH
>>* Marius Groeger
>> @@ -32,31 +36,16 @@
>>
>>   DECLARE_GLOBAL_DATA_PTR;
>>
>> -#if defined (CONFIG_SETUP_MEMORY_TAGS) || \
>> -defined (CONFIG_CMDLINE_TAG) || \
>> -defined (CONFIG_INITRD_TAG) || \
>> -defined (CONFIG_SERIAL_TAG) || \
>> -defined (CONFIG_REVISION_TAG)
>> -static void setup_start_tag (bd_t *bd);
>> -
>> -# ifdef CONFIG_SETUP_MEMORY_TAGS
>> -static void setup_memory_tags (bd_t *bd);
>> -# endif
>> -static void setup_commandline_tag (bd_t *bd, char *commandline);
>> -
>> -# ifdef CONFIG_INITRD_TAG
>> -static void setup_initrd_tag (bd_t *bd, ulong initrd_start,
>> -  ulong initrd_end);
>> -# endif
>> -static void setup_end_tag (bd_t *bd);
>> -
>> +static void (*kernel_entry)(int zero, int arch, uint params);
>
> NAK (see later on)
done.
>
>>   static struct tag *params;
>> -#endif /* CONFIG_SETUP_MEMORY_TAGS || CONFIG_CMDLINE_TAG || 
>> CONFIG_INITRD_TAG */
>>
>> -static ulong get_sp(void);
>> -#if defined(CONFIG_OF_LIBFDT)
>> -static int bootm_linux_fdt(int machid, bootm_headers_t *images);
>> -#endif
>> +static ulong get_sp(void)
>> +{
>> +ulong ret;
>> +
>> +asm("mov %0, sp" : "=r"(ret) : );
>> +return ret;
>> +}
>>
>>   void arch_lmb_reserve(struct lmb *lmb)
>>   {
>> @@ -80,85 +69,6 @@ void arch_lmb_reserve(struct lmb *lmb)
>>  gd->bd->bi_dram[0].start + gd->bd->bi_dram[0].size - sp);
>>   }
>>
>> -static void announce_and_cleanup(void)
>> -{
>> -printf("\nStarting kernel ...\n\n");
>> -
>> -#ifdef CONFIG_USB_DEVICE
>> -{
>> -extern void udc_disconnect(void);
>> -udc_disconnect();
>> -}
>> -#endif
>> -cleanup_before_linux();
>> -}
>
> I can not see why git decided to remove that here and add it later on ..
> did you change anything in announce_and_cleanup()?

Nope. I added something before and there were major changes in the file 
(moved large codejunks around)- not the best environment for a pretty 
diff i guess...
>
>> -
>> -int do_bootm_linux(int flag, int argc, char *argv[], bootm_headers_t 
>> *images)
>> -{
>> -bd_t*bd = gd->bd;
>> -char*s;
>> -int machid = bd->bi_arch_number;
>> -void(*kernel_entry)(int zero, int arch, uint params);
>> -
>> -#ifdef CONFIG_CMDLINE_TAG
>> -char *commandline = getenv ("bootargs");
>> -#endif
>> -
>> -if ((flag != 0)&&  (flag != BOOTM_STATE_OS_GO))
>> -return 1;
>> -
>> -s = getenv ("machid");
>> -if (s) {
>> -machid = simple_strtoul (s, NULL, 16);
>> -printf ("Using machid 0x%x from environment\n", machid);
>> -}
>> -
>> -show_boot_progress (15);
>> -
>> -#ifdef CONFIG_OF_LIBFDT
>> -if (images->ft_len)
>> -return bootm_linux_fdt(machid, images);
>> -#endif
>> -
>> -kernel_entry = (void (*)(int, int, uint))images->ep;
>> -
>> -debug ("## Transferring control to Linux (at address %08lx) ...\n",
>> -   (ulong) kernel_entry);
>> -
>> -#if defined (CONFIG_SETUP_MEMORY_TAGS) || \
>> -defined (CONFIG_CMDLINE_TAG) || \
>> -defined (CONFIG_INITRD_TAG) || \
>> -defined (CONFIG_SERIAL_TAG) || \
>> -defined (CONFIG_REVISION_TAG)
>> -setup_start_tag (bd);
>> -#ifdef CONFIG_SERIAL_TAG
>> -setup_serial_tag (¶ms);
>> -#endif
>> -#i

Re: [U-Boot] [PATCH] Correct call to eth_write_hwaddr()

[Wolfgang Denk]

Dear Simon Glass,

In message <1314719460-27858-1-git-send-email-...@chromium.org> you wrote:
> This fixes "Warning: failed to set MAC address" on platforms which rely on
> an 'ethaddr' environment variable to set the MAC address.
> 
> This bug was introduced by this commit:
> 
> 7616e785 Add Ethernet hardware MAC address framework to usbnet
> 
> Signed-off-by: Simon Glass 
> ---
>  net/eth.c |2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Until you walk a mile in another man's moccasins, you  can't  imagine
the smell.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] sf: winbond: Add support for the Winbond W25X40

[Wolfgang Denk]

Dear James Le Cuirot,

In message <1311014692-1420-1-git-send-email-ch...@aura-online.co.uk> you wrote:
> The Winbond W25X40 is now being used in the IP02 (and possibly IP04).
> Tested and working on the actual device.
> 
> Signed-off-by: James Le Cuirot 
> ---
>  drivers/mtd/spi/winbond.c |8 
>  1 files changed, 8 insertions(+), 0 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
As far as the laws of mathematics refer  to  reality,  they  are  not
certain;  and  as  far  as  they  are  certain,  they do not refer to
reality.   -- Albert Einstein
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH v4 3/3] MX25: Add initial support for MX25PDK

[Fabio Estevam]

Add the initial support for MX25PDK booting from SD card via internal boot.

Signed-off-by: Fabio Estevam 
---
Changes since v3:
- No changes
Changes since v2:
- No changes
Changes since v1:
- Keep mx25pdk entry to boards.cfg sorted
- Removed unused defines in mx25pdk.h
- Do not use the whole RAM space for memory test
 MAINTAINERS |1 +
 board/freescale/mx25pdk/Makefile|   49 ++
 board/freescale/mx25pdk/imximage.cfg|   73 +
 board/freescale/mx25pdk/lowlevel_init.S |   21 ++
 board/freescale/mx25pdk/mx25pdk.c   |   62 ++
 boards.cfg  |1 +
 include/configs/mx25pdk.h   |  106 +++
 7 files changed, 313 insertions(+), 0 deletions(-)
 create mode 100644 board/freescale/mx25pdk/Makefile
 create mode 100644 board/freescale/mx25pdk/imximage.cfg
 create mode 100644 board/freescale/mx25pdk/lowlevel_init.S
 create mode 100644 board/freescale/mx25pdk/mx25pdk.c
 create mode 100644 include/configs/mx25pdk.h

diff --git a/MAINTAINERS b/MAINTAINERS
index f895e9a..7f07b60 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -645,6 +645,7 @@ Kristoffer Ericson 
 
 Fabio Estevam 
 
+   mx25pdk i.MX25
mx31pdk i.MX31
mx53ard i.MX53
mx53smd i.MX53
diff --git a/board/freescale/mx25pdk/Makefile b/board/freescale/mx25pdk/Makefile
new file mode 100644
index 000..4d45ae3
--- /dev/null
+++ b/board/freescale/mx25pdk/Makefile
@@ -0,0 +1,49 @@
+#
+# Copyright (C) 2007, Guennadi Liakhovetski 
+#
+# (C) Copyright 2011 Freescale Semiconductor, Inc.
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License as
+# published by the Free Software Foundation; either version 2 of
+# the License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+# MA 02111-1307 USA
+#
+
+include $(TOPDIR)/config.mk
+
+LIB= $(obj)lib$(BOARD).o
+
+COBJS  := mx25pdk.o
+SOBJS  := lowlevel_init.o
+
+SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)
+OBJS   := $(addprefix $(obj),$(COBJS))
+SOBJS  := $(addprefix $(obj),$(SOBJS))
+
+$(LIB):$(obj).depend $(OBJS) $(SOBJS)
+   $(call cmd_link_o_target, $(OBJS) $(SOBJS))
+
+clean:
+   rm -f $(SOBJS) $(OBJS)
+
+distclean: clean
+   rm -f $(LIB) core *.bak .depend
+
+#
+
+# defines $(obj).depend target
+include $(SRCTREE)/rules.mk
+
+sinclude $(obj).depend
+
+#
diff --git a/board/freescale/mx25pdk/imximage.cfg 
b/board/freescale/mx25pdk/imximage.cfg
new file mode 100644
index 000..f7af7ff
--- /dev/null
+++ b/board/freescale/mx25pdk/imximage.cfg
@@ -0,0 +1,73 @@
+#
+# (C) Copyright 2009
+# Stefano Babic DENX Software Engineering sba...@denx.de.
+#
+# See file CREDITS for list of people who contributed to this
+# project.
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License as
+# published by the Free Software Foundation; either version 2 of
+# the License or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# Refer docs/README.imxmage for more details about how-to configure
+# and create imximage boot image
+#
+# The syntax is taken as close as possible with the kwbimage
+
+# Boot Device : one of
+# spi, sd (the board has no nand neither onenand)
+
+BOOT_FROM  sd
+
+# Device Configuration Data (DCD)
+#
+# Each entry must have the format:
+# Addr-type   AddressValue
+#
+# where:
+#  Addr-type register length (1,2 or 4 bytes)
+#  Address   absolute address of the register
+#  value value to be stored in the register
+
+# EIM config-CS5 init -- CPLD
+DATA 4 0xB8002050 0xD843
+DATA 4 0xB8002054 0x22252521
+DATA 4 0xB8002058 0x0A00
+
+# DDR2 init
+DATA 4 0xB8001004 0x0076E83A
+DATA 4 0xB8001010 0x0204
+DATA 4 0xB8001000 0x9221
+DATA 4 0x8f00 0x12344321
+DATA 4 0xB8001000 0xB221
+DATA 1 0x8200 0xda
+DATA 1 0x8300 0xda
+DATA 1 0x81000400 0xda
+DATA 1 0x8333 0xda
+
+DATA 4 0xB8001000 0x9221
+DATA 1 0x8400 0x12345678
+
+DATA 4 0xB8001000 0xA221
+DATA 4 0x8000 0x

[U-Boot] [PATCH v4 1/3] mx25: Introduce sys_proto.h

[Fabio Estevam]

Create a sys_proto.h for MX25 as done on other MX25 processors.

Signed-off-by: Fabio Estevam 
---
Changes since v3:
- No changes. This patch was introduced in v4

 arch/arm/cpu/arm926ejs/mx25/generic.c  |1 +
 arch/arm/include/asm/arch-mx25/imx-regs.h  |4 
 arch/arm/include/asm/arch-mx25/sys_proto.h |   27 +++
 3 files changed, 28 insertions(+), 4 deletions(-)
 create mode 100644 arch/arm/include/asm/arch-mx25/sys_proto.h

diff --git a/arch/arm/cpu/arm926ejs/mx25/generic.c 
b/arch/arm/cpu/arm926ejs/mx25/generic.c
index 76e4b5c..3feb449 100644
--- a/arch/arm/cpu/arm926ejs/mx25/generic.c
+++ b/arch/arm/cpu/arm926ejs/mx25/generic.c
@@ -31,6 +31,7 @@
 #ifdef CONFIG_MXC_MMC
 #include 
 #endif
+#include 
 
 /*
  *  get the system pll clock in Hz
diff --git a/arch/arm/include/asm/arch-mx25/imx-regs.h 
b/arch/arm/include/asm/arch-mx25/imx-regs.h
index 2ccb445..646c41e 100644
--- a/arch/arm/include/asm/arch-mx25/imx-regs.h
+++ b/arch/arm/include/asm/arch-mx25/imx-regs.h
@@ -34,10 +34,6 @@
 #define _IMX_REGS_H
 
 #ifndef __ASSEMBLY__
-#ifdef CONFIG_FEC_MXC
-extern void mx25_fec_init_pins(void);
-extern void imx_get_mac_from_fuse(unsigned char *mac);
-#endif
 
 /* Clock Control Module (CCM) registers */
 struct ccm_regs {
diff --git a/arch/arm/include/asm/arch-mx25/sys_proto.h 
b/arch/arm/include/asm/arch-mx25/sys_proto.h
new file mode 100644
index 000..d5bccf2
--- /dev/null
+++ b/arch/arm/include/asm/arch-mx25/sys_proto.h
@@ -0,0 +1,27 @@
+/*
+ * (C) Copyright 2009
+ * Stefano Babic, DENX Software Engineering, sba...@denx.de.
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef _SYS_PROTO_H_
+#define _SYS_PROTO_H_
+
+#ifdef CONFIG_FEC_MXC
+extern void mx25_fec_init_pins(void);
+extern void imx_get_mac_from_fuse(unsigned char *mac);
+#endif /* CONFIG_FEC_MXC */
+
+#endif /* _SYS_PROTO_H_ */
-- 
1.7.1


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH v4 2/3] MX25: tx25: Avoid the usage of extern in C file

[Fabio Estevam]

Avoid the usage of extern in C file as pointed out by checkpatch.

Signed-off-by: Fabio Estevam 
---
Changes since v3:
- Place mx25_uart1_init_pins inside sys_proto.h
Changes since v2:
- Place the extern in the MX25 imx-regs.h instead of inside the MX27 imx-regs.h
Changes since v1:
- No changes
 arch/arm/include/asm/arch-mx25/sys_proto.h |1 +
 board/karo/tx25/tx25.c |2 --
 2 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/arch/arm/include/asm/arch-mx25/sys_proto.h 
b/arch/arm/include/asm/arch-mx25/sys_proto.h
index d5bccf2..0280c57 100644
--- a/arch/arm/include/asm/arch-mx25/sys_proto.h
+++ b/arch/arm/include/asm/arch-mx25/sys_proto.h
@@ -23,5 +23,6 @@
 extern void mx25_fec_init_pins(void);
 extern void imx_get_mac_from_fuse(unsigned char *mac);
 #endif /* CONFIG_FEC_MXC */
+extern void mx25_uart1_init_pins(void);
 
 #endif /* _SYS_PROTO_H_ */
diff --git a/board/karo/tx25/tx25.c b/board/karo/tx25/tx25.c
index 25b99e8..c34672a 100644
--- a/board/karo/tx25/tx25.c
+++ b/board/karo/tx25/tx25.c
@@ -141,8 +141,6 @@ void tx25_fec_init(void)
 int board_init()
 {
 #ifdef CONFIG_MXC_UART
-   extern void mx25_uart1_init_pins(void);
-
mx25_uart1_init_pins();
 #endif
/* board id for linux */
-- 
1.7.1


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] NULL base_name for hwaddr env lookup

[stany MARCEL]

in net/eth.c there is this function call with test:

 if (eth_write_hwaddr(dev, NULL, eth_number))

If I understand correctly this test will always be true as NULL is used.

I think it should be "eth" instead of NULL.

Am I wrong ?

If not I can submit a patch.

Regards,

Stany
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH V3] Makefile : fix generation of cpu related asm-offsets.h

[Wolfgang Denk]

Dear Stefano,

In message <20110905134419.d820f158d...@gemini.denx.de> I wrote:
> 
> This one is not that much better:

Maybe it helps debugging - here is an observation:

> sed: can't read /work/wd/tmp-arm/arch/arm/cpu/arm1136/omap24xx/asm-offsets.s: 
> No such file or directory
> sed: can't read 
> /work/wd/tmp-arm/arch/arm/cpu/arm926ejs/davinci/asm-offsets.s: No such file 
> or directory
> sed: can't read /work/wd/tmp-arm/arch/arm/cpu/ixp//asm-offsets.s: No such 
> file or directory

Mind the inconsistency - see the double slash after "ixp".

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
"The combination of a number of things to make existence worthwhile."
"Yes, the philosophy of 'none,' meaning 'all.'"
-- Spock and Lincoln, "The Savage Curtain", stardate 5906.4
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] [COSMETIC] board/prodrive/alpr/fpga.c: Coding style cleanup

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1315222479-23960-1-git-send-email...@denx.de> you wrote:
> Signed-off-by: Wolfgang Denk 
> Cc: Stefan Roese 
> ---
> v2: fix compile issues; must use "do {} while (0)" in macros
> definitions if we want to omit the braces in if...else statements.
> Should have done this anyway.
> 
>  board/prodrive/alpr/fpga.c |   76 ++-
>  1 files changed, 39 insertions(+), 37 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
In general, they do what you want, unless you want consistency.
- Larry Wall in the perl man page
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH V3] Makefile : fix generation of cpu related asm-offsets.h

[Wolfgang Denk]

Dear Stefano Babic,

In message <1314726584-17203-1-git-send-email-sba...@denx.de> you wrote:
> commit 0edf8b5b2fa0d210ebc4d6da0fd1aceeb7e44e47 breaks
> building on a different directory with the O= parameter.
> The patch wil fix this issue, generating always asm-offsets.h before
> the other targets.
> 
> Signed-off-by: Stefano Babic 
> CC: Matthias Weisser 
> CC: Wolfgang Denk 
> ---
> 
> Note: V2 breaks *all* not ARM boards
> Put the generated asm-offsets.h into include/generated,
> because this directory is common to all architectures.

This one is not that much better:

Configuring for apollon board...
sed: can't read /work/wd/tmp-arm/arch/arm/cpu/arm1136/omap24xx/asm-offsets.s: 
No such file or directory
make: *** [/work/wd/tmp-arm/include/generated/asm-offsets.h] Error 1
make: *** Waiting for unfinished jobs
arm-linux-size: '/work/wd/tmp-arm/u-boot': No such file
Configuring for da850evm board...
sed: can't read /work/wd/tmp-arm/arch/arm/cpu/arm926ejs/davinci/asm-offsets.s: 
No such file or directory
make: *** [/work/wd/tmp-arm/include/generated/asm-offsets.h] Error 1
arm-linux-size: '/work/wd/tmp-arm/u-boot': No such file
Configuring for dvlhost board...
sed: can't read /work/wd/tmp-arm/arch/arm/cpu/ixp//asm-offsets.s: No such file 
or directory
make: *** [/work/wd/tmp-arm/include/generated/asm-offsets.h] Error 1
arm-linux-size: '/work/wd/tmp-arm/u-boot': No such file
Configuring for efikamx - Board: efikamx, Options: 
IMX_CONFIG=board/efikamx/imximage.cfg
   textdata bss dec hex filename
 1847924104  206516  395412   60894 /work/wd/tmp-arm/u-boot
Configuring for harmony board...
sed: can't read /work/wd/tmp-arm/arch/arm/cpu/armv7/tegra2/asm-offsets.s: No 
such file or directory
make: *** [/work/wd/tmp-arm/include/generated/asm-offsets.h] Error 1
make: *** Waiting for unfinished jobs
arm-linux-size: '/work/wd/tmp-arm/u-boot': No such file
Configuring for imx27lite board...
   textdata bss dec hex filename
 2427807812  236232  486824   76da8 /work/wd/tmp-arm/u-boot
Configuring for imx31_phycore board...
sed: can't read /work/wd/tmp-arm/arch/arm/cpu/arm1136/mx31/asm-offsets.s: No 
such file or directory
make: *** [/work/wd/tmp-arm/include/generated/asm-offsets.h] Error 1
make: *** Waiting for unfinished jobs
arm-linux-size: '/work/wd/tmp-arm/u-boot': No such file
Configuring for imx31_phycore_eet board...
sed: can't read /work/wd/tmp-arm/arch/arm/cpu/arm1136/mx31/asm-offsets.s: No 
such file or directory
make: *** [/work/wd/tmp-arm/include/generated/asm-offsets.h] Error 1
arm-linux-size: '/work/wd/tmp-arm/u-boot': No such file
Configuring for ixdp425 board...
sed: can't read /work/wd/tmp-arm/arch/arm/cpu/ixp//asm-offsets.s: No such file 
or directory
make: *** [/work/wd/tmp-arm/include/generated/asm-offsets.h] Error 1
arm-linux-size: '/work/wd/tmp-arm/u-boot': No such file
Configuring for ixdpg425 board...
sed: can't read /work/wd/tmp-arm/arch/arm/cpu/ixp//asm-offsets.s: No such file 
or directory
make: *** [/work/wd/tmp-arm/include/generated/asm-offsets.h] Error 1
arm-linux-size: '/work/wd/tmp-arm/u-boot': No such file
Configuring for jadecpu board...
sed: can't read /work/wd/tmp-arm/arch/arm/cpu/arm926ejs/mb86r0x/asm-offsets.s: 
No such file or directory
make: *** [/work/wd/tmp-arm/include/generated/asm-offsets.h] Error 1
arm-linux-size: '/work/wd/tmp-arm/u-boot': No such file
...

etc. etc.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Shakespeare's Law of Prototyping: (Hamlet III, iv, 156-160)
O, throw away the worser part of it,
And live the purer with the other half.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] i2c_no_probes: Fix initialization warnings

[Heiko Schocher]

Hello Sanjeev,

Sanjeev Premi wrote:
> This patch fixes the initialization warnings similar to
> the one below:
> cmd_i2c.o cmd_i2c.c -c
> cmd_i2c.c:109:1: warning: missing braces around initializer
> cmd_i2c.c:109:1: warning: (near initialization for 'i2c_no_probes[0]')
> 
> Signed-off-by: Sanjeev Premi 
> Cc: Albert ARIBAUD 
> ---
> 
>  The problem was originally found when compiling omap3_beagle[1].
>  All the changes were done with this bash script:
> 
>   [script_here]
>   list=search.lst
>   [ -f ${list} ] && \rm -rf ${list}
> 
>   grep CONFIG_SYS_I2C_NOPROBES ./include/configs/* | grep -v "{{" | cut -d":" 
> -f1 > ${list}
> 
>   for f in `cat ${list}` ;
>   do
> echo "Fixing $f..."
> sed -r -i 's/(CONFIG_SYS_I2C_NOPROBES\s*)\{{1}(.*)\}{1}/\1{{\2}}/g' "$f"
>   done
>   [/script_here]
> 
>   Compile tested for oma3_beagle only.

Compiling for example for the alpr board with your patch:

[hs@pollux u-boot]$ ./MAKEALL alpr
Configuring for alpr board...
cmd_i2c.c:115: warning: braces around scalar initializer
cmd_i2c.c:115: warning: (near initialization for 'i2c_no_probes[0]')
fpga.c:226: warning: initialization from incompatible pointer type
   textdata bss dec hex filename
 231746   18612   68328  318686   4dcde ./u-boot

- SUMMARY 
Boards compiled: 1
Boards with warnings or errors: 1 ( alpr )
--
[hs@pollux u-boot]$

-> so Sorry, NACK for this patch.

bye,
Heiko
-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/2] MX27: Update to autogenerated asm-offsets.h

[Wolfgang Denk]

Dear Stefano Babic,

In message <4e64cee5.8050...@denx.de> you wrote:
>
> > This commit breaks out-of-tree building of all i.MX27 boards:
> 
> It was not exactly this one, but yes, out-of-tree building was broken
> adding the asm-offsets.h.

Well, that's what git bisect says...

> I sent a patch to fix this issue:
> 
> http://patchwork.ozlabs.org/patch/112374/

Will check.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
God made machine language; all the rest is the work of man.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] omap3: beagle: Fix build warning

[Heiko Schocher]

Hello Sanjeev,

Premi, Sanjeev wrote:
>> -Original Message-
>> From: Albert ARIBAUD [mailto:albert.u.b...@aribaud.net] 
>> Sent: Monday, September 05, 2011 5:06 PM
>> To: Premi, Sanjeev
>> Cc: u-boot@lists.denx.de; Heiko Schocher; Wolfgang Denk
>> Subject: Re: [U-Boot] [PATCH] omap3: beagle: Fix build warning
>>
>> Hi Sanjeev,
>>
>> Le 05/09/2011 12:25, Sanjeev Premi a écrit :
>>> This patch fixes the warning dure to recent changes to the board
>>> configuration:
>>> cmd_i2c.o cmd_i2c.c -c
>>> cmd_i2c.c:109:1: warning: missing braces around initializer
>>> cmd_i2c.c:109:1: warning: (near initialization for 
>> 'i2c_no_probes[0]')
>>> Signed-off-by: Sanjeev Premi
>>> Cc: Jason Kridner
>>> ---
>>>
>>>   Patch was compile tested against the latest u-boot-arm.git at:
>>>   58c583b : net: Check network device driver name
>>>
>>>   include/configs/omap3_beagle.h |2 +-
>>>   1 files changed, 1 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/include/configs/omap3_beagle.h 
>> b/include/configs/omap3_beagle.h
>>> index 18c6deb..a891246 100644
>>> --- a/include/configs/omap3_beagle.h
>>> +++ b/include/configs/omap3_beagle.h
>>> @@ -118,7 +118,7 @@
>>>   #define CONFIG_I2C_MULTI_BUS  1
>>>
>>>   /* Probe all devices */
>>> -#define CONFIG_SYS_I2C_NOPROBES{0x0, 0x0}
>>> +#define CONFIG_SYS_I2C_NOPROBES{{0x0, 0x0}}
>>>
>>>   /* USB */
>>>   #define CONFIG_MUSB_UDC   1
>> Won't all board configs which use CONFIG_SYS_I2C_NOPROBES suffer from 
>> the same bug? I would hate to see an endless trickle of 
>> individual board 
>> config patches, and would much prefer a single patch to fix 
>> all boards 
> 
> [sp] Actually, not all boards suffer from this issue.
>  But yes, I can make a single patch and submit it
>  in next couple of hours.

Isn;t this issue introduced just from commit

author  Jason Kridner 
 Sat, 23 Jul 2011 04:42:44 + (23:42 -0500)
committer   Albert ARIBAUD 
 Sun, 4 Sep 2011 09:36:21 + (11:36 +0200)
commit  f74fc4ae6d6257ecdbc0049f6aa2e96212207f05

so ~1day old ... Hmm, I think, this is just a single board bugfix,
as this warning only raises, if CONFIG_I2C_MULTI_BUS and
CONFIG_SYS_I2C_NOPROBES is defined ... as introduced for the beagle
board through above commit ... other boards should be clean, as I tend
to do a MAKEALL after applying patches from ML ... but if you find
time and can check this, it would be nice!

bye,
Heiko
-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/2] MX27: Update to autogenerated asm-offsets.h

[Stefano Babic]

On 09/05/2011 03:13 PM, Wolfgang Denk wrote:
> Dear Stefano Babic,
> 
> In message <1310045827-14561-2-git-send-email-sba...@denx.de> you wrote:
>> On i.MX27, the asm-offsets.h file is not yet generated as it should be.
>>
>> Signed-off-by: Stefano Babic 
>> CC: Matthias Weisser 
> 
> This commit breaks out-of-tree building of all i.MX27 boards:
> 

It was not exactly this one, but yes, out-of-tree building was broken
adding the asm-offsets.h.

I sent a patch to fix this issue:

http://patchwork.ozlabs.org/patch/112374/

commit 0edf8b5b2fa0d210ebc4d6da0fd1aceeb7e44e47 breaks
building on a different directory with the O= parameter.
The patch will fix this issue, generating always asm-offsets.h before
the other targets.

Signed-off-by: Stefano Babic 
CC: Matthias Weisser 
CC: Wolfgang Denk 

This Patch should be reviewed. I have tested it with all architectures
(V2 of my patch broke PowerPc boards !), and I am quite sure it works -
not sure the solution I proposed is the best.

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 2/2] coldfire: Fix compilation with CONFIG_SYS_DRAMSZ1 defined

[Stany MARCEL]

A temp variable was used but not declared, with CONFIG_SYS_DRAMSZ1
defined. This variable is now declared in the functione when needed.

Signed-off-by: Stany MARCEL 
---
 board/freescale/m548xevb/m548xevb.c |3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/board/freescale/m548xevb/m548xevb.c 
b/board/freescale/m548xevb/m548xevb.c
index 4a2a5c7..fbc0888 100644
--- a/board/freescale/m548xevb/m548xevb.c
+++ b/board/freescale/m548xevb/m548xevb.c
@@ -43,6 +43,9 @@ phys_size_t initdram(int board_type)
volatile siu_t *siu = (siu_t *) (MMAP_SIU);
volatile sdram_t *sdram = (volatile sdram_t *)(MMAP_SDRAM);
u32 dramsize, i;
+#ifdef CONFIG_SYS_DRAMSZ1
+   u32 temp;
+#endif
 
siu->drv = CONFIG_SYS_SDRAM_DRVSTRENGTH;
 
-- 
1.7.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 1/2] coldfire: Fix compilation for most coldfire boards

[Stany MARCEL]

Many different patches to fix compilation of coldfire boards.
Compilation could now be done in a different directory from the
sources one. Simple board config switched from Makefile to boards.cfg
EP2500 broken board removed from boards.cfg. Link files corrected to
remove zlib.o and multiple defined symbols.

Signed-off-by: Stany MARCEL 
---
 MAKEALL  |6 --
 Makefile |  124 +-
 arch/m68k/cpu/mcf5227x/Makefile  |2 +-
 arch/m68k/cpu/mcf523x/Makefile   |2 +-
 arch/m68k/cpu/mcf532x/Makefile   |2 +-
 arch/m68k/cpu/mcf5445x/Makefile  |2 +-
 arch/m68k/cpu/mcf547x_8x/Makefile|2 +-
 board/BuS/EB+MCF-EV123/u-boot.lds|8 --
 board/cobra5272/u-boot.lds   |7 --
 board/esd/tasreg/u-boot.lds  |7 --
 board/freescale/m52277evb/.gitignore |1 +
 board/freescale/m5235evb/.gitignore  |1 +
 board/freescale/m5235evb/u-boot.16   |8 --
 board/freescale/m5235evb/u-boot.32   |   16 -
 board/freescale/m5249evb/u-boot.lds  |7 --
 board/freescale/m5253evbe/u-boot.lds |7 --
 board/freescale/m5272c3/u-boot.lds   |7 --
 board/freescale/m5275evb/u-boot.lds  |6 --
 board/freescale/m54451evb/.gitignore |1 +
 board/freescale/m54455evb/.gitignore |1 +
 board/idmr/u-boot.lds|7 --
 boards.cfg   |   21 ++-
 include/configs/M5329EVB.h   |8 +-
 23 files changed, 50 insertions(+), 203 deletions(-)
 create mode 100644 board/freescale/m52277evb/.gitignore
 create mode 100644 board/freescale/m5235evb/.gitignore
 create mode 100644 board/freescale/m54451evb/.gitignore
 create mode 100644 board/freescale/m54455evb/.gitignore

diff --git a/MAKEALL b/MAKEALL
index e72a019..2cb5b38 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -495,18 +495,12 @@ LIST_microblaze="$(boards_by_arch microblaze)"
 #
 
 LIST_coldfire="$(boards_by_arch m68k)
-   astro_mcf5373l  \
-   cobra5272   \
EB+MCF-EV123\
EB+MCF-EV123_internal   \
M52277EVB   \
M5235EVB\
-   M5329AFEE   \
-   M5373EVB\
M54451EVB   \
M54455EVB   \
-   M5475AFE\
-   M5485AFE\
 "
 
 #
diff --git a/Makefile b/Makefile
index d5a1f0a..39818f4 100644
--- a/Makefile
+++ b/Makefile
@@ -591,14 +591,11 @@ ucname= $(shell echo $(1) | sed -e 
's/\(.*\)_config/\U\1/')
 #
 ## Coldfire
 #
-
-astro_mcf5373l_config \
-astro_mcf5373l_RAM_config :unconfig
-   @$(MKCONFIG) -n $@ -t $@ astro_mcf5373l m68k mcf532x mcf5373l astro
-
 M52277EVB_config \
 M52277EVB_spansion_config \
 M52277EVB_stmicro_config : unconfig
+   @mkdir -p $(obj)include
+   @mkdir -p $(obj)board/freescale/m52277evb
@case "$@" in \
M52277EVB_config)   FLASH=SPANSION;; \
M52277EVB_spansion_config)  FLASH=SPANSION;; \
@@ -607,19 +604,21 @@ M52277EVB_stmicro_config :unconfig
if [ "$${FLASH}" = "SPANSION" ] ; then \
echo "#define CONFIG_SYS_SPANSION_BOOT" >> 
$(obj)include/config.h ; \
echo "CONFIG_SYS_TEXT_BASE = 0x" > 
$(obj)board/freescale/m52277evb/config.tmp ; \
-   cp $(obj)board/freescale/m52277evb/u-boot.spa 
$(obj)board/freescale/m52277evb/u-boot.lds ; \
+   cp board/freescale/m52277evb/u-boot.spa 
board/freescale/m52277evb/u-boot.lds ; \
fi; \
if [ "$${FLASH}" = "STMICRO" ] ; then \
echo "#define CONFIG_CF_SBF">> $(obj)include/config.h ; \
echo "#define CONFIG_SYS_STMICRO_BOOT"  >> 
$(obj)include/config.h ; \
echo "CONFIG_SYS_TEXT_BASE = 0x43E0" > 
$(obj)board/freescale/m52277evb/config.tmp ; \
-   cp $(obj)board/freescale/m52277evb/u-boot.stm 
$(obj)board/freescale/m52277evb/u-boot.lds ; \
+   cp board/freescale/m52277evb/u-boot.stm 
board/freescale/m52277evb/u-boot.lds ; \
fi
@$(MKCONFIG) -n $@ -a M52277EVB m68k mcf5227x m52277evb freescale
 
 M5235EVB_config \
 M5235EVB_Flash16_config \
 M5235EVB_Flash32_config:   unconfig
+   @mkdir -p $(obj)include
+   @mkdir -p $(obj)board/freescale/m5235evb
@case "$@" in \
M5235EVB_config)FLASH=16;; \
M5235EVB_Flash16_config)FLASH=16;; \
@@ -628,63 +627,40 @@ M5235EVB_Flash32_config:  unconfig
if [ "$${FLASH}" != "16" ] ; then \
echo "#define NORFLASH_PS32BIT  1" >> $(obj)include/config.h ; \
echo "CONFIG_SYS_TEXT_BASE =

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

[Fabio Estevam]

On Mon, Sep 5, 2011 at 10:05 AM, Marek Vasut  wrote:
...
>> extern void mx25_fec_init_pins(void);
>> extern void imx_get_mac_from_fuse(unsigned char *mac);
>> extern void mx25_uart1_init_pins(void);
>
> Ok, this might be a stupid one, but ...
>
> why use externs in header files ?

This will make checkpatch happy :-)

Regards,

Fabio Estevam
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/2] MX27: Update to autogenerated asm-offsets.h

[Wolfgang Denk]

Dear Stefano Babic,

In message <1310045827-14561-2-git-send-email-sba...@denx.de> you wrote:
> On i.MX27, the asm-offsets.h file is not yet generated as it should be.
> 
> Signed-off-by: Stefano Babic 
> CC: Matthias Weisser 

This commit breaks out-of-tree building of all i.MX27 boards:

MAKEALL_LOGDIR=/work/wd/tmp-arm-LOG \
BUILD_DIR=/work/wd/tmp-arm \
./MAKEALL imx27lite
Configuring for imx27lite board...
make[1]: *** No rule to make target 
`/home/wd/git/u-boot/work/include/autoconf.mk.dep', needed by
`/home/wd/git/u-boot/work/include/asm/arch/asm-offsets.h'.  Stop.
make: *** [/work/wd/tmp-arm/arch/arm/cpu/arm926ejs/mx27/libmx27.o] Error 2
make: *** Waiting for unfinished jobs
arm-linux-size: '/work/wd/tmp-arm/u-boot': No such file


Can you please have a look?

Thanks.

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Alles Gescheite ist schon gedacht worden, man muß nur versuchen,
es noch einmal zu denken.  -- Goethe, Maximen und Reflexionen
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] UBIFS: Change ubifsload to set the filesize variable

[Marek Vasut]

On Monday, September 05, 2011 01:03:49 PM Bastian Ruppert wrote:
> This is the same behaviour like tftp or fatload command.
> 
> Signed-off-by: Bastian Ruppert 
> CC: kmp...@infradead.org
> ---
>  fs/ubifs/ubifs.c |4 
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/fs/ubifs/ubifs.c b/fs/ubifs/ubifs.c
> index 5a5c739..2b9d2d2 100644
> --- a/fs/ubifs/ubifs.c
> +++ b/fs/ubifs/ubifs.c
> @@ -688,6 +688,7 @@ int ubifs_load(char *filename, u32 addr, u32 size)
>   int i;
>   int count;
>   int last_block_size = 0;
> + char buf [10];
> 
>   c->ubi = ubi_open_volume(c->vi.ubi_num, c->vi.vol_id, UBI_READONLY);
>   /* ubifs_findfile will resolve symlinks, so we know that we get
> @@ -719,6 +720,9 @@ int ubifs_load(char *filename, u32 addr, u32 size)
>   printf("Loading file '%s' to addr 0x%08x with size %d (0x%08x)...\n",
>  filename, addr, size, size);
> 
> + sprintf(buf, "%lX", size);
> + setenv("filesize", buf);
> +

Hi,

maybe you want to check set_local_var() too ?

>   page.addr = (void *)addr;
>   page.index = 0;
>   page.inode = inode;
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 1/2] mx31: make HSP clock for mx3fb driver available

[Marek Vasut]

On Monday, September 05, 2011 01:47:06 PM Helmut Raiger wrote:
> This additionally updates mx31/generic.c by
> - replacing __REG() macro accesses with readl() and writel()
> - providing macros for PDR0 and PLL bit accesses
> It also fixes a warning about the prototype of imx_get_uartclk(void)
> 
> Signed-off-by: Helmut Raiger 
> ---
> V2: uses macros and readl(), writel(), see commit message

This looks awesome. Please, when submitting V2, also add V2 to [PATCH 1/2] ... 
like [PATCH V2 1/2]. Minor thing though.

Acked-by: Marek Vasut 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 1/2] MX25: tx25: Avoid the usage of extern in C file

[Marek Vasut]

On Monday, September 05, 2011 01:37:17 PM Fabio Estevam wrote:
> On Mon, Sep 5, 2011 at 7:43 AM, Stefano Babic  wrote:
> ...
> 
> > What about to move the prototype ? For MX5/MX35 there is a sys_proto.h
> > (as it is done for other SOCs), in this case, well, it seems too much to
> > add a file for a single line. However, we could move it in clock.h and
> > creating a sys_proto.h if the number of exported functions will increase.
> 
> Ok, I can create a sys_proto.h for MX25 and put the following extern´s
> there:
> 
> extern void mx25_fec_init_pins(void);
> extern void imx_get_mac_from_fuse(unsigned char *mac);
> extern void mx25_uart1_init_pins(void);

Ok, this might be a stupid one, but ...

why use externs in header files ?

> 
> Regards,
> 
> Fabio Estevam
> ___
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH v2] UBIFS: Change ubifsload to set the filesize variable

[Bastian Ruppert]

This is the same behaviour like tftp or fatload command.

Signed-off-by: Bastian Ruppert 
CC: kmp...@infradead.org
---
 fs/ubifs/ubifs.c |6 +-
 1 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/fs/ubifs/ubifs.c b/fs/ubifs/ubifs.c
index 5a5c739..75a2edc 100644
--- a/fs/ubifs/ubifs.c
+++ b/fs/ubifs/ubifs.c
@@ -688,6 +688,7 @@ int ubifs_load(char *filename, u32 addr, u32 size)
int i;
int count;
int last_block_size = 0;
+   char buf [10];
 
c->ubi = ubi_open_volume(c->vi.ubi_num, c->vi.vol_id, UBI_READONLY);
/* ubifs_findfile will resolve symlinks, so we know that we get
@@ -739,8 +740,11 @@ int ubifs_load(char *filename, u32 addr, u32 size)
 
if (err)
printf("Error reading file '%s'\n", filename);
-   else
+   else {
+   sprintf(buf, "%lX", size);
+   setenv("filesize", buf);
printf("Done\n");
+   }
 
ubifs_iput(inode);
 
-- 
1.6.3.3

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] Aviso Urgente!

[Security Alert]

Estimado usuario de la cuenta de correo Web,

   Aviso Urgente!

Este mensaje es el mensaje de correo web del centro a todos los propietarios de 
las cuentas de correo web. Debido a la constante de velocidad de spam 
actualmente estamos realizando tareas de mantenimiento y la intensificación de 
nuestros servicios digitales de correo web para su conveniencia.

Estamos eliminando todas las cuentas de correo web no se utilizan para crear 
más espacio para las nuevas cuentas. Para evitar que su cuenta se cierre tendrá 
que poner al día siguiente para saber que es siempre una cuenta en la 
actualidad se utiliza con un duro protector de spam.

Correo Web confirma los detalles;
> Nombre de usuario:
> Contraseña:
> Fecha de nacimiento:

Se le enviará una confirmación de la contraseña alfanumérica nuevo para que 
sólo es válido durante este período y se puede cambiar después del proceso.

¡Advertencia! Cualquier propietario de la cuenta que se niega a actualizar su 
cuenta dentro de los siete días hábiles siguientes a la notificación de 
actualización perderá su cuenta permanentemente y que han dado consejos que 
usted envíe sus datos a nuestro buzón de correo a través de 
securityaler...@live.com

Gracias por su tiempo en la oficina!
Cuenta de correo electrónico del equipo de apoyo.
Código de advertencia: ID67565434.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 10/10] ARM: remove broken "shannon" board.

[Wolfgang Denk]

Signed-off-by: Wolfgang Denk 
Cc: Albert ARIBAUD 
Cc: Rolf Offermanns 
---
 MAINTAINERS   |4 -
 board/shannon/Makefile|   51 -
 board/shannon/config.mk   |   23 --
 board/shannon/flash.c |  475 -
 board/shannon/inferno.header  |  Bin 128 -> 0 bytes
 board/shannon/lowlevel_init.S |   92 
 board/shannon/shannon.c   |   99 -
 boards.cfg|1 -
 doc/README.scrapyard  |1 +
 include/configs/shannon.h |  223 ---
 10 files changed, 1 insertions(+), 968 deletions(-)
 delete mode 100644 board/shannon/Makefile
 delete mode 100644 board/shannon/config.mk
 delete mode 100644 board/shannon/flash.c
 delete mode 100644 board/shannon/inferno.header
 delete mode 100644 board/shannon/lowlevel_init.S
 delete mode 100644 board/shannon/shannon.c
 delete mode 100644 include/configs/shannon.h

diff --git a/MAINTAINERS b/MAINTAINERS
index 2d785fa..2f60a60 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -746,10 +746,6 @@ Nagendra T S  
 
am3517_craneARM ARMV7 (AM35x SoC)
 
-Rolf Offermanns 
-
-   shannon SA1100
-
 Kyungmin Park 
 
apollon ARM1136EJS
diff --git a/board/shannon/Makefile b/board/shannon/Makefile
deleted file mode 100644
index 23ac987..000
--- a/board/shannon/Makefile
+++ /dev/null
@@ -1,51 +0,0 @@
-#
-# (C) Copyright 2000-2006
-# Wolfgang Denk, DENX Software Engineering, w...@denx.de.
-#
-# See file CREDITS for list of people who contributed to this
-# project.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License as
-# published by the Free Software Foundation; either version 2 of
-# the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
-# MA 02111-1307 USA
-#
-
-include $(TOPDIR)/config.mk
-
-LIB= $(obj)lib$(BOARD).o
-
-COBJS  := shannon.o flash.o
-SOBJS  := lowlevel_init.o
-
-SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)
-OBJS   := $(addprefix $(obj),$(COBJS))
-SOBJS  := $(addprefix $(obj),$(SOBJS))
-
-$(LIB):$(obj).depend $(OBJS) $(SOBJS)
-   $(call cmd_link_o_target, $(OBJS) $(SOBJS))
-
-clean:
-   rm -f $(SOBJS) $(OBJS)
-
-distclean: clean
-   rm -f $(LIB) core *.bak $(obj).depend
-
-#
-
-# defines $(obj).depend target
-include $(SRCTREE)/rules.mk
-
-sinclude $(obj).depend
-
-#
diff --git a/board/shannon/config.mk b/board/shannon/config.mk
deleted file mode 100644
index 6afa2d3..000
--- a/board/shannon/config.mk
+++ /dev/null
@@ -1,23 +0,0 @@
-#
-# LART board with SA1100 cpu
-#
-# see http://www.lart.tudelft.nl/ for more information on LART
-#
-
-#
-# Tuxscreen has 4 banks of 4 MB DRAM each
-#
-# c000'
-# c800'
-# d000'
-# d800'
-#
-# Linux-Kernel is expected to be at c000'8000, entry c000'8000
-#
-# we load ourself to d838', the upper 1 MB of the last (4th) bank
-#
-# download areas is c800'
-#
-
-
-CONFIG_SYS_TEXT_BASE = 0xd838
diff --git a/board/shannon/flash.c b/board/shannon/flash.c
deleted file mode 100644
index 179ec5f..000
--- a/board/shannon/flash.c
+++ /dev/null
@@ -1,475 +0,0 @@
-/*
- * (C) Copyright 2002
- * Sysgo Real-Time Solutions, GmbH 
- * Alex Zuepke 
- *
- * See file CREDITS for list of people who contributed to this
- * project.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
- * MA 02111-1307 USA
- */
-
-#include 
-
-ulong myflush(void);
-
-
-#define FLASH_BANK_SIZE 0x40   /* 4 MB */
-#define MAIN_SECT_SIZE  0x2/* 128 KB */
-
-flash_info_tflash_info[CONFIG_SYS_MAX_FLASH_BANKS];
-
-
-#define CMD_READ_ARRAY 0x00F000F0
-#define CMD_UNLOCK10x00AA00AA
-#define CMD_UNLOCK20x00550055
-#defin

[U-Boot] [PATCH 06/10] ARM: remove broken "impa7" board.

[Wolfgang Denk]

Signed-off-by: Wolfgang Denk 
Cc: Albert ARIBAUD 
Cc: Marius Gröger 
---
 MAINTAINERS  |4 -
 MAKEALL  |1 -
 arch/arm/cpu/arm720t/cpu.c   |   29 +--
 arch/arm/cpu/arm720t/interrupts.c|   14 +-
 arch/arm/cpu/arm720t/start.S |   64 +-
 arch/arm/include/asm/arch-arm720t/hardware.h |2 -
 board/impa7/Makefile |   51 
 board/impa7/config.mk|   28 --
 board/impa7/flash.c  |  359 --
 board/impa7/impa7.c  |   71 -
 board/impa7/lowlevel_init.S  |   85 --
 boards.cfg   |1 -
 doc/README.scrapyard |1 +
 include/configs/impa7.h  |  174 -
 14 files changed, 6 insertions(+), 878 deletions(-)
 delete mode 100644 board/impa7/Makefile
 delete mode 100644 board/impa7/config.mk
 delete mode 100644 board/impa7/flash.c
 delete mode 100644 board/impa7/impa7.c
 delete mode 100644 board/impa7/lowlevel_init.S
 delete mode 100644 include/configs/impa7.h

diff --git a/MAINTAINERS b/MAINTAINERS
index b001f64..ca31239 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -657,10 +657,6 @@ Simon Guinot 
netspace_v2 ARM926EJS (Kirkwood SoC)
netspace_max_v2 ARM926EJS (Kirkwood SoC)
 
-Marius Gröger 
-
-   impa7   ARM720T (EP7211)
-
 Igor Grinberg 
 
cm-t35  ARM ARMV7 (OMAP3xx Soc)
diff --git a/MAKEALL b/MAKEALL
index 2876a26..350a524 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -301,7 +301,6 @@ LIST_SA="$(boards_by_cpu sa1100)"
 #
 
 LIST_ARM7="\
-   impa7   \
lpc2292sodimm   \
modnet50\
 "
diff --git a/arch/arm/cpu/arm720t/cpu.c b/arch/arm/cpu/arm720t/cpu.c
index 71d8d6b..974f288 100644
--- a/arch/arm/cpu/arm720t/cpu.c
+++ b/arch/arm/cpu/arm720t/cpu.c
@@ -36,10 +36,6 @@
 #include 
 #include 
 
-#if defined(CONFIG_IMPA7)
-static void cache_flush(void);
-#endif
-
 int cleanup_before_linux (void)
 {
/*
@@ -50,20 +46,7 @@ int cleanup_before_linux (void)
 * and we set the CPU-speed to 73 MHz - see start.S for details
 */
 
-#if defined(CONFIG_IMPA7)
-   disable_interrupts ();
-
-   /* turn off I-cache */
-   icache_disable();
-   dcache_disable();
-
-   /* flush I-cache */
-   cache_flush();
-#ifdef CONFIG_ARM7_REVD
-   /* go to high speed */
-   IO_SYSCON3 = (IO_SYSCON3 & ~CLKCTL) | CLKCTL_73;
-#endif
-#elif defined(CONFIG_NETARM) || defined(CONFIG_S3C4510B) || 
defined(CONFIG_LPC2292)
+#if defined(CONFIG_NETARM) || defined(CONFIG_S3C4510B) || 
defined(CONFIG_LPC2292)
disable_interrupts ();
/* Nothing more needed */
 #elif defined(CONFIG_INTEGRATOR) && defined(CONFIG_ARCH_INTEGRATOR)
@@ -73,13 +56,3 @@ int cleanup_before_linux (void)
 #endif
return 0;
 }
-
-#if defined(CONFIG_IMPA7)
-/* flush I/D-cache */
-static void cache_flush (void)
-{
-   unsigned long i = 0;
-
-   asm ("mcr p15, 0, %0, c7, c5, 0": :"r" (i));
-}
-#endif
diff --git a/arch/arm/cpu/arm720t/interrupts.c 
b/arch/arm/cpu/arm720t/interrupts.c
index 3cebf11..464dd30 100644
--- a/arch/arm/cpu/arm720t/interrupts.c
+++ b/arch/arm/cpu/arm720t/interrupts.c
@@ -149,18 +149,6 @@ int timer_init (void)
 
/* set timer 2 counter */
lastdec = TIMER_LOAD_VAL;
-#elif defined(CONFIG_IMPA7)
-   /* disable all interrupts */
-   IO_INTMR1 = 0;
-
-   /* operate timer 1 in prescale mode */
-   IO_SYSCON1 |= SYSCON1_TC1M;
-
-   /* select 2kHz clock source for timer 1 */
-   IO_SYSCON1 &= ~SYSCON1_TC1S;
-
-   /* set timer 1 counter */
-   lastdec = IO_TC1D = TIMER_LOAD_VAL;
 #elif defined(CONFIG_S3C4510B)
/* configure free running timer 0 */
PUT_REG( REG_TMOD, 0x0);
@@ -207,7 +195,7 @@ int timer_init (void)
  */
 
 
-#if defined(CONFIG_IMPA7) || defined(CONFIG_NETARM) || defined(CONFIG_LPC2292)
+#if defined(CONFIG_NETARM) || defined(CONFIG_LPC2292)
 
 ulong get_timer (ulong base)
 {
diff --git a/arch/arm/cpu/arm720t/start.S b/arch/arm/cpu/arm720t/start.S
index f2320ee..00fa8c9 100644
--- a/arch/arm/cpu/arm720t/start.S
+++ b/arch/arm/cpu/arm720t/start.S
@@ -272,25 +272,7 @@ _dynsym_start_ofs:
  *
  */
 
-#if defined(CONFIG_IMPA7)
-
-/* Interrupt-Controller base addresses */
-INTMR1:.word   0x8280 @ 32 bit size
-INTMR2:.word   0x80001280 @ 16 bit size
-INTMR3:.word   0x80002280 @  8 bit size
-
-/* SYSCONs */
-SYSCON1:   .word   0x8100
-SYSCON2:   .word   0x80001100
-SYSCON3:   .word   0x80002200
-
-#define CLKCTL0x6  /* mask */
-#define CLKCTL_18  0x0  /* 18.432 MHz */
-#define CLKCTL_36   

[U-Boot] [PATCH 07/10] ARM: remove broken "lart" board.

[Wolfgang Denk]

Signed-off-by: Wolfgang Denk 
Cc: Albert ARIBAUD 
Cc: Alex Züpke 
---
 MAINTAINERS|4 -
 board/lart/Makefile|   51 -
 board/lart/config.mk   |   23 --
 board/lart/flash.c |  476 
 board/lart/flashasm.S  |  177 
 board/lart/lart.c  |   76 ---
 board/lart/lowlevel_init.S |   94 -
 boards.cfg |1 -
 doc/README.scrapyard   |1 +
 include/configs/lart.h |  160 ---
 10 files changed, 1 insertions(+), 1062 deletions(-)
 delete mode 100644 board/lart/Makefile
 delete mode 100644 board/lart/config.mk
 delete mode 100644 board/lart/flash.c
 delete mode 100644 board/lart/flashasm.S
 delete mode 100644 board/lart/lart.c
 delete mode 100644 board/lart/lowlevel_init.S
 delete mode 100644 include/configs/lart.h

diff --git a/MAINTAINERS b/MAINTAINERS
index ca31239..d52e6f0 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -891,10 +891,6 @@ Richard Woodruff 
 
omap2420h4  ARM1136EJS
 
-Alex Züpke 
-
-   lartSA1100
-
 Syed Mohammed Khasim 
 Sughosh Ganu 
 
diff --git a/board/lart/Makefile b/board/lart/Makefile
deleted file mode 100644
index 463bc0b..000
--- a/board/lart/Makefile
+++ /dev/null
@@ -1,51 +0,0 @@
-#
-# (C) Copyright 2000-2006
-# Wolfgang Denk, DENX Software Engineering, w...@denx.de.
-#
-# See file CREDITS for list of people who contributed to this
-# project.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License as
-# published by the Free Software Foundation; either version 2 of
-# the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
-# MA 02111-1307 USA
-#
-
-include $(TOPDIR)/config.mk
-
-LIB= $(obj)lib$(BOARD).o
-
-COBJS  := lart.o flash.o
-SOBJS  := flashasm.o lowlevel_init.o
-
-SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)
-OBJS   := $(addprefix $(obj),$(COBJS))
-SOBJS  := $(addprefix $(obj),$(SOBJS))
-
-$(LIB):$(obj).depend $(OBJS) $(SOBJS)
-   $(call cmd_link_o_target, $(OBJS) $(SOBJS))
-
-clean:
-   rm -f $(SOBJS) $(OBJS)
-
-distclean: clean
-   rm -f $(LIB) core *.bak $(obj).depend
-
-#
-
-# defines $(obj).depend target
-include $(SRCTREE)/rules.mk
-
-sinclude $(obj).depend
-
-#
diff --git a/board/lart/config.mk b/board/lart/config.mk
deleted file mode 100644
index b6b5e4d..000
--- a/board/lart/config.mk
+++ /dev/null
@@ -1,23 +0,0 @@
-#
-# LART board with SA1100 cpu
-#
-# see http://www.lart.tudelft.nl/ for more information on LART
-#
-
-#
-# LART has 4 banks of 8 MB DRAM
-#
-# c000'
-# c100'
-# c800'
-# c900'
-#
-# Linux-Kernel is expected to be at c000'8000, entry c000'8000
-#
-# we load ourself to c178', the upper 1 MB of second bank
-#
-# download areas is c800'
-#
-
-
-CONFIG_SYS_TEXT_BASE = 0xc178
diff --git a/board/lart/flash.c b/board/lart/flash.c
deleted file mode 100644
index 408c884..000
--- a/board/lart/flash.c
+++ /dev/null
@@ -1,476 +0,0 @@
-/*
- * (C) Copyright 2002
- * Sysgo Real-Time Solutions, GmbH 
- * Marius Groeger 
- *
- * See file CREDITS for list of people who contributed to this
- * project.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
- * MA 02111-1307 USA
- */
-
-#include 
-
-ulong myflush(void);
-
-
-#define FLASH_BANK_SIZE 0x80
-#define MAIN_SECT_SIZE  0x2
-#define PARAM_SECT_SIZE 0x4000
-
-/* puzzle magic for lart
- * data_*_flash are def'd in flashasm.S
- */
-
-extern u32 data_from_flash(u32);
-extern u32 data_to_flash(u32);
-
-#define PUZZLE_FROM_FLASH(x)   data_from_flash((x))
-#define PUZZLE_TO_FLASH(x) data_to_flash((x))
-
-flash_info_tflash_info[CONFIG_SYS_MAX_FLASH_BANKS];
-
-
-#define CMD_READ_A

[U-Boot] [PATCH 04/10] ARM: remove broken "evb4510" board.

[Wolfgang Denk]

Signed-off-by: Wolfgang Denk 
Cc: Albert ARIBAUD 
Cc: Curt Brune 
---
 CREDITS   |1 -
 MAKEALL   |1 -
 board/evb4510/Makefile|   51 
 board/evb4510/config.mk   |   27 --
 board/evb4510/evb4510.c   |   64 -
 board/evb4510/flash.c |  539 -
 board/evb4510/lowlevel_init.S |  157 
 boards.cfg|1 -
 common/image.c|   11 -
 doc/README.scrapyard  |1 +
 include/configs/evb4510.h |  180 --
 11 files changed, 1 insertions(+), 1032 deletions(-)
 delete mode 100644 board/evb4510/Makefile
 delete mode 100644 board/evb4510/config.mk
 delete mode 100644 board/evb4510/evb4510.c
 delete mode 100644 board/evb4510/flash.c
 delete mode 100644 board/evb4510/lowlevel_init.S
 delete mode 100644 include/configs/evb4510.h

diff --git a/CREDITS b/CREDITS
index 34075a0..487a55e 100644
--- a/CREDITS
+++ b/CREDITS
@@ -82,7 +82,6 @@ D: Port to the gw8260 board
 N: Curt Brune
 E: c...@cucy.com
 D: Added support for Samsung S3C4510B CPU (ARM7tdmi based SoC)
-D: Added support for ESPD-Inc. EVB4510 Board
 W: http://www.cucy.com
 
 N: Jonathan De Bruyne
diff --git a/MAKEALL b/MAKEALL
index 40a1ecb..2876a26 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -301,7 +301,6 @@ LIST_SA="$(boards_by_cpu sa1100)"
 #
 
 LIST_ARM7="\
-   evb4510 \
impa7   \
lpc2292sodimm   \
modnet50\
diff --git a/board/evb4510/Makefile b/board/evb4510/Makefile
deleted file mode 100644
index 102eade..000
--- a/board/evb4510/Makefile
+++ /dev/null
@@ -1,51 +0,0 @@
-#
-# (C) Copyright 2000-2006
-# Wolfgang Denk, DENX Software Engineering, w...@denx.de.
-#
-# See file CREDITS for list of people who contributed to this
-# project.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License as
-# published by the Free Software Foundation; either version 2 of
-# the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
-# MA 02111-1307 USA
-#
-
-include $(TOPDIR)/config.mk
-
-LIB= $(obj)lib$(BOARD).o
-
-COBJS  := evb4510.o flash.o
-SOBJS  := lowlevel_init.o
-
-SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)
-OBJS   := $(addprefix $(obj),$(COBJS))
-SOBJS  := $(addprefix $(obj),$(SOBJS))
-
-$(LIB):$(obj).depend $(OBJS) $(SOBJS)
-   $(call cmd_link_o_target, $(OBJS) $(SOBJS))
-
-clean:
-   rm -f $(SOBJS) $(OBJS)
-
-distclean: clean
-   rm -f $(LIB) core *.bak $(obj).depend
-
-#
-
-# defines $(obj).depend target
-include $(SRCTREE)/rules.mk
-
-sinclude $(obj).depend
-
-#
diff --git a/board/evb4510/config.mk b/board/evb4510/config.mk
deleted file mode 100644
index 140c989..000
--- a/board/evb4510/config.mk
+++ /dev/null
@@ -1,27 +0,0 @@
-#
-# Copyright (c) 2004   Cucy Systems (http://www.cucy.com)
-# Curt Brune 
-#
-# (C) Copyright 2000-2004
-# Wolfgang Denk, DENX Software Engineering, w...@denx.de.
-#
-# See file CREDITS for list of people who contributed to this
-# project.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License as
-# published by the Free Software Foundation; either version 2 of
-# the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
-# MA 02111-1307 USA
-#
-
-CONFIG_SYS_TEXT_BASE = 0x007d
diff --git a/board/evb4510/evb4510.c b/board/evb4510/evb4510.c
deleted file mode 100644
index 13abbb7..000
--- a/board/evb4510/evb4510.c
+++ /dev/null
@@ -1,64 +0,0 @@
-/*
- * Copyright (c) 2004  Cucy Systems (http://www.cucy.com)
- * Curt Brune 
- *
- * See file CREDITS for list of people who contributed to this
- * project.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by th

[U-Boot] [PATCH 08/10] ARM: remove broken "lpc2292sodimm" board.

[Wolfgang Denk]

Signed-off-by: Wolfgang Denk 
Cc: Albert ARIBAUD 
---
 MAKEALL |1 -
 board/lpc2292sodimm/Makefile|   51 ---
 board/lpc2292sodimm/config.mk   |   30 
 board/lpc2292sodimm/flash.c |  252 ---
 board/lpc2292sodimm/lowlevel_init.S |   87 
 board/lpc2292sodimm/lpc2292sodimm.c |   57 
 boards.cfg  |1 -
 doc/README.scrapyard|1 +
 include/configs/lpc2292sodimm.h |  159 --
 9 files changed, 1 insertions(+), 638 deletions(-)
 delete mode 100644 board/lpc2292sodimm/Makefile
 delete mode 100644 board/lpc2292sodimm/config.mk
 delete mode 100644 board/lpc2292sodimm/flash.c
 delete mode 100644 board/lpc2292sodimm/lowlevel_init.S
 delete mode 100644 board/lpc2292sodimm/lpc2292sodimm.c
 delete mode 100644 include/configs/lpc2292sodimm.h

diff --git a/MAKEALL b/MAKEALL
index 350a524..c6e6d04 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -301,7 +301,6 @@ LIST_SA="$(boards_by_cpu sa1100)"
 #
 
 LIST_ARM7="\
-   lpc2292sodimm   \
modnet50\
 "
 
diff --git a/board/lpc2292sodimm/Makefile b/board/lpc2292sodimm/Makefile
deleted file mode 100644
index c1a5b0b..000
--- a/board/lpc2292sodimm/Makefile
+++ /dev/null
@@ -1,51 +0,0 @@
-#
-# (C) Copyright 2007
-# Wolfgang Denk, DENX Software Engineering, w...@denx.de.
-#
-# See file CREDITS for list of people who contributed to this
-# project.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License as
-# published by the Free Software Foundation; either version 2 of
-# the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
-# MA 02111-1307 USA
-#
-
-include $(TOPDIR)/config.mk
-
-LIB= $(obj)lib$(BOARD).o
-
-COBJS  := flash.o lpc2292sodimm.o
-SOBJTS := lowlevel_init.o
-
-SRCS   := $(SOBJTS:.o=.S) $(COBJS:.o=.c)
-OBJS   := $(addprefix $(obj),$(COBJS))
-SOBJS  := $(addprefix $(obj),$(SOBJTS))
-
-$(LIB):$(obj).depend $(OBJS) $(SOBJS)
-   $(call cmd_link_o_target, $(OBJS) $(SOBJS))
-
-clean:
-   rm -f $(SOBJS) $(OBJS)
-
-distclean: clean
-   rm -f $(LIB) core *.bak $(obj).depend
-
-#
-
-# defines $(obj).depend target
-include $(SRCTREE)/rules.mk
-
-sinclude $(obj).depend
-
-#
diff --git a/board/lpc2292sodimm/config.mk b/board/lpc2292sodimm/config.mk
deleted file mode 100644
index 4891792..000
--- a/board/lpc2292sodimm/config.mk
+++ /dev/null
@@ -1,30 +0,0 @@
-#
-# (C) Copyright 2000
-# Sysgo Real-Time Solutions, GmbH 
-# Marius Groeger 
-#
-# (C) Copyright 2000
-# Wolfgang Denk, DENX Software Engineering, w...@denx.de.
-#
-# See file CREDITS for list of people who contributed to this
-# project.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License as
-# published by the Free Software Foundation; either version 2 of
-# the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
-# MA 02111-1307 USA
-#
-
-#address where u-boot will be relocated
-#CONFIG_SYS_TEXT_BASE = 0x0
-CONFIG_SYS_TEXT_BASE = 0x8150
diff --git a/board/lpc2292sodimm/flash.c b/board/lpc2292sodimm/flash.c
deleted file mode 100644
index fd5389f..000
--- a/board/lpc2292sodimm/flash.c
+++ /dev/null
@@ -1,252 +0,0 @@
-/*
- * (C) Copyright 2006 Embedded Artists AB 
- *
- * Modified to use the routines in arch/arm/cpu/arm720t/lpc2292/flash.c by
- * Gary Jennejohn 
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABI

[U-Boot] [PATCH 02/10] ARM: remove broken "dnp1110" board.

[Wolfgang Denk]

Signed-off-by: Wolfgang Denk 
Cc: Albert ARIBAUD 
Cc: Alex Züpke 
---
 MAINTAINERS   |1 -
 board/dnp1110/Makefile|   51 -
 board/dnp1110/config.mk   |   17 --
 board/dnp1110/dnp1110.c   |   67 ---
 board/dnp1110/flash.c |  422 -
 board/dnp1110/lowlevel_init.S |  135 -
 boards.cfg|1 -
 doc/README.scrapyard  |1 +
 include/configs/dnp1110.h |  162 
 9 files changed, 1 insertions(+), 856 deletions(-)
 delete mode 100644 board/dnp1110/Makefile
 delete mode 100644 board/dnp1110/config.mk
 delete mode 100644 board/dnp1110/dnp1110.c
 delete mode 100644 board/dnp1110/flash.c
 delete mode 100644 board/dnp1110/lowlevel_init.S
 delete mode 100644 include/configs/dnp1110.h

diff --git a/MAINTAINERS b/MAINTAINERS
index 3bfe3fb..16edb67 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -903,7 +903,6 @@ Richard Woodruff 
 Alex Züpke 
 
lartSA1100
-   dnp1110 SA1110
 
 Syed Mohammed Khasim 
 Sughosh Ganu 
diff --git a/board/dnp1110/Makefile b/board/dnp1110/Makefile
deleted file mode 100644
index 8a86c0f..000
--- a/board/dnp1110/Makefile
+++ /dev/null
@@ -1,51 +0,0 @@
-#
-# (C) Copyright 2000-2006
-# Wolfgang Denk, DENX Software Engineering, w...@denx.de.
-#
-# See file CREDITS for list of people who contributed to this
-# project.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License as
-# published by the Free Software Foundation; either version 2 of
-# the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
-# MA 02111-1307 USA
-#
-
-include $(TOPDIR)/config.mk
-
-LIB= $(obj)lib$(BOARD).o
-
-COBJS  := dnp1110.o flash.o
-SOBJS  := lowlevel_init.o
-
-SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)
-OBJS   := $(addprefix $(obj),$(COBJS))
-SOBJS  := $(addprefix $(obj),$(SOBJS))
-
-$(LIB):$(obj).depend $(OBJS) $(SOBJS)
-   $(call cmd_link_o_target, $(OBJS) $(SOBJS))
-
-clean:
-   rm -f $(SOBJS) $(OBJS)
-
-distclean: clean
-   rm -f $(LIB) core *.bak $(obj).depend
-
-#
-
-# defines $(obj).depend target
-include $(SRCTREE)/rules.mk
-
-sinclude $(obj).depend
-
-#
diff --git a/board/dnp1110/config.mk b/board/dnp1110/config.mk
deleted file mode 100644
index ccf8277..000
--- a/board/dnp1110/config.mk
+++ /dev/null
@@ -1,17 +0,0 @@
-#
-# DNP/1110 board with SA1100 cpu
-#
-# http://www.dilnetpc.com
-#
-
-#
-# DILNETPC has 1 banks of 32 MB DRAM
-#
-# c000'
-#
-# Linux-Kernel is expected to be at c000'8000, entry c000'8000
-#
-# we load ourself to c1f8', the upper 1 MB of the first (only) bank
-#
-
-CONFIG_SYS_TEXT_BASE = 0xc1f8
diff --git a/board/dnp1110/dnp1110.c b/board/dnp1110/dnp1110.c
deleted file mode 100644
index c215f5f..000
--- a/board/dnp1110/dnp1110.c
+++ /dev/null
@@ -1,67 +0,0 @@
-/*
- * (C) Copyright 2002
- * Sysgo Real-Time Solutions, GmbH 
- * Marius Groeger 
- *
- * See file CREDITS for list of people who contributed to this
- * project.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
- * MA 02111-1307 USA
- */
-
-#include 
-#include 
-#include 
-
-DECLARE_GLOBAL_DATA_PTR;
-
-/*
- * Miscelaneous platform dependent initialisations
- */
-
-int board_init (void)
-{
-   /* memory and cpu-speed are setup before relocation */
-   /* so we do _nothing_ here */
-
-   /* arch number of DNP1110-Board */
-   gd->bd->bi_arch_number = MACH_TYPE_DNP1110;
-
-   /* flash vpp on */
-   PPDR |= 0x80;   /* assumes LCD controller is off */
-   PPSR |= 0x80;
-
-   return 0;
-}
-
-int dram_init (void)
-{
-   gd->bd->bi_dram[0].start = PHYS_SDRAM_1;
-   gd->bd->bi_dram[0].size = 

[U-Boot] [PATCH 00/10] ARM board removal, next round

[Wolfgang Denk]

So here we go - current master still (or again) has a number of
broken, non-compiling ARM boards.  Scrap them.

Signed-off-by: Wolfgang Denk 
Cc: Albert ARIBAUD 
Cc: Alex Züpke 
Cc: Curt Brune 
Cc: George G. Davis 
Cc: Marius Gröger 
Cc: Rolf Offermanns 
Cc: Thomas Elste 

Wolfgang Denk (10):
  ARM: remove broken "SMN42" board.
  ARM: remove broken "dnp1110" board.
  ARM: remove broken "ep7312" board.
  ARM: remove broken "evb4510" board.
  ARM: remove broken "gcplus" board.
  ARM: remove broken "impa7" board.
  ARM: remove broken "lart" board.
  ARM: remove broken "lpc2292sodimm" board.
  ARM: remove broken "modnet50" board.
  ARM: remove broken "shannon" board.

 CREDITS  |6 -
 MAINTAINERS  |   22 -
 MAKEALL  |   14 -
 arch/arm/cpu/arm720t/cpu.c   |   29 +--
 arch/arm/cpu/arm720t/interrupts.c|   14 +-
 arch/arm/cpu/arm720t/start.S |   64 +---
 arch/arm/include/asm/arch-arm720t/hardware.h |6 -
 board/dnp1110/Makefile   |   51 ---
 board/dnp1110/config.mk  |   17 -
 board/dnp1110/dnp1110.c  |   67 
 board/dnp1110/flash.c|  422 
 board/dnp1110/lowlevel_init.S|  135 ---
 board/ep7312/Makefile|   55 ---
 board/ep7312/config.mk   |   28 --
 board/ep7312/ep7312.c|   66 
 board/ep7312/flash.c |  343 
 board/ep7312/lowlevel_init.S |   95 -
 board/evb4510/Makefile   |   51 ---
 board/evb4510/config.mk  |   27 --
 board/evb4510/evb4510.c  |   64 ---
 board/evb4510/flash.c|  539 --
 board/evb4510/lowlevel_init.S|  157 
 board/gcplus/Makefile|   53 ---
 board/gcplus/config.mk   |   13 -
 board/gcplus/flash.c |  438 -
 board/gcplus/gcplus.c|   82 
 board/gcplus/lowlevel_init.S |   77 
 board/impa7/Makefile |   51 ---
 board/impa7/config.mk|   28 --
 board/impa7/flash.c  |  359 -
 board/impa7/impa7.c  |   71 
 board/impa7/lowlevel_init.S  |   85 
 board/lart/Makefile  |   51 ---
 board/lart/config.mk |   23 --
 board/lart/flash.c   |  476 ---
 board/lart/flashasm.S|  177 -
 board/lart/lart.c|   76 
 board/lart/lowlevel_init.S   |   94 -
 board/lpc2292sodimm/Makefile |   51 ---
 board/lpc2292sodimm/config.mk|   30 --
 board/lpc2292sodimm/flash.c  |  252 
 board/lpc2292sodimm/lowlevel_init.S  |   87 
 board/lpc2292sodimm/lpc2292sodimm.c  |   57 ---
 board/modnet50/Makefile  |   51 ---
 board/modnet50/config.mk |   29 --
 board/modnet50/flash.c   |  538 -
 board/modnet50/lowlevel_init.S   |  204 --
 board/modnet50/modnet50.c|   49 ---
 board/shannon/Makefile   |   51 ---
 board/shannon/config.mk  |   23 --
 board/shannon/flash.c|  475 ---
 board/shannon/inferno.header |  Bin 128 -> 0 bytes
 board/shannon/lowlevel_init.S|   92 -
 board/shannon/shannon.c  |   99 -
 board/siemens/SMN42/Makefile |   51 ---
 board/siemens/SMN42/config.mk|   30 --
 board/siemens/SMN42/flash.c  |  475 ---
 board/siemens/SMN42/lowlevel_init.S  |  123 --
 board/siemens/SMN42/smn42.c  |   57 ---
 boards.cfg   |   10 -
 common/image.c   |   11 -
 doc/README.modnet50  |   62 ---
 doc/README.scrapyard |   10 +
 include/configs/SMN42.h  |  201 --
 include/configs/dnp1110.h|  162 
 include/configs/ep7312.h |  173 
 include/configs/evb4510.h|  180 -
 include/configs/gcplus.h |  185 -
 include/configs/impa7.h  |  174 -
 include/configs/lart.h   |  160 
 include/configs/lpc2292sodimm.h  |  159 
 include/configs/modnet50.h 

[U-Boot] [PATCH 03/10] ARM: remove broken "ep7312" board.

[Wolfgang Denk]

Signed-off-by: Wolfgang Denk 
Cc: Albert ARIBAUD 
Cc: Marius Gröger 
---
 MAINTAINERS  |1 -
 MAKEALL  |1 -
 arch/arm/cpu/arm720t/cpu.c   |6 +-
 arch/arm/cpu/arm720t/interrupts.c|4 +-
 arch/arm/cpu/arm720t/start.S |6 +-
 arch/arm/include/asm/arch-arm720t/hardware.h |4 -
 board/ep7312/Makefile|   55 
 board/ep7312/config.mk   |   28 --
 board/ep7312/ep7312.c|   66 -
 board/ep7312/flash.c |  343 --
 board/ep7312/lowlevel_init.S |   95 ---
 boards.cfg   |1 -
 doc/README.scrapyard |1 +
 include/configs/ep7312.h |  173 -
 14 files changed, 9 insertions(+), 775 deletions(-)
 delete mode 100644 board/ep7312/Makefile
 delete mode 100644 board/ep7312/config.mk
 delete mode 100644 board/ep7312/ep7312.c
 delete mode 100644 board/ep7312/flash.c
 delete mode 100644 board/ep7312/lowlevel_init.S
 delete mode 100644 include/configs/ep7312.h

diff --git a/MAINTAINERS b/MAINTAINERS
index 16edb67..83d3656 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -664,7 +664,6 @@ Simon Guinot 
 Marius Gröger 
 
impa7   ARM720T (EP7211)
-   ep7312  ARM720T (EP7312)
 
 Igor Grinberg 
 
diff --git a/MAKEALL b/MAKEALL
index e35af42..40a1ecb 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -301,7 +301,6 @@ LIST_SA="$(boards_by_cpu sa1100)"
 #
 
 LIST_ARM7="\
-   ep7312  \
evb4510 \
impa7   \
lpc2292sodimm   \
diff --git a/arch/arm/cpu/arm720t/cpu.c b/arch/arm/cpu/arm720t/cpu.c
index ff992aa..71d8d6b 100644
--- a/arch/arm/cpu/arm720t/cpu.c
+++ b/arch/arm/cpu/arm720t/cpu.c
@@ -36,7 +36,7 @@
 #include 
 #include 
 
-#if defined(CONFIG_IMPA7) || defined(CONFIG_EP7312) || 
defined(CONFIG_ARMADILLO)
+#if defined(CONFIG_IMPA7)
 static void cache_flush(void);
 #endif
 
@@ -50,7 +50,7 @@ int cleanup_before_linux (void)
 * and we set the CPU-speed to 73 MHz - see start.S for details
 */
 
-#if defined(CONFIG_IMPA7) || defined(CONFIG_EP7312) || 
defined(CONFIG_ARMADILLO)
+#if defined(CONFIG_IMPA7)
disable_interrupts ();
 
/* turn off I-cache */
@@ -74,7 +74,7 @@ int cleanup_before_linux (void)
return 0;
 }
 
-#if defined(CONFIG_IMPA7) || defined(CONFIG_EP7312) || 
defined(CONFIG_ARMADILLO)
+#if defined(CONFIG_IMPA7)
 /* flush I/D-cache */
 static void cache_flush (void)
 {
diff --git a/arch/arm/cpu/arm720t/interrupts.c 
b/arch/arm/cpu/arm720t/interrupts.c
index fa9c5a2..3cebf11 100644
--- a/arch/arm/cpu/arm720t/interrupts.c
+++ b/arch/arm/cpu/arm720t/interrupts.c
@@ -149,7 +149,7 @@ int timer_init (void)
 
/* set timer 2 counter */
lastdec = TIMER_LOAD_VAL;
-#elif defined(CONFIG_IMPA7) || defined(CONFIG_EP7312) || 
defined(CONFIG_ARMADILLO)
+#elif defined(CONFIG_IMPA7)
/* disable all interrupts */
IO_INTMR1 = 0;
 
@@ -207,7 +207,7 @@ int timer_init (void)
  */
 
 
-#if defined(CONFIG_IMPA7) || defined(CONFIG_EP7312) || defined(CONFIG_NETARM) 
|| defined(CONFIG_ARMADILLO) || defined(CONFIG_LPC2292)
+#if defined(CONFIG_IMPA7) || defined(CONFIG_NETARM) || defined(CONFIG_LPC2292)
 
 ulong get_timer (ulong base)
 {
diff --git a/arch/arm/cpu/arm720t/start.S b/arch/arm/cpu/arm720t/start.S
index e9a094f..f2320ee 100644
--- a/arch/arm/cpu/arm720t/start.S
+++ b/arch/arm/cpu/arm720t/start.S
@@ -272,7 +272,7 @@ _dynsym_start_ofs:
  *
  */
 
-#if defined(CONFIG_IMPA7) || defined(CONFIG_EP7312) || 
defined(CONFIG_ARMADILLO)
+#if defined(CONFIG_IMPA7)
 
 /* Interrupt-Controller base addresses */
 INTMR1:.word   0x8280 @ 32 bit size
@@ -301,7 +301,7 @@ MEMMAP_ADR: .word   MEMMAP
 #endif
 
 cpu_init_crit:
-#if defined(CONFIG_IMPA7) || defined(CONFIG_EP7312) || 
defined(CONFIG_ARMADILLO)
+#if defined(CONFIG_IMPA7)
 
/*
 * mask all IRQs by clearing all bits in the INTMRs
@@ -634,7 +634,7 @@ fiq:
 
 #endif
 
-#if defined(CONFIG_IMPA7) || defined(CONFIG_EP7312) || 
defined(CONFIG_ARMADILLO)
+#if defined(CONFIG_IMPA7)
.align  5
 .globl reset_cpu
 reset_cpu:
diff --git a/arch/arm/include/asm/arch-arm720t/hardware.h 
b/arch/arm/include/asm/arch-arm720t/hardware.h
index 9bee19f..d89effa 100644
--- a/arch/arm/include/asm/arch-arm720t/hardware.h
+++ b/arch/arm/include/asm/arch-arm720t/hardware.h
@@ -28,10 +28,6 @@
 #include 
 #elif defined(CONFIG_IMPA7)
 /* include IMPA7 specific hardware file if there was one */
-#elif defined(CONFIG_EP7312)
-/* include EP7312 specific hardware file if there was one */
-#elif defined(CONFIG_ARMADILLO)
-/* include armadillo specific hardware fi

[U-Boot] [PATCH 09/10] ARM: remove broken "modnet50" board.

[Wolfgang Denk]

Signed-off-by: Wolfgang Denk 
Cc: Albert ARIBAUD 
Cc: Thomas Elste 
---
 CREDITS|5 -
 MAINTAINERS|4 -
 MAKEALL|9 -
 board/modnet50/Makefile|   51 
 board/modnet50/config.mk   |   29 ---
 board/modnet50/flash.c |  538 
 board/modnet50/lowlevel_init.S |  204 ---
 board/modnet50/modnet50.c  |   49 
 boards.cfg |1 -
 doc/README.modnet50|   62 -
 doc/README.scrapyard   |1 +
 include/configs/modnet50.h |  196 ---
 12 files changed, 1 insertions(+), 1148 deletions(-)
 delete mode 100644 board/modnet50/Makefile
 delete mode 100644 board/modnet50/config.mk
 delete mode 100644 board/modnet50/flash.c
 delete mode 100644 board/modnet50/lowlevel_init.S
 delete mode 100644 board/modnet50/modnet50.c
 delete mode 100644 doc/README.modnet50
 delete mode 100644 include/configs/modnet50.h

diff --git a/CREDITS b/CREDITS
index 487a55e..e8e923a 100644
--- a/CREDITS
+++ b/CREDITS
@@ -133,11 +133,6 @@ N: Dave Ellis
 E: d...@sixnetio.com
 D: EEPROM Speedup, SXNI855T port
 
-N: Thomas Elste
-E: i...@elste.org
-D: Port for the ModNET50 Board, NET+50 CPU Port
-W: http://www.imms.de
-
 N: Daniel Engstr?m
 E: dan...@omicron.se
 D: x86 port, Support for sc520_cdp board
diff --git a/MAINTAINERS b/MAINTAINERS
index d52e6f0..2d785fa 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -628,10 +628,6 @@ Wolfgang Denk 
imx27lite   i.MX27
qongi.MX31
 
-Thomas Elste 
-
-   modnet50ARM720T (NET+50)
-
 Kristoffer Ericson 
 
jornada SA1110
diff --git a/MAKEALL b/MAKEALL
index c6e6d04..2e39d2c 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -297,14 +297,6 @@ LIST_ppc=" \
 LIST_SA="$(boards_by_cpu sa1100)"
 
 #
-## ARM7 Systems
-#
-
-LIST_ARM7="\
-   modnet50\
-"
-
-#
 ## ARM9 Systems
 #
 
@@ -400,7 +392,6 @@ LIST_ixp="$(boards_by_cpu ixp)
 
 LIST_arm=" \
${LIST_SA}  \
-   ${LIST_ARM7}\
${LIST_ARM9}\
${LIST_ARM10}   \
${LIST_ARM11}   \
diff --git a/board/modnet50/Makefile b/board/modnet50/Makefile
deleted file mode 100644
index d5a541f..000
--- a/board/modnet50/Makefile
+++ /dev/null
@@ -1,51 +0,0 @@
-#
-# (C) Copyright 2000-2006
-# Wolfgang Denk, DENX Software Engineering, w...@denx.de.
-#
-# See file CREDITS for list of people who contributed to this
-# project.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License as
-# published by the Free Software Foundation; either version 2 of
-# the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
-# MA 02111-1307 USA
-#
-
-include $(TOPDIR)/config.mk
-
-LIB= $(obj)lib$(BOARD).o
-
-COBJS  := modnet50.o flash.o
-SOBJS  := lowlevel_init.o
-
-SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)
-OBJS   := $(addprefix $(obj),$(COBJS))
-SOBJS  := $(addprefix $(obj),$(SOBJS))
-
-$(LIB):$(obj).depend $(OBJS) $(SOBJS)
-   $(call cmd_link_o_target, $(OBJS) $(SOBJS))
-
-clean:
-   rm -f $(SOBJS) $(OBJS)
-
-distclean: clean
-   rm -f $(LIB) core *.bak $(obj).depend
-
-#
-
-# defines $(obj).depend target
-include $(SRCTREE)/rules.mk
-
-sinclude $(obj).depend
-
-#
diff --git a/board/modnet50/config.mk b/board/modnet50/config.mk
deleted file mode 100644
index 4e4d305..000
--- a/board/modnet50/config.mk
+++ /dev/null
@@ -1,29 +0,0 @@
-#
-# (C) Copyright 2000
-# Sysgo Real-Time Solutions, GmbH 
-# Marius Groeger 
-#
-# (C) Copyright 2000-2004
-# Wolfgang Denk, DENX Software Engineering, w...@denx.de.
-#
-# See file CREDITS for list of people who contributed to this
-# project.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License as
-# published by the Free Software Foundation; either version 2 of
-# the License, or (at your option) any later version.
-#
-# This program is distributed in the h

[U-Boot] [PATCH 01/10] ARM: remove broken "SMN42" board.

[Wolfgang Denk]

Signed-off-by: Wolfgang Denk 
Cc: Albert ARIBAUD 
---
 MAKEALL |1 -
 board/siemens/SMN42/Makefile|   51 
 board/siemens/SMN42/config.mk   |   30 ---
 board/siemens/SMN42/flash.c |  475 ---
 board/siemens/SMN42/lowlevel_init.S |  123 -
 board/siemens/SMN42/smn42.c |   57 -
 boards.cfg  |1 -
 doc/README.scrapyard|1 +
 include/configs/SMN42.h |  201 ---
 9 files changed, 1 insertions(+), 939 deletions(-)
 delete mode 100644 board/siemens/SMN42/Makefile
 delete mode 100644 board/siemens/SMN42/config.mk
 delete mode 100644 board/siemens/SMN42/flash.c
 delete mode 100644 board/siemens/SMN42/lowlevel_init.S
 delete mode 100644 board/siemens/SMN42/smn42.c
 delete mode 100644 include/configs/SMN42.h

diff --git a/MAKEALL b/MAKEALL
index e72a019..e35af42 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -306,7 +306,6 @@ LIST_ARM7=" \
impa7   \
lpc2292sodimm   \
modnet50\
-   SMN42   \
 "
 
 #
diff --git a/board/siemens/SMN42/Makefile b/board/siemens/SMN42/Makefile
deleted file mode 100644
index c054a3e..000
--- a/board/siemens/SMN42/Makefile
+++ /dev/null
@@ -1,51 +0,0 @@
-#
-# (C) Copyright 2007
-# Wolfgang Denk, DENX Software Engineering, w...@denx.de.
-#
-# See file CREDITS for list of people who contributed to this
-# project.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License as
-# published by the Free Software Foundation; either version 2 of
-# the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
-# MA 02111-1307 USA
-#
-
-include $(TOPDIR)/config.mk
-
-LIB= $(obj)lib$(BOARD).o
-
-COBJS  := flash.o smn42.o
-SOBJTS := lowlevel_init.o
-
-SRCS   := $(SOBJTS:.o=.S) $(COBJS:.o=.c)
-OBJS   := $(addprefix $(obj),$(COBJS))
-SOBJS  := $(addprefix $(obj),$(SOBJTS))
-
-$(LIB):$(obj).depend $(OBJS) $(SOBJS)
-   $(call cmd_link_o_target, $(OBJS) $(SOBJS))
-
-clean:
-   rm -f $(SOBJS) $(OBJS)
-
-distclean: clean
-   rm -f $(LIB) core *.bak $(obj).depend
-
-#
-
-# defines $(obj).depend target
-include $(SRCTREE)/rules.mk
-
-sinclude $(obj).depend
-
-#
diff --git a/board/siemens/SMN42/config.mk b/board/siemens/SMN42/config.mk
deleted file mode 100644
index 4891792..000
--- a/board/siemens/SMN42/config.mk
+++ /dev/null
@@ -1,30 +0,0 @@
-#
-# (C) Copyright 2000
-# Sysgo Real-Time Solutions, GmbH 
-# Marius Groeger 
-#
-# (C) Copyright 2000
-# Wolfgang Denk, DENX Software Engineering, w...@denx.de.
-#
-# See file CREDITS for list of people who contributed to this
-# project.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License as
-# published by the Free Software Foundation; either version 2 of
-# the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
-# MA 02111-1307 USA
-#
-
-#address where u-boot will be relocated
-#CONFIG_SYS_TEXT_BASE = 0x0
-CONFIG_SYS_TEXT_BASE = 0x8150
diff --git a/board/siemens/SMN42/flash.c b/board/siemens/SMN42/flash.c
deleted file mode 100644
index fc91574..000
--- a/board/siemens/SMN42/flash.c
+++ /dev/null
@@ -1,475 +0,0 @@
-/*
- * (C) Copyright 2006 Embedded Artists AB 
- *
- * (C) Copyright 2007 Gary Jennejohn ga...@denx.de
- * Modified to use the routines in arch/arm/cpu/arm720t/lpc2292/flash.c.
- * Heavily modified to support the SMN42 board from Siemens
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY