From mboxrd@z Thu Jan  1 00:00:00 1970
From: Tom Rini <trini@ti.com>
Date: Wed, 7 Mar 2012 11:45:00 -0700
Subject: [U-Boot] [PATCH V3] OMAP5: reset: Use cold reset in case of
	5430ES1.0
In-Reply-To: <CAOMWX4drSDQ+=gpqH7sBrxb0-j5Vt2V=rE4Udh7_JscHhuJxbA@mail.gmail.com>
References: <1331103129-3558-1-git-send-email-r.sricharan@ti.com>
	<CAOMWX4drSDQ+=gpqH7sBrxb0-j5Vt2V=rE4Udh7_JscHhuJxbA@mail.gmail.com>
Message-ID: <4F57ACAC.8020208@ti.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 03/07/2012 11:22 AM, Menon, Nishanth wrote:
> On Wed, Mar 7, 2012 at 00:52, R Sricharan<r.sricharan@ti.com>  wrote:
>> Warm reset is not functional in case of omap5430ES1.0.
>> So use cold reset instead.
>>
>> Signed-off-by: R Sricharan<r.sricharan@ti.com>
>> ---
>>   [v3]
>>      Addressed Tom Rini's comments.<trini@ti.com>
>>
>>   arch/arm/cpu/armv7/omap-common/reset.S |    3 +++
>>   arch/arm/cpu/armv7/omap5/hwinit.c      |   15 +++++++++++++++
>>   2 files changed, 18 insertions(+), 0 deletions(-)
>>
>> diff --git a/arch/arm/cpu/armv7/omap-common/reset.S b/arch/arm/cpu/armv7/omap-common/reset.S
>> index 838b122..f2a522a 100644
>> --- a/arch/arm/cpu/armv7/omap-common/reset.S
>> +++ b/arch/arm/cpu/armv7/omap-common/reset.S
>> @@ -23,6 +23,8 @@
>>
>>   #include<config.h>
>>
>> +#ifndef CONFIG_OMAP54XX
>> +.type  reset_cpu, %function
>>   .global reset_cpu
>>   reset_cpu:
>>         ldr     r1, rstctl                      @ get addr for global reset
>> @@ -36,3 +38,4 @@ rstctl:
>>         .word   PRM_RSTCTRL
>>   rstbit:
>>         .word   PRM_RSTCTRL_RESET
>> +#endif
>
> instead of doing the #ifdeffery, why not make this a C file with weak
> and override it in omap5 as needed?

Agreed, thanks.

-- 
Tom