From mboxrd@z Thu Jan  1 00:00:00 1970
From: Luis Araneda <luaraneda@gmail.com>
Date: Wed, 18 Jul 2018 14:02:56 -0400
Subject: [U-Boot] [RFC PATCH 0/4] arm: zynq: implement FPGA load from SPL
In-Reply-To: <8d595d25-d5d2-b434-6cb2-2522dd07cf43@xilinx.com>
References: <20180718074141.16539-1-luaraneda@gmail.com>
 <8d595d25-d5d2-b434-6cb2-2522dd07cf43@xilinx.com>
Message-ID: <CAHbBuxrQcTp2wUnSzD0bFVaEyG3YCKqSfVBcJs7AMSWiHZEYfQ@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Michal,

On Wed, Jul 18, 2018 at 4:00 AM Michal Simek <michal.simek@xilinx.com> wrote:
> Can you please also send defconfig/config changes?
> Separate patch is fine.

The changes required to the defconfigs test/support this are:
CONFIG_SPL_LOAD_FIT=y
CONFIG_SPL_FPGA_SUPPORT=y

I didn't send them because just changing the defconfig isn't enough, I
had to manually create a FIT image to replace the default u-boot.img,
which is generated
by U-Boot's Makefile.
The .its file (to generate the FIT image) has the following FPGA node:
fpga-1 {
  description = "Zybo Z7-20 FPGA image";
  data = /incbin/("zybo-z7-20-preboot.bin");
  type = "fpga";
  arch = "arm";
  compression = "none";
  load = <0x30000000>;
};

I run-tested the changes on a Zybo Z7-20, and compile-tested on a Zybo.

I have an idea to automate the FIT generation. The build system could
scan for the existence of a file, for example
"board/xilinx/zynq/<board>/preboot.bin", and add the fpga node
automatically (to .its) if the file exists. Because I think that
storing .bin files in the U-Boot repository is infeasible. That will
require additions and modifications to the current way the build
system works, and I'm still thinking how to implement them.

By the way, sorry for changing the subject, but I realized that you
use two e-mail accounts. Do you have a preferred one? I'm asking
because I sent the Zybo-Z7 support series (and the I2C DM one) to the
@monstr.eu account, which you hasn't responded yet, and this series to
the @xilinx.com account, which you replied quickly. Maybe you are just
waiting for other people to review/test it.

Thanks,

Luis Araneda.