Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758785AbcLORId (ORCPT ); Thu, 15 Dec 2016 12:08:33 -0500 Received: from mx1.redhat.com ([209.132.183.28]:45064 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753645AbcLORIa (ORCPT ); Thu, 15 Dec 2016 12:08:30 -0500 Date: Thu, 15 Dec 2016 10:08:29 -0700 From: Alex Williamson To: Auger Eric Cc: kwankhede@nvidia.com, cjia@nvidia.com, linux-kernel@vger.kernel.org, kvm@vger.kernel.org Subject: Re: [PATCH] vfio-mdev: Fix mtty sample driver building Message-ID: <20161215100829.431a0cd6@t450s.home> In-Reply-To: References: <20161214193859.32293.43760.stgit@gimli.home> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.27]); Thu, 15 Dec 2016 17:08:30 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2256 Lines: 71 On Thu, 15 Dec 2016 10:36:02 +0100 Auger Eric wrote: > Hi, > > On 14/12/2016 20:39, Alex Williamson wrote: > > This sample driver was originally under Documentation/ and was moved > > to samples, but build support was never adjusted for the new location. > > > > Signed-off-by: Alex Williamson > > --- > > samples/Kconfig | 7 +++++++ > > samples/Makefile | 3 ++- > > samples/vfio-mdev/Makefile | 14 +------------- > > 3 files changed, 10 insertions(+), 14 deletions(-) > > > > diff --git a/samples/Kconfig b/samples/Kconfig > > index a6d2a43..b124f62 100644 > > --- a/samples/Kconfig > > +++ b/samples/Kconfig > > @@ -105,4 +105,11 @@ config SAMPLE_BLACKFIN_GPTIMERS > > help > > Build samples of blackfin gptimers sample module. > > > > +config SAMPLE_VFIO_MDEV_MTTY > > + tristate "Build VFIO mtty example mediated device sample code -- loadable modules only" > > + depends on VFIO_MDEV_DEVICE && m > > + help > > + Build a virtual tty sample driver for use as a VFIO > > + mediated device > > + > > endif # SAMPLES > > diff --git a/samples/Makefile b/samples/Makefile > > index e17d66d..86a137e 100644 > > --- a/samples/Makefile > > +++ b/samples/Makefile > > @@ -2,4 +2,5 @@ > > > > obj-$(CONFIG_SAMPLES) += kobject/ kprobes/ trace_events/ livepatch/ \ > > hw_breakpoint/ kfifo/ kdb/ hidraw/ rpmsg/ seccomp/ \ > > - configfs/ connector/ v4l/ trace_printk/ blackfin/ > > + configfs/ connector/ v4l/ trace_printk/ blackfin/ \ > > + vfio-mdev/ > > diff --git a/samples/vfio-mdev/Makefile b/samples/vfio-mdev/Makefile > > index a932edb..cbbd868 100644 > > --- a/samples/vfio-mdev/Makefile > > +++ b/samples/vfio-mdev/Makefile > > @@ -1,13 +1 @@ > > -# > > -# Makefile for mtty.c file > > -# > > -KERNEL_DIR:=/lib/modules/$(shell uname -r)/build > > - > > -obj-m:=mtty.o > > - > > -modules clean modules_install: > > - $(MAKE) -C $(KERNEL_DIR) SUBDIRS=$(PWD) $@ > > - > > -default: modules > > - > > -module: modules > > +obj-$(CONFIG_SAMPLE_VFIO_MDEV_MTTY) += mtty.o > > Test-compiled on ARM aarch64 > > Reviewed-by: Eric Auger > Tested-by: Eric Auger Thanks for both! Alex