Re: [PATCH v1 21/26] imx8mn: synchronise device tree with linux

Adam Ford
2022-08-12T12:49:26Z

On Wed, Aug 3, 2022 at 8:02 AM Marcel Ziswiler
<marcel.ziswiler@toradex.com> wrote:
>
> Hi Adam
>
> On Sun, 2022-07-31 at 11:58 -0500, Adam Ford wrote:
> > On Thu, Jul 21, 2022 at 8:44 AM Marcel Ziswiler <marcel@ziswiler.com> wrote:
> > >
> > > From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> > >
> > > Synchronise device tree with linux v5.19-rc5.
> > >
> > > Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> >
> > For what it's worth to others who may have been impacted, this patch
> > broke my nano booting because the imx8mn.dtsi now has the UART's as
> > subnodes to spba1 which wasn't being included in SPL's device tree.
> > It's likely to impact others who are booting with DM_SERIAL enabled on
> > their Nano.
>
> Sorry about that.
>
> > To fix, I had to add a small line of code to my
> > imx8mn-beacon-kit-u-boot.dtsi file with the following:
> >
> > &spba1 {
> > u-boot,dm-spl;
> > };
> >
> > With that, my board boots again. If anyone else is using DM_SERIAL
> > and having issues no longer starting, give that a try.
> >
> > Since there are few nano boards with common -u-boot.dtsi entries, it
> > seems like it makes sense to have a common imx8mn-u-boot.dtsi file
> > like we do for Mini and Plus. I'll try to work on that as I have
> > time. (hopefully today)
>
> Thanks for your help in correcting this.

Marcel,

Have you seen the patch series I posted at [1]?
I know Tim Harvey replied 'tested-by' to the cover letter. Might you
have a chance to test it?

[1] - https://patchwork.ozlabs.org/project/uboot/list/?series=312016
>
> Cheers
>
> Marcel