From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg Kroah-Hartman Subject: Re: [PATCH -next] misc: fix various misc/mic/host/ build errors Date: Wed, 16 Oct 2013 13:37:27 -0700 Message-ID: <20131016203727.GB1633@kroah.com> References: <1381762088-18880-1-git-send-email-treding@nvidia.com> <525C481E.1010908@infradead.org> <20131016192648.GA19637@kroah.com> <525EEBD4.7070800@infradead.org> <20131016194440.GA14416@kroah.com> <20131016203024.GA6579@mithrandir> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail.linuxfoundation.org ([140.211.169.12]:37246 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1761602Ab3JPUh2 (ORCPT ); Wed, 16 Oct 2013 16:37:28 -0400 Content-Disposition: inline In-Reply-To: <20131016203024.GA6579@mithrandir> Sender: linux-next-owner@vger.kernel.org List-ID: To: Thierry Reding Cc: Randy Dunlap , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Mark Brown , Harshavardhan R Kharche , Ashutosh Dixit , Nikhil Rao , Sudeep Dutt On Wed, Oct 16, 2013 at 10:30:25PM +0200, Thierry Reding wrote: > On Wed, Oct 16, 2013 at 12:44:40PM -0700, Greg Kroah-Hartman wrote: > > On Wed, Oct 16, 2013 at 12:41:08PM -0700, Randy Dunlap wrote: > > > On 10/16/13 12:26, Greg Kroah-Hartman wrote: > > > > On Mon, Oct 14, 2013 at 12:38:06PM -0700, Randy Dunlap wrote: > > > >> From: Randy Dunlap > > > >> > > > >> Fix various build errors (on x86_64) for drivers/misc/mic/host/: > > > >> > > > >> drivers/misc/mic/host/mic_main.c:378:3: error: too many arguments to function 'sysfs_get_dirent' > > > >> drivers/misc/mic/host/mic_intr.h:134:58: warning: 'struct pci_dev' declared inside parameter list [enabled by default] > > > >> drivers/misc/mic/host/mic_intr.h:134:58: warning: its scope is only this definition or declaration, which is probably not what you want [enabled by default] > > > >> drivers/misc/mic/host/mic_intr.h:135:58: warning: 'struct pci_dev' declared inside parameter list [enabled by default] > > > >> drivers/misc/mic/host/mic_device.h:169:2: error: implicit declaration of function 'ioread32' [-Werror=implicit-function-declaration] > > > >> drivers/misc/mic/host/mic_device.h: In function 'mic_mmio_read': > > > >> drivers/misc/mic/host/mic_device.h:169:2: error: implicit declaration of function 'ioread32' [-Werror=implicit-function-declaration] > > > >> drivers/misc/mic/host/mic_device.h: In function 'mic_mmio_write': > > > >> drivers/misc/mic/host/mic_device.h:183:2: error: implicit declaration of function 'iowrite32' [-Werror=implicit-function-declaration] > > > >> > > > >> Signed-off-by: Randy Dunlap > > > >> Cc: Harshavardhan R Kharche > > > >> Cc: Ashutosh Dixit > > > >> Cc: Greg Kroah-Hartman > > > >> Cc: Nikhil Rao > > > >> Cc: Sudeep Dutt > > > > > > > > This is a -next issue only at this point in time, I thought it was in > > > > the -next tree as a "fixup" already? > > > > > > Are you asking the driver maintainers? > > > > No, the linux-next maintainers. > > > > The driver maintainers have nothing to do with this, it's a sysfs change > > in my driver-core tree that causes this issue. > > Well, we do carry a patch to fix this, but that generally doesn't get > carried to Linus' tree automatically as far as I know. So I guess it's > fine for now, but when the merge window opens you'll need to make sure > that Linus merges the branches in the correct order so that the build > doesn't break in between. Yes, I'll do that as part of my normal tree submission process. > I think the typical way to solve this kind of build dependencies between > trees is for one tree to provide a stable branch that can be merged into > the other. That ensures that each tree can still be built independently. I don't do that for my different trees, sorry, especially for staging drivers, they can break all the time, no one really cares :) thanks, greg k-h