From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg Ungerer Subject: Re: [PATCH] m68knommu: add missing make rule for Palm/Pilot bitmap boot logo Date: Thu, 22 Dec 2011 17:27:06 +1000 Message-ID: <4EF2DBCA.6020806@snapgear.com> References: <1324537383-16741-1-git-send-email-gerg@snapgear.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from sncsmrelay2.nai.com ([67.97.80.206]:24983 "EHLO sncsmrelay2.nai.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752288Ab1LVH0H convert rfc822-to-8bit (ORCPT ); Thu, 22 Dec 2011 02:26:07 -0500 In-Reply-To: Sender: linux-m68k-owner@vger.kernel.org List-Id: linux-m68k@vger.kernel.org To: Geert Uytterhoeven Cc: linux-m68k@vger.kernel.org, uclinux-dev@uclinux.org, Greg Ungerer Hi Geert, On 22/12/11 17:18, Geert Uytterhoeven wrote: > On Thu, Dec 22, 2011 at 08:03, wrote: >> From: Greg Ungerer >> >> Compiling for the m68knommu/68328 Palm/Pilot target you get: >> >> =C3=A1AS =C3=A1 =C3=A1 =C3=A1arch/m68k/platform/68328/head-pilot.o >> arch/m68k/platform/68328/head-pilot.S:37:23: fatal error: bootlogo.r= h: No such file or directory >> >> The Makefile contains rules to make the bootlogo.rh file from the C = source >> bootlogo.h. But it is missing a dependency that compiling head-pilot= =2ES needs >> bootlogo.rh. >> >> Signed-off-by: Greg Ungerer >> --- >> =C3=A1arch/m68k/platform/68328/Makefile | =C3=A1 =C3=A13 +++ >> =C3=A11 files changed, 3 insertions(+), 0 deletions(-) >> >> diff --git a/arch/m68k/platform/68328/Makefile b/arch/m68k/platform/= 68328/Makefile >> index e4dfd8f..704e36d 100644 >> --- a/arch/m68k/platform/68328/Makefile >> +++ b/arch/m68k/platform/68328/Makefile >> @@ -16,6 +16,9 @@ obj-$(CONFIG_ROM) =C3=A1 =C3=A1 +=3D romvec.o >> =C3=A1extra-y =C3=A1 =C3=A1 =C3=A1 =C3=A1 =C3=A1 =C3=A1 =C3=A1 =C3=A1= =C3=A1 =C3=A1 =C3=A1 =C3=A1:=3D head.o >> =C3=A1extra-$(CONFIG_M68328) +=3D bootlogo.rh head.o >> >> +$(obj)/head-pilot.o: $(obj)/bootlogo.rh >> +$(obj)/head-rom.o: $(obj)/bootlogo.rh >> + >> =C3=A1$(obj)/bootlogo.rh: $(src)/bootlogo.h >> =C3=A1 =C3=A1 =C3=A1 =C3=A1perl $(src)/bootlogo.pl< $(src)/bootlogo= =2Eh> $(obj)/bootlogo.rh > > Nothing serious, you just made me wondering: > 1. Any chance bootlogo.h can be generated from a more edit-friendly= format, > cfr. drivers/video/logo/? Probably. This is the way the code came to me originally, and I haven't looked at it in quite a while. > 2. Why do you include the *.rh file from an assembler source file, = instead of > including the *.h file from a C source file? Good question. I am not sure why it was done this way. I'll see if we can do this a little more cleanly. Regards Greg -----------------------------------------------------------------------= - Greg Ungerer -- Principal Engineer EMAIL: gerg@snapgear.co= m SnapGear Group, McAfee PHONE: +61 7 3435 288= 8 8 Gardner Close FAX: +61 7 3217 532= 3 Milton, QLD, 4064, Australia WEB: http://www.SnapGear.co= m