On 12/3/25 16:19, Kory Maincent wrote:
On Tue, 2 Dec 2025 17:55:15 +0530
Swamil Jain <[email protected]> wrote:
Hi Kory,
Thanks for the series.
On 11/26/25 23:05, Kory Maincent (TI.com) wrote:
The starting point for this work was adding support for the HDMI cape:
https://www.seeedstudio.com/Seeed-Studio-BeagleBone-Green-HDMI-Cape.html
This will be sent in a later series.
Initially, Miguel proposed modifying the ite-it66121 bridge to support
the legacy behavior without the DRM_BRIDGE_ATTACH_NO_CONNECTOR flag:
https://lore.kernel.org/lkml/[email protected]/
This patch was NAK'd as we don't want to add more legacy code. Maxime
requested that the tilcdc driver be updated to use
DRM_BRIDGE_ATTACH_NO_CONNECTOR instead.
While working on this update, I discovered that the tilcdc driver
contained significant amounts of legacy code that needed cleaning.
Since this driver was developed alongside the tda998x driver for
several AM335x boards, the tda998x driver also required cleanup and
support for the DRM_BRIDGE_ATTACH_NO_CONNECTOR flag.
This series is based on the tilcdc fix sent to mainline:
https://lore.kernel.org/lkml/[email protected]/
Patch 1-7: Convert tilcdc binding to YAML and remove the ti,tilcdc,panel
sub-binding and driver
Patch 8-16: Clean up tilcdc driver
Patch 17-19: Clean up tda998x driver
Patch 20: Add DRM_BRIDGE_ATTACH_NO_CONNECTOR support to tda998x
Patch 21: Add DRM_BRIDGE_ATTACH_NO_CONNECTOR support to tilcdc
Signed-off-by: Kory Maincent (TI.com) <[email protected]>
---
Kory Maincent (TI.com) (21):
dt-bindings: display: tilcdc: Convert to DT schema
dt-bindings: display: tilcdc: Add fifo-threshold property
drm/tilcdc: Remove simulate_vesa_sync flag
drm/tilcdc: Add support for DRM bus flags and simplify panel config
ARM: dts: omap: Bind panel to panel-dpi instead of ti,tilcdc,panel
driver dt-bindings: display: tilcdc: Remove panel binding
drm/tilcdc: Remove tilcdc panel driver
drm/tilcdc: Remove component framework support
drm/tilcdc: Remove tilcdc_panel_info structure
drm/tilcdc: Remove redundant #endif/#ifdef in debugfs code
drm/tilcdc: Remove unused encoder and connector tracking arrays
drm/tilcdc: Rename external_encoder and external_connector to
encoder and connector drm/tilcdc: Rename tilcdc_external to tilcdc_encoder
drm/tilcdc: Remove the useless module list support
drm/tilcdc: Modernize driver initialization and cleanup paths
drm/tilcdc: Remove the use of drm_device private_data
drm/bridge: tda998x: Remove component support
drm/bridge: tda998x: Move tda998x_create/destroy into probe and
remove drm/bridge: tda998x: Remove useless tda998x_connector_destroy wrapper
drm/bridge: tda998x: Add support for DRM_BRIDGE_ATTACH_NO_CONNECTOR
drm/tilcdc: Add support for DRM_BRIDGE_ATTACH_NO_CONNECTOR
.../devicetree/bindings/display/tilcdc/panel.txt | 66 ---
.../devicetree/bindings/display/tilcdc/tilcdc.txt | 82 ----
.../devicetree/bindings/display/tilcdc/tilcdc.yaml | 103 +++++
arch/arm/boot/dts/ti/davinci/da850-evm.dts | 26 +-
arch/arm/boot/dts/ti/omap/am335x-guardian.dts | 25 +-
arch/arm/boot/dts/ti/omap/am335x-pdu001.dts | 21 +-
arch/arm/boot/dts/ti/omap/am335x-pepper.dts | 22 +-
arch/arm/boot/dts/ti/omap/am335x-sbc-t335.dts | 25 +-
arch/arm/boot/dts/ti/omap/am335x-sl50.dts | 25 +-
drivers/gpu/drm/bridge/tda998x_drv.c | 251 ++++++------
drivers/gpu/drm/tilcdc/Makefile | 3 +-
drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 117 ++----
drivers/gpu/drm/tilcdc/tilcdc_drv.c | 456
+++++++-------------- drivers/gpu/drm/tilcdc/tilcdc_drv.h |
88 +--- drivers/gpu/drm/tilcdc/tilcdc_encoder.c | 93 +++++
.../tilcdc/{tilcdc_external.h => tilcdc_encoder.h} | 5 +-
drivers/gpu/drm/tilcdc/tilcdc_external.c | 179 --------
drivers/gpu/drm/tilcdc/tilcdc_panel.c | 408
------------------ drivers/gpu/drm/tilcdc/tilcdc_panel.h | 15
- drivers/gpu/drm/tilcdc/tilcdc_plane.c | 2 +-
drivers/gpu/drm/tilcdc/tilcdc_regs.h | 8 +-
21 files changed, 589 insertions(+), 1431 deletions(-)
---
base-commit: 670bacfc7579bdd79a3069cfb5ab60a6a7923003
I was trying to test the patches, unable to find the base-commit, are
you using drm-misc-next?
It is based on the tilcdc fix as explained in the cover letter.
https://lore.kernel.org/lkml/[email protected]/
Therefore you won't be able to find this base commit hash.
Sorry I missed it. Thanks for pointing.
Regards,
Swamil.
Regards,