On 4/25/22 15:05, Tom Rini wrote:
On Mon, Apr 25, 2022 at 03:02:29PM +0200, Marek Vasut wrote:
On 4/25/22 14:46, Fabio Estevam wrote:
Hi Marek,
On 24/04/2022 18: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.
As DM_SERIAL is mandatory now, we should get rid of CONFIG_MXC_UART_BASE.
I would prefer a patch that removes CONFIG_MXC_UART_BASE instead.
DM is mandatory in SPL too ? I doubt it.
It is strongly encouraged, but not mandatory. It is not used on I guess imx27/31/5, but is for all of the imx8 families. So CONFIG_MXC_UART_BASE still needs to be moved out of board.h files (and perhaps out of CONFIG namespace since it's not configurable, it's part of the SoC) and perhaps the path for imx8* is to drop the references since it's all DM?
MX8M SPL is not DM serial either. But I think I will just postpone this cleanup until I have time to finish it.