Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752830AbcDOI1n (ORCPT ); Fri, 15 Apr 2016 04:27:43 -0400 Received: from spamalot.elettra.trieste.it ([140.105.206.208]:50038 "EHLO spamalot.elettra.eu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751010AbcDOI1l (ORCPT ); Fri, 15 Apr 2016 04:27:41 -0400 MIME-Version: 1.0 In-Reply-To: <1459950325-10327-1-git-send-email-alessio.bogani@elettra.eu> References: <1459950325-10327-1-git-send-email-alessio.bogani@elettra.eu> Date: Fri, 15 Apr 2016 10:27:17 +0200 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [RFC PATCH 1/1] powerpc/fsl: Fix build of the dtb embedded kernel images From: Alessio Igor Bogani To: Scott Wood , Kumar Gala , linuxppc-dev@lists.ozlabs.org Cc: Alessio Igor Bogani , LKML Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1571 Lines: 42 Hi, On 6 April 2016 at 15:45, Alessio Igor Bogani wrote: > The commit dc37374 move a lot of device tree files into fsl directory > fixing Makefile for cuImage target only. Unfortunately there are others > target which require to embebbed device tree into the kernel image > (i.e. dtbImage.%). So use a more generic approach. > > Signed-off-by: Alessio Igor Bogani > --- > arch/powerpc/boot/Makefile | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/arch/powerpc/boot/Makefile b/arch/powerpc/boot/Makefile > index 6116510..8fe78a3 100644 > --- a/arch/powerpc/boot/Makefile > +++ b/arch/powerpc/boot/Makefile > @@ -362,9 +362,6 @@ $(obj)/cuImage.initrd.%: vmlinux $(obj)/%.dtb $(wrapperbits) > $(obj)/cuImage.%: vmlinux $(obj)/%.dtb $(wrapperbits) > $(call if_changed,wrap,cuboot-$*,,$(obj)/$*.dtb) > > -$(obj)/cuImage.%: vmlinux $(obj)/fsl/%.dtb $(wrapperbits) > - $(call if_changed,wrap,cuboot-$*,,$(obj)/fsl/$*.dtb) > - > $(obj)/simpleImage.initrd.%: vmlinux $(obj)/%.dtb $(wrapperbits) > $(call if_changed,wrap,simpleboot-$*,,$(obj)/$*.dtb,$(obj)/ramdisk.image.gz) > > @@ -381,6 +378,9 @@ $(obj)/treeImage.%: vmlinux $(obj)/%.dtb $(wrapperbits) > $(obj)/%.dtb: $(src)/dts/%.dts FORCE > $(call if_changed_dep,dtc) > > +$(obj)/%.dtb: $(src)/dts/fsl/%.dts FORCE > + $(call if_changed_dep,dtc) > + > # If there isn't a platform selected then just strip the vmlinux. > ifeq (,$(image-y)) > image-y := vmlinux.strip Any comments? Ciao, Alessio