On 20.05.22 10:11, Stefano Babic wrote:
On 20.05.22 09:45, Marek Vasut wrote:
On 5/20/22 09:30, Stefano Babic wrote:
Hi Marek,
On 24.04.22 23:44, Marek Vasut wrote:
Introduce helper macro UART_BASE_ADDR(n), which returns Nth UART base address. Convert all board configurations to this new macro. This is the first step toward switching CONFIG_MXC_UART_BASE to Kconfig. This is a clean up, no functional change.
The new macro contains compile-time test to verify N is in suitable range. The test works such that it multiplies constant N by constant double-negation of size of a non-empty structure, i.e. it multiplies constant N by constant 1 in each successful compilation case.
The non-empty structure may contain C11 _Static_assert(), make use of this and place the kernel variant of static assert in there, so that it performs the compile-time check for N in the correct range. Note that it is not possible to directly use static_assert in compound statements, hence this convoluted construct.
Signed-off-by: Marek Vasut marex@denx.de Cc: Fabio Estevam festevam@denx.de Cc: Peng Fan peng.fan@nxp.com Cc: Stefano Babic sbabic@denx.de
I have not found the reason, but this breaks MX51:
All this UART_BASE_ADDR stuff is postponed, just drop for now.
Ok, understood - I remove them.
Well, there is at least one patch depending on this:
http://patchwork.ozlabs.org/project/uboot/patch/20220505074341.24086-2-peng....
I will first try just to remove the i.MX5 patch, it seems to me that the MX51 is the only architecture that cannot be built.
Regards, Stefano
Best regards, Stefano