From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2a00:1450:400c:c0c::244; helo=mail-wr0-x244.google.com; envelope-from=leif.lindholm@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-wr0-x244.google.com (mail-wr0-x244.google.com [IPv6:2a00:1450:400c:c0c::244]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id D63F82096966D for ; Fri, 25 May 2018 11:34:03 -0700 (PDT) Received: by mail-wr0-x244.google.com with SMTP id w18-v6so10747757wrn.6 for ; Fri, 25 May 2018 11:34:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=4Lvxp132JmHlyypk9JlV0LwhLLLLVv70/EGH8jicjmc=; b=FmYMyTgnhXQDSWlbW72NWva/wmLnDkuLYqC10s1QQbCKIgiJra+1RmPHvjk4JzhN49 3dcHYG+FFyqz8HY3JaUYuN6ikDMaZ4NMSXOXanEBbPZ4EAM5wcyROSEO/wJy2GXlQOA8 mstCS82YH62O8EQBI5I7DHxSyMndbImpGL/lw= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=4Lvxp132JmHlyypk9JlV0LwhLLLLVv70/EGH8jicjmc=; b=bQlPoSX1XePsyQxuCZkFQFDW0rks4601yBnwkLexRxwrC9S1AUPXEFbuCY7zqPlXDf 3R1xFMqVBM33jIWyj4jRYEeU3WGMJNSJwuYCdsJHCz93oul3dSo4EzAWPTLkpBZgPjrv C1BmIV3xsEUDxCmvac/SauQ18eIZPFYYaxJOkg6W8a1Nw7IHhtDek7EoxNTXxgxjelSk 791vLSj8G10L91ymjHU/Fst0vXn3GQWg2rtl0yHPM9+B0lx+wgDOPEZtp0IBvAgRHUEF vnyIT7DGwEiT0ls99Rok0RnyvKGoLymslIrYrTK0Tew4X8fRMcY+3hQiarVc2P+MTde8 MBiw== X-Gm-Message-State: ALKqPwcvSBxWahIM+bmbZ93XDUtRKxVuTieoKSl23MyzZnuMJp73eo8y iqys7+8RQD0v2dvAyJ+RA6fOLEQ83hU= X-Google-Smtp-Source: AB8JxZoc8727fJpeQMS1ACquYl6/3BzFIA3CuiGQnXWevw/Wlp94AvQCBYjrDC6TLJkijhu3VjL72Q== X-Received: by 2002:adf:8672:: with SMTP id 47-v6mr2995365wrw.102.1527273242280; Fri, 25 May 2018 11:34:02 -0700 (PDT) Received: from bivouac.eciton.net (bivouac.eciton.net. [2a00:1098:0:86:1000:23:0:2]) by smtp.gmail.com with ESMTPSA id 42-v6sm49332667wrx.24.2018.05.25.11.34.01 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 25 May 2018 11:34:01 -0700 (PDT) Date: Fri, 25 May 2018 19:33:59 +0100 From: Leif Lindholm To: Haojian Zhuang Cc: edk2-devel@lists.01.org, Ard Biesheuvel Message-ID: <20180525183359.bl4h4dhgcyaxnweo@bivouac.eciton.net> References: <1527122790-23592-1-git-send-email-haojian.zhuang@linaro.org> <1527122790-23592-4-git-send-email-haojian.zhuang@linaro.org> MIME-Version: 1.0 In-Reply-To: <1527122790-23592-4-git-send-email-haojian.zhuang@linaro.org> User-Agent: NeoMutt/20170113 (1.7.2) Subject: Re: [PATCH v6 edk2-platforms 3/6] Platform/HiKey960: enable virtual keyboard X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 25 May 2018 18:34:04 -0000 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, May 24, 2018 at 08:46:27AM +0800, Haojian Zhuang wrote: > Enable virtual keyboard on HiKey960 platform. It checks two > conditions, such as pattern in memory and GPIO pin setting. > Since code is ported from non-open > > The hardcoding code is taken from non-open reference code. > Can't fix it for lack of documents. > > Cc: Leif Lindholm > Cc: Ard Biesheuvel > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Haojian Zhuang Reviewed-by: Leif Lindholm > --- > Platform/Hisilicon/HiKey960/HiKey960.dsc | 5 + > Platform/Hisilicon/HiKey960/HiKey960.fdf | 5 + > Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.inf | 1 + > Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.h | 10 ++ > Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.c | 97 ++++++++++++++++++++ > 5 files changed, 118 insertions(+) > > diff --git a/Platform/Hisilicon/HiKey960/HiKey960.dsc b/Platform/Hisilicon/HiKey960/HiKey960.dsc > index 6cc1c1edf453..79e68754976d 100644 > --- a/Platform/Hisilicon/HiKey960/HiKey960.dsc > +++ b/Platform/Hisilicon/HiKey960/HiKey960.dsc > @@ -182,6 +182,11 @@ [Components.common] > Platform/Hisilicon/HiKey960/HiKey960GpioDxe/HiKey960GpioDxe.inf > ArmPlatformPkg/Drivers/PL061GpioDxe/PL061GpioDxe.inf > > + # > + # Virtual Keyboard > + # > + EmbeddedPkg/Drivers/VirtualKeyboardDxe/VirtualKeyboardDxe.inf > + > Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.inf > > # > diff --git a/Platform/Hisilicon/HiKey960/HiKey960.fdf b/Platform/Hisilicon/HiKey960/HiKey960.fdf > index b7d70b010598..d65f77878575 100644 > --- a/Platform/Hisilicon/HiKey960/HiKey960.fdf > +++ b/Platform/Hisilicon/HiKey960/HiKey960.fdf > @@ -123,6 +123,11 @@ [FV.FvMain] > INF Platform/Hisilicon/HiKey960/HiKey960GpioDxe/HiKey960GpioDxe.inf > INF ArmPlatformPkg/Drivers/PL061GpioDxe/PL061GpioDxe.inf > > + # > + # Virtual Keyboard > + # > + INF EmbeddedPkg/Drivers/VirtualKeyboardDxe/VirtualKeyboardDxe.inf > + > INF Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.inf > > # > diff --git a/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.inf b/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.inf > index a1a7d005ce8b..46a9a5803e3d 100644 > --- a/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.inf > +++ b/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.inf > @@ -39,6 +39,7 @@ [LibraryClasses] > > [Protocols] > gEmbeddedGpioProtocolGuid > + gPlatformVirtualKeyboardProtocolGuid > > [Guids] > gEfiEndOfDxeEventGroupGuid > diff --git a/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.h b/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.h > index 9a4c66f42c50..211eea55aa54 100644 > --- a/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.h > +++ b/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.h > @@ -21,6 +21,7 @@ > > #include > #include > +#include > #include > #include > #include > @@ -29,6 +30,15 @@ > #include > #include > > +#include > +#include > + > +#define ADB_REBOOT_ADDRESS 0x32100000 > +#define ADB_REBOOT_BOOTLOADER 0x77665500 > +#define ADB_REBOOT_NONE 0x77665501 > + > +#define DETECT_SW_FASTBOOT 68 // GPIO8_4 > + > enum { > BOOT_MODE_RECOVERY = 0, > BOOT_MODE_MASK = 1, > diff --git a/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.c b/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.c > index 7c1705241e88..7ff2f118128d 100644 > --- a/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.c > +++ b/Platform/Hisilicon/HiKey960/HiKey960Dxe/HiKey960Dxe.c > @@ -14,6 +14,8 @@ > > #include "HiKey960Dxe.h" > > +STATIC EMBEDDED_GPIO *mGpio; > + > STATIC > VOID > InitSdCard ( > @@ -166,6 +168,94 @@ OnEndOfDxe ( > > EFI_STATUS > EFIAPI > +VirtualKeyboardRegister ( > + IN VOID > + ) > +{ > + EFI_STATUS Status; > + > + Status = gBS->LocateProtocol ( > + &gEmbeddedGpioProtocolGuid, > + NULL, > + (VOID **) &mGpio > + ); > + if (EFI_ERROR (Status)) { > + return Status; > + } > + return EFI_SUCCESS; > +} > + > +EFI_STATUS > +EFIAPI > +VirtualKeyboardReset ( > + IN VOID > + ) > +{ > + EFI_STATUS Status; > + > + if (mGpio == NULL) { > + return EFI_INVALID_PARAMETER; > + } > + // > + // Configure GPIO68 as GPIO function > + // > + MmioWrite32 (0xe896c108, 0); > + Status = mGpio->Set (mGpio, DETECT_SW_FASTBOOT, GPIO_MODE_INPUT); > + return Status; > +} > + > +BOOLEAN > +EFIAPI > +VirtualKeyboardQuery ( > + IN VIRTUAL_KBD_KEY *VirtualKey > + ) > +{ > + EFI_STATUS Status; > + UINTN Value = 0; > + > + if ((VirtualKey == NULL) || (mGpio == NULL)) { > + return FALSE; > + } > + if (MmioRead32 (ADB_REBOOT_ADDRESS) == ADB_REBOOT_BOOTLOADER) { > + goto Done; > + } else { > + Status = mGpio->Get (mGpio, DETECT_SW_FASTBOOT, &Value); > + if (EFI_ERROR (Status) || (Value != 0)) { > + return FALSE; > + } > + } > +Done: > + VirtualKey->Signature = VIRTUAL_KEYBOARD_KEY_SIGNATURE; > + VirtualKey->Key.ScanCode = SCAN_NULL; > + VirtualKey->Key.UnicodeChar = L'f'; > + return TRUE; > +} > + > +EFI_STATUS > +EFIAPI > +VirtualKeyboardClear ( > + IN VIRTUAL_KBD_KEY *VirtualKey > + ) > +{ > + if (VirtualKey == NULL) { > + return EFI_INVALID_PARAMETER; > + } > + if (MmioRead32 (ADB_REBOOT_ADDRESS) == ADB_REBOOT_BOOTLOADER) { > + MmioWrite32 (ADB_REBOOT_ADDRESS, ADB_REBOOT_NONE); > + WriteBackInvalidateDataCacheRange ((VOID *)ADB_REBOOT_ADDRESS, 4); > + } > + return EFI_SUCCESS; > +} > + > +PLATFORM_VIRTUAL_KBD_PROTOCOL mVirtualKeyboard = { > + VirtualKeyboardRegister, > + VirtualKeyboardReset, > + VirtualKeyboardQuery, > + VirtualKeyboardClear > +}; > + > +EFI_STATUS > +EFIAPI > HiKey960EntryPoint ( > IN EFI_HANDLE ImageHandle, > IN EFI_SYSTEM_TABLE *SystemTable > @@ -194,5 +284,12 @@ HiKey960EntryPoint ( > if (EFI_ERROR (Status)) { > return Status; > } > + > + Status = gBS->InstallProtocolInterface ( > + &ImageHandle, > + &gPlatformVirtualKeyboardProtocolGuid, > + EFI_NATIVE_INTERFACE, > + &mVirtualKeyboard > + ); > return Status; > } > -- > 2.7.4 >