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@xxxxxxxxxxxxxxx > > wrote: > > From: Laurent Pinchart < > > laurent.pinchart@xxxxxxxxxxxxxxxx > > > > > > > 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