* [U-Boot] [PATCH] FIX: ARM: IMX: Add compatible 'fsl, imx53-uart' to NXP's serial driver
@ 2019-07-09 14:04 Lukasz Majewski
2019-07-09 14:23 ` Marek Vasut
0 siblings, 1 reply; 4+ messages in thread
From: Lukasz Majewski @ 2019-07-09 14:04 UTC (permalink / raw)
To: u-boot
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' as a compatible for the aforementioned
serial driver (it is also defined in the Linux kernel).
Signed-off-by: Lukasz Majewski <lukma@denx.de>
---
drivers/serial/serial_mxc.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/serial/serial_mxc.c b/drivers/serial/serial_mxc.c
index a435e68005..af40576f66 100644
--- a/drivers/serial/serial_mxc.c
+++ b/drivers/serial/serial_mxc.c
@@ -342,6 +342,7 @@ static int mxc_serial_ofdata_to_platdata(struct udevice *dev)
}
static const struct udevice_id mxc_serial_ids[] = {
+ { .compatible = "fsl,imx53-uart" },
{ .compatible = "fsl,imx6sx-uart" },
{ .compatible = "fsl,imx6ul-uart" },
{ .compatible = "fsl,imx7d-uart" },
--
2.11.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] FIX: ARM: IMX: Add compatible 'fsl, imx53-uart' to NXP's serial driver
2019-07-09 14:04 [U-Boot] [PATCH] FIX: ARM: IMX: Add compatible 'fsl, imx53-uart' to NXP's serial driver Lukasz Majewski
@ 2019-07-09 14:23 ` Marek Vasut
2019-07-09 15:06 ` Lukasz Majewski
0 siblings, 1 reply; 4+ messages in thread
From: Marek Vasut @ 2019-07-09 14:23 UTC (permalink / raw)
To: u-boot
On 7/9/19 4:04 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' as a compatible for the aforementioned
> serial driver (it is also defined in the Linux kernel).
>
> Signed-off-by: Lukasz Majewski <lukma@denx.de>
The tags in subject should be "serial: mxc: ..."
> ---
>
> drivers/serial/serial_mxc.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/serial/serial_mxc.c b/drivers/serial/serial_mxc.c
> index a435e68005..af40576f66 100644
> --- a/drivers/serial/serial_mxc.c
> +++ b/drivers/serial/serial_mxc.c
> @@ -342,6 +342,7 @@ static int mxc_serial_ofdata_to_platdata(struct udevice *dev)
> }
>
> static const struct udevice_id mxc_serial_ids[] = {
Add fsl,imx21-uart to catch all the remaining cases ?
> + { .compatible = "fsl,imx53-uart" },
> { .compatible = "fsl,imx6sx-uart" },
> { .compatible = "fsl,imx6ul-uart" },
> { .compatible = "fsl,imx7d-uart" },
>
[...]
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] FIX: ARM: IMX: Add compatible 'fsl, imx53-uart' to NXP's serial driver
2019-07-09 14:23 ` Marek Vasut
@ 2019-07-09 15:06 ` Lukasz Majewski
2019-07-09 15:41 ` Marek Vasut
0 siblings, 1 reply; 4+ messages in thread
From: Lukasz Majewski @ 2019-07-09 15:06 UTC (permalink / raw)
To: u-boot
Hi Marek,
> On 7/9/19 4:04 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' as a compatible for the
> > aforementioned serial driver (it is also defined in the Linux
> > kernel).
> >
> > Signed-off-by: Lukasz Majewski <lukma@denx.de>
>
> The tags in subject should be "serial: mxc: ..."
>
> > ---
> >
> > drivers/serial/serial_mxc.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/serial/serial_mxc.c
> > b/drivers/serial/serial_mxc.c index a435e68005..af40576f66 100644
> > --- a/drivers/serial/serial_mxc.c
> > +++ b/drivers/serial/serial_mxc.c
> > @@ -342,6 +342,7 @@ static int mxc_serial_ofdata_to_platdata(struct
> > udevice *dev) }
> >
> > static const struct udevice_id mxc_serial_ids[] = {
>
> Add fsl,imx21-uart to catch all the remaining cases ?
>
I've posted v2 of this patch.
Just one question - I've noticed that there was some work done for
m53 menlo board. It seems like it could be affected by this issue (if it
uses serial console in u-boot).
> > + { .compatible = "fsl,imx53-uart" },
> > { .compatible = "fsl,imx6sx-uart" },
> > { .compatible = "fsl,imx6ul-uart" },
> > { .compatible = "fsl,imx7d-uart" },
> >
>
> [...]
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190709/fe69d9c0/attachment.sig>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] FIX: ARM: IMX: Add compatible 'fsl, imx53-uart' to NXP's serial driver
2019-07-09 15:06 ` Lukasz Majewski
@ 2019-07-09 15:41 ` Marek Vasut
0 siblings, 0 replies; 4+ messages in thread
From: Marek Vasut @ 2019-07-09 15:41 UTC (permalink / raw)
To: u-boot
On 7/9/19 5:06 PM, Lukasz Majewski wrote:
> Hi Marek,
>
>> On 7/9/19 4:04 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' as a compatible for the
>>> aforementioned serial driver (it is also defined in the Linux
>>> kernel).
>>>
>>> Signed-off-by: Lukasz Majewski <lukma@denx.de>
>>
>> The tags in subject should be "serial: mxc: ..."
>>
>>> ---
>>>
>>> drivers/serial/serial_mxc.c | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/drivers/serial/serial_mxc.c
>>> b/drivers/serial/serial_mxc.c index a435e68005..af40576f66 100644
>>> --- a/drivers/serial/serial_mxc.c
>>> +++ b/drivers/serial/serial_mxc.c
>>> @@ -342,6 +342,7 @@ static int mxc_serial_ofdata_to_platdata(struct
>>> udevice *dev) }
>>>
>>> static const struct udevice_id mxc_serial_ids[] = {
>>
>> Add fsl,imx21-uart to catch all the remaining cases ?
>>
>
> I've posted v2 of this patch.
>
> Just one question - I've noticed that there was some work done for
> m53 menlo board. It seems like it could be affected by this issue (if it
> uses serial console in u-boot).
Odd, when I tested it, I had no problem with serial console, but it
would make sense that it is affected. I'll take a look.
[...]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-07-09 15:41 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-09 14:04 [U-Boot] [PATCH] FIX: ARM: IMX: Add compatible 'fsl, imx53-uart' to NXP's serial driver Lukasz Majewski
2019-07-09 14:23 ` Marek Vasut
2019-07-09 15:06 ` Lukasz Majewski
2019-07-09 15:41 ` Marek Vasut
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.