Skip to content

RFC drm/i2c: Move tda998x to a couple encoder/connector

Jean-Francois Moine dri-devel linux-arm-kernel linux-devicetree linux-media 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 RFC Latest6 patches 6 repliesIn review
  2. v1 RFC5 patches 0 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6drm/i2c: tda998x: Add the required port propertyDocumentation/devicetree/bindings/drm/i2c/tda998x.txt+10 -10 replies
2/6drm/i2c: tda998x: Move tda998x to a couple encoder/connectordrivers/gpu/drm/i2c/tda998x_drv.c+188 -1355 replies
3/6drm/tilcd: dts: Add the video output portDocumentation/devicetree/bindings/drm/tilcdc/tilcdc.txt+14 -00 replies
4/6drm/tilcdc: Change the interface with the tda998x driverdrivers/gpu/drm/tilcdc/Makefile drivers/gpu/drm/tilcdc/tilcdc_drv.c drivers/gpu/drm/tilcdc/tilcdc_slave.c +1 more+68 -4500 replies
5/6drm/tilcd: dts: Remove unused slave descriptionDocumentation/devicetree/bindings/drm/tilcdc/slave.txt+0 -180 replies
6/6ARM: AM33XX: dts: Change the tda998x descriptionarch/arm/boot/dts/am335x-base0033.dts arch/arm/boot/dts/am335x-boneblack.dts+35 -140 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [PATCH RFC v2 0/6] drm/i2c: Move tda998x to a couple encoder/connector

The 'slave encoder' structure of the tda998x driver asks for glue between the DRM driver and the encoder/connector structures.

Changing the tda998x driver to a simple encoder/connector simplifies the code of the tilcdc driver. This change is permitted by Russell's infrastructure for componentised subsystems.

The proposed patch set does not include changes to the Armada DRM driver. These changes should already have been prepared by Russell King, as told in the message
https://www.mail-archive.com/linux-media at vger.kernel.org/msg71202.html

The tilcdc part of this patch set has not been tested.

This patch set applies after the patchs:
drm/i2c: tda998x: Fix lack of required reg in DT documentation drm/i2c: tda998x: Change the compatible strings

- v2
- fix lack of call to component_bind_all() in tilcdc_drv.c
- add tda998x configuration for non-DT systems

Jean-Francois Moine (6):
drm/i2c: tda998x: Add the required port property drm/i2c: tda998x: Move tda998x to a couple encoder/connector drm/tilcd: dts: Add the video output port drm/tilcdc: Change the interface with the tda998x driver drm/tilcd: dts: Remove unused slave description
ARM: AM33XX: dts: Change the tda998x description

 .../devicetree/bindings/drm/i2c/tda998x.txt        |  11 +-
 .../devicetree/bindings/drm/tilcdc/slave.txt       |  18 -
 .../devicetree/bindings/drm/tilcdc/tilcdc.txt      |  14 +
 arch/arm/boot/dts/am335x-base0033.dts              |  28 +-
 arch/arm/boot/dts/am335x-boneblack.dts             |  21 +-
 drivers/gpu/drm/i2c/tda998x_drv.c                  | 323 +++++++++-------
 drivers/gpu/drm/tilcdc/Makefile                    |   1 -
 drivers/gpu/drm/tilcdc/tilcdc_drv.c                |  85 ++++-
 drivers/gpu/drm/tilcdc/tilcdc_slave.c              | 406 ---------------------
 drivers/gpu/drm/tilcdc/tilcdc_slave.h              |  26 --
 10 files changed, 315 insertions(+), 618 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/drm/tilcdc/slave.txt
 delete mode 100644 drivers/gpu/drm/tilcdc/tilcdc_slave.c
 delete mode 100644 drivers/gpu/drm/tilcdc/tilcdc_slave.h
Signature · 1 line
1.9.1