On Tue, Nov 21, 2017 at 07:46:17AM +0100, Marcin Wojtas wrote:
> Hi,
>
> I submit v2 of the Armada variable support with the style of
> the MvFvbDxe driver fixed and other minor modifications. Depex
> configuration was moved from 4/4 to previous patches. Details
> can be found in the changelog and commit messages.
>
> Patches are available in the github:
> https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commits/var-upstream-r20171121
>
> I'm looking forward to the comments or remarks.
For 2-4/4:
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
> Best regards,
> Marcin
>
> Changelog:
> v1 -> v2:
> * Change patches order in order to safely apply related Depex modifications
>
> * 1/4:
> - Fix style (line breaking, openining brackets, alignment, typos)
> - Rework MvFvbSetAttributes - because it can return only EFI_SUCCESS /
> EFI_INVALID_PARAMETER and should return FVB attributes,
> add debug information, that operation is not supported and return
> default values via MvFvbGetAttributes.
> - Return EFI_SUCCESS, in case NumBytes == 0 in MvFvbRead.
>
> * 2/4:
> - Move Depex configuration from 4/4
>
> * 3/4:
> - Move Depex configuration from 4/4
> - Reword and break commit message.
>
> * 4/4:
> - Remove Depex configuration
>
> Marcin Wojtas (4):
> Platform/Marvell: Introduce MvFvbDxe variable support driver
> Marvell/Drivers: MvSpiFlash: Enable using driver in RT
> Marvell/Drivers: MvSpiDxe: Enable using driver in RT
> Marvell/Armada: Enable variables support
>
> Platform/Marvell/Armada/Armada.dsc.inc | 25 +-
> Platform/Marvell/Armada/Armada70x0.fdf | 6 +-
> Platform/Marvell/Drivers/Spi/Devices/MvSpiFlash.c | 58 +-
> Platform/Marvell/Drivers/Spi/Devices/MvSpiFlash.h | 1 +
> Platform/Marvell/Drivers/Spi/Devices/MvSpiFlash.inf | 11 +-
> Platform/Marvell/Drivers/Spi/MvSpiDxe.c | 50 +-
> Platform/Marvell/Drivers/Spi/MvSpiDxe.h | 2 +
> Platform/Marvell/Drivers/Spi/MvSpiDxe.inf | 9 +-
> Platform/Marvell/Drivers/Spi/Variables/MvFvbDxe.c | 1056 ++++++++++++++++++++
> Platform/Marvell/Drivers/Spi/Variables/MvFvbDxe.h | 114 +++
> Platform/Marvell/Drivers/Spi/Variables/MvFvbDxe.inf | 91 ++
> Platform/Marvell/Include/Protocol/Spi.h | 7 +
> Platform/Marvell/Marvell.dec | 4 +
> 13 files changed, 1420 insertions(+), 14 deletions(-)
> create mode 100644 Platform/Marvell/Drivers/Spi/Variables/MvFvbDxe.c
> create mode 100644 Platform/Marvell/Drivers/Spi/Variables/MvFvbDxe.h
> create mode 100644 Platform/Marvell/Drivers/Spi/Variables/MvFvbDxe.inf
>
> --
> 2.7.4
>
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel