devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Sebastian Reichel <sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
Cc: linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	"Benoît Cousson"
	<bcousson-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	"Marcel Partap" <mpartap-hi6Y0CQ0nG0@public.gmane.org>,
	"Michael Scott"
	<michael.scott-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	"Tomi Valkeinen" <tomi.valkeinen-l0cyMroinI0@public.gmane.org>
Subject: Re: [PATCH 6/6] ARM: dts: omap4-droid4: Add LCD
Date: Sun, 19 Mar 2017 23:55:42 +0100	[thread overview]
Message-ID: <20170319225542.ts53qpwdhdrmy2ac@earth> (raw)
In-Reply-To: <20170319161030.GA20572-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>

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

Hi,

On Sun, Mar 19, 2017 at 09:10:30AM -0700, Tony Lindgren wrote:
> * Sebastian Reichel <sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> [170318 18:31]:
> > Hi,
> > 
> > On Sat, Mar 04, 2017 at 09:43:59PM -0800, Tony Lindgren wrote:
> > > The LCD panel on droid 4 is a command mode LCD. The binding follows
> > > the standard omapdrm binding and the changes needed for omapdrm command
> > > mode panels are posted separately.
> > > 
> > > Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> > > Cc: Marcel Partap <mpartap-hi6Y0CQ0nG0@public.gmane.org>
> > > Cc: Michael Scott <michael.scott-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
> > > Cc: Sebastian Reichel <sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> > > Cc: Tomi Valkeinen <tomi.valkeinen-l0cyMroinI0@public.gmane.org>
> > > Signed-off-by: Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
> > 
> > Tested-By: Sebastian Reichel <sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> > 
> > With a non-modular kernel lcd is not working with omapdrm if HDMI
> > is enabled. After dropping HDMI in droid4's dts file everything
> > worked as expected. I assume both work properly with a modular
> > kernel?
> 
> Yes with loadable modules both work just fine. If things do not
> work properly as built-in, chances are there's some unhandled
> dependency that needs -EPROBE_DEFER somewhere for a regulator
> or a clock.

I think that would also result in problems with disabled HDMI.
I guess the problem is, that omapdrm is initialized too early.
AFAIK omapdrm is not hotplug-capable.

-- Sebastian

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

  parent reply	other threads:[~2017-03-19 22:55 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-05  5:43 [PATCH 0/6] Device tree changes to add more droid 4 devices Tony Lindgren
     [not found] ` <20170305054359.8293-1-tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-03-05  5:43   ` [PATCH 1/6] ARM: dts: omap4-droid4: Add touchscreen Tony Lindgren
2017-03-05  5:43   ` [PATCH 2/6] ARM: dts: omap4-droid4: Add LCD backlight Tony Lindgren
     [not found]     ` <20170305054359.8293-3-tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-03-05 23:24       ` Kim, Milo
2017-03-19  1:18       ` Sebastian Reichel
2017-03-20  2:30         ` Kim, Milo
2017-03-05  5:43   ` [PATCH 3/6] ARM: dts: omap4-droid4: Add GPIO poweroff Tony Lindgren
     [not found]     ` <20170305054359.8293-4-tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-03-19  1:19       ` Sebastian Reichel
2017-03-05  5:43   ` [PATCH 4/6] ARM: dts: omap4-droid4: Add tmp105 sensor for droid 4 Tony Lindgren
     [not found]     ` <20170305054359.8293-5-tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-03-19  1:26       ` Sebastian Reichel
2017-03-05  5:43   ` [PATCH 5/6] ARM: dts: omap4-droid4: Add HDMI support Tony Lindgren
     [not found]     ` <20170305054359.8293-6-tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-03-16 22:43       ` Sebastian Reichel
2017-03-05  5:43   ` [PATCH 6/6] ARM: dts: omap4-droid4: Add LCD Tony Lindgren
     [not found]     ` <20170305054359.8293-7-tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-03-19  1:29       ` Sebastian Reichel
2017-03-19 16:10         ` Tony Lindgren
     [not found]           ` <20170319161030.GA20572-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2017-03-19 22:55             ` Sebastian Reichel [this message]
2017-03-20  7:52               ` Tomi Valkeinen
     [not found]                 ` <9c4a5d88-4d2a-2120-9fdf-f3fbba69147c-l0cyMroinI0@public.gmane.org>
2017-03-23  6:31                   ` Sebastian Reichel
2017-03-23  7:08                     ` Tomi Valkeinen
     [not found]                       ` <b4666cd6-9925-6143-41c3-591e68c48456-l0cyMroinI0@public.gmane.org>
2017-03-23  8:42                         ` Sebastian Reichel

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=20170319225542.ts53qpwdhdrmy2ac@earth \
    --to=sre-dgejt+ai2ygdnm+yrofe0a@public.gmane.org \
    --cc=bcousson-rdvid1DuHRBWk0Htik3J/w@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=michael.scott-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=mpartap-hi6Y0CQ0nG0@public.gmane.org \
    --cc=tomi.valkeinen-l0cyMroinI0@public.gmane.org \
    --cc=tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.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 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).