mbox series

[v3,0/5] dt-bindings: display: ti,tfp410.txt: convert to yaml

Message ID 20200611102356.31563-1-ricardo.canuelo@collabora.com (mailing list archive)
Headers show
Series dt-bindings: display: ti,tfp410.txt: convert to yaml | expand

Message

Ricardo Cañuelo June 11, 2020, 10:23 a.m. UTC
This series converts the DT binding for the TI TFP410 DPI-to-DVI encoder
to json-schema.

This also fixes a minor bug in the ti-tfp410 driver that would cause a
wrong calculation of the setup and hold times when the de-skew feature
is enabled. The retrieval of the de-skew value from the DT has also been
updated to reflect the binding changes.

Changes in v3:

  - ti,tfp410.yaml
    - Original translation to yaml separated into its own commit with no
      additional changes (Laurent Pinchart).
    - Redefinition of ti,deskew property in its own commit (Laurent).
    - Removal of unnecessary text (Laurent).
    - New conditional constraint for ti,deskew (Laurent).
    - "ports" node set as optional (Rob Herring).

  - dove-sbc-a510.dts: removed. This comes from Rob's suggestion that
    the "ports" node should be optional instead of defining empty port
    nodes in DTs files.

  - ti-tfp410.c
    - Original changes split into two commits. The first one changes the
      datatype and retrieval of the deskew property. The second one
      fixes the calculation of hold and setup times (Laurent).

The bindings have been tested with:

  make dt_binding_check ARCH=<arch> DT_SCHEMA_FILES=<...ti,tfp410.yaml>
  make dtbs_check ARCH=<arch> DT_SCHEMA_FILES=<...ti,tfp410.yaml>

for <arch> = arm and arm64.
This uncovered a number of dts files that use the TFP410 but not through
I2C and don't define the ti,deskew property. These should be fixed at
some point.

Ricardo Cañuelo (5):
  dt-bindings: display: ti,tfp410.txt: convert to yaml
  dt-bindings: display: ti,tfp410.yaml: Redefine ti,deskew property
  drm/bridge: tfp410: fix de-skew value retrieval from DT
  drm/bridge: tfp410: Fix setup and hold time calculation
  dt-bindings: display: ti,tfp410.yaml: make the ports node optional

 .../bindings/display/bridge/ti,tfp410.txt     |  66 ---------
 .../bindings/display/bridge/ti,tfp410.yaml    | 130 ++++++++++++++++++
 drivers/gpu/drm/bridge/ti-tfp410.c            |  10 +-
 3 files changed, 135 insertions(+), 71 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/display/bridge/ti,tfp410.txt
 create mode 100644 Documentation/devicetree/bindings/display/bridge/ti,tfp410.yaml