mbox series

[0/2] Introduce h/vsync-disable properties for ti-sn65dsi83

Message ID 20250306091133.46888-1-alessandro.zini@siemens.com (mailing list archive)
Headers show
Series Introduce h/vsync-disable properties for ti-sn65dsi83 | expand

Message

Zini, Alessandro March 6, 2025, 9:11 a.m. UTC
From: Alessandro Zini <alessandro.zini@siemens.com>

This patch series adds support for disabling the generation of h/vsync signals
on the ti-sn65dsi83 bridge.

This is required on some panels which are driven in DE-only mode but do not
ignore sync packets, and instead require them to be low-voltage level or ground.

A discussion (1) on TI's E2E forum confirms that this may be required for some
panels.

(1) https://e2e.ti.com/support/interface-group/interface/f/interface-forum/1475734/sn65dsi84-disable-hsync-and-vsync

Alessandro Zini (2):
  dt-bindings: drm/bridge: ti-sn65dsi83: add h/vsync-disable bindings
  drm/bridge: ti-sn65dsi83: add h/vsync-disable support

 .../bindings/display/bridge/ti,sn65dsi83.yaml    | 12 ++++++++++++
 drivers/gpu/drm/bridge/ti-sn65dsi83.c            | 16 ++++++++++++++--
 2 files changed, 26 insertions(+), 2 deletions(-)

Comments

Rob Herring (Arm) March 7, 2025, 9:24 p.m. UTC | #1
On Thu, Mar 06, 2025 at 10:11:31AM +0100, A. Zini wrote:
> From: Alessandro Zini <alessandro.zini@siemens.com>
> 
> This patch series adds support for disabling the generation of h/vsync signals
> on the ti-sn65dsi83 bridge.
> 
> This is required on some panels which are driven in DE-only mode but do not
> ignore sync packets, and instead require them to be low-voltage level or ground.
> 
> A discussion (1) on TI's E2E forum confirms that this may be required for some
> panels.

If this is a property of the panel then it should be in the panel's 
endpoint or implicit with implicit being preferred. It should not be 
defined in some bridge binding.

Rob