On Tue, Jan 17, 2023 at 04:24:33PM +0100, Mårten Lindahl wrote: > In order to allow the chip_spec array reference the function pointers > for interrupts, the code for these functions need to be moved above the > chip_spec array. > > This is a prestep to support a more generic setup of interrupts. ... > + u16 buffer[8] __aligned(8) = {0}; /* 1x16-bit + naturally aligned ts */ I understand that this just a code move without a single change, but I have to ask. Don't we use the specific struct layout for this? Also, 0 is redundant. P.S. Maybe considered as a followup change(s). -- With Best Regards, Andy Shevchenko