On 05/04/2015 07:32 PM, Kamil Debski wrote:

Hi,

The sixth version of this patchset addresses recent comments on the mailing list. Please see the changelog below for details.

Just in case people are wondering what happened to this: about a month ago I took over from Kamil and I am working hard to get a v7 posted. I'm currently working on a utility to send messages over the cec line and once that's done I plan on posting a new version which should be close to the final version.

One of the main problems is making sure that the framework covers the full CEC 2.0 functionality (or at least, making sure that any missing pieces can be added seamlessly later). CEC 2.0 is a lot more strict in what should and what shouldn't be implemented, so this takes time.

Regards,

Hans

Best wishes, Kamil Debski

Changes since v5

• drop struct cec_timeval in favour of a __u64 that keeps the timestamp in ns
• remove userspace documentation from Documentation/cec.txt as userspace API is described in the DocBook
• add missing documentation for the passthrough mode to the DocBook
• add information about the number of events that can be queued
• fix misspelling of reply
• fix behaviour of posting an event in cec_received_msg, such that the behaviour is consistent with the documentation

Changes since v4

• add sequence numbering to transmitted messages
• add sequence number handling to event hanlding
• add passthrough mode
• change reserved field sizes
• fixed CEC version defines and addec CEC 2.0 commands
• add DocBook documentation

Changes since v3

• remove the promiscuous mode
• rewrite the devicetree patches
• fixes, expansion and partial rewrite of the documentation
• reorder of API structures and addition of reserved fields
• use own struct to report time (32/64 bit safe)
• fix of handling events
• add cec.h to include/uapi/linux/Kbuild
• fixes in the adv76xx driver (add missing methods, change adv7604 to adv76xx)
• cleanup of debug messages in s5p-cec driver
• remove non necessary claiming of a gpio in the s5p-cec driver
• cleanup headers of the s5p-cec driver

Changes since v2

• added promiscuous mode
• added new key codes to the input framework
• add vendor ID reporting
• add the possibility to clear assigned logical addresses
• cleanup of the rc cec map

Changes since v1

• documentation edited and moved to the Documentation folder
• added key up/down message handling
• add missing CEC commands to the cec.h file

Background

The work on a common CEC framework was started over three years ago by Hans Verkuil. Unfortunately the work has stalled. As I have received the task of creating a driver for the CEC interface module present on the Exynos range of SoCs, I got in touch with Hans. He replied that the work stalled due to his lack of time.

Original RFC by Hans Verkuil/Martin Bugge

https://www.mail-archive.com/linux-media@vger.kernel.org/msg28735.html

Hans Verkuil (5): cec: add HDMI CEC framework DocBook/media: add CEC documentation v4l2-subdev: add HDMI CEC ops cec: adv7604: add cec support. cec: adv7511: add cec support.

Kamil Debski (6): dts: exynos4*: add HDMI CEC pin definition to pinctrl dts: exynos4: add node for the HDMI CEC device dts: exynos4412-odroid*: enable the HDMI CEC device HID: add HDMI CEC specific keycodes rc: Add HDMI CEC protoctol handling cec: s5p-cec: Add s5p-cec driver

