linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] drm/panel: st7789v: Add a dependency on the backlight framework
@ 2017-05-30 21:01 Maxime Ripard
  2017-06-10 20:10 ` Maxime Ripard
  2017-06-14 18:22 ` Thierry Reding
  0 siblings, 2 replies; 3+ messages in thread
From: Maxime Ripard @ 2017-05-30 21:01 UTC (permalink / raw)
  To: Thierry Reding, Daniel Vetter, David Airlie
  Cc: dri-devel, linux-kernel, Maxime Ripard

The driver depends on the backlight functions, but we have no dependency
on it in Kconfig. Add this dependency to avoid breakages.

Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
---
 drivers/gpu/drm/panel/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/panel/Kconfig b/drivers/gpu/drm/panel/Kconfig
index 3e29a9903303..ab40d0fbd457 100644
--- a/drivers/gpu/drm/panel/Kconfig
+++ b/drivers/gpu/drm/panel/Kconfig
@@ -100,6 +100,7 @@ config DRM_PANEL_SHARP_LS043T1LE01
 config DRM_PANEL_SITRONIX_ST7789V
 	tristate "Sitronix ST7789V panel"
 	depends on OF && SPI
+	depends on BACKLIGHT_CLASS_DEVICE
 	help
 	  Say Y here if you want to enable support for the Sitronix
 	  ST7789V controller for 240x320 LCD panels
-- 
2.9.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] drm/panel: st7789v: Add a dependency on the backlight framework
  2017-05-30 21:01 [PATCH] drm/panel: st7789v: Add a dependency on the backlight framework Maxime Ripard
@ 2017-06-10 20:10 ` Maxime Ripard
  2017-06-14 18:22 ` Thierry Reding
  1 sibling, 0 replies; 3+ messages in thread
From: Maxime Ripard @ 2017-06-10 20:10 UTC (permalink / raw)
  To: Thierry Reding, Daniel Vetter, David Airlie; +Cc: dri-devel, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 393 bytes --]

On Tue, May 30, 2017 at 11:01:36PM +0200, Maxime Ripard wrote:
> The driver depends on the backlight functions, but we have no dependency
> on it in Kconfig. Add this dependency to avoid breakages.
> 
> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>

Ping?
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] drm/panel: st7789v: Add a dependency on the backlight framework
  2017-05-30 21:01 [PATCH] drm/panel: st7789v: Add a dependency on the backlight framework Maxime Ripard
  2017-06-10 20:10 ` Maxime Ripard
@ 2017-06-14 18:22 ` Thierry Reding
  1 sibling, 0 replies; 3+ messages in thread
From: Thierry Reding @ 2017-06-14 18:22 UTC (permalink / raw)
  To: Maxime Ripard; +Cc: Daniel Vetter, David Airlie, dri-devel, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 485 bytes --]

On Tue, May 30, 2017 at 11:01:36PM +0200, Maxime Ripard wrote:
> The driver depends on the backlight functions, but we have no dependency
> on it in Kconfig. Add this dependency to avoid breakages.
> 
> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
> ---
>  drivers/gpu/drm/panel/Kconfig | 1 +
>  1 file changed, 1 insertion(+)

Arnd had sent the same patch a couple of days before you did, I've now
applied his patch to drm-misc-next.

Thanks,
Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-06-14 18:22 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-30 21:01 [PATCH] drm/panel: st7789v: Add a dependency on the backlight framework Maxime Ripard
2017-06-10 20:10 ` Maxime Ripard
2017-06-14 18:22 ` Thierry Reding

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).