Re: [PATCH v2 0/3] drm/i2c: tda998x: move under drivers/gpu/drm/bridge

From: Hans Verkuil
Date: Fri Jan 10 2025 - 07:46:12 EST


On 10/01/2025 13:31, Dmitry Baryshkov wrote:
> TDA998x is the HDMI bridge driver, incorporating drm_connector and
> optional drm_encoder (created via the component bind API by the TICLDC
> and HDLCD drivers). TDA9950 is an I2C-CEC translator, being present
> on-die on the TDA9989 and TDA19989 chips.
>
> Move TDA9950 driver to drivers/media/cec/i2c and TDA998x driver to
> drivers/gpu/drm/bridge/.
>
> Note to Hans and Mauro: I'd ask for all patches to be merged through
> drm-misc tree rather than splitting them between drm-misc and media
> trees. The patchset depends on the patches that are a part of
> drm-misc-next, but are not going to hit 6.14. Please specify if you are
> fine with that proposal and if you'd need an immutable branch for
> linux-media.

I'm fine that this goes through drm-misc. I don't see a need for an
immutable branch.

Regards,

Hans

>
> Note to RMK: I didn't change that, but I'd like to propose to use
> drm-misc tree for this bridge.
>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
> ---
> Changes in v2:
> - Moved TDA9950 driver to drivers/media/cec/i2c (Hans, Laurent)
> - Moved TDA998x driver to drivers/gpu/drm/bridges istead of tda/ subdir
> (Laurent)
> - Link to v1: https://lore.kernel.org/r/20241215-drm-move-tda998x-v1-0-7817122b1d73@xxxxxxxxxx
>
> ---
> Dmitry Baryshkov (3):
> drm/i2c: tda998x: drop support for platform_data
> media: cec: move driver for TDA9950 from drm/i2c
> drm/i2c: move TDA drivers under drivers/gpu/drm/bridge
>
> MAINTAINERS | 3 +-
> drivers/gpu/drm/arm/Kconfig | 1 +
> drivers/gpu/drm/bridge/Kconfig | 8 +++++
> drivers/gpu/drm/bridge/Makefile | 4 +++
> drivers/gpu/drm/{i2c => bridge}/tda998x_drv.c | 49 +++------------------------
> drivers/gpu/drm/i2c/Kconfig | 18 ----------
> drivers/gpu/drm/i2c/Makefile | 4 ---
> drivers/media/cec/i2c/Kconfig | 9 +++++
> drivers/media/cec/i2c/Makefile | 1 +
> drivers/{gpu/drm => media/cec}/i2c/tda9950.c | 0
> include/drm/i2c/tda998x.h | 40 ----------------------
> 11 files changed, 28 insertions(+), 109 deletions(-)
> ---
> base-commit: e91570cf7c1b3e406878e7b1efbe823d039d1659
> change-id: 20241214-drm-move-tda998x-8fb4321434d2
>
> Best regards,