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

[Tom Rini]

On Fri, Feb 17, 2023 at 03:40:28PM +0100, Marek Vasut wrote:
> 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.

Yes, it's still on the board to call arch_misc_init() inside
spl_board_init().

-- 
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/20230217/7d8d0971/attachment.sig>