[PATCH 18/41] ARM: restart: ixp4xx: use new restart hook

Krzysztof Halasa khc at pm.waw.pl
Fri Nov 18 02:49:01 EST 2011


Russell King - ARM Linux <linux at arm.linux.org.uk> writes:

>> Hook these platforms restart code into the new restart hook rather
>> than using arch_reset().

Looks good basically, except a detail:

>> --- a/arch/arm/mach-ixp4xx/common.c
>> +++ b/arch/arm/mach-ixp4xx/common.c
>> @@ -500,3 +500,23 @@ static void __init ixp4xx_clockevent_init(void)
>>  
>>  	clockevents_register_device(&clockevent_ixp4xx);
>>  }
>> +
>> +void ixp4xx_restart(char mode, const char *cmd)
>> +{
>> +	if ( 1 && mode == 's') {
           ^^^^^^^
This one. Yeah, I can see where does it come from.
Otherwise,
Acked-by: Krzysztof Hałasa <khc at pm.waw.pl>

>> +		/* Jump into ROM at address 0 */
>> +		soft_restart(0);

OTOH I probably need to investigate this soft_restart, "when time
permits", that's another story of course.
-- 
Krzysztof Halasa



More information about the linux-arm-kernel mailing list