[U-Boot] rockchip: rk3399: set fdtfile

[Jonathan Gray <jsg@jsg.id.au>]

On Mon, Mar 19, 2018 at 08:17:49AM +0100, klaus.goger at theobroma-systems.com wrote:
> 
> > On 17.03.2018, at 09:48, Mark Kettenis <kettenis@openbsd.org> wrote:
> > 
> > Signed-off-by: Mark Kettenis <kettenis@openbsd.org>
> > Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> > ---
> > include/configs/rk3399_common.h | 1 +
> > 1 file changed, 1 insertion(+)
> > 
> > diff --git a/include/configs/rk3399_common.h b/include/configs/rk3399_common.h
> > index d700bf2549..9a5de3a7e3 100644
> > --- a/include/configs/rk3399_common.h
> > +++ b/include/configs/rk3399_common.h
> > @@ -59,6 +59,7 @@
> > #include <config_distro_bootcmd.h>
> > #define CONFIG_EXTRA_ENV_SETTINGS \
> > 	ENV_MEM_LAYOUT_SETTINGS \
> > +	"fdtfile=rockchip/" CONFIG_DEFAULT_DEVICE_TREE ".dtb\0" \
> > 	"partitions=" PARTS_DEFAULT \
> > 	BOOTENV
> > 
> 
> This assumes that the devicetree files in U-Boot and the OS are the same.
> As it???s our goal to keep Linux devicetrees in sync with U-Boot this patch should
> be fine in the long-term. But at the moment this only applies for rk3399-firefly (on
> Linux).
> 
> Mark, where would one find the devicetrees included in OpenBSD?
> I looked in sys/arch/amd64 but couldn???t find any.

The ideal case is the firmware (U-Boot, EDK2 etc) supplies a device tree
and resides on it's own storage.  Then we could have single install
images that work everywhere without having to distribute board specific
U-Boot and dtb files.

As most arm systems aren't ideal there are ports/packages that build
device trees from the linux source and package up those provided
as part of raspberrypi-firmware.

https://cvsweb.openbsd.org/cgi-bin/cvsweb/ports/sysutils/dtb/
https://cvsweb.openbsd.org/cgi-bin/cvsweb/ports/sysutils/raspberrypi-firmware/

Then there are some U-Boot targets with builtin device trees.

https://cvsweb.openbsd.org/cgi-bin/cvsweb/ports/sysutils/u-boot/
https://cvsweb.openbsd.org/cgi-bin/cvsweb/ports/sysutils/u-boot-pinebook/

The parts of the tree that build install images with U-Boot/dtbs:

https://cvsweb.openbsd.org/cgi-bin/cvsweb/src/distrib/armv7/
https://cvsweb.openbsd.org/cgi-bin/cvsweb/src/distrib/arm64/

> 
> Anyhow, I???m fine with that change.
> 
> Tested-by: Klaus Goger <klaus.goger@theobroma-systems.com>
> 
> ??? 
> Klaus
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot