search for: nouveau_led_resume

...struct nouveau_led * nouveau_led(struct drm_device *dev) { return nouveau_drm(dev)->led; } /* nouveau_led.c */ -#if IS_ENABLED(CONFIG_LEDS_CLASS) +#if IS_REACHABLE(CONFIG_LEDS_CLASS) int nouveau_led_init(struct drm_device *dev); void nouveau_led_suspend(struct drm_device *dev); void nouveau_led_resume(struct drm_device *dev); void nouveau_led_fini(struct drm_device *dev); #else static inline int nouveau_led_init(struct drm_device *dev) { return 0; }; static inline void nouveau_led_suspend(struct drm_device *dev) { }; static inline void nouveau_led_resume(struct drm_device *dev) { }; stati...

...c +++ b/drm/nouveau/nouveau_led.c @@ -107,7 +107,8 @@ nouveau_led_suspend(struct drm_device *dev) { struct nouveau_drm *drm = nouveau_drm(dev); - led_classdev_suspend(&drm->led->led); + if (drm->led) + led_classdev_suspend(&drm->led->led); } void @@ -115,8 +116,8 @@ nouveau_led_resume(struct drm_device *dev) { struct nouveau_drm *drm = nouveau_drm(dev); - led_classdev_resume(&drm->led->led); - + if (drm->led) + led_classdev_resume(&drm->led->led); } void -- 2.8.0

...cli; int ret; + nouveau_led_suspend(dev); + if (dev->mode_config.num_crtc) { NV_INFO(drm, "suspending console...\n"); nouveau_fbcon_set_suspend(dev, 1); @@ -638,6 +643,8 @@ nouveau_do_resume(struct drm_device *dev, bool runtime) nouveau_fbcon_set_suspend(dev, 0); } + nouveau_led_resume(dev); + return 0; } diff --git a/drm/nouveau/nouveau_drm.h b/drm/nouveau/nouveau_drm.h index 5c363ed..b148dcb 100644 --- a/drm/nouveau/nouveau_drm.h +++ b/drm/nouveau/nouveau_drm.h @@ -166,6 +166,9 @@ struct nouveau_drm { struct nouveau_hwmon *hwmon; struct nouveau_debugfs *debugfs; + /*...

...dev) > { > return nouveau_drm(dev)->led; > } > > /* nouveau_led.c */ > -#if IS_ENABLED(CONFIG_LEDS_CLASS) > +#if IS_REACHABLE(CONFIG_LEDS_CLASS) > int nouveau_led_init(struct drm_device *dev); > void nouveau_led_suspend(struct drm_device *dev); > void nouveau_led_resume(struct drm_device *dev); > void nouveau_led_fini(struct drm_device *dev); > #else > static inline int nouveau_led_init(struct drm_device *dev) { return 0; }; > static inline void nouveau_led_suspend(struct drm_device *dev) { }; > static inline void nouveau_led_resume(struct drm...

...cli; int ret; + nouveau_led_suspend(dev); + if (dev->mode_config.num_crtc) { NV_INFO(drm, "suspending console...\n"); nouveau_fbcon_set_suspend(dev, 1); @@ -649,6 +654,8 @@ nouveau_do_resume(struct drm_device *dev, bool runtime) nouveau_fbcon_set_suspend(dev, 0); } + nouveau_led_resume(dev); + return 0; } diff --git a/drm/nouveau/nouveau_drv.h b/drm/nouveau/nouveau_drv.h index 822a021..c0e2b32 100644 --- a/drm/nouveau/nouveau_drv.h +++ b/drm/nouveau/nouveau_drv.h @@ -166,6 +166,9 @@ struct nouveau_drm { struct nouveau_hwmon *hwmon; struct nouveau_debugfs *debugfs; + /*...

...rivers/gpu/drm/nouveau/nouveau.o: In function `nouveau_do_suspend': tvnv17.c:(.text.nouveau_do_suspend+0x10): undefined reference to `nouveau_led_suspend' drivers/gpu/drm/nouveau/nouveau.o: In function `nouveau_do_resume': tvnv17.c:(.text.nouveau_do_resume+0xf0): undefined reference to `nouveau_led_resume' drivers/gpu/drm/nouveau/nouveau.o: In function `nouveau_drm_unload': tvnv17.c:(.text.nouveau_drm_unload+0x34): undefined reference to `nouveau_led_fini' drivers/gpu/drm/nouveau/nouveau.o: In function `nouveau_drm_load': tvnv17.c:(.text.nouveau_drm_load+0x7d0): undefined reference t...

...{ > NV_INFO(drm, "suspending console...\n"); > nouveau_fbcon_set_suspend(dev, 1); > @@ -638,6 +643,8 @@ nouveau_do_resume(struct drm_device *dev, bool runtime) > nouveau_fbcon_set_suspend(dev, 0); > } > > + nouveau_led_resume(dev); > + > return 0; > } > > diff --git a/drm/nouveau/nouveau_drm.h b/drm/nouveau/nouveau_drm.h > index 5c363ed..b148dcb 100644 > --- a/drm/nouveau/nouveau_drm.h > +++ b/drm/nouveau/nouveau_drm.h > @@ -166,6 +166,9 @@ struct nouveau_drm { > struct nou...

...{ > NV_INFO(drm, "suspending console...\n"); > nouveau_fbcon_set_suspend(dev, 1); > @@ -638,6 +643,8 @@ nouveau_do_resume(struct drm_device *dev, bool runtime) > nouveau_fbcon_set_suspend(dev, 0); > } > > + nouveau_led_resume(dev); > + > return 0; > } > > diff --git a/drm/nouveau/nouveau_drm.h b/drm/nouveau/nouveau_drm.h > index 5c363ed..b148dcb 100644 > --- a/drm/nouveau/nouveau_drm.h > +++ b/drm/nouveau/nouveau_drm.h > @@ -166,6 +166,9 @@ struct nouveau_drm { > struct nou...

...{ > NV_INFO(drm, "suspending console...\n"); > nouveau_fbcon_set_suspend(dev, 1); > @@ -638,6 +643,8 @@ nouveau_do_resume(struct drm_device *dev, bool runtime) > nouveau_fbcon_set_suspend(dev, 0); > } > > + nouveau_led_resume(dev); > + > return 0; > } > > diff --git a/drm/nouveau/nouveau_drm.h b/drm/nouveau/nouveau_drm.h > index 5c363ed..b148dcb 100644 > --- a/drm/nouveau/nouveau_drm.h > +++ b/drm/nouveau/nouveau_drm.h > @@ -166,6 +166,9 @@ struct nouveau_drm { > struct nou...

This fixes the auto-magic detection of LEDS_CLASS by fixing the case where nouveau would be built-in and the LEDS_CLASS would be built as as module. Cc: <stable at vger.kernel.org> # 4.9.x- Reported-by: Intel's 0-DAY Signed-off-by: Martin Peres <martin.peres at free.fr> --- drivers/gpu/drm/nouveau/Kbuild | 2 +- drivers/gpu/drm/nouveau/Kconfig | 9 +++++++++ 2 files changed, 10

On Tuesday, November 8, 2016 10:46:07 AM CET Ilia Mirkin wrote: > > diff --git a/drivers/gpu/drm/nouveau/Kconfig b/drivers/gpu/drm/nouveau/Kconfig > > index 78631fb61adf..715cd6f4dc31 100644 > > --- a/drivers/gpu/drm/nouveau/Kconfig > > +++ b/drivers/gpu/drm/nouveau/Kconfig > > @@ -46,6 +46,14 @@ config NOUVEAU_DEBUG > > The paranoia and spam levels

...++ b/drivers/gpu/drm/nouveau/nouveau_led.h @@ -42,7 +42,7 @@ nouveau_led(struct drm_device *dev) } /* nouveau_led.c */ -#if IS_ENABLED(CONFIG_LEDS_CLASS) +#if IS_REACHABLE(CONFIG_LEDS_CLASS) int nouveau_led_init(struct drm_device *dev); void nouveau_led_suspend(struct drm_device *dev); void nouveau_led_resume(struct drm_device *dev); -- 2.10.2

...veau_do_suspend(struct drm_device *dev, bool runtime) if (dev->mode_config.num_crtc) { NV_DEBUG(drm, "resuming display...\n"); nouveau_display_resume(dev, runtime); + +fail_fbcon: + NV_DEBUG(drm, "resuming console...\n"); + nouveau_fbcon_set_suspend(dev, 0); } + + nouveau_led_resume(dev); + return ret; } -- 2.17.1

