From: H Hartley Sweeten Date: Tue, 23 Apr 2013 18:05:06 +0000 (-0700) Subject: staging: comedi: ni_labpc: fix legacy driver build X-Git-Tag: Ubuntu-5.2.0-15.16~15862^2 X-Git-Url: https://git.proxmox.com/?a=commitdiff_plain;h=a0eeed40100b2eca57afdf99d76c57dedd218a8a;p=mirror_ubuntu-eoan-kernel.git staging: comedi: ni_labpc: fix legacy driver build CONFIG_COMEDI_NI_LABPC_ISA is a tristate option. #if IS_ENABLED() needs to be used instead of #ifdef in order to build the legacy specific code in this driver. The local variable 'board' is not used in labpc_attach(). Remove it. The labpc_detach() function is only referenced in this file. Make it static. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/comedi/drivers/ni_labpc.c b/drivers/staging/comedi/drivers/ni_labpc.c index bb68d8fa83a3..3d978f34d212 100644 --- a/drivers/staging/comedi/drivers/ni_labpc.c +++ b/drivers/staging/comedi/drivers/ni_labpc.c @@ -234,7 +234,7 @@ static inline void labpc_writeb(unsigned int byte, unsigned long address) writeb(byte, (void __iomem *)address); } -#ifdef CONFIG_COMEDI_NI_LABPC_ISA +#if IS_ENABLED(CONFIG_COMEDI_NI_LABPC_ISA) static const struct labpc_boardinfo labpc_boards[] = { { .name = "lab-pc-1200", @@ -1710,10 +1710,9 @@ void labpc_common_detach(struct comedi_device *dev) } EXPORT_SYMBOL_GPL(labpc_common_detach); -#ifdef CONFIG_COMEDI_NI_LABPC_ISA +#if IS_ENABLED(CONFIG_COMEDI_NI_LABPC_ISA) static int labpc_attach(struct comedi_device *dev, struct comedi_devconfig *it) { - const struct labpc_boardinfo *board = comedi_board(dev); struct labpc_private *devpriv; unsigned int irq = it->options[1]; unsigned int dma_chan = it->options[2]; @@ -1756,7 +1755,7 @@ static int labpc_attach(struct comedi_device *dev, struct comedi_devconfig *it) return 0; } -void labpc_detach(struct comedi_device *dev) +static void labpc_detach(struct comedi_device *dev) { struct labpc_private *devpriv = dev->private;