From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753300Ab0ICGx4 (ORCPT ); Fri, 3 Sep 2010 02:53:56 -0400 Received: from dalsmrelay2.nai.com ([205.227.136.216]:59747 "HELO dalsmrelay2.nai.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1752227Ab0ICGxy (ORCPT ); Fri, 3 Sep 2010 02:53:54 -0400 Message-ID: <4C809B63.2040802@snapgear.com> Date: Fri, 3 Sep 2010 16:53:23 +1000 From: Greg Ungerer User-Agent: Thunderbird 2.0.0.24 (X11/20100411) MIME-Version: 1.0 To: David Howells CC: linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/3] Fix M68K irqflags References: <4C804F3A.8020001@snapgear.com> <19747.1283422807@redhat.com> <20100902102203.19806.39028.stgit@warthog.procyon.org.uk> <7077.1283496259@redhat.com> In-Reply-To: <7077.1283496259@redhat.com> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 03 Sep 2010 06:53:41.0968 (UTC) FILETIME=[BEAE7D00:01CB4B34] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org David Howells wrote: > Greg Ungerer wrote: > >>> - DEFINE(THREAD_SIZE, THREAD_SIZE); >>> - >> This breaks final link due to missing THREAD_SIZE definition in head.S: >> >> LD vmlinux >> arch/m68knommu/platform/coldfire/head.o: In function `_clear_bss': >> (.head.text+0xa4): undefined reference to `THREAD_SIZE' >> >> >> Without that asm-offsets.c change it works on my ColdFire boards. >> I am happy with it: > > With that, I get a duplicate reference to THREAD_SIZE. head.S should include > asm/thread_info.h rather than THREAD_SIZE being redefined in asm-offsets. > > CC init/do_mounts_rd.o > In file included from /warthog/nfs/linux-2.6-fscache/arch/m68k/include/asm/asm-offsets.h:1, > from arch/m68knommu/platform/coldfire/entry.S:36: > include/generated/asm-offsets.h:46:1: warning: "THREAD_SIZE" redefined > In file included from /warthog/nfs/linux-2.6-fscache/arch/m68k/include/asm/thread_info.h:2, > from arch/m68knommu/platform/coldfire/entry.S:32: > /warthog/nfs/linux-2.6-fscache/arch/m68k/include/asm/thread_info_no.h:27:1: warning: this is the location of the previous definition Yes, I get that too. I just hadn't tracked down why that was yet. > Unfortunately, I don't have a binutils that can handle the funny Coldfire > instructions. Can you point me at a Coldfire toolchain? This is what I currently use: http://www.uclinux.org/pub/uClinux/m68k-elf-tools/tools-20080626/m68k-uclinux-tools-20080626.sh Its just a binutils-2.18 and gcc-4.2.4 configured for m68k-uclinux target. Regards Greg ------------------------------------------------------------------------ Greg Ungerer -- Principal Engineer EMAIL: gerg@snapgear.com SnapGear Group, McAfee PHONE: +61 7 3435 2888 8 Gardner Close FAX: +61 7 3217 5323 Milton, QLD, 4064, Australia WEB: http://www.SnapGear.com