Re: [PATCH v3 0/3] drm/panel-edp: remove several legacy compatibles used by the driver

From: Jeffrey Hugo
Date: Fri May 31 2024 - 13:46:32 EST


On 5/31/2024 11:43 AM, Dmitry Baryshkov wrote:
On Fri, May 31, 2024 at 10:18:07AM -0600, Jeffrey Hugo wrote:
On 5/30/2024 5:12 PM, 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).

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

Ok to drop the sharp one I added. It should be able to be handled by the
(newish) edp-panel, but I think the TI bridge driver needs some work for the
specific platform (no I2C connection) to verify.

Thanks. I'm tempted to merge the series as is now and drop
sharp,ld-d5116z01b once you can confirm that it can be handled by
edp-panel on your platform.


Sounds good to me.

-Jeff