From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Wed, 3 Mar 2021 14:08:26 -0500 Subject: [PATCH 2/4] board: ns3: Remove superfluous reset logic In-Reply-To: References: Message-ID: <20210303190826.GK1310@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > Cc: Rayagonda Kokatanur > Signed-off-by: Harald Seiler > Reviewed-by: Simon Glass (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: