dri-devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Boris Brezillon <boris.brezillon@bootlin.com>
To: Thierry Reding <thierry.reding@gmail.com>,
	dri-devel@lists.freedesktop.org
Cc: David Airlie <airlied@linux.ie>,
	Boris Brezillon <boris.brezillon@bootlin.com>
Subject: [RFC PATCH 0/3] drm/panel: Support panel detection
Date: Mon, 30 Apr 2018 16:43:20 +0200	[thread overview]
Message-ID: <20180430144323.9233-1-boris.brezillon@bootlin.com> (raw)

Some panels are connected through extension boards an provide an easy
way for the main board to detect when they are present (like checking
for a working I2C communication with a device and making sure a
specific reg in this device has a consistent value).

When this is the case, we might want to support dynamic panel detection
and only expose the display (and its display modes) when the panel is
detected, similar to the monitor detection we use for regular
connectors (HDMI, DVI, ...).

This solves a problem we have on the Rpi when the panel is not
connected to the board but described in the DT. This prevents the whole
display pipeline from being exposed because one of the element (the
panel) is missing.

This was posted as an RFC because I'm not sure dynamically detecting
panels or supporting panel hotplug is actually something we want to do.

Thierry, and DRM folks in general, feel free to give your opinion on
this approach, and/or propose others solution to solve the "DSI panel
not connected" problem we have on RPi. Oh, and before someone suggests
it, IIRC, Eric wanted to avoid relying on DT overlays to do that, I
don't remember why though.

Regards,

Boris

Boris Brezillon (3):
  drm/panel: Support panel detection
  drm/bridge: panel: Make use of the panel detection infrastructure
  drm/panel: rpi-touchscreen: Implement ->detect()

 drivers/gpu/drm/bridge/panel.c                     | 13 +++++
 .../gpu/drm/panel/panel-raspberrypi-touchscreen.c  | 62 ++++++++++++++++++----
 include/drm/drm_panel.h                            | 12 +++++
 3 files changed, 78 insertions(+), 9 deletions(-)

-- 
2.14.1

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

             reply	other threads:[~2018-04-30 14:43 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-30 14:43 Boris Brezillon [this message]
2018-04-30 14:43 ` [RFC PATCH 1/3] drm/panel: Support panel detection Boris Brezillon
2018-04-30 16:08   ` Daniel Vetter
2018-04-30 16:34     ` Boris Brezillon
2018-04-30 14:43 ` [RFC PATCH 2/3] drm/bridge: panel: Make use of the panel detection infrastructure Boris Brezillon
2018-04-30 14:43 ` [RFC PATCH 3/3] drm/panel: rpi-touchscreen: Implement ->detect() Boris Brezillon
2018-04-30 17:22 ` [RFC PATCH 0/3] drm/panel: Support panel detection Eric Anholt
2018-04-30 17:43   ` Boris Brezillon
2018-04-30 19:48     ` Daniel Vetter

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=20180430144323.9233-1-boris.brezillon@bootlin.com \
    --to=boris.brezillon@bootlin.com \
    --cc=airlied@linux.ie \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=thierry.reding@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox