RE: [PATCH v6 00/11] HDMI CEC framework

-- Kamil Debski Samsung R&D Institute Poland > -----Original Message----- > From: Kamil Debski [mailto:k.debski@samsung.com] > Sent: Monday, May 04, 2015 7:33 PM > To: dri-devel@lists.freedesktop.org; linux-media@vger.kernel.org > Cc: m.szyprowski@samsung.com; k.debski@samsung.com; > mchehab@osg.samsung.com; hverkuil@xs4all.nl; kyungmin.park@samsung.com; > thomas@tommie-lie.de; sean@mess.org; dmitry.torokhov@gmail.com; linux- > input@vger.kernel.org; linux-samsung-soc@vger.kernel.org; > lars@opdenkamp.eu > Subject: [PATCH v6 00/11] > > Hi, > > The sixth version of this patchset addresses recent comments on the > mailing list. Please see the changelog below for details. > > 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 > > -- > 1.7.9.5