[U-Boot] [PATCH v2] serial: mxc: Add compatibles to NXP's serial driver (for imx21 and imx53)

[U-Boot] [PATCH v2] serial: mxc: Add compatibles to NXP's serial driver (for imx21 and imx53)

[Lukasz Majewski]

This patch fixes not enabled uart2 (and hence serial console) on i.MX53
devices. After following commit 1d255904c306 ("ARM: dts: imx: imx53:
Synchronize iMX53 DT with Linux") from the uart2 compatible the
'fsl,imx7d-uart' has been removed (which was correct).

However, the root cause of the problem was the commit 98d62e618bb9 ("arm:
imx: add i.MX53 Beckhoff CX9020 Embedded PC") which introduced this
compatible.

Moreover, without this patch all i.MX53 UARTs are not usable as neither
'fsl,imx53-uart' nor 'fsl,imx21-uart' are in compatible IDs in
drivers/serial/serial_mxc.c file.

The fix is to add 'fsl,imx53-uart' and 'fsl,imx21-uart' as compatibles
for the aforementioned serial driver (those are also defined in the Linux
kernel).

Signed-off-by: Lukasz Majewski <lukma@denx.de>

---

Changes in v2:
- Add 'fsl,imx21-uart' to the compatible table
- Change the subject tags to 'serial: mxc:"

 drivers/serial/serial_mxc.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/serial/serial_mxc.c b/drivers/serial/serial_mxc.c
index a435e68005..e73e979897 100644
--- a/drivers/serial/serial_mxc.c
+++ b/drivers/serial/serial_mxc.c
@@ -342,6 +342,8 @@ static int mxc_serial_ofdata_to_platdata(struct udevice *dev)
 }
 
 static const struct udevice_id mxc_serial_ids[] = {
+	{ .compatible = "fsl,imx21-uart" },
+	{ .compatible = "fsl,imx53-uart" },
 	{ .compatible = "fsl,imx6sx-uart" },
 	{ .compatible = "fsl,imx6ul-uart" },
 	{ .compatible = "fsl,imx7d-uart" },
-- 
2.11.0

[U-Boot] [PATCH v2] serial: mxc: Add compatibles to NXP's serial driver (for imx21 and imx53)

[Marek Vasut]

On 7/9/19 5:00 PM, Lukasz Majewski wrote:
> This patch fixes not enabled uart2 (and hence serial console) on i.MX53
> devices. After following commit 1d255904c306 ("ARM: dts: imx: imx53:
> Synchronize iMX53 DT with Linux") from the uart2 compatible the
> 'fsl,imx7d-uart' has been removed (which was correct).
> 
> However, the root cause of the problem was the commit 98d62e618bb9 ("arm:
> imx: add i.MX53 Beckhoff CX9020 Embedded PC") which introduced this
> compatible.
> 
> Moreover, without this patch all i.MX53 UARTs are not usable as neither
> 'fsl,imx53-uart' nor 'fsl,imx21-uart' are in compatible IDs in
> drivers/serial/serial_mxc.c file.
> 
> The fix is to add 'fsl,imx53-uart' and 'fsl,imx21-uart' as compatibles
> for the aforementioned serial driver (those are also defined in the Linux
> kernel).
> 
> Signed-off-by: Lukasz Majewski <lukma@denx.de>

Acked-by: Marek Vasut <marex@denx.de>

[U-Boot] [PATCH v2] serial: mxc: Add compatibles to NXP's serial driver (for imx21 and imx53)

[sbabic at denx.de]

> This patch fixes not enabled uart2 (and hence serial console) on i.MX53
> devices. After following commit 1d255904c306 ("ARM: dts: imx: imx53:
> Synchronize iMX53 DT with Linux") from the uart2 compatible the
> 'fsl,imx7d-uart' has been removed (which was correct).
> However, the root cause of the problem was the commit 98d62e618bb9 ("arm:
> imx: add i.MX53 Beckhoff CX9020 Embedded PC") which introduced this
> compatible.
> Moreover, without this patch all i.MX53 UARTs are not usable as neither
> 'fsl,imx53-uart' nor 'fsl,imx21-uart' are in compatible IDs in
> drivers/serial/serial_mxc.c file.
> The fix is to add 'fsl,imx53-uart' and 'fsl,imx21-uart' as compatibles
> for the aforementioned serial driver (those are also defined in the Linux
> kernel).
> Signed-off-by: Lukasz Majewski <lukma@denx.de>
> Acked-by: Marek Vasut <marex@denx.de>

Applied to u-boot-imx, master, thanks !

Best regards,
Stefano Babic

-- 
=====================================================================
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: sbabic at denx.de
=====================================================================