All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Mark Brown <broonie@kernel.org>
Cc: Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>,
	linux-fbdev@vger.kernel.org, linux-omap@vger.kernel.org,
	linaro-kernel@lists.linaro.org, Mark Brown <broonie@linaro.org>
Subject: Re: [PATCH] OMAPDSS: Add missing dependency on backlight for DSI-CM panel drier
Date: Thu, 26 Sep 2013 08:36:26 +0000	[thread overview]
Message-ID: <5243F20A.9000707@ti.com> (raw)
In-Reply-To: <1380108661-22702-1-git-send-email-broonie@kernel.org>

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

On 25/09/13 14:31, Mark Brown wrote:
> From: Mark Brown <broonie@linaro.org>
> 
> The DSI-CM driver uses the backlight class so needs to build depend on it.
> 
> Signed-off-by: Mark Brown <broonie@linaro.org>
> ---
>  drivers/video/omap2/displays-new/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/video/omap2/displays-new/Kconfig b/drivers/video/omap2/displays-new/Kconfig
> index 6c90885..10b25e7 100644
> --- a/drivers/video/omap2/displays-new/Kconfig
> +++ b/drivers/video/omap2/displays-new/Kconfig
> @@ -35,6 +35,7 @@ config DISPLAY_PANEL_DPI
>  
>  config DISPLAY_PANEL_DSI_CM
>  	tristate "Generic DSI Command Mode Panel"
> +	depends on BACKLIGHT_CLASS_DEVICE
>  	help
>  	  Driver for generic DSI command mode panels.
>  
> 

Thanks, I'll queue for 3.12 fixes.

I wish we could "select" instead of "depends on"...

 Tomi



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 901 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Mark Brown <broonie@kernel.org>
Cc: Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>,
	linux-fbdev@vger.kernel.org, linux-omap@vger.kernel.org,
	linaro-kernel@lists.linaro.org, Mark Brown <broonie@linaro.org>
Subject: Re: [PATCH] OMAPDSS: Add missing dependency on backlight for DSI-CM panel drier
Date: Thu, 26 Sep 2013 11:36:26 +0300	[thread overview]
Message-ID: <5243F20A.9000707@ti.com> (raw)
In-Reply-To: <1380108661-22702-1-git-send-email-broonie@kernel.org>

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

On 25/09/13 14:31, Mark Brown wrote:
> From: Mark Brown <broonie@linaro.org>
> 
> The DSI-CM driver uses the backlight class so needs to build depend on it.
> 
> Signed-off-by: Mark Brown <broonie@linaro.org>
> ---
>  drivers/video/omap2/displays-new/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/video/omap2/displays-new/Kconfig b/drivers/video/omap2/displays-new/Kconfig
> index 6c90885..10b25e7 100644
> --- a/drivers/video/omap2/displays-new/Kconfig
> +++ b/drivers/video/omap2/displays-new/Kconfig
> @@ -35,6 +35,7 @@ config DISPLAY_PANEL_DPI
>  
>  config DISPLAY_PANEL_DSI_CM
>  	tristate "Generic DSI Command Mode Panel"
> +	depends on BACKLIGHT_CLASS_DEVICE
>  	help
>  	  Driver for generic DSI command mode panels.
>  
> 

Thanks, I'll queue for 3.12 fixes.

I wish we could "select" instead of "depends on"...

 Tomi



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 901 bytes --]

  parent reply	other threads:[~2013-09-26  8:36 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-25 11:31 [PATCH] OMAPDSS: Add missing dependency on backlight for DSI-CM panel drier Mark Brown
2013-09-25 11:31 ` Mark Brown
2013-09-26  0:59 ` Jingoo Han
2013-09-26  0:59   ` Jingoo Han
2013-09-26  8:36 ` Tomi Valkeinen [this message]
2013-09-26  8:36   ` Tomi Valkeinen
2013-09-26 10:12   ` Mark Brown
2013-09-26 10:12     ` Mark Brown
2013-09-26 10:21     ` Tomi Valkeinen
2013-09-26 10:21       ` Tomi Valkeinen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5243F20A.9000707@ti.com \
    --to=tomi.valkeinen@ti.com \
    --cc=broonie@kernel.org \
    --cc=broonie@linaro.org \
    --cc=linaro-kernel@lists.linaro.org \
    --cc=linux-fbdev@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=plagnioj@jcrosoft.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.