dt-bindings status

dt-bindings status

[Alexander Shiyan]

Hello.

DTS files from the current kernel require "include/dt-bindings".
Are there plans to add these parts of the kernel tree in the barebox?
Thanks.

---
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: dt-bindings status

[Sascha Hauer]

Hi Alexander,

On Wed, Nov 06, 2013 at 01:01:59PM +0400, Alexander Shiyan wrote:
> Hello.
> 
> DTS files from the current kernel require "include/dt-bindings".
> Are there plans to add these parts of the kernel tree in the barebox?
> Thanks.

We should add this for barebox aswell once we need it. Right now I
haven't missed it. If you need it feel free to add it.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: dt-bindings status

[Sebastian Hesselbarth]

On 11/06/13 10:57, Sascha Hauer wrote:
> On Wed, Nov 06, 2013 at 01:01:59PM +0400, Alexander Shiyan wrote:
>> Hello.
>>
>> DTS files from the current kernel require "include/dt-bindings".
>> Are there plans to add these parts of the kernel tree in the barebox?
>> Thanks.
>
> We should add this for barebox aswell once we need it. Right now I
> haven't missed it. If you need it feel free to add it.

I miss it! :)

I haven't found a way to tell dtc's cpp that those includes are in
include/dt-bindings, but I didn't look very hard.

Currently, I have them in arch/arm/dts/dt-bindings which works but
is not the right place, for sure.

I'd be happy if Alexander comes up with a patch.

Sebastian

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: dt-bindings status

[Sascha Hauer]

On Fri, Nov 08, 2013 at 01:52:27PM +0100, Sebastian Hesselbarth wrote:
> On 11/06/13 10:57, Sascha Hauer wrote:
> >On Wed, Nov 06, 2013 at 01:01:59PM +0400, Alexander Shiyan wrote:
> >>Hello.
> >>
> >>DTS files from the current kernel require "include/dt-bindings".
> >>Are there plans to add these parts of the kernel tree in the barebox?
> >>Thanks.
> >
> >We should add this for barebox aswell once we need it. Right now I
> >haven't missed it. If you need it feel free to add it.
> 
> I miss it! :)
> 
> I haven't found a way to tell dtc's cpp that those includes are in
> include/dt-bindings, but I didn't look very hard.

I just found out:

From 3808314a4014c201e4047ddf44187fadcedaaebd Mon Sep 17 00:00:00 2001
From: Sascha Hauer <s.hauer@pengutronix.de>
Date: Mon, 18 Nov 2013 13:59:17 +0100
Subject: [PATCH] ARM: dtb: create dt-bindings link

For making #include <dt-bindings/...> work

We already have the necessary -I flag in dtc_cpp_flags, no nothing
needs to be done here.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 arch/arm/dts/include/dt-bindings | 1 +
 1 file changed, 1 insertion(+)
 create mode 120000 arch/arm/dts/include/dt-bindings

diff --git a/arch/arm/dts/include/dt-bindings b/arch/arm/dts/include/dt-bindings
new file mode 120000
index 0000000..0cecb3d
--- /dev/null
+++ b/arch/arm/dts/include/dt-bindings
@@ -0,0 +1 @@
+../../../../include/dt-bindings
\ No newline at end of file
-- 
1.8.4.2

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox