| Message ID | 20240614-edp-panel-drop-v4-0-4e0a112eec46@linaro.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | drm/panel-edp: remove several legacy compatibles used by the driver | expand |
On Fri, 14 Jun 2024 03:02:19 +0300, Dmitry Baryshkov wrote: > There are two ways to describe an eDP panel in device tree. The > recommended way is to add a device on the AUX bus, ideally using the > edp-panel compatible. The legacy way is to define a top-level platform > device for the panel. > > > [ ... ] Acked-by: Maxime Ripard <mripard@kernel.org> Thanks! Maxime
On Fri, 14 Jun 2024 03:02:19 +0300, Dmitry Baryshkov wrote: > There are two ways to describe an eDP panel in device tree. The > recommended way is to add a device on the AUX bus, ideally using the > edp-panel compatible. The legacy way is to define a top-level platform > device for the panel. > > Document that adding support for eDP panels in a legacy way is strongly > discouraged (if not forbidden at all). > > [...] Applied to drm-misc-next, thanks! [1/3] drm/panel-edp: add fat warning against adding new panel compatibles commit: faeebafe1d273ed6271dfb6555cd90ee730343c1 [2/3] dt-bindings: display: panel-edp-legacy: drop several eDP panels commit: 73458101e5681a7d766184cb4e16cb7b87571359 [3/3] drm/panel-edp: drop several legacy panels commit: 51e1fb144f17c277309ed5a1c6f0d921df0064d7 Best regards,
There are two ways to describe an eDP panel in device tree. The recommended way is to add a device on the AUX bus, ideally using the edp-panel compatible. The legacy way is to define a top-level platform device for the panel. Document that adding support for eDP panels in a legacy way is strongly discouraged (if not forbidden at all). While we are at it, also drop legacy compatible strings and bindings for five panels. These compatible strings were never used by a DT file present in Linux kernel and most likely were never used with the upstream Linux kernel. The following compatibles were never used by the devices supported by the upstream kernel and are a subject to possible removal: - lg,lp097qx1-spa1 - samsung,lsn122dl01-c01 - sharp,ld-d5116z01b I'm considering dropping them later, unless there is a good reason not to do so. To: Douglas Anderson <dianders@chromium.org> To: Neil Armstrong <neil.armstrong@linaro.org> To: Jessica Zhang <quic_jesszhan@quicinc.com> To: Sam Ravnborg <sam@ravnborg.org> To: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> To: Maxime Ripard <mripard@kernel.org> To: Thomas Zimmermann <tzimmermann@suse.de> To: David Airlie <airlied@gmail.com> To: Daniel Vetter <daniel@ffwll.ch> To: Rob Herring <robh@kernel.org> To: Krzysztof Kozlowski <krzk+dt@kernel.org> To: Conor Dooley <conor+dt@kernel.org> Cc: dri-devel@lists.freedesktop.org Cc: linux-kernel@vger.kernel.org Cc: linux-arm-msm@vger.kernel.org Cc: linux-rockchip@lists.infradead.org Cc: Jeffrey Hugo <quic_jhugo@quicinc.com> Cc: devicetree@vger.kernel.org Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> Changes in v4: - Rebased on top of drm-misc-next - Added devicetree maintainers to cc list, missed them for v2/v3 - Link to v3: https://lore.kernel.org/r/20240531-edp-panel-drop-v3-0-4c98b2b95e3a@linaro.org Changes in v3: - Rephrased the warning comment, following some of Doug's suggestions. - Link to v2: https://lore.kernel.org/r/20240529-edp-panel-drop-v2-0-fcfc457fc8dd@linaro.org Changes in v2: - Actually drop support for five panels acked by Doug Andersson - Link to v1: https://lore.kernel.org/r/20240523-edp-panel-drop-v1-1-045d62511d09@linaro.org --- Dmitry Baryshkov (3): drm/panel-edp: add fat warning against adding new panel compatibles dt-bindings: display: panel-edp-legacy: drop several eDP panels drm/panel-edp: drop several legacy panels .../bindings/display/panel/panel-edp-legacy.yaml | 10 -- drivers/gpu/drm/panel/panel-edp.c | 192 +++------------------ 2 files changed, 25 insertions(+), 177 deletions(-) --- base-commit: 9324410846e13595d453b7f34508b1f6b15fb1a7 change-id: 20240523-edp-panel-drop-00aa239b3c6b Best regards,