[PATCH v3 0/3] fpga: zynqmp: Adding support of loading authenticated images

[Oleksandr Suvorov]

Done. The v4 is sent.

On Mon, Nov 15, 2021, 09:59 Michal Simek <michal.simek at 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 at 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 at foundries.io>
> >>>
> https://patchwork.ozlabs.org/project/uboot/patch/20211015091506.2602-1-jorge@foundries.io/
> >>>
> https://patchwork.ozlabs.org/project/uboot/patch/20211005111324.19749-3-jorge@foundries.io/
> >>>
> >>> 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
>
>