Re: [PATCH] qxl_driver.c: fix compiler warning 'defined but not used' qxl_kernel_mode_enabled

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> On Mon, Jun 09, 2014 at 04:08:37PM +0300, Alon Levy wrote:
> > Signed-off-by: Alon Levy <alevy@xxxxxxxxxx>
> > ---
> >  src/qxl_driver.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/src/qxl_driver.c b/src/qxl_driver.c
> > index b9aa0e9..d9bf447 100644
> > --- a/src/qxl_driver.c
> > +++ b/src/qxl_driver.c
> > @@ -1242,7 +1242,7 @@ qxl_init_scrn (ScrnInfoPtr pScrn, Bool kms)
> >      pScrn->ValidMode        = NULL;
> >  }
> >  
> > -#ifdef XF86DRM_MODE
> > +#if defined(XF86DRM_MODE) && !defined(XSPICE)
> >  static char *
> >  CreatePCIBusID(const struct pci_device *dev)
> >  {
> > --
> > 1.9.3
> 
> ACK, though we could also ensure that XF86DRM_MODE is never defined when
> XSPICE is ?
> 

You're right, but it's more work - mainly deciding what I want done - should configure not allow both drm and xspice? probably. Later patch?
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]