From 3c1e53ce8f52429067a6664576e0bd5611d4e764 Mon Sep 17 00:00:00 2001 From: Ronald Cron Date: Wed, 25 Feb 2015 19:34:07 +0000 Subject: [PATCH] EmbeddedPkg/FdtPlatformDxe: Add 'setfdt' EFI Shell command The 'setfdt' EFI Shell command allows to define a new FDT as the prefered one for development purposes. It allows to trigger the run of the FDT installation process as well. Please refer to the README.txt file for more comprehensive description. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ronald Cron Reviewed-by: Olivier Martin git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@16934 6f19259b-4bc3-4df7-8a09-765794883524 --- .../Drivers/FdtPlatformDxe/FdtPlatform.c | 108 +++++++++++++++++- .../Drivers/FdtPlatformDxe/FdtPlatformDxe.inf | 8 ++ .../Drivers/FdtPlatformDxe/FdtPlatformDxe.uni | Bin 0 -> 6674 bytes EmbeddedPkg/Drivers/FdtPlatformDxe/README.txt | 24 +++- EmbeddedPkg/EmbeddedPkg.dsc | 5 + 5 files changed, 139 insertions(+), 6 deletions(-) create mode 100644 EmbeddedPkg/Drivers/FdtPlatformDxe/FdtPlatformDxe.uni diff --git a/EmbeddedPkg/Drivers/FdtPlatformDxe/FdtPlatform.c b/EmbeddedPkg/Drivers/FdtPlatformDxe/FdtPlatform.c index 8608fcf5ba..e777b0f7f7 100644 --- a/EmbeddedPkg/Drivers/FdtPlatformDxe/FdtPlatform.c +++ b/EmbeddedPkg/Drivers/FdtPlatformDxe/FdtPlatform.c @@ -20,11 +20,15 @@ #include #include #include - +#include #include +#include +#include #include #include +#include +#include #include #include @@ -34,6 +38,7 @@ // // Internal types // + STATIC VOID OnEndOfDxe ( IN EFI_EVENT Event, IN VOID *Context @@ -45,6 +50,48 @@ STATIC EFI_STATUS InstallFdt ( IN CONST CHAR16* TextDevicePath ); +STATIC SHELL_STATUS EFIAPI ShellDynCmdSetFdtHandler ( + IN EFI_SHELL_DYNAMIC_COMMAND_PROTOCOL *This, + IN EFI_SYSTEM_TABLE *SystemTable, + IN EFI_SHELL_PARAMETERS_PROTOCOL *ShellParameters, + IN EFI_SHELL_PROTOCOL *Shell + ); + +STATIC CHAR16* EFIAPI ShellDynCmdSetFdtGetHelp ( + IN EFI_SHELL_DYNAMIC_COMMAND_PROTOCOL *This, + IN CONST CHAR8 *Language + ); + +STATIC SHELL_STATUS UpdateFdtTextDevicePath ( + IN EFI_SHELL_PROTOCOL *Shell, + IN CONST CHAR16 *FilePath + ); + +STATIC SHELL_STATUS EfiCodeToShellCode ( + IN EFI_STATUS Status + ); + +// +// Internal variables +// + +STATIC CONST EFI_SHELL_DYNAMIC_COMMAND_PROTOCOL mShellDynCmdProtocolSetFdt = { + L"setfdt", // Name of the command + ShellDynCmdSetFdtHandler, // Handler + ShellDynCmdSetFdtGetHelp // GetHelp +}; + +STATIC CONST EFI_GUID mFdtPlatformDxeHiiGuid = { + 0x8afa7610, 0x62b1, 0x46aa, + {0xb5, 0x34, 0xc3, 0xde, 0xff, 0x39, 0x77, 0x8c} + }; +STATIC CONST SHELL_PARAM_ITEM ParamList[] = { + {L"-i", TypeFlag }, + {NULL , TypeMax } +}; + +STATIC EFI_HANDLE mFdtPlatformDxeHiiHandle; + /** Main entry point of the FDT platform driver. @@ -53,7 +100,10 @@ STATIC EFI_STATUS InstallFdt ( @param[in] *SystemTable A pointer to the EFI System table. @retval EFI_SUCCESS The driver was initialized. - @retval EFI_OUT_OF_RESOURCES The "End of DXE" event could not be allocated. + @retval EFI_OUT_OF_RESOURCES The "End of DXE" event could not be allocated or + there was not enough memory in pool to install + the Shell Dynamic Command protocol. + @retval EFI_LOAD_ERROR Unable to add the HII package. **/ EFI_STATUS @@ -80,6 +130,57 @@ FdtPlatformEntryPoint ( &EndOfDxeEvent ); + if (EFI_ERROR (Status)) { + return Status; + } + + // + // If the development features are enabled, install the dynamic shell + // command "setfdt" to be able to define a device path for the FDT + // that has precedence over the device paths defined by + // "PcdFdtDevicePaths". + // + + if (FeaturePcdGet (PcdOverridePlatformFdt)) { + // + // Register the strings for the user interface in the HII Database. + // This shows the way to the multi-language support, even if + // only the English language is actually supported. The strings to register + // are stored in the "FdtPlatformDxeStrings[]" array. This array is + // built by the building process from the "*.uni" file associated to + // the present driver (cf. FdtPlatfromDxe.inf). Examine your Build + // folder under your package's DEBUG folder and you will find the array + // defined in a xxxStrDefs.h file. + // + mFdtPlatformDxeHiiHandle = HiiAddPackages ( + &mFdtPlatformDxeHiiGuid, + ImageHandle, + FdtPlatformDxeStrings, + NULL + ); + + if (mFdtPlatformDxeHiiHandle != NULL) { + Status = gBS->InstallMultipleProtocolInterfaces ( + &ImageHandle, + &gEfiShellDynamicCommandProtocolGuid, + &mShellDynCmdProtocolSetFdt, + NULL + ); + if (EFI_ERROR (Status)) { + HiiRemovePackages (mFdtPlatformDxeHiiHandle); + } + } else { + Status = EFI_LOAD_ERROR; + } + if (EFI_ERROR (Status)) { + DEBUG (( + EFI_D_WARN, + "Unable to install \"setfdt\" EFI Shell command - %r \n", + Status + )); + } + } + return Status; } @@ -352,8 +453,6 @@ Error : return Status; } -======= - /** This is the shell command "setfdt" handler function. This function handles the command when it is invoked in the shell. @@ -739,4 +838,3 @@ EfiCodeToShellCode ( return ShellStatus; } ->>>>>>> 4ac4fed... EmbeddedPkg/FdtPlatformDxe: Fix typo issue diff --git a/EmbeddedPkg/Drivers/FdtPlatformDxe/FdtPlatformDxe.inf b/EmbeddedPkg/Drivers/FdtPlatformDxe/FdtPlatformDxe.inf index ea45baa2e3..64d14a26e7 100644 --- a/EmbeddedPkg/Drivers/FdtPlatformDxe/FdtPlatformDxe.inf +++ b/EmbeddedPkg/Drivers/FdtPlatformDxe/FdtPlatformDxe.inf @@ -15,6 +15,7 @@ [Defines] INF_VERSION = 0x00010006 BASE_NAME = FdtPlatformDxe + MODULE_UNI_FILE = FdtPlatformDxe.uni FILE_GUID = 6e9a4c69-57c6-4fcd-b083-4f2c3bdb6051 MODULE_TYPE = DXE_DRIVER VERSION_STRING = 0.1 @@ -22,11 +23,14 @@ [Sources.common] FdtPlatform.c + FdtPlatformDxe.uni [Packages] EmbeddedPkg/EmbeddedPkg.dec ArmPkg/ArmPkg.dec MdePkg/MdePkg.dec + MdeModulePkg/MdeModulePkg.dec + ShellPkg/ShellPkg.dec [LibraryClasses] UefiDriverEntryPoint @@ -36,9 +40,13 @@ DebugLib UefiBootServicesTableLib UefiRuntimeServicesTableLib + HiiLib + ShellLib [Protocols] + gEfiDevicePathToTextProtocolGuid gEfiDevicePathFromTextProtocolGuid + gEfiShellDynamicCommandProtocolGuid [Guids] gEfiEndOfDxeEventGroupGuid diff --git a/EmbeddedPkg/Drivers/FdtPlatformDxe/FdtPlatformDxe.uni b/EmbeddedPkg/Drivers/FdtPlatformDxe/FdtPlatformDxe.uni new file mode 100644 index 0000000000000000000000000000000000000000..2f8ddc272a4e3b99568f41cf0f277d6fa3d2d3c5 GIT binary patch literal 6674 zcmb`LTTfd@6ou!xQvbt|QIs?dkVJio(nzM5V1*cj?X*=ASr}}v1P0j-gg@VQeX~8D zbF*_y5yIy-m$mm^`!@gnv+cHB;-0%-_4l*;Nl)90YvTTJ*Y3vMyNMgQb9dop8vD$h zx?gmC?OwW9?l<>B_jPydKI(d)5od18zqxdmy2hDP|1@w@H_+Q#{W9{c+tJ*2i#-E| z#C5daNTU-Wml99hh zrfWA$&D(Kpx2xxa45q~0c^#bUw3?2BDYugE{t7qbzl zziU(;RaQf$&3ldNd)*8rgXsWlv(*0ZWF}j@@oa)Yre(%Yyw0~YcOnaI=d3W*o4&B0 zxy{^{Ev<5){~%?!BG0~S2-T5zbm-pex9+*vlui?Am1{?Dn_^m5cMY%Uu50Pq(EGST zv_ezNZ)il@uY9OGIL>cPSJq|Knyx$knWq;EY>E3*-@$e0cjOrl0a;7$n_87M4*l-O zdICG|*t4TijA2(BwWqPn{9D%#`T}yit})-mq-eOm7VE^-o6^ORc*UN(^0%P3{58?I zYv~bz|dZoKJg&A-taeEp&)5>_pOkBdASK8sb<`mbid#l1oe08>T)waIDTJv6f zaOj`1zGJ&ZuWJZRQ<&Of$UumQJ5R-;fky_;r+Q-%0^i0Pix`hJ{?>brd1)g1nI9qB zeJSQm@$~sTV%mzoyA8iKPa9r4$YPS1=J170@d6vwWEH;SX;Y!ErRD|?HeDEpd!8d5 z$-66#^gI{%QkR9XaZCKSU9!Is$v4S=E=d#pM;st&>X|)>li;y^LyWZ@6ETRtFZya& zH~gGvvnn`|qAL%>$6%#-B#5iv3`js2@xsDjw~z~9GO>+3^M4WVNykm;IAS#oKMvxd zv`&@*Ya*s4DZIv{Ds1S=A9v@`V;K{wYPqXednQz9wVVlX8h;^&lA{?*o_ZKVBKC%O zy^J5#7!t9AESqTqdzPVp>z;^J?=izxJiUD!xI~p~fQO!2NQ&Jj{R77$d6nBV*8fS)L&`JK{P(Oh=zNtdpxX3|iTTbXT1S%Q zDBG$mTjJ7@P*cgSM*G=g`6|qxz4l>xZxR2$7G?)suzsv-^Lrp)ku4M7VZ=%=MDMJ1 z)4AVzMRL+qfB9X4XGXm2S?1SaWID6W#I2xr>aYCj`R5=x6;4~xksc)KAU_%l(4DlE z|MTBoiBa?%^o%e$Mvm&}E$$GJ(2P|YCQ+#IvK2BNhJ6_??m|`!RdgPwSn-RSh+jASXSrxR#%fJMECAO z7VKeX40cLDbg~YKSP*+|B9L|3)K&CXVV`o0>+&7!$YFki1?ikPM*uT@5xkyg3~#X$ z{k&;7>r(+2(GspRlN@nX;AXJ7oy9D((Nqy)pqgsr5g7aW^WAM`Rc+=qm4AMn@|D1B z&LCocV6o9N9CR~{k>T)C57;70WKyhaVJyB;D zJiO=?hVjgM6x>gEE&j$ClbwJCYnsl>QC77B^-X!s<|L*gDUKIaE)o^9 zA{m~V=fKxIp`UGQr0J{pd+2?DGmyZ=XR-h_MtQH1#h2c%u5%}9zi^sW+>1BG{jhmo z+MftP=!=){lCSfrq3v`H4cVy^T+LSSM8b1fjp#PkJ|<&P-)VU)_w^KOI5FdVt4yLj zp^sg%MXgIobLuq;YsigO0q_;A67#IGdf4)jQW%e73!R@VQLFNm&F->KdB z-K%KZ&I)_laZil0^FZDm`@Sl$s;-sM?3G`a=xygUK?~swB;Dz)mX_)ybZ=zm^1fnU z`%KiG-TABeu33XK-GO3m-k;L;txE>FL%hTK#g=%JKTE00Vxe*iSt