Re: [PATCH v1] toradex: {aquila, smarc, verdin}-imx95: add board_fix_fdt

Stefan Eichenberger <[email protected]>
Newsgroups org.u-boot-project.lists.u-boot
Message-ID <ao1IFKQq7Lni4Kx4@eichest-laptop>
Hi Fabio,

On Mon, Aug 24, 2026 at 10:24:41PM -0300, Fabio Estevam wrote:
> Hi Stefan,
> 
> On Fri, Aug 21, 2026 at 4:10 AM Stefan Eichenberger <[email protected]> wrote:
> 
> > +#if IS_ENABLED(CONFIG_OF_BOARD_FIXUP)
> > +int board_fix_fdt(void *fdt)
> > +{
> > +       /* Remove nodes based on fuses. */
> > +       return imx9_uboot_fixup_by_fuse(fdt);
> 
> Shouldn't this be added to the common i.MX95 SoC file instead of
> adding it per board?

We discussed this internally and decided to implement it in the same way
as NXP does in board/nxp/imx95_frdm/imx95_frdm.c, but I'm not sure why
it was necessary to put it in the board file. Perhaps because it can
only be done in board_fix_fdt, which might be overwritten in the board
file anyway? If we were to do this for the entire architecture, we would
need to define a weak board_fix_fdt, which could be overwritten, if I
understand correctly.

Regards,
Stefan
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.