[U-Boot] [PATCH 21/21] Remove legacy do_reset() function

Moffett, Kyle D Kyle.D.Moffett at boeing.com
Tue Mar 8 00:00:23 CET 2011


Hi!

On Mar 07, 2011, at 16:55, Graeme Russ wrote:
> On Tue, Mar 8, 2011 at 4:37 AM, Kyle Moffett <Kyle.D.Moffett at boeing.com> wrote:
>> All of the users of the legacy do_reset() function have been converted
>> to __arch_restart() or __board_restart() as appropriate, so the
>> compatibility calls to do_reset() may be removed.
>> 
>> In addition the do_generic_reset() function is renamed to the now-unused
>> name do_reset().
> 
> If it is unused, why not remove it completely?

The "reset" command used to invoke "do_reset()".  Partway through this patch series I changed it to invoke a new "do_generic_reset()" function to preserve bisection.

The calltrace looked like this:

=> do_generic_reset()
   => system_restart()
      => __board_restart()
      => __arch_restart()
      => do_reset()

Now that nothing talks about "do_reset()" anymore, I delete that old function and rename "do_generic_reset()" over top of it:

=> do_reset()  [Same as old do_generic_reset()]
   => system_restart()
      => __board_restart()
      => __arch_restart()

Cheers,
Kyle Moffett


More information about the U-Boot mailing list