From mboxrd@z Thu Jan 1 00:00:00 1970 From: Fabio Estevam Date: Wed, 30 Dec 2020 12:24:30 -0300 Subject: [PATCH 2/2] ARM: dts: imx8mm-beacon: add UHS and HS400/HS400ES properties In-Reply-To: References: <20201230141421.2860212-1-aford173@gmail.com> <20201230141421.2860212-2-aford173@gmail.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, Dec 30, 2020 at 11:45 AM Adam Ford wrote: > 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] IMHO the device trees should be the same for Linux and U-Boot as much as we can. Of course, it is OK to add custom U-Boot properties, such as u-boot,dm-spl inside -u-boot.dtsi file, but adding MMC related properties only in U-Boot dtsi does not seem correct. It would be nice if someone could investigate what is missing in the U-Boot sdhc driver to handle these modes by default.