[PATCH 3/6] System restart: add a scope value to the system restart feature

Juergen Borleis jbe at pengutronix.de
Thu Jun 25 00:06:05 PDT 2015


Hi Sascha,

On Wednesday 24 June 2015 20:34:40 Sascha Hauer wrote:
> On Mon, Jun 22, 2015 at 12:33:21PM +0200, Juergen Borleis wrote:
> > Some systems have more than one feature to restart it. Maybe these
> > restarts features are all equal or it is very important which restart
> > feauture is used to restart the system in a reliable manner. For example
> > if external devices must be reset in conjunction with the SoC some SoC's
> > internal restart features cannot be used. Then an external restart
> > feature must be forced instead by using the restart feature with the
> > wider scope.
>
> It seems this patch misses several conversions from reset_cpu to
> restart_machine:
>
> ./arch/mips/lib/bootm.c:23:     reset_cpu(0);
> ./arch/ppc/lib/ppclinux.c:90:   reset_cpu(0);
> ./arch/sandbox/os/common.c:144:                 reset_cpu(0);
> ./arch/sandbox/os/common.c:152:                 reset_cpu(0);
> ./arch/arm/mach-at91/bootstrap.c:148:   reset_cpu(0);
> ./arch/arm/mach-tegra/tegra20-pmc.c:221:        reset_cpu(0);
> ./arch/blackfin/lib/traps.c:94: reset_cpu(0);
> ./arch/blackfin/lib/traps.c:104:        reset_cpu(0);

Hmm, interesting. I did a sed replacement in the whole source tree. Don't know 
why they slipped through.

> ./arch/arm/mach-at91/at91sam9g45_reset.S
> ./arch/arm/mach-at91/at91sam9_reset.S

Ups, assembler...

Regards,
Juergen

-- 
Pengutronix e.K.                              | Juergen Borleis             |
Industrial Linux Solutions                    | http://www.pengutronix.de/  |



More information about the barebox mailing list