Done. The v4 is sent.
On Mon, Nov 15, 2021, 09:59 Michal Simek michal.simek@xilinx.com wrote:
On 11/5/21 22:16, Oleksandr Suvorov wrote:
Hello Michal,
On Wed, Nov 3, 2021 at 1:45 PM Michal Simek michal.simek@xilinx.com
wrote:
On 11/2/21 14:49, Oleksandr Suvorov wrote:
This patchset introduces support for the authenticated FPGA images on ZynqMP boards, besides that introducing common way to pass the compatible property to any fpga driver.
It bases on the initial work by Jorge Ramirez-Ortiz <
jorge@foundries.io>
https://patchwork.ozlabs.org/project/uboot/patch/20211015091506.2602-1-jorge...
https://patchwork.ozlabs.org/project/uboot/patch/20211005111324.19749-3-jorg...
Changes in v3:
- remove the patch which introduced CMD_SPL_FPGA_LOAD_SECURE.
- fix mixing definitions/declarations.
- replace strcmp() calls with more secure strncmp().
- document the "u-boot,zynqmp-fpga-ddrauth" compatible string.
- fix code style by check-patch recommendations.
Changes in v2:
- add function fit_fpga_load() to simplify calls of fpga_load() from contexts without a compatible attribute.
- move all ZynqMP-specific logic to drivers/fpga/zynqmppl.c
- prepare for passing a "compatible" FDT property to any fpga driver.
Oleksandr Suvorov (3): fpga: add option for loading FPGA secure bitstreams fpga: add fit_fpga_load function fpga: zynqmp: support loading authenticated images
cmd/Kconfig | 3 ++- common/Kconfig.boot | 4 +-- common/spl/spl_fit.c | 6 ++--- doc/uImage.FIT/source_file_format.txt | 5 +++- drivers/fpga/Kconfig | 14 ++++++++++ drivers/fpga/fpga.c | 37
+++++++++++++++++++++------
drivers/fpga/xilinx.c | 2 +- drivers/fpga/zynqmppl.c | 25 ++++++++++++++++-- include/fpga.h | 4 +++ 9 files changed, 81 insertions(+), 19 deletions(-)
Applied.
Thanks!
Unfortunately (my bad) we tested not the latest variant of the patchset. And this one, applied, has a bug. If you can rewrite a tree of a branch where the patchset is applied, please, remove it and I'll send the next version, fixed. Otherwise, I'll send new patches with a fix for the bug. Both variants are ready. Sorry for that :(
Please send new version and I will deal with it.
Thanks, Michal