Hi Michal,

On 05/04/2022 10:16, Michal Orzel wrote:
  #if defined(BUILD_ID)
@@ -109,12 +104,7 @@ SECTIONS
         *(.data.schedulers)
         __end_schedulers_array = .;
-#ifdef CONFIG_HYPFS
-       . = ALIGN(8);

This will be replaced with POINTER_ALIGN which is 4-byte on Arm32. AFAICT, there are no 64-bit value used in struct param_hypfs. So it should be fine.

That said, I think this is worth mentioning in the commit message.

The rest of this patch looks good to me.

Cheers,

--
Julien Grall

Reply via email to