Documentation/DocBook/media/Makefile | 4 +- Documentation/DocBook/media/v4l/biblio.xml | 10 + Documentation/DocBook/media/v4l/cec-api.xml | 74 ++ Documentation/DocBook/media/v4l/cec-func-close.xml | 59 + Documentation/DocBook/media/v4l/cec-func-ioctl.xml | 73 ++ Documentation/DocBook/media/v4l/cec-func-open.xml | 94 ++ Documentation/DocBook/media/v4l/cec-func-poll.xml | 89 ++ .../DocBook/media/v4l/cec-ioc-g-adap-log-addrs.xml | 275 +++++ .../DocBook/media/v4l/cec-ioc-g-adap-phys-addr.xml | 78 ++ .../DocBook/media/v4l/cec-ioc-g-adap-state.xml | 87 ++ Documentation/DocBook/media/v4l/cec-ioc-g-caps.xml | 173 +++ .../DocBook/media/v4l/cec-ioc-g-event.xml | 125 ++ .../DocBook/media/v4l/cec-ioc-g-passthrough.xml | 88 ++ .../DocBook/media/v4l/cec-ioc-g-vendor-id.xml | 70 ++ .../DocBook/media/v4l/cec-ioc-receive.xml | 185 +++ Documentation/DocBook/media_api.tmpl | 6 +- Documentation/cec.txt | 165 +++ .../devicetree/bindings/media/s5p-cec.txt | 33 + arch/arm/boot/dts/exynos4.dtsi | 12 + arch/arm/boot/dts/exynos4210-pinctrl.dtsi | 7 + arch/arm/boot/dts/exynos4412-odroid-common.dtsi | 4 + arch/arm/boot/dts/exynos4x12-pinctrl.dtsi | 7 + drivers/media/Kconfig | 6 + drivers/media/Makefile | 2 + drivers/media/cec.c | 1191 ++++++++++++++++++++ drivers/media/i2c/adv7511.c | 347 +++++- drivers/media/i2c/adv7604.c | 207 +++- drivers/media/platform/Kconfig | 10 + drivers/media/platform/Makefile | 1 + drivers/media/platform/s5p-cec/Makefile | 4 + drivers/media/platform/s5p-cec/exynos_hdmi_cec.h | 37 + .../media/platform/s5p-cec/exynos_hdmi_cecctrl.c | 208 ++++ drivers/media/platform/s5p-cec/regs-cec.h | 96 ++ drivers/media/platform/s5p-cec/s5p_cec.c | 283 +++++ drivers/media/platform/s5p-cec/s5p_cec.h | 76 ++ drivers/media/rc/keymaps/Makefile | 1 + drivers/media/rc/keymaps/rc-cec.c | 144 +++ drivers/media/rc/rc-main.c | 1 + include/media/adv7511.h | 6 +- include/media/cec.h | 142 +++ include/media/rc-core.h | 1 + include/media/rc-map.h | 5 +- include/media/v4l2-subdev.h | 8 + include/uapi/linux/Kbuild | 1 + include/uapi/linux/cec.h | 332 ++++++ include/uapi/linux/input.h | 12 + 46 files changed, 4824 insertions(+), 15 deletions(-) create mode 100644 Documentation/DocBook/media/v4l/cec-api.xml create mode 100644 Documentation/DocBook/media/v4l/cec-func-close.xml create mode 100644 Documentation/DocBook/media/v4l/cec-func-ioctl.xml create mode 100644 Documentation/DocBook/media/v4l/cec-func-open.xml create mode 100644 Documentation/DocBook/media/v4l/cec-func-poll.xml create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-g-adap-log-addrs.xml create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-g-adap-phys-addr.xml create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-g-adap-state.xml create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-g-caps.xml create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-g-event.xml create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-g-passthrough.xml create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-g-vendor-id.xml create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-receive.xml create mode 100644 Documentation/cec.txt create mode 100644 Documentation/devicetree/bindings/media/s5p-cec.txt create mode 100644 drivers/media/cec.c create mode 100644 drivers/media/platform/s5p-cec/Makefile create mode 100644 drivers/media/platform/s5p-cec/exynos_hdmi_cec.h create mode 100644 drivers/media/platform/s5p-cec/exynos_hdmi_cecctrl.c create mode 100644 drivers/media/platform/s5p-cec/regs-cec.h create mode 100644 drivers/media/platform/s5p-cec/s5p_cec.c create mode 100644 drivers/media/platform/s5p-cec/s5p_cec.h create mode 100644 drivers/media/rc/keymaps/rc-cec.c create mode 100644 include/media/cec.h create mode 100644 include/uapi/linux/cec.h