Am Donnerstag, dem 09.12.2021 um 12:37 +0530 schrieb Jagan Teki:
On Thu, Nov 18, 2021 at 2:50 PM Alexander Stein < alexander.stein@ew.tq-group.com
wrote: From: Laurent Pinchart < laurent.pinchart@ideasonboard.com
The SN65DSI8x EN signal may be tied to VCC, or otherwise controlled by means not available to the kernel. Make the GPIO optional.
Sorry, I couldn't understand what it means. Does it mean VCC enabled designs no need to enable GPIO? I've a design that do support both EN and VCC.
The patches 1 & 2 are only about the "enable" gpio for the bridge, it's unrelated to the VCC regulator introduced in patch 3 & 4. Maybe the commit message should say:
The SN65DSI8x EN signal may be hard-wired to VCC, or otherwise
controlled[...] But I copied the message from bbda1704fc15 ("drm/bridge: ti-sn65dsi86: Make enable GPIO optional").
This is for use-cases where there is no GPIO the kernel can use, to control the EN pad of the bridge. Thus make this gpio optional in bindings and driver.
HTH Alexander