Re: [PATCH 13/14] ARM: optee-early: drop start_optee_early()

[Sascha Hauer <s.hauer@pengutronix.de>]

On Fri, Jun 27, 2025 at 05:21:03PM +0200, Ahmad Fatoum wrote:
> Hi,
> 
> On 6/27/25 16:07, Sascha Hauer wrote:
> > Now that all users of start_optee_early() use the SoC variants
> > make start_optee_early() a static function. While at it move the zeroing
> > of the data_location which was duplicated in the SoC specific functions
> > to start_optee_early().
> 
> start_optee_early() is documented in Documentation/user/optee.rst and
> would need to be updated.

I adjusted the documentation in "ARM: optee-early: add mx6_start_optee_early helper".
Apparently I failed to catch all occurences.

> 
> 
> > +	/* We expect this function to be called with data caches disabled */
> > +	BUG_ON(get_cr() & CR_C);
> 
> Did you mean to drop the BUG_ON() in start_optee_early().

My series was originally based on master and at some point I rebased on
v2025.06.1 which gave me this artifact which doesn't belong into this
patch. I'll rebase on master again next time.

Sascha

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |