On Thu, Jan 17, 2019 at 12:15:10PM +0100, Marcin Wojtas wrote:
> Introduce new callback that can provide information
> about GPIO SoC controllers, as well as on-board
> I2C IO expanders. According ArmadaSoCDescLib
> ArmadaBoardDescLib routines are used for
> obtaining required data.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Marcin Wojtas
Reviewed-by: Leif Lindholm
> ---
> Silicon/Marvell/Include/Protocol/BoardDesc.h | 8
> Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.c | 50
> 2 files changed, 58 insertions(+)
>
> diff --git a/Silicon/Marvell/Include/Protocol/BoardDesc.h
> b/Silicon/Marvell/Include/Protocol/BoardDesc.h
> index 1d57a16..02905ea 100644
> --- a/Silicon/Marvell/Include/Protocol/BoardDesc.h
> +++ b/Silicon/Marvell/Include/Protocol/BoardDesc.h
> @@ -50,6 +50,13 @@ EFI_STATUS
>
> typedef
> EFI_STATUS
> +(EFIAPI *MV_BOARD_GPIO_DESCRIPTION_GET) (
> + IN MARVELL_BOARD_DESC_PROTOCOL*This,
> + IN OUT MV_BOARD_GPIO_DESCRIPTION **GpioDescription
> + );
> +
> +typedef
> +EFI_STATUS
> (EFIAPI *MV_BOARD_DESC_I2C_GET) (
>IN MARVELL_BOARD_DESC_PROTOCOL *This,
>IN OUT MV_BOARD_I2C_DESC **I2cDesc
> @@ -113,6 +120,7 @@ struct _MARVELL_BOARD_DESC_PROTOCOL {
>MV_BOARD_DESC_UTMI_GET BoardDescUtmiGet;
>MV_BOARD_DESC_XHCI_GET BoardDescXhciGet;
>MV_BOARD_DESC_FREE BoardDescFree;
> + MV_BOARD_GPIO_DESCRIPTION_GET GpioDescriptionGet;
> };
>
> #endif // __MARVELL_BOARD_DESC_PROTOCOL_H__
> diff --git a/Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.c
> b/Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.c
> index f71bfc4..973c362 100644
> --- a/Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.c
> +++ b/Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.c
> @@ -35,6 +35,8 @@ SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
>
> MV_BOARD_DESC *mBoardDescInstance;
>
> +STATIC MV_BOARD_GPIO_DESCRIPTION *mGpioDescription;
> +
> STATIC
> EFI_STATUS
> MvBoardDescComPhyGet (
> @@ -100,6 +102,53 @@ MvBoardDescComPhyGet (
>
> STATIC
> EFI_STATUS
> +MvBoardGpioDescriptionGet (
> + IN MARVELL_BOARD_DESC_PROTOCOL*This,
> + IN OUT MV_BOARD_GPIO_DESCRIPTION **GpioDescription
> + )
> +{
> + UINTN SoCGpioCount, GpioExpanderCount;
> + MV_GPIO_EXPANDER *GpioExpanders;
> + GPIO_CONTROLLER *SoCGpio;
> + EFI_STATUS Status;
> +
> + /* Use existing structure if already created. */
> + if (mGpioDescription != NULL) {
> +*GpioDescription = mGpioDescription;
> +return EFI_SUCCESS;
> + }
> +
> + /* Get SoC data about all available GPIO controllers. */
> + Status = ArmadaSoCGpioGet (&SoCGpio, &SoCGpioCount);
> + if (EFI_ERROR (Status)) {
> +return Status;
> + }
> +
> + /* Get per-board information about all available GPIO expanders. */
> + Status = ArmadaBoardGpioExpanderGet (&GpioExpanders, &GpioExpanderCount);
> + if (EFI_ERROR (Status)) {
> +return Status;
> + }
> +
> + /* Allocate and fill board description. */
> + mGpioDescription = AllocateZeroPool (sizeof (MV_BOARD_GPIO_DESCRIPTION));
> + if (mGpioDescription == NULL) {
> +DEBUG ((DEBUG_ERROR, "%a: Cannot allocate memory\n", __FUNCTION__));
> +return EFI_OUT_OF_RESOURCES;
> + }
> +
> + mGpioDescription->SoCGpio = SoCGpio;
> + mGpioDescription->GpioDeviceCount = SoCGpioCount;
> + mGpioDescription->GpioExpanders = GpioExpanders;
> + mGpioDescription->GpioExpanderCount = GpioExpanderCount;
> +
> + *GpioDescription = mGpioDescription;
> +
> + return EFI_SUCCESS;
> +}
> +
> +STATIC
> +EFI_STATUS
> MvBoardDescI2cGet (
>IN MARVELL_BOARD_DESC_PROTOCOL *This,
>IN OUT MV_BOARD_I2C_DESC **I2cDesc
> @@ -571,6 +620,7 @@ MvBoardDescInitProtocol (
>BoardDescProtocol->BoardDescUtmiGet = MvBoardDescUtmiGet;
>BoardDescProtocol->BoardDescXhciGet = MvBoardDescXhciGet;
>BoardDescProtocol->BoardDescFree = MvBoardDescFree;
> + BoardDescProtocol->GpioDescriptionGet = MvBoardGpioDescriptionGet;
>
>return EFI_SUCCESS;
> }
> --
> 2.7.4
>
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel