All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Anholt <eric@anholt.net>
To: dri-devel@lists.freedesktop.org
Subject: -EPROBE_DEFER and failed DSI panel probe
Date: Wed, 15 Nov 2017 12:26:48 -0800	[thread overview]
Message-ID: <87k1yr9tkn.fsf@anholt.net> (raw)


[-- Attachment #1.1: Type: text/plain, Size: 1256 bytes --]

I'm happy to have the DSI panel finally working on VC4 (just waiting on
https://lists.freedesktop.org/archives/dri-devel/2017-October/156407.html),
but now I've got another problem to solve.  It would be great if I could
include the DSI panel in our upstream DT, so that it automatically
worked when you plugged one in.  However, right now we return
-EPROBE_DEFER during bind unless the panel has actually shown up.  This
means that if you don't have the panel actually connected, you get this
sequence at startup:

[   10.719929] [drm] Initialized
[   10.829510] vc4_hdmi 3f902000.hdmi: vc4-hdmi-hifi <-> 3f902000.hdmi mapping ok
[   10.844043] vc4-drm soc:gpu: bound 3f902000.hdmi (ops vc4_hdmi_ops [vc4])
[   10.848626] vc4-drm soc:gpu: bound 3f806000.vec (ops vc4_vec_ops [vc4])
[   10.850214] vc4-drm soc:gpu: failed to bind 3f700000.dsi (ops vc4_dsi_ops [vc4]): -517
[   10.856559] vc4-drm soc:gpu: master bind failed: -517

[...]

[   10.967718] rpi_touchscreen 3-0045: Atmel I2C read failed: -6

Once the panel driver fails to probe, we never get asked to re-bind vc4,
and drm_of_find_panel_or_bridge looks like it would just give us
-EPROBE_DEFER again since the panel still wasn't registered.

Does anyone have any suggestions for handling this?

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

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

             reply	other threads:[~2017-11-15 20:26 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20171115202702epcas5p3a0fd46223a5ad440e897e05f855b85bd@epcas5p3.samsung.com>
2017-11-15 20:26 ` Eric Anholt [this message]
2017-11-16  8:34   ` -EPROBE_DEFER and failed DSI panel probe Andrzej Hajda
2017-11-16 20:27     ` Eric Anholt
2017-11-17  7:14       ` Andrzej Hajda
2017-11-18  1:16         ` Eric Anholt
2017-11-20  7:53           ` Andrzej Hajda
2017-11-20 10:11             ` Daniel Vetter
2017-11-20 20:31               ` Eric Anholt
2018-02-08 16:06             ` Boris Brezillon
2017-11-20 20:16   ` Rob Herring

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=87k1yr9tkn.fsf@anholt.net \
    --to=eric@anholt.net \
    --cc=dri-devel@lists.freedesktop.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.