On Tue, 7 Jan 2025 at 15:58, Simon Glass sjg@chromium.org wrote:
Hi Ilias,
On Tue, 7 Jan 2025 at 06:37, Ilias Apalodimas ilias.apalodimas@linaro.org wrote:
Hi Simon,
On Mon, 6 Jan 2025 at 16:48, Simon Glass sjg@chromium.org wrote:
The allocate/free-pages functions return an address, so there is no need to convert it. Fix this.
Signed-off-by: Simon Glass sjg@chromium.org Fixes: c824a96d76d ("efi_loader: Use the log with memory-related...")
I can't find that commit in -master. Is there anything I am missing?
Sort-of. You, Tom and Heinrich NAKed this series. See [1]
Ok, please don't cc me on patches that are not destined for master or next. I have no interest in out-of-tree work
Thanks /Ilias
lib/efi_loader/efi_log.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/lib/efi_loader/efi_log.c b/lib/efi_loader/efi_log.c index 4b9c6727fe7..ad490bb431c 100644 --- a/lib/efi_loader/efi_log.c +++ b/lib/efi_loader/efi_log.c @@ -340,8 +340,7 @@ void show_rec(int seq, struct efil_rec_hdr *rec_hdr) show_ulong("pgs", (ulong)rec->pages); show_addr("mem", (ulong)rec->memory); if (rec_hdr->ended) {
show_addr("*mem",(ulong)map_to_sysmem((void *)rec->e_memory));
show_addr("*mem", rec->e_memory); show_ret(rec_hdr->e_ret); } break;@@ -349,7 +348,7 @@ void show_rec(int seq, struct efil_rec_hdr *rec_hdr) case EFILT_FREE_PAGES: { struct efil_free_pages *rec = start;
show_addr("mem", map_to_sysmem((void *)rec->memory));
show_addr("mem", rec->memory); show_ulong("pag", (ulong)rec->pages); if (rec_hdr->ended) show_ret(rec_hdr->e_ret);-- 2.34.1
Regards, Simon
[1] https://lore.kernel.org/u-boot/CAC_iWjKtaN54B98OKbkoXkC_GmKJ=x+M4=UY_O6roSOp...