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

[Michal Simek]

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