[PATCH 2/4] board: ns3: Remove superfluous reset logic

Tom Rini trini at konsulko.com
Wed Mar 3 20:08:26 CET 2021


On Tue, Dec 15, 2020 at 04:47:50PM +0100, Harald Seiler wrote:

> The current implementation of reset_cpu() in the ns3 board code does not
> archieve what it is supposed to (according to the comments), due to
> a number of reasons:
> 
>  1. The argument to reset_cpu() is _not_ actually passed from the
>     `reset` command, but is set to 0 in all call-sites (in this
>     specific case, see arch/arm/lib/reset.c).  Thus, performing
>     different kinds of resets based on its value will not work as
>     expected.
> 
>  2. Contrary to its documentation, the passed argument is not
>     interpreted, but a static `L3_RESET` define is used.  The other
>     comment properly notes that this will always perform a L3 reset,
>     though.
> 
>  3. The "parsing" of the static `L3_RESET` value is not even using the
>     upper and lower nibble as stated in the comment, but uses the last
>     two decimal digits of the value.
> 
> This is currently one of the only implementations left in U-Boot, which
> make "use" of the value passed to reset_cpu().  As this is done under
> false assumption (the value does not have any meaning anymore), it makes
> sense to bring it into line with the rest and start ignoring the
> parameter.
> 
> This is a preparation for removal of the reset_cpu() parameter across
> the entire tree in a later patch.
> 
> Fixes: b5a152e7ca0b ("board: ns3: default reset type to L3")
> Cc: Bharat Gooty <bharat.gooty at broadcom.com>
> Cc: Rayagonda Kokatanur <rayagonda.kokatanur at broadcom.com>
> Signed-off-by: Harald Seiler <hws at denx.de>
> Reviewed-by: Simon Glass <sjg at chromium.org>

(Er, 1/4 had the wrong branch ...)
Applied to u-boot/next, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210303/88c612a3/attachment.sig>


More information about the U-Boot mailing list