From: shawn.guo@freescale.com (Shawn Guo)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2] staging: imx-drm-core: skip components whose parent device is disabled
Date: Sat, 19 Apr 2014 13:53:08 +0800 [thread overview]
Message-ID: <20140419055205.GX2523@dragon> (raw)
In-Reply-To: <20140418204249.GG24070@n2100.arm.linux.org.uk>
On Fri, Apr 18, 2014 at 09:42:49PM +0100, Russell King - ARM Linux wrote:
> On Mon, Apr 14, 2014 at 10:02:26AM +0800, Shawn Guo wrote:
> > In a board setup which disables LDB device node completely by changing
> > status to 'disabled', and only enables HDMI device, we're running into
> > the problem that imx-drm master never succeeds in binding, and hence
> > HDMI does not come up either.
> >
> > &ldb {
> > status = "disabled";
> >
> > lvds-channel at 1 {
> > ...
> > status = "okay";
> > };
> > };
> >
> > The imx-drm-core should really skip the LVDS channels no matter what
> > lvds-channel's status is, if LDB device is disabled. Let's consider
> > such setup a misconfiguration, give a warning in there and not add the
> > component.
> >
> > Signed-off-by: Shawn Guo <shawn.guo@freescale.com>
> > ---
> > Changes since v1:
> > * Put a warning on such misconfiguration
>
> Thanks Shawn, I've added it to my queue for Greg.
Thanks, Russell.
BTW, can you take a look at the following two patches, or are they
already in your queue?
imx-drm: imx-drm-core: Fix imx_drm_encoder_get_mux_id
https://lkml.org/lkml/2014/4/7/58
mx-drm: imx-tve: correct DDC property name to 'ddc-i2c-bus'
('i' is missing from the prefix word, and it should be 'imx-drm')
http://www.spinics.net/lists/arm-kernel/msg321057.html
Shawn
next prev parent reply other threads:[~2014-04-19 5:53 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-14 2:02 [PATCH v2] staging: imx-drm-core: skip components whose parent device is disabled Shawn Guo
2014-04-14 8:01 ` Philipp Zabel
2014-04-18 20:42 ` Russell King - ARM Linux
2014-04-19 5:53 ` Shawn Guo [this message]
2014-04-19 8:42 ` Russell King - ARM Linux
2014-04-19 11:00 ` Shawn Guo
2014-04-21 14:22 ` Olof Johansson
2014-04-21 23:31 ` Shawn Guo
2014-04-22 20:49 ` Arnd Bergmann
2014-04-22 21:55 ` Russell King - ARM Linux
2014-04-23 7:44 ` Shawn Guo
2014-04-23 9:00 ` Russell King - ARM Linux
2014-04-24 21:36 ` Arnd Bergmann
2014-04-26 10:20 ` Russell King - ARM Linux
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=20140419055205.GX2523@dragon \
--to=shawn.guo@freescale.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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.