Re: [PATCH] powerpc: dts: use #include "..." to include local DT
Masahiro Yamadawrites: > Hi Michael, > > Ping. Please apply this patch. > > I need this to clean up Makefiles in the next development cycle. Sorry for some reason it didn't land in patchwork, so I keep forgetting about it. Have merged it now for 4.13. cheers
Re: [PATCH] powerpc: dts: use #include "..." to include local DT
Masahiro Yamadawrites: > 2017-06-14 15:45 GMT+09:00 Michael Ellerman : >> >> Acked-by: Michael Ellerman > > I have not seen it in linux-next yet. > > Who will pick it up? In the original patch you said: Fix them to remove -I$(srctree)/arch/$(SRCARCH)/boot/dts path from dtc_cpp_flags. So I assumed there was a series somewhere that did that and included this patch. But if there isn't then I can just merge it. cheers
Re: [PATCH] powerpc: dts: use #include "..." to include local DT
Masahiro Yamadawrites: > 2017-06-13 19:21 GMT+09:00 Michael Ellerman : >> Masahiro Yamada writes: >>> >>> (+Anatolij Gustschin ) >>> >>> Ping. >>> I am not 100% sure who is responsible for this, >>> but somebody, could take a look at this patch, please? >> >> Have you tested it actually works? >> >> It sounds reasonable, and if it behaves as you describe there is no >> change in behaviour, right? > > I do not have access to hardware, > but it is pretty easy to test this patch. > > $ make O=foo ARCH=powerpc CROSS_COMPILE=powerpc-linux- dts/ac14xx.dtb > > gave me the DTB output. > > The binary comparison matched with/without this patch, > so I am sure there is no change in behavior. > > Likewise for mpc5121ads and pdm360ng. Thanks. Acked-by: Michael Ellerman cheers
Re: [PATCH] powerpc: dts: use #include "..." to include local DT
Hi, On Wed, 24 May 2017 14:12:24 +0900 Masahiro Yamada yamada.masah...@socionext.com wrote: >Most of DT files in PowerPC use #include "..." to make pre-processor >include DT in the same directory, but we have 3 exceptional files >that use #include <...> for that. > >Fix them to remove -I$(srctree)/arch/$(SRCARCH)/boot/dts path from >dtc_cpp_flags. > >Signed-off-by: Masahiro YamadaTested-by: Anatolij Gustschin
Re: [PATCH] powerpc: dts: use #include "..." to include local DT
On Tue, 13 Jun 2017 20:21:45 +1000 Michael Ellerman m...@ellerman.id.au wrote: >Masahiro Yamadawrites: ... >> Ping. >> I am not 100% sure who is responsible for this, >> but somebody, could take a look at this patch, please? > >Have you tested it actually works? > >It sounds reasonable, and if it behaves as you describe there is no >change in behaviour, right? yes, these dtbs build with this patch and I've tested with mpc5121ads.dtb. Thanks, Anatolij
Re: [PATCH] powerpc: dts: use #include "..." to include local DT
Masahiro Yamadawrites: > Hi > > (+Anatolij Gustschin ) > > > Ping. > I am not 100% sure who is responsible for this, > but somebody, could take a look at this patch, please? Have you tested it actually works? It sounds reasonable, and if it behaves as you describe there is no change in behaviour, right? cheers > 2017-05-24 14:12 GMT+09:00 Masahiro Yamada : >> Most of DT files in PowerPC use #include "..." to make pre-processor >> include DT in the same directory, but we have 3 exceptional files >> that use #include <...> for that. >> >> Fix them to remove -I$(srctree)/arch/$(SRCARCH)/boot/dts path from >> dtc_cpp_flags. >> >> Signed-off-by: Masahiro Yamada >> --- >> >> arch/powerpc/boot/dts/ac14xx.dts | 2 +- >> arch/powerpc/boot/dts/mpc5121ads.dts | 2 +- >> arch/powerpc/boot/dts/pdm360ng.dts | 2 +- >> 3 files changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/arch/powerpc/boot/dts/ac14xx.dts >> b/arch/powerpc/boot/dts/ac14xx.dts >> index 27fcabc2f857..83bcfd865167 100644 >> --- a/arch/powerpc/boot/dts/ac14xx.dts >> +++ b/arch/powerpc/boot/dts/ac14xx.dts >> @@ -10,7 +10,7 @@ >> */ >> >> >> -#include >> +#include "mpc5121.dtsi" >> >> / { >> model = "ac14xx"; >> diff --git a/arch/powerpc/boot/dts/mpc5121ads.dts >> b/arch/powerpc/boot/dts/mpc5121ads.dts >> index 75888ce2c792..73c30621429b 100644 >> --- a/arch/powerpc/boot/dts/mpc5121ads.dts >> +++ b/arch/powerpc/boot/dts/mpc5121ads.dts >> @@ -9,7 +9,7 @@ >> * option) any later version. >> */ >> >> -#include >> +#include "mpc5121.dtsi" >> >> / { >> model = "mpc5121ads"; >> diff --git a/arch/powerpc/boot/dts/pdm360ng.dts >> b/arch/powerpc/boot/dts/pdm360ng.dts >> index 0cec7244abe7..445b88114009 100644 >> --- a/arch/powerpc/boot/dts/pdm360ng.dts >> +++ b/arch/powerpc/boot/dts/pdm360ng.dts >> @@ -13,7 +13,7 @@ >> * option) any later version. >> */ >> >> -#include >> +#include "mpc5121.dtsi" >> >> / { >> model = "pdm360ng"; >> -- >> 2.7.4 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe devicetree" in >> the body of a message to majord...@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html > > > > -- > Best Regards > Masahiro Yamada