On 2015-12-03 15:57, Nikolay Martynov wrote: > HIi > > 2015-12-03 9:10 GMT-05:00 Felix Fietkau <[email protected]>: >> On 2015-12-03 00:42, Nikolay Martynov wrote: >>> Current way of compuling dts files involves calling C preprocessor on >>> main dts file only. This means that dtsi includes cannot have C-style >>> includes. >>> >>> This patch addresses this problem. It uses approach similar to one >>> use in linux kernel: it preprocesses all dtsi's in current dir into >>> tmp dir and then uses that tmp dir as include dir for main dts compilation. >>> >>> Note: this patch preprocesses onlt *.dtsi, not *.dts, so only *.dtsi >>> can be includes, but it looks like all current architectures follow this >>> convention. >>> >>> This approach should be compatible with all current architectures. >>> >>> This patch also updates ramips arch to use new dtsi comilation code. >>> >>> Signed-off-by: Nikolay Martynov <[email protected]> >>> --- >>> include/image.mk | 22 +++++++++++++++++----- >>> target/linux/ramips/image/Makefile | 2 +- >>> 2 files changed, 18 insertions(+), 6 deletions(-) >>> >>> --- a/target/linux/ramips/image/Makefile >>> +++ b/target/linux/ramips/image/Makefile >>> @@ -39,7 +39,7 @@ define Device/Default >>> endef >>> >>> define Build/patch-dtb >>> - $(LINUX_DIR)/scripts/dtc/dtc -O dtb -o [email protected] ../dts/$(DTS).dts >>> + $(call >>> Image/BuildDTB,../dts/$(DTS).dts,[email protected],-I$(LINUX_DIR)/arch/mips/boot/dts >>> -I$(LINUX_DIR)/arch/mips/boot/dts/include) >> Why did you add those -I options? Isn't that already handled by the >> common preprocessor options in Image/PreprocessDTS? > Unfortunately they are not the same. Kernel arch seems to be mipsel > which doesn't match directory name it has to look in. I think this is > ralink specific. Ah, I see what you mean. This is not ramips specific, it's simply a bug in the code inherited from the Image/BuildDTB template. Please change references to arch/$(ARCH) over to arch/$(LINUX_KARCH), and it should work afterwards.
- Felix _______________________________________________ openwrt-devel mailing list [email protected] https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
