Re: [PATCH] vfio-mdev: Fix mtty sample driver building

[Alex Williamson]

On Thu, 15 Dec 2016 10:36:02 +0100
Auger Eric <eric.auger@xxxxxxxxxx> 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 <alex.williamson@xxxxxxxxxx>
> > ---
> >  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 <eric.auger@xxxxxxxxxx>
> Tested-by: Eric Auger <eric.auger@xxxxxxxxxx>

Thanks for both!

Alex