On 9/9/21 10:57 AM, Simon Glass wrote:
Hi Heinrich,

On Wed, 8 Sept 2021 at 12:25, Heinrich Schuchardt <xypron.g...@gmx.de> wrote:



On 9/8/21 3:34 PM, Simon Glass wrote:
Add a message here so that both paths of memory allocation are reported.

Signed-off-by: Simon Glass <s...@chromium.org>
---

   lib/efi/efi_app.c | 1 +
   1 file changed, 1 insertion(+)

diff --git a/lib/efi/efi_app.c b/lib/efi/efi_app.c
index e0666d1fae2..c8f4784eb7d 100644
--- a/lib/efi/efi_app.c
+++ b/lib/efi/efi_app.c
@@ -97,6 +97,7 @@ static efi_status_t setup_memory(struct efi_priv *priv)
                       return ret;
               priv->use_pool_for_malloc = true;
       } else {
+             printf("(using allocated RAM address %lx) ", (ulong)addr);

Shouldn't this be log_debug()?

It matches the one a few lines higher. I suppose it could be a debug
statement, but allocation is an area that seems to different on
different UEFIs, so I have it more prominent for the moment, until we
figure it out.

If you want it more prominent, isn't it log_info() that you were promoting?

Best regards

Heinrich

Reply via email to