Re: [PATCH 0/4] video/exynos/cec: add HDMI state notifier & use in s5p-cec

[Marek Szyprowski]

Hi Hans,


On 2016-12-13 16:08, Hans Verkuil wrote:

From: Hans Verkuil 

This patch series adds the HDMI notifier code, based on Russell's code:

https://patchwork.kernel.org/patch/9277043/

It adds support for it to the exynos_hdmi drm driver, adds support for
it to the CEC framework and finally adds support to the s5p-cec driver,
which now can be moved out of staging.

Tested with my Odroid U3 exynos4 devboard.

Comments are welcome. I'd like to get this in for the 4.11 kernel as
this is a missing piece needed to integrate CEC drivers.

Benjamin, can you look at doing the same notifier integration for your
st-cec driver as is done for s5p-cec? It would be good to be able to
move st-cec out of staging at the same time.


Thanks for working on this and taking it from by TODO list! :)

Please add:
Tested-by: Marek Szyprowski 

If you plan to send an updated version, please send it also to
linux-samsung-...@vger.kernel.org, Krzysztof and Inki to get their acks
for the bindings, dtsi and drm parts.

This HDMI notifier framework will probably be also useful for integrating
HDMI audio support for Samsung ASoC driver.


Regards,

Hans

Hans Verkuil (4):
   video: add HDMI state notifier support
   exynos_hdmi: add HDMI notifier support
   cec: integrate HDMI notifier support
   s5p-cec: add hdmi-notifier support, move out of staging

  .../devicetree/bindings/media/s5p-cec.txt  |   2 +
  arch/arm/boot/dts/exynos4.dtsi |   1 +
  drivers/gpu/drm/exynos/Kconfig |   1 +
  drivers/gpu/drm/exynos/exynos_hdmi.c   |  24 +++-
  drivers/media/cec/cec-core.c   |  50 
  drivers/media/platform/Kconfig |  18 +++
  drivers/media/platform/Makefile|   1 +
  .../media => media/platform}/s5p-cec/Makefile  |   0
  .../platform}/s5p-cec/exynos_hdmi_cec.h|   0
  .../platform}/s5p-cec/exynos_hdmi_cecctrl.c|   0
  .../media => media/platform}/s5p-cec/regs-cec.h|   0
  .../media => media/platform}/s5p-cec/s5p_cec.c |  35 +-
  .../media => media/platform}/s5p-cec/s5p_cec.h |   3 +
  drivers/staging/media/Kconfig  |   2 -
  drivers/staging/media/Makefile |   1 -
  drivers/staging/media/s5p-cec/Kconfig  |   9 --
  drivers/staging/media/s5p-cec/TODO |   7 --
  drivers/video/Kconfig  |   3 +
  drivers/video/Makefile |   1 +
  drivers/video/hdmi-notifier.c  | 134 +
  include/linux/hdmi-notifier.h  | 109 +
  include/media/cec.h|  15 +++
  22 files changed, 389 insertions(+), 27 deletions(-)
  rename drivers/{staging/media => media/platform}/s5p-cec/Makefile (100%)
  rename drivers/{staging/media => media/platform}/s5p-cec/exynos_hdmi_cec.h 
(100%)
  rename drivers/{staging/media => 
media/platform}/s5p-cec/exynos_hdmi_cecctrl.c (100%)
  rename drivers/{staging/media => media/platform}/s5p-cec/regs-cec.h (100%)
  rename drivers/{staging/media => media/platform}/s5p-cec/s5p_cec.c (89%)
  rename drivers/{staging/media => media/platform}/s5p-cec/s5p_cec.h (97%)
  delete mode 100644 drivers/staging/media/s5p-cec/Kconfig
  delete mode 100644 drivers/staging/media/s5p-cec/TODO
  create mode 100644 drivers/video/hdmi-notifier.c
  create mode 100644 include/linux/hdmi-notifier.h



Best regards
--
Marek Szyprowski, PhD
Samsung R Institute Poland

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 0/4] video/exynos/cec: add HDMI state notifier & use in s5p-cec

[Hans Verkuil]

From: Hans Verkuil 

This patch series adds the HDMI notifier code, based on Russell's code:

https://patchwork.kernel.org/patch/9277043/

It adds support for it to the exynos_hdmi drm driver, adds support for
it to the CEC framework and finally adds support to the s5p-cec driver,
which now can be moved out of staging.

Tested with my Odroid U3 exynos4 devboard.

Comments are welcome. I'd like to get this in for the 4.11 kernel as
this is a missing piece needed to integrate CEC drivers.

Benjamin, can you look at doing the same notifier integration for your
st-cec driver as is done for s5p-cec? It would be good to be able to
move st-cec out of staging at the same time.

Regards,

Hans

Hans Verkuil (4):
  video: add HDMI state notifier support
  exynos_hdmi: add HDMI notifier support
  cec: integrate HDMI notifier support
  s5p-cec: add hdmi-notifier support, move out of staging

 .../devicetree/bindings/media/s5p-cec.txt  |   2 +
 arch/arm/boot/dts/exynos4.dtsi |   1 +
 drivers/gpu/drm/exynos/Kconfig |   1 +
 drivers/gpu/drm/exynos/exynos_hdmi.c   |  24 +++-
 drivers/media/cec/cec-core.c   |  50 
 drivers/media/platform/Kconfig |  18 +++
 drivers/media/platform/Makefile|   1 +
 .../media => media/platform}/s5p-cec/Makefile  |   0
 .../platform}/s5p-cec/exynos_hdmi_cec.h|   0
 .../platform}/s5p-cec/exynos_hdmi_cecctrl.c|   0
 .../media => media/platform}/s5p-cec/regs-cec.h|   0
 .../media => media/platform}/s5p-cec/s5p_cec.c |  35 +-
 .../media => media/platform}/s5p-cec/s5p_cec.h |   3 +
 drivers/staging/media/Kconfig  |   2 -
 drivers/staging/media/Makefile |   1 -
 drivers/staging/media/s5p-cec/Kconfig  |   9 --
 drivers/staging/media/s5p-cec/TODO |   7 --
 drivers/video/Kconfig  |   3 +
 drivers/video/Makefile |   1 +
 drivers/video/hdmi-notifier.c  | 134 +
 include/linux/hdmi-notifier.h  | 109 +
 include/media/cec.h|  15 +++
 22 files changed, 389 insertions(+), 27 deletions(-)
 rename drivers/{staging/media => media/platform}/s5p-cec/Makefile (100%)
 rename drivers/{staging/media => media/platform}/s5p-cec/exynos_hdmi_cec.h 
(100%)
 rename drivers/{staging/media => media/platform}/s5p-cec/exynos_hdmi_cecctrl.c 
(100%)
 rename drivers/{staging/media => media/platform}/s5p-cec/regs-cec.h (100%)
 rename drivers/{staging/media => media/platform}/s5p-cec/s5p_cec.c (89%)
 rename drivers/{staging/media => media/platform}/s5p-cec/s5p_cec.h (97%)
 delete mode 100644 drivers/staging/media/s5p-cec/Kconfig
 delete mode 100644 drivers/staging/media/s5p-cec/TODO
 create mode 100644 drivers/video/hdmi-notifier.c
 create mode 100644 include/linux/hdmi-notifier.h

-- 
2.10.2

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html