[PATCH 2/2] ARM: dts: imx8mm-beacon: add UHS and HS400/HS400ES properties

Adam Ford aford173 at gmail.com
Wed Dec 30 15:45:40 CET 2020


On Wed, Dec 30, 2020 at 8:22 AM Fabio Estevam <festevam at gmail.com> wrote:
>
> Hi Adam,
>
> On Wed, Dec 30, 2020 at 11:14 AM Adam Ford <aford173 at 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


More information about the U-Boot mailing list