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

Oleksandr Suvorov oleksandr.suvorov at foundries.io
Fri Nov 5 22:16:44 CET 2021


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 :(
-- 
Best regards,

Oleksandr Suvorov
Software Engineer
T: +380 63 8489656
E: oleksandr.suvorov at foundries.io
W: www.foundries.io


More information about the U-Boot mailing list