Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756179Ab3GCS1O (ORCPT ); Wed, 3 Jul 2013 14:27:14 -0400 Received: from mail-qa0-f43.google.com ([209.85.216.43]:49380 "EHLO mail-qa0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754184Ab3GCS1N (ORCPT ); Wed, 3 Jul 2013 14:27:13 -0400 MIME-Version: 1.0 X-Originating-IP: [173.13.129.225] In-Reply-To: <20130627094650.GC4283@n2100.arm.linux.org.uk> References: <51C97F40.5030101@arm.com> <20130625184539.GB2718@n2100.arm.linux.org.uk> <20130626044842.GR23141@intel.com> <20130626081909.GD2718@n2100.arm.linux.org.uk> <20130626103930.GS23141@intel.com> <20130627094650.GC4283@n2100.arm.linux.org.uk> Date: Wed, 3 Jul 2013 11:27:12 -0700 Message-ID: Subject: Re: amba-pl08x and 'get_signal' namespace collision/build error From: Olof Johansson To: Vinod Koul Cc: Mark Brown , Jonathan Austin , "linux-kernel@vger.kernel.org" , Steve Capper , Catalin Marinas , LinusW , Will Deacon , Al Viro , "linux-arm-kernel@lists.infradead.org" , Russell King - ARM Linux Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2079 Lines: 54 Hi, On Thu, Jun 27, 2013 at 2:46 AM, Russell King - ARM Linux wrote: > On Wed, Jun 26, 2013 at 04:09:30PM +0530, Vinod Koul wrote: >> On Wed, Jun 26, 2013 at 09:19:09AM +0100, Russell King - ARM Linux wrote: >> > On Wed, Jun 26, 2013 at 10:18:42AM +0530, Vinod Koul wrote: >> > > Yup. This is fixed in slave-dma tree by a patch from mark by renaming it. >> > > >> > > This should not show in the -next tree >> > >> > Except, that change probably is probably responsible for this new error: >> > >> > drivers/dma/amba-pl08x.c: In function 'pl08x_request_mux': >> > drivers/dma/amba-pl08x.c:304:13: error: expected identifier before '(' token >> This was pushed four days ago and havent seen any error report expect this. >> >> Anyway I rebuild my -next and looks fine for me >> >> CC drivers/dma/dmaengine.o >> CC drivers/dma/virt-dma.o >> CC drivers/dma/iovlock.o >> CC [M] drivers/dma/dmatest.o >> CC drivers/dma/amba-pl08x.o >> LD drivers/dma/built-in.o >> >> Can you share your config which failed for you? >> >> Mark, I believe you are testing for this, have you seen above error? > > Right, what's going on is that I have a commit in my tree which adds an > include to asm/pgtable.h which introduces a #define for get_signal. > This becomes visible to amba-pl08x.c. > > However, my build test tree fails *every* time because what I'm building > is Linus' tree, my tree plus arm-soc, and it doesn't include your tree. > This makes my build testing for the next merge window impossible. > > So, I need the fix to pl08x.c merged into my tree. This has now hit the mainline kernel, and several defconfigs (nhk815, lpc32xx and the spear ones) are broken there. Vinod, when are you sending up your pull request with the fix? It'd be good to see it go in soon. -Olof -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/