RE: [PATCH 3.16 00/17] 3.16.1-stable review

From: Kukjin Kim
Date: Sun Aug 10 2014 - 20:22:34 EST


Olof Johansson wrote:
>
> On Sat, Aug 9, 2014 at 6:21 PM, Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote:
> > Hi Linus,
> >
> > On Sat, 9 Aug 2014 10:43:51 -0700 Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote:
> >>
> >> On Sat, Aug 9, 2014 at 10:15 AM, Guenter Roeck <linux@xxxxxxxxxxxx> wrote:
> >> >
> >> > ERROR (phandle_references): Reference to non-existent node or label
> >> > "disp_pd"
> >> >
> >> > ERROR: Input tree has errors, aborting (use -f to force output)
> >> > make[1]: *** [arch/arm/boot/dts/exynos5420-arndale-octa.dtb] Error 2
> >>
> >> Ok, so it's something else that was apparently known about in -next,
> >> but not mentioned when pushing to me. Oh well.
>
> Yeah, I had missed this one since I thought the DRM guys would fix it
> in their tree when it was reported.
>
Yeah I was not aware of the build error when I sent my tree to arm-soc for this
merge window...anyway, sorry for the error because of exynos stuff and I'll try
to look at patches more closely.

> It didn't show up for me since when I did my build/test before sending
> pull requests since that was done with a mainline branch that hadn't
> merged DRM yet.
>
> >> The new disp_dt use was added by commit 5a8da524049c ("ARM: dts:
> >> exynos5420: add dsi node"), while commit d51cad7df871 ("ARM: dts:
> >> remove display power domain for exynos5420") removed the node.
> >>
> >> I'll leave it broken until the ARM people get around to fixing it.
> >> Likely by just removing the disp_dt line from the dsi node, but I'll
> >> let the proper people sort it out.
> >
> > Yeah, that's the correct fix (it was even tested). Olof has the patch
> > for you.
>
> Yep, I've got the patch to do that in my current fixes branch that
> I'll send tonight.
>
Thanks, Olof.

And Sorry, Linus.

- Kukjin

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/