On Wed, Feb 17, 2010 at 10:42:43AM +0100, Daniel Mack wrote: > Fix the following compile time error: > > drivers/built-in.o: In function `nouveau_connector_detect': > /home/daniel/src/linux/jup/linux-2.6/drivers/gpu/drm/nouveau/nouveau_connector.c:243: > undefined reference to `acpi_lid_open' > > Signed-off-by: Daniel Mack <dan...@caiaq.de> > Cc: David Airlie <airl...@linux.ie> > Cc: Ben Skeggs <bske...@redhat.com> > Cc: Francisco Jerez <curroje...@riseup.net> > Cc: Maarten Maathuis <madman2...@gmail.com> > Cc: Xavier Chantry <shinin...@gmail.com> > Cc: Marcin Slusarz <marcin.slus...@gmail.com> > Cc: dri-devel@lists.sourceforge.net > --- > drivers/gpu/drm/nouveau/nouveau_connector.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/gpu/drm/nouveau/nouveau_connector.c > b/drivers/gpu/drm/nouveau/nouveau_connector.c > index d2f6335..c74d45d 100644 > --- a/drivers/gpu/drm/nouveau/nouveau_connector.c > +++ b/drivers/gpu/drm/nouveau/nouveau_connector.c > @@ -239,7 +239,7 @@ nouveau_connector_detect(struct drm_connector *connector) > if (connector->connector_type == DRM_MODE_CONNECTOR_LVDS) > nv_encoder = find_encoder_by_type(connector, OUTPUT_LVDS); > if (nv_encoder && nv_connector->native_mode) { > -#ifdef CONFIG_ACPI > +#ifdef CONFIG_ACPI_BUTTON > if (!nouveau_ignorelid && !acpi_lid_open()) > return connector_status_disconnected; > #endif > --
You are fixing CONFIG_ACPI_BUTTON=m vs CONFIG_DRM_NOUVEAU=y compilation, but breaking runtime behaviour of CONFIG_ACPI_BUTTON=m + CONFIG_DRM_NOUVEAU=m. I think this code should be compiled when: #if defined(CONFIG_ACPI_BUTTON) || (defined(CONFIG_ACPI_BUTTON_MODULE) && defined(CONFIG_DRM_NOUVEAU_MODULE)) Marcin ------------------------------------------------------------------------------ Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel