[PATCH] sandbox: use 'poweroff' command for exit

Antony Pavlov antonynpavlov at gmail.com
Mon Jan 22 06:08:31 PST 2018


On Mon, 22 Jan 2018 14:40:20 +0100
Sam Ravnborg <sam at ravnborg.org> wrote:

> Hi Antony
> 
> On Mon, Jan 22, 2018 at 01:01:31PM +0300, Antony Pavlov wrote:
> > The 'poweroff' command is more appropriate for exit
> > barebox than 'reset'.
> 
> Changelog says "reset" but code says "restart".

The command name is 'reset'.
No matter what is the file's name.
I can't see 'restart' command in barebox mainline master branch.

> Could the restart command be useful for others than sandbox?
> In other words, is it correct to delete this command?

I see no reason for dropping 'reset' command.
Sometimes 'reset' and 'poweroff' commands are very handy.

> 	
> 	Sam
> 
> >  arch/sandbox/board/restart.c           | 17 -----------------


-- 
Best regards,
  Antony Pavlov



More information about the barebox mailing list