On Wed, Dec 30, 2020 at 8:22 AM Fabio Estevam <feste...@gmail.com> wrote: > > Hi Adam, > > On Wed, Dec 30, 2020 at 11:14 AM Adam Ford <aford...@gmail.com> wrote: > > > &usdhc2 { > > u-boot,dm-spl; > > + sd-uhs-sdr104; > > + sd-uhs-ddr50; > > Shouldn't these properties be part of the main dtsi instead of adding > them to the u-boot.dtsi one?
Linux doesn't appear to need it, and no 64-bit imx boards in Linux appear to use them. Some layerscape boards appear to use these flags. If it's not necessary for Linux, but it is necessary for U-Boot, it seems like the -u-boot.dtsi file is the right place to put it. It's consistent with that was done for the imx8mm-evk-u-boot.dtsi and others [1] [1] - https://gitlab.denx.de/u-boot/u-boot/-/commit/50b1a69cee0dc06d0c713a5a978998f2b4a9cb31 adam