Re: [PATCH edk2-platforms 0/5] Add Secure96 mezzanine support

[Leif Lindholm <leif.lindholm@linaro.org>]

On Thu, Feb 15, 2018 at 05:20:49PM +0000, Ard Biesheuvel wrote:
> This series adds preliminary support for the Secure96 mezzanine board,
> an expansion board that can be plugged into the low speed connector on
> the Socionext SynQuacer based Developer Box platform.
> 
> I have attempted to implement this in a reusable way, i.e., the secure96
> specific parts are in separate drivers which could theoretically be
> imported by other platforms as well.
> 
> I am presenting this to discuss the approach. My end goal is to wire up
> the Atmel SHA204A on this board in UEFI so it can be used as a random
> number generator, but this should be mostly orthogonal (and if it isn't,
> we can add it on top).

So, ideally I would like to see a lot more reusable code for handling
overlays, but as this is the first mezzanine board to be added I'm
actually pretty happy for it to go in[1].

We may want a whiteboard session at Linaro Connect to discuss the
generic problem.

/
    Leif

[1] Apart from any potential modifications to 1-2/5.
    Oh, and there's a typo in SynQuacer in the subject of 1-2/5.

> Ard Biesheuvel (5):
>   Silicon/SynQuaver/DeviceTree: add node for SPI controller
>   Silicon/SynQuaver/DeviceTree: add node for I2C controller
>   Platform: add support for 96boards Secure96 mezzanine adapter
>   Silicon/SynQuacer/PlatformDxe: add menu option to select mezzanine
>   Platform/Socionext/DeveloperBox: add Secure96 support
> 
>  Platform/96boards/Secure96/DeviceTree/DeviceTree.inf               |  40 +++++
>  Platform/96boards/Secure96/DeviceTree/Secure96.dts                 |  74 ++++++++++
>  Platform/96boards/Secure96/Secure96.dec                            |  56 +++++++
>  Platform/96boards/Secure96/Secure96Dxe/Secure96Dxe.c               | 153 ++++++++++++++++++++
>  Platform/96boards/Secure96/Secure96Dxe/Secure96Dxe.inf             |  51 +++++++
>  Platform/Socionext/DeveloperBox/DeveloperBox.dsc                   |  21 +++
>  Platform/Socionext/DeveloperBox/DeveloperBox.fdf                   |   6 +
>  Silicon/Socionext/SynQuacer/DeviceTree/SynQuacer.dtsi              |  36 +++++
>  Silicon/Socionext/SynQuacer/Drivers/PlatformDxe/PlatformDxe.c      |   8 +
>  Silicon/Socionext/SynQuacer/Drivers/PlatformDxe/PlatformDxe.inf    |   2 +
>  Silicon/Socionext/SynQuacer/Drivers/PlatformDxe/PlatformDxeHii.uni |   6 +
>  Silicon/Socionext/SynQuacer/Drivers/PlatformDxe/PlatformDxeHii.vfr |   8 +
>  Silicon/Socionext/SynQuacer/Include/Platform/VarStore.h            |   6 +-
>  13 files changed, 466 insertions(+), 1 deletion(-)
>  create mode 100644 Platform/96boards/Secure96/DeviceTree/DeviceTree.inf
>  create mode 100644 Platform/96boards/Secure96/DeviceTree/Secure96.dts
>  create mode 100644 Platform/96boards/Secure96/Secure96.dec
>  create mode 100644 Platform/96boards/Secure96/Secure96Dxe/Secure96Dxe.c
>  create mode 100644 Platform/96boards/Secure96/Secure96Dxe/Secure96Dxe.inf
> 
> -- 
> 2.11.0
>