From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753405AbdK1Mbk (ORCPT ); Tue, 28 Nov 2017 07:31:40 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:44408 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753170AbdK1Mbi (ORCPT ); Tue, 28 Nov 2017 07:31:38 -0500 Date: Tue, 28 Nov 2017 13:31:42 +0100 From: Greg Kroah-Hartman To: Chris Coffey Cc: devel@driverdev.osuosl.org, Christian Gromm , linux-kernel@vger.kernel.org, Michael Fabry Subject: Re: [PATCH] staging: most: Fix build errors Message-ID: <20171128123142.GA23875@kroah.com> References: <20171127211239.6873-1-cmc@babblebit.net> <20171128072607.GB10757@kroah.com> <20171128121247.yeklq5dagwos3mcd@deb-660> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171128121247.yeklq5dagwos3mcd@deb-660> User-Agent: Mutt/1.9.1 (2017-09-22) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Nov 28, 2017 at 12:12:47PM +0000, Chris Coffey wrote: > On Tue, Nov 28, 2017 at 08:26:07AM +0100, Greg Kroah-Hartman wrote: > > On Mon, Nov 27, 2017 at 09:12:39PM +0000, Chris Coffey wrote: > > > This patch fixes build errors that occur when trying to build the > > > MOST modules. For example, the cdev module build fails with: > > > > > > make[3]: *** No rule to make target 'drivers/staging/most/aim-cdev/cdev.o', needed by 'drivers/staging/most/aim-cdev/aim_cdev.o'. Stop. > > > scripts/Makefile.build:569: recipe for target 'drivers/staging/most/aim-cdev' failed > > > make[2]: *** [drivers/staging/most/aim-cdev] Error 2 > > > scripts/Makefile.build:569: recipe for target 'drivers/staging/most' failed > > > make[1]: *** [drivers/staging/most] Error 2 > > > Makefile:1502: recipe for target '_module_drivers/staging' failed > > > make: *** [_module_drivers/staging] Error 2 > > > > How are you seeing this error? What is the build command line that you > > provided to do this? It should just "not build" right now at all. > > > > I did a git pull to get the latest staging updates, enabled a bunch of > staging drivers, and then ran a build on the staging modules with 'make > modules M=drivers/staging'. Unfortunately now I'm unable to reproduce > the error (of course). If you run 'make' it would have fixed up the dependancies and not caused a build error. That makes sense how you saw it, but it's not there for anyone else so all is fine. > > > Update the Makefile variables and paths with the new directory > > > structure to fix these errors. > > > > > > Signed-off-by: Chris Coffey > > > --- > > > drivers/staging/most/Makefile | 14 +++++++------- > > > 1 file changed, 7 insertions(+), 7 deletions(-) > > > > I think a patch was sent yesterday to fix these up, did you see it in > > the archives? > > I checked the archives before making the patch, and didn't see anything > MOST-related in there for the past few days. The patch below made the > build succeed and generate all of the MOST modules. Subject: [PATCH 1/3] staging: most: fix Makefile is the patch sent yesterday that should resolve this. I'm going to go apply it right now... thanks, greg k-h