From: Albert ARIBAUD <albert.u.boot@aribaud.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 09/16] ARM: remove broken "versatile" boards.
Date: Tue, 30 Aug 2011 10:24:03 +0200 [thread overview]
Message-ID: <4E5C9E23.6050104@aribaud.net> (raw)
In-Reply-To: <4E5C9C49.2050008@aribaud.net>
Stefano,
Le 30/08/2011 10:16, Albert ARIBAUD a ?crit :
> Le 30/08/2011 09:16, Stefano Babic a ?crit :
>
>> On 08/30/2011 08:47 AM, Albert ARIBAUD wrote:
>>> Hi Stefano,
>>
>> Hi Albert,
>>
>>> Building all versatile boards here with ELDK4.2 on a Natty Ubuntu, I get
>>> warnings of two sorts:
>>>
>>> 1) in the QEMU build, you needlessly cast a pointer as 'volatile',
>>
>> This is wrong...
>>
>>> which
>>> I missed on review but which my compiler does not like -- I did a
>>> general removal patch for code in U-Boot that calls get_ram_size() with
>>> a volatile-cast argument.
>>
>> I saw the same issue rebuilding all IMX board for the scb9328 - because
>> this was the only board with this warning I found, I sent a patch for it
>> and I have already applied to u-boot-imx. I sent to you a "pull
>> request", but let me know if you prefer I drop my patch from u-boot-imx
>> before you merge into u-boot-arm.
>
> I'll pick it with the IMX pull req, and I will apply Wolfgang's patch
> set afterward, minus versatile if the board is completely fixed.
Sorry, I realize there might have been a misunderstanding. The global
fix patch I did was applied some time ago and any spurious volatile left
must be fixed by new patches, so your patch is welcome anyway.
Note that you also need to provide a fix for versatile.c.
Amicalement,
--
Albert.
next prev parent reply other threads:[~2011-08-30 8:24 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-08-24 14:12 [U-Boot] [STATUS] ARM: board removal Albert ARIBAUD
2011-08-24 16:09 ` Reinhard Meyer
2011-08-24 17:19 ` Albert ARIBAUD
2011-08-24 18:11 ` Wolfgang Denk
2011-08-25 13:03 ` Albert ARIBAUD
2011-08-25 20:36 ` Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 00/16] " Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 01/16] ARM: remove broken "armadillo" board Wolfgang Denk
2011-08-26 15:41 ` Marek Vasut
2011-08-26 19:57 ` Wolfgang Denk
2011-08-26 20:49 ` Marek Vasut
2011-08-26 22:29 ` Wolfgang Denk
2011-08-30 6:06 ` Albert ARIBAUD
2011-08-30 7:20 ` Marek Vasut
2011-08-30 9:32 ` Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 02/16] ARM: remove broken "B2" board Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 03/16] ARM: remove broken "edb93xx" boards Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 04/16] ARM: remove broken "lpd7a40x" boards Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 05/16] ARM: remove broken "mx1fs2" board Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 06/16] ARM: remove broken "netstar" board Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 07/16] ARM: remove broken "sbc2410x" board Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 08/16] ARM: remove broken "smdk2400" board Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 09/16] ARM: remove broken "versatile" boards Wolfgang Denk
2011-08-26 12:56 ` Stefano Babic
2011-08-30 6:47 ` Albert ARIBAUD
2011-08-30 7:16 ` Stefano Babic
2011-08-30 8:16 ` Albert ARIBAUD
2011-08-30 8:24 ` Albert ARIBAUD [this message]
2011-08-26 12:25 ` [U-Boot] [PATCH v2 10/16] ARM: remove broken "voiceblue" board Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 11/16] ARM: remove broken "at91cap9adk" board Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 12/16] ARM: remove broken "cmc_pu2" board Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 13/16] ARM: remove broken "csb637" board Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 14/16] ARM: remove broken "kb9202" board Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 15/16] ARM: remove broken "m501sk" board Wolfgang Denk
2011-08-26 12:25 ` [U-Boot] [PATCH v2 16/16] ARM: remove broken "at91rm9200dk" board Wolfgang Denk
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4E5C9E23.6050104@aribaud.net \
--to=albert.u.boot@aribaud.net \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.