[PATCH 1/4] ARM: imx: imx8m: Introduce and use UART_BASE_ADDR(n)

Tom Rini trini at konsulko.com
Mon Apr 25 15:39:38 CEST 2022


On Mon, Apr 25, 2022 at 03:13:53PM +0200, Marek Vasut wrote:
> 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.

Right, sorry, I meant DM enabled in SPL to start with.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20220425/ddd413c2/attachment.sig>


More information about the U-Boot mailing list