On 2/13/19 11:45 PM, Dalon L Westergreen wrote: > On Wed, 2019-02-13 at 17:10 +0100, Marek Vasut wrote: >> On 2/13/19 3:18 PM, [email protected] wrote: >>> From: Tien Fong Chee <[email protected]> >>> >>> Add default fitImage file bundling FPGA bitstreams for Arria10. >>> >>> Signed-off-by: Tien Fong Chee <[email protected]> >>> >>> --- >>> >>> changes for v8 >>> - Changed the FPGA node name to fpga-core and fpga-periph for both core and >>> periph bitstreams respectively. >>> --- >>> board/altera/arria10-socdk/fit_spl_fpga.its | 39 >>> +++++++++++++++++++++++++++++ >>> 1 file changed, 39 insertions(+) >>> create mode 100644 board/altera/arria10-socdk/fit_spl_fpga.its >>> >>> diff --git a/board/altera/arria10-socdk/fit_spl_fpga.its >>> b/board/altera/arria10-socdk/fit_spl_fpga.its >>> new file mode 100644 >>> index 0000000..8ce175b >>> --- /dev/null >>> +++ b/board/altera/arria10-socdk/fit_spl_fpga.its >>> @@ -0,0 +1,39 @@ >>> +// SPDX-License-Identifier: GPL-2.0 >>> + /* >>> + * Copyright (C) 2019 Intel Corporation <www.intel.com> >>> + * >>> + */ >>> + >>> +/dts-v1/; >>> + >>> +/ { >>> + description = "FIT image with FPGA bistream"; >>> + #address-cells = <1>; >>> + >>> + images { >>> + fpga-core@1 { >>> + description = "FPGA core bitstream"; >>> + data = /incbin/("../../../ghrd_10as066n2.core.rbf"); >>> + type = "fpga"; >>> + arch = "arm"; >>> + compression = "none"; >>> + load = <0x400>; >> >> Is the load address required ? >> >>> + }; >>> + >>> + fpga-periph@2 { >>> + description = "FPGA peripheral bitstream"; >>> + data = /incbin/("../../../ghrd_10as066n2.periph.rbf"); >>> + type = "fpga"; >>> + arch = "arm"; >>> + compression = "none"; >>> + }; >>> + }; >>> + >>> + configurations { >>> + default = "config-1"; >>> + config-1 { >>> + description = "Boot with FPGA early IO release config"; >>> + fpga = "fpga-periph@2", "fpga-core@1"; >> >> Don't you need to load the core first ? > > No, the periphery is first. This brings up the dram and i/o.
Then why do we have periph@2 above ? Shouldn't those two images be swapped to make this look less confusing ? -- Best regards, Marek Vasut _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
