diff mbox

[U-Boot,v3,2/2] arm: a320: enable tagged list support

Message ID 1311044214-1652-1-git-send-email-ratbert.chuang@gmail.com
State Accepted
Headers show

Commit Message

Po-Yu Chuang July 19, 2011, 2:56 a.m. UTC
From: Po-Yu Chuang <ratbert@faraday-tech.com>


Signed-off-by: Po-Yu Chuang <ratbert@faraday-tech.com>
---
v2:
add a preceding patch to fix multiline comment style

v3:
rebase

 include/configs/a320evb.h |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

Comments

Albert ARIBAUD Aug. 4, 2011, 9:32 a.m. UTC | #1
Hi Po Yu Chuang,

On 19/07/2011 04:56, Po-Yu Chuang wrote:
> From: Po-Yu Chuang<ratbert@faraday-tech.com>
>
>
> Signed-off-by: Po-Yu Chuang<ratbert@faraday-tech.com>
> ---
> v2:
> add a preceding patch to fix multiline comment style
>
> v3:
> rebase
>
>   include/configs/a320evb.h |    6 ++++++
>   1 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/include/configs/a320evb.h b/include/configs/a320evb.h
> index f832d52..62913b5 100644
> --- a/include/configs/a320evb.h
> +++ b/include/configs/a320evb.h
> @@ -25,6 +25,12 @@
>   #include<asm/arch/a320.h>
>
>   /*
> + * Linux kernel tagged list
> + */
> +#define CONFIG_CMDLINE_TAG
> +#define CONFIG_SETUP_MEMORY_TAGS
> +
> +/*
>    * CPU and Board Configuration Options
>    */
>   #undef CONFIG_USE_IRQ		/* we don't need IRQ/FIQ stuff */

Applied to u-boot-arm (after local fix to context lines -- my fault 
entirely, waited too long before applying it), thanks!

Amicalement,
Po-Yu Chuang Aug. 4, 2011, 9:44 a.m. UTC | #2
Dear Albert,

On Thu, Aug 4, 2011 at 5:32 PM, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Hi Po Yu Chuang,
>
> On 19/07/2011 04:56, Po-Yu Chuang wrote:
>>
>> From: Po-Yu Chuang<ratbert@faraday-tech.com>
>>
>>
>> Signed-off-by: Po-Yu Chuang<ratbert@faraday-tech.com>
>> ---
>> v2:
>> add a preceding patch to fix multiline comment style
>>
>> v3:
>> rebase
>>
>>  include/configs/a320evb.h |    6 ++++++
>>  1 files changed, 6 insertions(+), 0 deletions(-)
>>
>> diff --git a/include/configs/a320evb.h b/include/configs/a320evb.h
>> index f832d52..62913b5 100644
>> --- a/include/configs/a320evb.h
>> +++ b/include/configs/a320evb.h
>> @@ -25,6 +25,12 @@
>>  #include<asm/arch/a320.h>
>>
>>  /*
>> + * Linux kernel tagged list
>> + */
>> +#define CONFIG_CMDLINE_TAG
>> +#define CONFIG_SETUP_MEMORY_TAGS
>> +
>> +/*
>>   * CPU and Board Configuration Options
>>   */
>>  #undef CONFIG_USE_IRQ         /* we don't need IRQ/FIQ stuff */
>
> Applied to u-boot-arm (after local fix to context lines -- my fault
> entirely, waited too long before applying it), thanks!

Just to remind you.

There is a preceding patch "arm: a320: fix multiline comment style"
according to Wolfgang's request.

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

I guess that's why you need local fix.

Best regards,
Po-Yu Chuang
Albert ARIBAUD Aug. 4, 2011, 9:57 a.m. UTC | #3
Hi Po Yu Chuang,

On 04/08/2011 11:44, Po-Yu Chuang wrote:
> Dear Albert,
>
> On Thu, Aug 4, 2011 at 5:32 PM, Albert ARIBAUD
> <albert.u.boot@aribaud.net>  wrote:
>> Hi Po Yu Chuang,
>>
>> On 19/07/2011 04:56, Po-Yu Chuang wrote:
>>>
>>> From: Po-Yu Chuang<ratbert@faraday-tech.com>
>>>
>>>
>>> Signed-off-by: Po-Yu Chuang<ratbert@faraday-tech.com>
>>> ---
>>> v2:
>>> add a preceding patch to fix multiline comment style
>>>
>>> v3:
>>> rebase
>>>
>>>   include/configs/a320evb.h |    6 ++++++
>>>   1 files changed, 6 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/include/configs/a320evb.h b/include/configs/a320evb.h
>>> index f832d52..62913b5 100644
>>> --- a/include/configs/a320evb.h
>>> +++ b/include/configs/a320evb.h
>>> @@ -25,6 +25,12 @@
>>>   #include<asm/arch/a320.h>
>>>
>>>   /*
>>> + * Linux kernel tagged list
>>> + */
>>> +#define CONFIG_CMDLINE_TAG
>>> +#define CONFIG_SETUP_MEMORY_TAGS
>>> +
>>> +/*
>>>    * CPU and Board Configuration Options
>>>    */
>>>   #undef CONFIG_USE_IRQ         /* we don't need IRQ/FIQ stuff */
>>
>> Applied to u-boot-arm (after local fix to context lines -- my fault
>> entirely, waited too long before applying it), thanks!
>
> Just to remind you.
>
> There is a preceding patch "arm: a320: fix multiline comment style"
> according to Wolfgang's request.
>
> http://patchwork.ozlabs.org/patch/105374/
>
> I guess that's why you need local fix.

Thanks -- I'll reorder u-boot-arm to apply the style fix first.

> Best regards,
> Po-Yu Chuang

Amicalement,
diff mbox

Patch

diff --git a/include/configs/a320evb.h b/include/configs/a320evb.h
index f832d52..62913b5 100644
--- a/include/configs/a320evb.h
+++ b/include/configs/a320evb.h
@@ -25,6 +25,12 @@ 
 #include <asm/arch/a320.h>
 
 /*
+ * Linux kernel tagged list
+ */
+#define CONFIG_CMDLINE_TAG
+#define CONFIG_SETUP_MEMORY_TAGS
+
+/*
  * CPU and Board Configuration Options
  */
 #undef CONFIG_USE_IRQ		/* we don't need IRQ/FIQ stuff */