...nouveau/nouveau.o: In function `nouveau_do_suspend': > tvnv17.c:(.text.nouveau_do_suspend+0x10): undefined reference to `nouveau_led_suspend' > drivers/gpu/drm/nouveau/nouveau.o: In function `nouveau_do_resume': > tvnv17.c:(.text.nouveau_do_resume+0xf0): undefined reference to `nouveau_led_resume' > drivers/gpu/drm/nouveau/nouveau.o: In function `nouveau_drm_unload': > tvnv17.c:(.text.nouveau_drm_unload+0x34): undefined reference to `nouveau_led_fini' > drivers/gpu/drm/nouveau/nouveau.o: In function `nouveau_drm_load': > tvnv17.c:(.text.nouveau_drm_load+0x7d0): u...

...NFO(drm, "suspending console...\n"); >> nouveau_fbcon_set_suspend(dev, 1); >> @@ -638,6 +643,8 @@ nouveau_do_resume(struct drm_device *dev, bool runtime) >> nouveau_fbcon_set_suspend(dev, 0); >> } >> >> + nouveau_led_resume(dev); >> + >> return 0; >> } >> >> diff --git a/drm/nouveau/nouveau_drm.h b/drm/nouveau/nouveau_drm.h >> index 5c363ed..b148dcb 100644 >> --- a/drm/nouveau/nouveau_drm.h >> +++ b/drm/nouveau/nouveau_drm.h >> @@ -166,6 +166,9 @@ struct...

On 23 August 2016 at 00:42, Martin Peres <martin.peres at free.fr> wrote: > v2: > - guard LED framework calls with ifdef CONFIG_LEDS_CLASS > IIRC kernel has the tendency of using static inlines in the headers when CONFIG_foo is not set. Worth using that and removing the ifdef from the source file ? -Emil

This is the latest version of https://patchwork.freedesktop.org/series/46815/ I moved everything out of fb_helper and back into nouveau, because it seems that other drivers actually do have this handled already as far as I can tell. Lyude Paul (13): drm/nouveau: Fix bogus drm_kms_helper_poll_enable() placement drm/nouveau: Remove duplicate poll_enable() in pmops_runtime_suspend()