[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20220205001342.3155839-1-dianders@chromium.org>
Date: Fri, 4 Feb 2022 16:13:39 -0800
From: Douglas Anderson <dianders@...omium.org>
To: dri-devel@...ts.freedesktop.org
Cc: Daniel Vetter <daniel@...ll.ch>,
Javier Martinez Canillas <javierm@...hat.com>,
robert.foss@...aro.org, lschyi@...omium.org,
Sam Ravnborg <sam@...nborg.org>, jjsu@...omium.org,
Douglas Anderson <dianders@...omium.org>,
Andrzej Hajda <andrzej.hajda@...el.com>,
David Airlie <airlied@...ux.ie>,
Jernej Skrabec <jernej.skrabec@...il.com>,
Jonas Karlman <jonas@...boo.se>,
Laurent Pinchart <Laurent.pinchart@...asonboard.com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>,
Neil Armstrong <narmstrong@...libre.com>,
Thierry Reding <thierry.reding@...il.com>,
Thomas Zimmermann <tzimmermann@...e.de>,
linux-kernel@...r.kernel.org
Subject: [PATCH v2 0/3] drm/panel-edp: Debugfs for panel-edp
The main goal of this series is the final patch in the series: to
allow test code to reliably find out if we ended up hitting the
"fallback" case of the generic edp-panel driver where we don't
recognize a panel and choose to use super conservative timing.
Version 1 of the patch actually landed but was quickly reverted since
it was pointed out that it should have been done in debugfs, not
sysfs.
As discussed on IRC [1], we want this support to be under the
"connector" directory of debugfs but there was no existing way to do
that. Thus patch #2 in the series was born to try to plumb this
through. It was asserted that it would be OK to rely on a fairly
modern display pipeline for this plumbing and perhaps fail to populate
the debugfs file if we're using older/deprecated pipelines.
Patch #1 in the series was born because the bridge chip I was using
was still using an older/deprecated pipeline. While this doesn't get
us fully to a modern pipeline for ti-sn65dsi86 (it still doesn't move
to "NO_CONNECTOR") it hopefully moves us in the right direction.
This was tested on sc7180-trogdor devices with _both_ the ti-sn65dsi86
and the parade-ps8640 bridge chips (since some devices have one, some
the other). The parade-ps8640 already uses supports "NO_CONNECTOR",
luckily.
[1] https://people.freedesktop.org/~cbrill/dri-log/?channel=dri-devel&date=2022-02-02
Changes in v2:
- ("ti-sn65dsi86: Use drm_bridge_connector") new for v2.
- ("drm: Plumb debugfs_init through to panels") new for v2.
- Now using debugfs, not sysfs
Douglas Anderson (3):
drm/bridge: ti-sn65dsi86: Use drm_bridge_connector
drm: Plumb debugfs_init through to panels
drm/panel-edp: Allow querying the detected panel via debugfs
drivers/gpu/drm/bridge/panel.c | 12 +++++
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 72 +++++---------------------
drivers/gpu/drm/drm_bridge_connector.c | 15 ++++++
drivers/gpu/drm/drm_debugfs.c | 3 ++
drivers/gpu/drm/panel/panel-edp.c | 37 +++++++++++--
include/drm/drm_bridge.h | 7 +++
include/drm/drm_connector.h | 7 +++
include/drm/drm_panel.h | 8 +++
8 files changed, 98 insertions(+), 63 deletions(-)
--
2.35.0.263.gb82422642f-goog
Powered by blists - more mailing lists