On 04/13/2018 10:01 AM, Kever Yang wrote:
Hi Heinrich,

     Does this fdtfile will used for kernel? There should be a fdt config
for kernel dtb in boot config file,

and we should not use U-Boot dtb for kernel, they are different.


Thanks,
- Kever
On 04/13/2018 03:54 PM, Kever Yang wrote:
Hi Heinrich,

     What kind of issue do you met without this patch?

I think distro boot always work fine in my side without this patch.

Run make tinker-rk3288_defconfig

In the environment you get:
  soc=rockchip
  board=tinker_rk3288

According to doc/README.pxe:

  If the fdtfile environment variable is set, <path>/<fdtfile> is
  retrieved. Otherwise, the filename is generated from the soc and
  board environment, i.e. <path>/<soc>-<board>.dtb is retrieved.

The logic is implemented in file cmd/pxe.c.

So instead of retrieving tinker_rk3288.dtb we will try to read rockchip-tinker_rk3288.dtb which is incorrect.

If you want to look downstream, Debian package flash-kernel uses ${fdtfile} as the preferred way to identify the dtb file. The variable is prerequisite for images that shall run on different SOCs.
See /etc/flash-kernel/bootscript/bootscr.uboot-generic.

Best regards

Heinrich


Thanks,
- Kever
On 04/13/2018 06:45 AM, Heinrich Schuchardt wrote:
All rk3288 default configs define CONFIG_DEFAULT_DEVICE_TREE.
So we can use it to define ${fdtfile} in rk3288_common.h.

This variable is needed by the distro boot command.

Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
---
  include/configs/rk3288_common.h | 1 +
  1 file changed, 1 insertion(+)

diff --git a/include/configs/rk3288_common.h b/include/configs/rk3288_common.h
index 78595b86ec..762a206143 100644
--- a/include/configs/rk3288_common.h
+++ b/include/configs/rk3288_common.h
@@ -73,6 +73,7 @@
  #define CONFIG_EXTRA_ENV_SETTINGS \
        "fdt_high=0x0fffffff\0" \
        "initrd_high=0x0fffffff\0" \
+       "fdtfile=" CONFIG_DEFAULT_DEVICE_TREE ".dtb\0" \
        "partitions=" PARTS_DEFAULT \
        ENV_MEM_LAYOUT_SETTINGS \
        ROCKCHIP_DEVICE_SETTINGS \

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot



_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to