Re: [PATCH] arc: Add "include chroot" for DT bindings

From: Alexey Brodkin
Date: Tue May 30 2017 - 13:04:38 EST


Hi Rob,

On Tue, 2017-05-30 at 11:56 -0500, Rob Herring wrote:
> On Tue, May 30, 2017 at 8:22 AM, Alexey Brodkin
> <Alexey.Brodkin@xxxxxxxxxxxx> wrote:
> >
> > Basically this extends
> > c58299aa8754 "kbuild: create an "include chroot" for DT bindings" for
> > ARC where we extensively use Device Tree and there're good reasons
> > to use DT bindings, especially if those are required.
> >
> > Otherwise on attempt to compile .dts with something like:
> > -------------------->8---------------------
> > Â#include <dt-bindings/net/ti-dp83867.h>
> > -------------------->8---------------------
> >
> > we inevitably see this compilation failure:
> > -------------------->8---------------------
> > arch/arc/boot/dts/hsdk.dts:16:40: fatal error: dt-bindings/net/ti-dp83867.h: No such file or directory
> > Â#include <dt-bindings/net/ti-dp83867.h>
> > ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ^
> > compilation terminated.
> > -------------------->8---------------------
> >
> > Signed-off-by: Alexey Brodkin <abrodkin@xxxxxxxxxxxx>
> > Cc: Stephen Warren <swarren@xxxxxxxxxxxxx>
> > Cc: Rob Herring <robh+dt@xxxxxxxxxx>
> > ---
> > Âarch/arc/boot/dts/include/dt-bindings | 1 +
> > Â1 file changed, 1 insertion(+)
> > Âcreate mode 120000 arch/arc/boot/dts/include/dt-bindings
>
> I believe commit d5d332d3f7e8 ("devicetree: Move include prefixes from
> arch to separate directory") does what you want.

Indeed that helps!
Thanks a lot for the pointer!

-Alexey