Instead of using an ambiguous parameter name 'size' and having a comment to clarify what's meant by it, just rename it nelems to make clear that it's the number of elements that should be specified as argument. Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> --- v1 -> v2: - no change --- include/linux/align.h | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/include/linux/align.h b/include/linux/align.h index 3b0c60d6e9ae..ff3134db9ea1 100644 --- a/include/linux/align.h +++ b/include/linux/align.h @@ -15,12 +15,9 @@ /* * The STACK_ALIGN_ARRAY macro is used to allocate a buffer on the stack that * meets a minimum alignment requirement. - * - * Note that the size parameter is the number of array elements to allocate, - * not the number of bytes. - */ -#define STACK_ALIGN_ARRAY(type, name, size, align) \ - char __##name[sizeof(type) * (size) + (align) - 1]; \ - type *name = (type *)ALIGN((unsigned long)__##name, align) + */ +#define STACK_ALIGN_ARRAY(type, name, nelems, align) \ + char __##name[sizeof(type) * (nelems) + (align) - 1]; \ + type *name = (type *)ALIGN((uintptr_t)__##name, align) #endif -- 2.39.5
