Hi all,
After merging the imx-mxs tree, today's linux-next build (arm
multi_v7_defconfig) failed like this:
Error: arch/arm/boot/dts/imx6ull-colibri.dtsi:536.4-5 syntax error
FATAL ERROR: Unable to parse input tree
make[2]: *** [scripts/Makefile.lib:285: arch/arm/boot/dts/imx6ull-colibri-wifi-eval-v3.dtb] Error 1
Error: arch/arm/boot/dts/imx6ull-colibri.dtsi:536.4-5 syntax error
FATAL ERROR: Unable to parse input tree
make[2]: *** [scripts/Makefile.lib:285: arch/arm/boot/dts/imx6ull-colibri-eval-v3.dtb] Error 1
arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi:422.28-424.7: Warning (graph_endpoint): /soc/aips-bus@2100000/i2c@21a0000/edp-bridge@68/ports/port@2/endpoint: graph connection to node '/panel/port/endpoint' is not bidirectional
arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi:422.28-424.7: Warning (graph_endpoint): /soc/aips-bus@2100000/i2c@21a0000/edp-bridge@68/ports/port@2/endpoint: graph connection to node '/panel/port/endpoint' is not bidirectional
Caused by commit
e7ebb215cb78 ("ARM: dts: colibri-imx6ull: correct wrong pinmuxing and add comments")
The warning was (probably) introduced by commit
e79295edf1df ("ARM: dts: imx6: RDU2: link eDP bridge to panel")
I have used the imx-mxs tree from next-20191211 for today.
--
Cheers,
Stephen Rothwell
Hi Stephen,
On Thu, Dec 12, 2019 at 08:35:56AM +1100, Stephen Rothwell wrote:
> Hi all,
>
> After merging the imx-mxs tree, today's linux-next build (arm
> multi_v7_defconfig) failed like this:
Sorry for the breakage.
>
> Error: arch/arm/boot/dts/imx6ull-colibri.dtsi:536.4-5 syntax error
> FATAL ERROR: Unable to parse input tree
> make[2]: *** [scripts/Makefile.lib:285: arch/arm/boot/dts/imx6ull-colibri-wifi-eval-v3.dtb] Error 1
> Error: arch/arm/boot/dts/imx6ull-colibri.dtsi:536.4-5 syntax error
> FATAL ERROR: Unable to parse input tree
> make[2]: *** [scripts/Makefile.lib:285: arch/arm/boot/dts/imx6ull-colibri-eval-v3.dtb] Error 1
> arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi:422.28-424.7: Warning (graph_endpoint): /soc/aips-bus@2100000/i2c@21a0000/edp-bridge@68/ports/port@2/endpoint: graph connection to node '/panel/port/endpoint' is not bidirectional
> arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi:422.28-424.7: Warning (graph_endpoint): /soc/aips-bus@2100000/i2c@21a0000/edp-bridge@68/ports/port@2/endpoint: graph connection to node '/panel/port/endpoint' is not bidirectional
>
> Caused by commit
>
> e7ebb215cb78 ("ARM: dts: colibri-imx6ull: correct wrong pinmuxing and add comments")
>
> The warning was (probably) introduced by commit
>
> e79295edf1df ("ARM: dts: imx6: RDU2: link eDP bridge to panel")
I have just taken these two commits out.
>
> I have used the imx-mxs tree from next-20191211 for today.
Thanks.
Shawn
Hi Shawn,
On Do, 2019-12-12 at 09:33 +0800, Shawn Guo wrote:
> Hi Stephen,
>
> On Thu, Dec 12, 2019 at 08:35:56AM +1100, Stephen Rothwell wrote:
> > Hi all,
> >
> > After merging the imx-mxs tree, today's linux-next build (arm
> > multi_v7_defconfig) failed like this:
>
> Sorry for the breakage.
>
[...]
> > The warning was (probably) introduced by commit
> >
> > e79295edf1df ("ARM: dts: imx6: RDU2: link eDP bridge to panel")
>
> I have just taken these two commits out.
The commit message of this change should maybe have been clearer, but
the warning is expected. The bootloader fixes the DT before passing it
on to Linux by completing the other direction of link if eDP is
enabled.
How do we want to handle this? If you totally dislike the warning I can
get around it by not adding the link and adding some more complexity to
the bootloader, but given the choice I would like to keep the
incomplete link in the Linux DT and keep bootloader complexity down.
Regards,
Lucas
On Thu, Dec 12, 2019 at 10:45:34AM +0100, Lucas Stach wrote:
> Hi Shawn,
>
> On Do, 2019-12-12 at 09:33 +0800, Shawn Guo wrote:
> > Hi Stephen,
> >
> > On Thu, Dec 12, 2019 at 08:35:56AM +1100, Stephen Rothwell wrote:
> > > Hi all,
> > >
> > > After merging the imx-mxs tree, today's linux-next build (arm
> > > multi_v7_defconfig) failed like this:
> >
> > Sorry for the breakage.
> >
> [...]
> > > The warning was (probably) introduced by commit
> > >
> > > e79295edf1df ("ARM: dts: imx6: RDU2: link eDP bridge to panel")
> >
> > I have just taken these two commits out.
>
> The commit message of this change should maybe have been clearer, but
> the warning is expected. The bootloader fixes the DT before passing it
> on to Linux by completing the other direction of link if eDP is
> enabled.
>
> How do we want to handle this? If you totally dislike the warning I can
> get around it
Yes, please. I do not like the warning.
Shawn