Re: [PATCHv2 0/6] drm/omap: hdmi: improve hdmi4 CEC, add CEC for hdmi5

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 25/08/2024 23:31, H. Nikolaus Schaller wrote:
Hi,
CEC features are useful to e.g. control HDMI monitor standby.

But I wonder what happened to this series?

I could find some reviewed-by: and acked-by: in [1] but it wasn't merged upstream
for unidentifiable reasons.

We apparently had merged this series some years ago into our LetuxOS distro kernel
and now we found it to be broken (NULL dereference) at least for omap5uevm
(and likely Pyra Handheld) after rebasing to v6.11-rc (it was already broken
since v6.9-rc1). Fixes were not difficult, but it would be better if it were
part of upstream.

There was a v3:

20210428132545.1205162-1-hverkuil-cisco@xxxxxxxxx

I see there was a concern from Laurent in:

YLjMZiX71mcQNQdO@xxxxxxxxxxxxxxxxxxxxxxxxxx

And we need an ack from the bridge maintainers for the drm_bridge parts. But the series is three years old, so I think someone would have to rebase on top of mainline and re-test and re-send first.

 Tomi

BR and thanks,
Nikolaus

[1] https://lore.kernel.org/r/all/20210302162403.983585-4-hverkuil-cisco@xxxxxxxxx/T/

Am 02.03.2021 um 17:23 schrieb Hans Verkuil <hverkuil-cisco@xxxxxxxxx>:

This series improves the drm_bridge support for CEC by introducing two
new bridge ops in the first patch, and using those in the second patch.

This makes it possible to call cec_s_conn_info() and set
CEC_CAP_CONNECTOR_INFO for the CEC adapter, so userspace can associate
the CEC adapter with the corresponding DRM connector.

The third patch simplifies CEC physical address handling by using the
cec_s_phys_addr_from_edid helper function that didn't exist when this
code was originally written.

The fourth patch adds the cec clock to ti,omap5-dss.txt.

The fifth patch the missing cec clock to the dra7 and omap5 device tree,
and the last patch adds CEC support to the OMAP5 driver.

Tested with a Pandaboard and a Beagle X15 board.

Regards,

Hans

Changes since v1:

- as per suggestion from Laurent, changed cec_init/exit to
  connector_attach/_detach which are just called for all
  bridges. The DRM_BRIDGE_OP_CEC was dropped.

- added patch to add the cec clock to ti,omap5-dss.txt

- swapped the order of the last two patches

- incorporated Tomi's suggestions for the hdmi5 CEC support.

Hans Verkuil (6):
  drm: drm_bridge: add connector_attach/detach bridge ops
  drm/omapdrm/dss/hdmi4: switch to the connector bridge ops
  drm/omapdrm/dss/hdmi4: simplify CEC Phys Addr handling
  dt-bindings: display: ti: ti,omap5-dss.txt: add cec clock
  dra7.dtsi/omap5.dtsi: add cec clock
  drm/omapdrm/dss/hdmi5: add CEC support

.../bindings/display/ti/ti,omap5-dss.txt      |   4 +-
arch/arm/boot/dts/dra7.dtsi                   |   5 +-
arch/arm/boot/dts/omap5.dtsi                  |   5 +-
drivers/gpu/drm/drm_bridge_connector.c        |   9 +
drivers/gpu/drm/omapdrm/Kconfig               |   8 +
drivers/gpu/drm/omapdrm/Makefile              |   1 +
drivers/gpu/drm/omapdrm/dss/hdmi.h            |   1 +
drivers/gpu/drm/omapdrm/dss/hdmi4.c           |  40 ++--
drivers/gpu/drm/omapdrm/dss/hdmi4_cec.c       |  13 +-
drivers/gpu/drm/omapdrm/dss/hdmi4_cec.h       |  12 +-
drivers/gpu/drm/omapdrm/dss/hdmi5.c           |  63 +++++-
drivers/gpu/drm/omapdrm/dss/hdmi5_cec.c       | 209 ++++++++++++++++++
drivers/gpu/drm/omapdrm/dss/hdmi5_cec.h       |  42 ++++
drivers/gpu/drm/omapdrm/dss/hdmi5_core.c      |  35 ++-
drivers/gpu/drm/omapdrm/dss/hdmi5_core.h      |  33 ++-
include/drm/drm_bridge.h                      |  27 +++
16 files changed, 453 insertions(+), 54 deletions(-)
create mode 100644 drivers/gpu/drm/omapdrm/dss/hdmi5_cec.c
create mode 100644 drivers/gpu/drm/omapdrm/dss/hdmi5_cec.h

--
2.30.1







[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux