* [PATCH] ArmPkg/PlatformBootManagerLib: add missing newline
@ 2017-12-09 9:56 Ard Biesheuvel
2017-12-11 11:48 ` Leif Lindholm
0 siblings, 1 reply; 3+ messages in thread
From: Ard Biesheuvel @ 2017-12-09 9:56 UTC (permalink / raw)
To: edk2-devel; +Cc: leif.lindholm, Ard Biesheuvel
Add a missing newline to the version string output that is displayed
on the serial console, or the next line will be appended to it.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
---
ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c b/ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c
index ccdb3e0a5d78..61ab61ccc780 100644
--- a/ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c
+++ b/ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c
@@ -549,7 +549,7 @@ PlatformBootManagerAfterConsole (
Status = BootLogoEnableLogo ();
if (EFI_ERROR (Status)) {
if (FirmwareVerLength > 0) {
- Print (VERSION_STRING_PREFIX L"%s",
+ Print (VERSION_STRING_PREFIX L"%s\n",
PcdGetPtr (PcdFirmwareVersionString));
}
Print (L"Press ESCAPE for boot options ");
--
2.11.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ArmPkg/PlatformBootManagerLib: add missing newline
2017-12-09 9:56 [PATCH] ArmPkg/PlatformBootManagerLib: add missing newline Ard Biesheuvel
@ 2017-12-11 11:48 ` Leif Lindholm
2017-12-11 18:06 ` Ard Biesheuvel
0 siblings, 1 reply; 3+ messages in thread
From: Leif Lindholm @ 2017-12-11 11:48 UTC (permalink / raw)
To: Ard Biesheuvel; +Cc: edk2-devel
On Sat, Dec 09, 2017 at 09:56:24AM +0000, Ard Biesheuvel wrote:
> Add a missing newline to the version string output that is displayed
> on the serial console, or the next line will be appended to it.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
> ---
> ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c b/ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c
> index ccdb3e0a5d78..61ab61ccc780 100644
> --- a/ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c
> +++ b/ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c
> @@ -549,7 +549,7 @@ PlatformBootManagerAfterConsole (
> Status = BootLogoEnableLogo ();
> if (EFI_ERROR (Status)) {
> if (FirmwareVerLength > 0) {
> - Print (VERSION_STRING_PREFIX L"%s",
> + Print (VERSION_STRING_PREFIX L"%s\n",
> PcdGetPtr (PcdFirmwareVersionString));
> }
> Print (L"Press ESCAPE for boot options ");
> --
> 2.11.0
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ArmPkg/PlatformBootManagerLib: add missing newline
2017-12-11 11:48 ` Leif Lindholm
@ 2017-12-11 18:06 ` Ard Biesheuvel
0 siblings, 0 replies; 3+ messages in thread
From: Ard Biesheuvel @ 2017-12-11 18:06 UTC (permalink / raw)
To: Leif Lindholm; +Cc: edk2-devel@lists.01.org
On 11 December 2017 at 11:48, Leif Lindholm <leif.lindholm@linaro.org> wrote:
> On Sat, Dec 09, 2017 at 09:56:24AM +0000, Ard Biesheuvel wrote:
>> Add a missing newline to the version string output that is displayed
>> on the serial console, or the next line will be appended to it.
>>
>> Contributed-under: TianoCore Contribution Agreement 1.1
>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>
> Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
>
Thanks. Pushed as 26b99f3bb3f1fc34962e203ba431447dcd2ad946
>> ---
>> ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c b/ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c
>> index ccdb3e0a5d78..61ab61ccc780 100644
>> --- a/ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c
>> +++ b/ArmPkg/Library/PlatformBootManagerLib/PlatformBm.c
>> @@ -549,7 +549,7 @@ PlatformBootManagerAfterConsole (
>> Status = BootLogoEnableLogo ();
>> if (EFI_ERROR (Status)) {
>> if (FirmwareVerLength > 0) {
>> - Print (VERSION_STRING_PREFIX L"%s",
>> + Print (VERSION_STRING_PREFIX L"%s\n",
>> PcdGetPtr (PcdFirmwareVersionString));
>> }
>> Print (L"Press ESCAPE for boot options ");
>> --
>> 2.11.0
>>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-12-11 18:01 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-12-09 9:56 [PATCH] ArmPkg/PlatformBootManagerLib: add missing newline Ard Biesheuvel
2017-12-11 11:48 ` Leif Lindholm
2017-12-11 18:06 ` Ard Biesheuvel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox