Re: [platforms: PATCH v4 2/7] Marvell/Library: ArmadaBoardDescLib: Extend SDMMC information

[Ard Biesheuvel <ard.biesheuvel@linaro.org>]

On Sat, 10 Nov 2018 at 00:02, Marcin Wojtas <mw@semihalf.com> wrote:
>
> From: Tomasz Michalec <tm@semihalf.com>
>
> Added fields specific for Xenon host controller and declaration
> of ArmadaBoardDescSdMmcGet function.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Marcin Wojtas <mw@semihalf.com>

Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>

> ---
>  Silicon/Marvell/Include/Library/ArmadaBoardDescLib.h | 22 +++++++++++++++++++-
>  1 file changed, 21 insertions(+), 1 deletion(-)
>
> diff --git a/Silicon/Marvell/Include/Library/ArmadaBoardDescLib.h b/Silicon/Marvell/Include/Library/ArmadaBoardDescLib.h
> index ee8e06e..8e29a68 100644
> --- a/Silicon/Marvell/Include/Library/ArmadaBoardDescLib.h
> +++ b/Silicon/Marvell/Include/Library/ArmadaBoardDescLib.h
> @@ -55,9 +55,21 @@ typedef struct {
>  //
>  // SDMMC devices per-board description
>  //
> +typedef enum {
> +  RemovableSlot,
> +  EmbeddedSlot,
> +  SharedBusSlot,
> +  UnknownSlot
> +} MV_SDMMC_SLOT_TYPE;
> +
>  typedef struct {
>    MV_SOC_SDMMC_DESC *SoC;
> -  UINTN              SdMmcDevCount;
> +  UINTN    SdMmcDevCount;
> +  BOOLEAN  Xenon1v8Enabled;
> +  BOOLEAN  Xenon8BitBusEnabled;
> +  BOOLEAN  XenonSlowModeEnabled;
> +  UINT8    XenonTuningStepDivisor;
> +  MV_SDMMC_SLOT_TYPE SlotType;
>  } MV_BOARD_SDMMC_DESC;
>
>  //
> @@ -84,4 +96,12 @@ typedef struct {
>    UINTN             UtmiDevCount;
>    UINTN             UtmiPortType;
>  } MV_BOARD_UTMI_DESC;
> +
> +EFI_STATUS
> +EFIAPI
> +ArmadaBoardDescSdMmcGet (
> +  IN OUT UINTN               *SdMmcDevCount,
> +  IN OUT MV_BOARD_SDMMC_DESC **SdMmcDesc
> +  );
> +
>  #endif /* __ARMADA_SOC_DESC_LIB_H__ */
> --
> 2.7.4
>