Re: [PATCH v1] toradex: {aquila, smarc, verdin}-imx95: add board_fix_fdt
Fabio Estevam <[email protected]>
| Newsgroups | org.u-boot-project.lists.u-boot |
|---|---|
| Message-ID | <CAOMZO5Df=ZoziqoYrAB-3rS+P-giWHjYrhKNBeQE6UKi2a+pBA@mail.gmail.com> |
Hi Stefan, On Tue, Aug 25, 2026 at 4:45 AM Stefan Eichenberger <[email protected]> wrote: > 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. It seems we should handle this similarly to arch/arm/mach-imx/imx8m/soc.c, where peripherals are disabled according to the SoC fuses. Since this is a property of the SoC rather than a particular board, I would prefer to implement the fuse-based fixup once in the common i.MX95 SoC code instead of adding the same board_fix_fdt() implementation to every i.MX95 board. This would also avoid requiring each new i.MX95 board to duplicate this code in the future. Could you please move this to the common i. MX95 SoC code? If there is a technical reason we can't do that, we can revisit the per-board approach. Thanks, Fabio Estevam