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 <ma...@denx.de>
Cc: Fabio Estevam <feste...@denx.de>
Cc: Peng Fan <peng....@nxp.com>
Cc: Stefano Babic <sba...@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....@oss.nxp.com/

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



--
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=====================================================================

Reply via email to