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,
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.

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 Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux