??????[PATCH v1] linker_lists:Update the alignment using CONFIG_LINKER_LIST_ALIGN

[??????]

Dear Simon Glass,

I noticed that the attached patch remains unmerged. While I believe it may 
have undergone prior auditing, I'm unable to locate the corresponding review 
records. Could you kindly confirm its status and initiate the merging process 
if appropriate? Please let me know if you need any further information or 
clarification regarding this request.

Best regards,
Liya



------------------ ???????? ------------------
??????:                                                                         
                                               "1425075683"                     
                                                               
<1425075...@qq.com&gt;;
????????:&nbsp;2025??1??18??(??????) ????11:16
??????:&nbsp;"u-boot"<u-boot@lists.denx.de&gt;;
????:&nbsp;"Simon Glass"<s...@chromium.org&gt;;"Tom 
Rini"<tr...@konsulko.com&gt;;"??????"<1425075...@qq.com&gt;;
????:&nbsp;[PATCH v1] linker_lists:Update the alignment using 
CONFIG_LINKER_LIST_ALIGN



From: Liya Huang <1425075...@qq.com&gt;

This patch updates the alignment of linker lists to use the
CONFIG_LINKER_LIST_ALIGN macro instead of a hardcoded value.
This ensures that the alignment is consistent with the configuration.
Replace __attribute__((unused)) with __maybe_unused and
__always_unused to eliminate the warning of checkpatch.pl.

Signed-off-by: Liya Huang <1425075...@qq.com&gt;
---

Changes in v1:
- Due to an error, the patch was resubmitted, leaving the code unchanged

&nbsp;include/linker_lists.h | 21 +++++++++++++--------
&nbsp;1 file changed, 13 insertions(+), 8 deletions(-)

diff --git a/include/linker_lists.h b/include/linker_lists.h
index f9a2ee0c76..e8791430d7 100644
--- a/include/linker_lists.h
+++ b/include/linker_lists.h
@@ -68,8 +68,9 @@
&nbsp; *&nbsp;&nbsp; };
&nbsp; */
&nbsp;#define ll_entry_declare(_type, _name, _list)                             
\
-       _type _u_boot_list_2_##_list##_2_##_name __aligned(4)           \
-                       __attribute__((unused))                         \
+       _type _u_boot_list_2_##_list##_2_##_name                \
+                       __aligned(CONFIG_LINKER_LIST_ALIGN)             \
+                       __maybe_unused                                          
        \
&nbsp;                  __section("__u_boot_list_2_"#_list"_2_"#_name)
&nbsp;
&nbsp;/**
@@ -91,8 +92,9 @@
&nbsp; *&nbsp;&nbsp; };
&nbsp; */
&nbsp;#define ll_entry_declare_list(_type, _name, _list)                        
\
-       _type _u_boot_list_2_##_list##_2_##_name[] __aligned(4)         \
-                       __attribute__((unused))                         \
+       _type _u_boot_list_2_##_list##_2_##_name[]              \
+                       __aligned(CONFIG_LINKER_LIST_ALIGN)             \
+                       __maybe_unused                                          
        \
&nbsp;                  __section("__u_boot_list_2_"#_list"_2_"#_name)
&nbsp;
&nbsp;/*
@@ -125,7 +127,7 @@
&nbsp;#define ll_entry_start(_type, _list)                                      
\
&nbsp;({                                                                        
\
&nbsp;  static char start[0] __aligned(CONFIG_LINKER_LIST_ALIGN)        \
-               __attribute__((unused))                                 \
+               __always_unused                 \
&nbsp;          __section("__u_boot_list_2_"#_list"_1");                        
\
&nbsp;  _type * tmp = (_type *)&amp;start;                                      
\
&nbsp;  asm("":"+r"(tmp));                                              \
@@ -153,7 +155,8 @@
&nbsp; */
&nbsp;#define ll_entry_end(_type, _list)                                        
\
&nbsp;({                                                                        
\
-       static char end[0] __aligned(4) __attribute__((unused))         \
+       static char end[0] __aligned(CONFIG_LINKER_LIST_ALIGN)  \
+               __always_unused                 \
&nbsp;          __section("__u_boot_list_2_"#_list"_3");                        
\
&nbsp;  _type * tmp = (_type *)&amp;end;                                        
\
&nbsp;  asm("":"+r"(tmp));                                              \
@@ -249,7 +252,8 @@
&nbsp; */
&nbsp;#define ll_start(_type)                                                   
\
&nbsp;({                                                                        
\
-       static char start[0] __aligned(4) __attribute__((unused))       \
+       static char start[0] __aligned(CONFIG_LINKER_LIST_ALIGN)        \
+               __always_unused                                                 
        \
&nbsp;          __section("__u_boot_list_1");                           \
&nbsp;  _type * tmp = (_type *)&amp;start;                                      
\
&nbsp;  asm("":"+r"(tmp));                                              \
@@ -274,7 +278,8 @@
&nbsp; */
&nbsp;#define ll_end(_type)                                                     
\
&nbsp;({                                                                        
\
-       static char end[0] __aligned(4) __attribute__((unused))         \
+       static char end[0] __aligned(CONFIG_LINKER_LIST_ALIGN)  \
+               __always_unused                                                 
        \
&nbsp;          __section("__u_boot_list_3");                           \
&nbsp;  _type * tmp = (_type *)&amp;end;                                        
\
&nbsp;  asm("":"+r"(tmp));                                              \
-- 
2.25.1