[v3] drm/i2c: tda998x: move under drivers/gpu/drm/bridge

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

Posted by Dmitry Baryshkov 11 months, 1 week ago

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: In a comment for v2 Hans has acked merging these patches through
drm-misc(-next) with no additional immutable branch.

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@linaro.org>
---
Changes in v3:
- Dropped irrelevant sentence from the tda9998 commit message (Hans)
- Link to v2: https://lore.kernel.org/r/20250110-drm-move-tda998x-v2-0-ef2ae5832d21@linaro.org

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@linaro.org

---
Dmitry Baryshkov (3):
      drm/i2c: tda998x: drop support for platform_data
      media: cec: move driver for TDA9950 from drm/i2c
      drm/i2c: move TDA998x driver 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,
-- 
Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

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

Posted by Dmitry Baryshkov 10 months, 1 week ago

On Mon, 13 Jan 2025 11:53:43 +0200, 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/.
> 
> [...]

Applied to drm-misc-next, thanks!

[1/3] drm/i2c: tda998x: drop support for platform_data
      commit: b7c5169ab9ddc3432161412bc2de28111306fbb3
[2/3] media: cec: move driver for TDA9950 from drm/i2c
      commit: caa6f4a75e9f8eaebd814a9c2d1602e0926f473d
[3/3] drm/i2c: move TDA998x driver under drivers/gpu/drm/bridge
      commit: 325ba852d148434c5bcb06d513af1933a7f77b70

Best regards,
-- 
With best wishes
Dmitry

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

Posted by Dmitry Baryshkov 10 months, 1 week ago

On Mon, Jan 13, 2025 at 11:53:43AM +0200, 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: In a comment for v2 Hans has acked merging these patches through
> drm-misc(-next) with no additional immutable branch.
> 
> 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@linaro.org>
> ---
> Changes in v3:
> - Dropped irrelevant sentence from the tda9998 commit message (Hans)
> - Link to v2: https://lore.kernel.org/r/20250110-drm-move-tda998x-v2-0-ef2ae5832d21@linaro.org
> 
> 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@linaro.org
> 
> ---
> Dmitry Baryshkov (3):
>       drm/i2c: tda998x: drop support for platform_data
>       media: cec: move driver for TDA9950 from drm/i2c
>       drm/i2c: move TDA998x driver under drivers/gpu/drm/bridge

Dear drm-misc maintainers, is it okay to apply the patchset?

Hans previously (in response to v2 cover letter) acked mergine of the
CEC patch without additional immutable branch.

-- 
With best wishes
Dmitry