[PATCHv2 12/13] global: Add a select for ARCH_MISC_INIT where used

Marek Vasut marex at denx.de
Fri Feb 17 15:40:28 CET 2023

On 2/17/23 15:00, Tom Rini wrote:
> On Fri, Feb 17, 2023 at 02:33:28PM +0100, Marek Vasut wrote:
>> On 2/17/23 13:50, Tom Rini wrote:
>>> When we have an arch_misc_init function, we can select it to ensure it's
>>> used.
>>> In that there is a functional change here, it's that on i.MX6 it's
>>> always been possible to populate "serial#" with something meaningful,
>>> but not every platform was taking the hook, and now it is.
>>> Signed-off-by: Tom Rini <trini at konsulko.com>
>>> ---
>>> Cc: Marcel Ziswiler <marcel.ziswiler at toradex.com>
>>> Cc: Marek Vasut <marex at denx.de>
>>> Cc: Olaf Mandel <o.mandel at menlosystems.com>
>>> Cc: Stefano Babic <sbabic at denx.de>
>>> Cc: Fabio Estevam <festevam at gmail.com>
>>> Cc: NXP i.MX U-Boot Team <uboot-imx at nxp.com>
>>> Changes in v2:
>>> - Always select ARCH_MISC_INIT on IMX8M as dealing with FSL_CAAM or not
>>>     needs further SoC-specific clean up work. This replaces patches 10 and
>>>     11 from before so that if the user enables FSL_CAAM then they will
>>>     continue to get the functionality they had before. As Marek noted in the
>>>     thread, further clean-up in this area would be good.
>> The SPL code does not automatically call arch_misc_init() if ARCH_MISC_INIT
>> is selected , so this won't work as-is. Is there another patch on which I
>> wasn't CCed which deals with that ?
> This preserves the exiting behavior, nothing more.

Where does the arch_misc_init() get called from in SPL ?
If that remains in board code for now, then yeah, fine by me.

More information about the U-Boot mailing list