On 14/10/2020 10:21, Pali Rohár wrote:

<snip>

diff --git a/doc/README.marvell b/doc/README.marvell
index 5416bc3035..be07f31f8c 100644
--- a/doc/README.marvell
+++ b/doc/README.marvell
@@ -43,8 +43,11 @@ Build Procedure
           In order to prevent this, the required device-tree MUST be set 
during compilation.
           All device-tree files are located in ./arch/arm/dts/ folder.
-       For other DB boards (MacchiatoBin, EspressoBin and 3700 DB board) 
compile u-boot with
-       just default device-tree from defconfig using:
+       For the EspressoBin board with populated eMMC device use
+               # make DEVICE_TREE=armada-3720-espressobin-emmc
+
+       For other DB boards (MacchiatoBin, EspressoBin without soldered eMMC 
and 3700 DB board)
+       compile u-boot with just default device-tree from defconfig using:

Hello! Does not it really make sense to do autodetection of eMMC
presence and enable it in U-Boot code only when needed and therefore
avoid having two DTS files and needs for specifying DEVICE_TREE variable
and therefore variant of Espressobin, as I stated in previous emails?

I think this just complicates build process... E.g. we already have a
code in U-Boot which detects V5 vs V7 variant.

I still like the idea, but I'm still hesitating to just enable the emmc everywhere. There's the issue with powering down the emmc block if it's not populated. How would that work once we switch to atf for comphy control?

From a different angle: If we could just do that, why does Linux have different devicetrees?

Regards,
Andre

Reply via email to