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